julia_v_iluhina

Untitled

Aug 1st, 2016
67
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.99 KB | None | 0 0
  1.  
  2.     private void addTasks(String ... taskText){
  3.     private void deleteTask(String taskText){
  4.     private void toggle(String taskText){
  5.  
  6.     private void toogleAll(){
  7.     private void clearCompleted(){
  8. /*
  9.     Это - методы для выполнения действий над тасками
  10.  
  11.     в именах одних методов - ты уточнил - что работаем с тасками
  12.     в других именах - не стал
  13.  
  14.     лучше быть последовательным)
  15.  
  16.     допускаю, тут особняком стоит clearCompleted() - термин взяли с UI (User Interface)
  17.     и он наилучший тут
  18.  
  19.     а для остальных - должна быть какая-то одна логика
  20.     я описывала в прошлом ревью варианты, и какой из них более предпочтительный с моей точки зрения
  21.  
  22.     а когда в именах одних методов уточнили, а в других - не уточнили
  23.     так возникают сразу вопросы - addTasks и deleteTask - понятно, работаем с тасками
  24.     а вот тут, интересно с чем - toggle и toogleAll
  25.     потому - важно придерживаться одной логики в нейминге
  26.  
  27.     когда параметр метода - не текст таски, а текстЫ тасок - это надо уточнить
  28.     не String ... taskText
  29.     а  String ... taskTexts
  30.  
  31.     чтобы переименовать класс/методпеременную/параметр
  32.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.vwuqi54t6fyg
  33.  
  34. */
  35. *****************************************************
  36.     private void assertTasksAre(String ... taskText){
  37.         $$("#todo-list li label").shouldHave(exactTexts(taskText));
  38.     }
  39. /*
  40.     уточнять до label - не нужно
  41.     использовать переменную tasks  - нужно
  42.     уже к списку тасок и его внутренностям - обращаемся исключительно через tasks
  43.  
  44.     про имя параметра - см выше
  45. */
  46. ************************************
  47.  
  48.     private void isEmptyList(){
  49.         tasks.shouldBe(empty);
  50.     }
  51. /*
  52.     для метода, выполняющего проверку - лучше начать имя с assert
  53.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.tezjc2whgbl7
  54.    
  55.     что мы проверяем - мы проверяем сто список тасок пуст - про это в имени не написано
  56.     в имени метода assertTasksAre - мы список тасок называли Tasks
  57.    
  58.     логично и в этом случае также назвать
  59.    
  60.     assertTasksAreEmpty - немного двусмысленно (не понятно - что пусто - то ли таски в списке, то ли сам список), но в принципе приемлемо
  61.     assertTasksListIsEmpty - уже однозначнее, но тут для списка тасок применили другой термин - TasksList , хотя везде применяли Tasks,
  62.     тоже можно использовать
  63.     assertNoTasks - достаточно точно, используем те же термины, мне такой вариант нравится больше всех перечисленных
  64.    
  65.     мне - нравится пара имен - assertTasks & assertNoTasks - все лишнее убрали, оставили только суть
  66.    
  67.     ты - можешь использовать какой-то из этих или придумать свой,
  68.     описала логику рассуждений специально
  69.  
  70. */
Advertisement
Add Comment
Please, Sign In to add comment