julia_v_iluhina

Untitled

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