Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- ну... паттерн немного не выдержан, конечно
- посмотри еще на мои пояснения и на приведенные линки - в прошлом ревью
- все методы - кроме build - просто собирают данные
- build - создает строящийся объект
- и уже в рамках конструктора этого объекта - уже согласно всем собранным данным что-то там делается
- это касается абсолютно всех действий, которые нужно сделать
- делает - построенный объект
- а строитель - только строит этот объект, и более ничего не делает
- читай про Single Responsibility Principle )
- и строим мы = точно не Task )
- подумай над именем класса
- и еще на подумать
- вызовем given().completedTasks("a").activeTasks("b").completedTasks("c")
- и я жду при таком вызове - что именно в таком порядке таски создадутся
- и в таких состояниях
- а в текущей реализации - сначала добавятся активные, потом - закомпличеные
- по поводу не отображения тасок - нужно после заполнения локал сториджа выполнить JavaScript-команду location.reload()
- просто рефреша не хватит при такой реализации перехода по фильтрам
- и executeJavaScript("localStorage.clear()"); - не нужно вообще
- подумай почему
- также позаботься про DRY )
- много повторений
- пока это сделай)
Advertisement
Add Comment
Please, Sign In to add comment