julia_v_iluhina

Untitled

Dec 26th, 2016
91
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.06 KB | None | 0 0
  1.     public Task (String description, boolean done, String title, int id, String uri) {
  2.         ...
  3.         this.uri = uri + "/" + id;
  4.     }
  5. /*
  6.     я бы все равно не стала внутри класса Task - зашивать логику работы с uri
  7.     Не надо классу-контейнеру знать - как этот uri получается
  8.  
  9.     У этого класса - одна функция - быть контейнером
  10.     И нечего ему знать как генерить uri для таски
  11.  
  12.     Что ему передали - то и будет uri для таски
  13.     Single Responsibility Principle
  14.  
  15.     считаю вот такой вариант - лучше
  16.     public Task (String description, boolean done, String title, String uri)
  17.  
  18.     и соответственно - передавать будешь в качестве uri - TasksApi.uri + "/" + id
  19. */
  20. ************************
  21.     @Before
  22.     public void resetRestfulBase() {
  23.         reset();
  24.     }
  25. //или
  26.     @Before
  27.     public void reset() {
  28.         TasksApi.reset();
  29.     }
  30. /*
  31.     в целом - букв примерно столько же)
  32.     но - такой вариант мне кажется - буквы более полезные ) - от них больше полезной уточняющей информации
  33.  
  34.     я бы вызывала методы ресурса-модуля - указывая его имя
  35.     и сэкономила
  36.     на именах методов тест-класса(не testReadTasks(), а testRead())
  37.     и самого ресурса(тут ты уже это сделал)
  38.  
  39.     но вызывая методы ресурса - указывала бы имя класса для ресурса
  40. */
  41. *************************
  42. public void testCreateTask()
  43. public void testCreateNotFilledTask()
  44. //сравни
  45. public void testCreateWithFullInformation()
  46. public void testCreateWithTitle()
  47. /*
  48.     второй вариант - однозначнее, понятнее
  49. */
Advertisement
Add Comment
Please, Sign In to add comment