Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- /*
- практически идеально
- уточнить селектор + добавить одну проверку
- и моя придирка)
- */
- *****************
- $("strong")
- /*
- лучше уточни селектор
- идентифицировать элемент только по типу элемента - слишком рискованно
- добавится еще элемент такого же типа - и все... хатка поломается)
- http://joxi.ru/DrlQ5oLh4aOVnm
- у родительского элемента есть id
- это будет хорошим надежным уточнением
- */
- ********************************************
- ...
- page.filterCompleted();
- page.assertVisibleTasks("a edited", "b");
- page.toggle("b");
- /*
- вот тут после переоткрытия таски - нужна проверка
- следующее действие не проверяет переоткрытие таски
- */
- page.clearCompleted();
- ...
- ************************************
- http://joxi.ru/1A5zNxjuKeyGJr
- /*
- это придирка)
- на самом деле - лучше послушайся
- хорошо оформленный код - сильно облегчает работу в команде)
- а если он еще и просто хороший - вообще цены тебе не будет)
- тем более - что правила оформлени не сложные и многое можно подправлять очень быстро
- см рисунок,
- 1 - я бы в одно-двустрочных методах такие пропуски не делала
- а вот между методами - стоит пропускать строку - облегчает чтение кода
- выделила прямоугольниками слева это
- а в многострочных методах пропуски строк между блоками кода - тоже облегчают
- ты это очень хорошо используешь как раз)
- про пропуски строк - тоже есть конвеншенсы
- https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
- про это надо отдельно самому заботиться
- реформатирование кода тут уже не поможет)
- */
Advertisement
Add Comment
Please, Sign In to add comment