julia_v_iluhina

Untitled

Oct 19th, 2016
98
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 1.89 KB | None | 0 0
  1. ну... паттерн немного не выдержан, конечно
  2. посмотри еще на мои пояснения и на приведенные линки - в прошлом ревью
  3.  
  4. все методы - кроме build - просто собирают данные
  5. build - создает строящийся объект
  6.  
  7. и уже в рамках конструктора этого объекта - уже согласно всем собранным данным что-то там делается
  8. это касается абсолютно всех действий, которые нужно сделать
  9.  
  10. делает - построенный объект
  11. а строитель - только строит этот объект, и более ничего не делает
  12.  
  13. читай про Single Responsibility Principle )
  14.  
  15. и строим мы = точно не Task )
  16. подумай над именем класса
  17.  
  18. и еще на подумать
  19. вызовем given().completedTasks("a").activeTasks("b").completedTasks("c")
  20. и я жду при таком вызове - что именно в таком порядке таски создадутся
  21. и в таких состояниях
  22. а в текущей реализации - сначала добавятся активные, потом - закомпличеные
  23.  
  24. по поводу не отображения тасок - нужно после заполнения локал сториджа выполнить JavaScript-команду location.reload()
  25. просто рефреша не хватит при такой реализации перехода по фильтрам
  26.  
  27. и executeJavaScript("localStorage.clear()"); - не нужно вообще
  28. подумай почему
  29.  
  30. также позаботься про DRY )
  31. много повторений
  32.  
  33. пока это сделай)
Advertisement
Add Comment
Please, Sign In to add comment