Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- private void addTasks(String ... taskText){
- private void deleteTask(String taskText){
- private void toggle(String taskText){
- private void toogleAll(){
- private void clearCompleted(){
- /*
- Это - методы для выполнения действий над тасками
- в именах одних методов - ты уточнил - что работаем с тасками
- в других именах - не стал
- лучше быть последовательным)
- допускаю, тут особняком стоит clearCompleted() - термин взяли с UI (User Interface)
- и он наилучший тут
- а для остальных - должна быть какая-то одна логика
- я описывала в прошлом ревью варианты, и какой из них более предпочтительный с моей точки зрения
- а когда в именах одних методов уточнили, а в других - не уточнили
- так возникают сразу вопросы - addTasks и deleteTask - понятно, работаем с тасками
- а вот тут, интересно с чем - toggle и toogleAll
- потому - важно придерживаться одной логики в нейминге
- когда параметр метода - не текст таски, а текстЫ тасок - это надо уточнить
- не String ... taskText
- а String ... taskTexts
- чтобы переименовать класс/методпеременную/параметр
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.vwuqi54t6fyg
- */
- *****************************************************
- private void assertTasksAre(String ... taskText){
- $$("#todo-list li label").shouldHave(exactTexts(taskText));
- }
- /*
- уточнять до label - не нужно
- использовать переменную tasks - нужно
- уже к списку тасок и его внутренностям - обращаемся исключительно через tasks
- про имя параметра - см выше
- */
- ************************************
- private void isEmptyList(){
- tasks.shouldBe(empty);
- }
- /*
- для метода, выполняющего проверку - лучше начать имя с assert
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.tezjc2whgbl7
- что мы проверяем - мы проверяем сто список тасок пуст - про это в имени не написано
- в имени метода assertTasksAre - мы список тасок называли Tasks
- логично и в этом случае также назвать
- assertTasksAreEmpty - немного двусмысленно (не понятно - что пусто - то ли таски в списке, то ли сам список), но в принципе приемлемо
- assertTasksListIsEmpty - уже однозначнее, но тут для списка тасок применили другой термин - TasksList , хотя везде применяли Tasks,
- тоже можно использовать
- assertNoTasks - достаточно точно, используем те же термины, мне такой вариант нравится больше всех перечисленных
- мне - нравится пара имен - assertTasks & assertNoTasks - все лишнее убрали, оставили только суть
- ты - можешь использовать какой-то из этих или придумать свой,
- описала логику рассуждений специально
- */
Advertisement
Add Comment
Please, Sign In to add comment