julia_v_iluhina

Untitled

Sep 22nd, 2016
90
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.71 KB | None | 0 0
  1.     ElementsCollection tasks = $$("#todo-list li");
  2. /*
  3.     Есть селекторо поточнее - "#todo-list>li"
  4.     тут можно и так оставить
  5.  
  6.     в некоторых случаях это может быть важным
  7. */
  8. ***************************************************
  9.     private void assertTasksAre(String... task){
  10. /*
  11.     на самом деле мы не таскУ передаем в качестве параметров
  12.     а таскИ
  13.     или еще точнее - текстЫ тасок
  14.     taskTexts - тут более уместный термин
  15.  
  16.     то же самое касается и других методов, принимающих текст/таксты тасок в качестве параметра
  17.     taskText / taskTexts
  18.  
  19.     кстати, для метода add - ты как таз такой термин и применила)
  20.  
  21.     старайся придерживаться правила
  22.     для одного понятия использовать один термин
  23.  
  24.     тогда код будет понятнее/однозначнее
  25.  
  26.     как переименовать параметр / метод/ класс / переменную
  27.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.uaiq15o3yjor
  28. */
  29. *******************************
  30.     private void deleteTask(String task){
  31. /*
  32.     в именах большинства вспомогательных методов-действий - ты не уточняла - что работаем с тасками -
  33.     а тут - уточняешь
  34.  
  35.     тут за счет того - что мы работаем только с тасками - можно из имен
  36.     вспомогательных методов-действий убрать слово Task
  37.  
  38.     получишь
  39.         add
  40.         delete
  41.         markCompleted
  42.         toggleAll
  43.         clearCompleted
  44.  
  45.     заметь, мы тут не говорим о методах-проверках
  46.     с ними - другая история )
  47.     даже в таком простом приложении - проверок будет несколько
  48.     и потому важно в имени метода-мпроверки - точно описать - что проверяем
  49. */
  50. ***************************************************
  51.     private void markCompleted(String task){
  52.         tasks.find(exactText(task)).find(".toggle").click();
  53.     }
  54. /*
  55.     что касается термина markCompleted - можно лаконичнее - complete
  56.  
  57.     но и этого нам будет мало
  58.  
  59.     примени этот метод к уже запомпличеной таске
  60.     разве метод в таком случае выполнит complete?
  61.     а имя метода должно четко отражать - что делает метод
  62.     вне зависимости от контекста вызова
  63.  
  64.     вот посмотри про однозначность имен методов
  65.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.tmkuktenqqdi
  66.  
  67.  
  68.     подумай, какое имя для метода с такой реализацией - будет однозначным
  69. */
  70. *************************************************
  71.     private void shouldBeEmpty(){
  72. /*
  73.     вот это почитай)
  74.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.sswzyp7qkm9o
  75.    
  76.     и прими окончательное решение по именам методов-проверок
  77. */
Advertisement
Add Comment
Please, Sign In to add comment