julia_v_iluhina

Untitled

Nov 7th, 2016
77
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.84 KB | None | 0 0
  1.     public void assertTasksVisible(String... taskTexts) {
  2.         tasks.filter(visible).shouldHave(exactTexts(taskTexts));
  3.     }
  4.  
  5.     public void assertTasksInvisible() {
  6.         tasks.filter(visible).shouldBe(empty);
  7.     }
  8.  
  9.     public void assertTasksAre(String... taskTexts) {
  10.         tasks.shouldHave(exactTexts(taskTexts));
  11.     }
  12.  
  13.     public void assertNoTasks() {
  14.         tasks.shouldBe(empty);
  15.     }
  16.  
  17. /*
  18.     для проверки текстов тасок - у нас есть методы
  19.     assertTasksAre
  20.     assertTasksVisible
  21.  
  22.     а для проверки - что список тасок пуст (или список видимых тасок пуст)
  23.     assertNoTasks()
  24.     assertTasksInvisible()
  25.  
  26.     разброд и шатание)
  27.  
  28.     ранее у нас была пара
  29.     assertTasksAre
  30.     assertNoTasks()
  31.     В общем-то - хорошие имена для методов, достаточно точно отражают суть
  32.  
  33.     появляется вторая пара
  34.     в которой все, что добавляется
  35.     это то, что работаем именно с видимыми тасками
  36.     т е не с Tasks, а с VisibleTasks
  37.     напрашивается логичное решение - и следующую пару методов - назвать - руководствуясь той же логикой
  38.     assertVisibleTasksAre
  39.     assertNoVisibleTasks()
  40.  
  41.     получается - что мы останемся в тех же терминах и одной логике в нейминге
  42.  
  43.     название assertTasksInvisible() - вообще может запутать
  44.     мы не проверяем не видимых тасок (их может вообще не быть)
  45.     мы проверяем - видимые таски
  46.     а именно - факт, что их нету
  47.    
  48.     это подравняй обязательно
  49. */
  50. **********************************************
  51.     public SelenideElement startEdit(String oldTaskText, String newTaskText){
  52.         ...
  53.         SelenideElement editedTask = ...
  54.         return editedTask;
  55.     }
  56. /*
  57.     сравни с вариантом
  58. */
  59.     public SelenideElement startEdit(String oldTaskText, String newTaskText){
  60.         ...
  61.         return ...;
  62.     }
  63. /*
  64.     второй вариант - лаконичнее
  65.     и не сказать, что сложнее
  66.  
  67.     разве что - используя такую переменную - ты с помощью удачно подобранного имени
  68.     можешь получше объяснить - что это такое
  69.  
  70.     только вот это название editedTask - не идеальное
  71.     edited - это и редактируемая, и отредактированная (а это очень разные вещи, в нашем случае - отличительные)
  72.     по мне, так название только путает)
  73.  
  74.     идеального варианта не вижу
  75.  
  76.     для меня, кстати, это тоже индикатор
  77.     если я нормального названия не могу подобрать для локальной переменной
  78.     то я смотрю - а могу я от нее избавиться без потери наглядности
  79.     как правило - могу)
  80.  
  81.     не настаиваю на переписывании в 2 строки
  82.     но советую)
  83. */
  84. *****************************
  85. /*
  86.     про переходы по фильтрам
  87.     http://pastebin.com/svZarY1D, строки 166-174
  88.     да, тоже не настаиваю
  89.    
  90.     я надеялась, что после просмотра видео ты изменишь свое мнение)
  91. */
Advertisement
Add Comment
Please, Sign In to add comment