Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public ElementsCollection tasksToDo = $$("#todo-list li");
- ....
- public void delete(String taskText) {
- $$(tasksToDo).find(exactText(taskText)).hover().find((".destroy")).click();
- }
- /*
- насчет имени переменной tasksToDo
- tasks - более чем достаточно)
- tasksToDo - ничем не точнее
- так зачем тогда эти уточнения дополнительные?
- других тасок у нас нету)
- tasksToDo - это уже коллекция элементов
- (она уже = $$("#todo-list li"))
- по сути - используя $$(tasksToDo)
- ты используешь $$($$("#todo-list li"))
- а этого точно делать не нужно
- просто -
- там, где ты ранее применял $$("#todo-list li")
- применяй tasksToDo
- и получишь код типа
- tasksToDo.find(exactText(taskText)).hover().find((".destroy")).click();
- в остальных местах кода - учти это
- */
- **********************************
- public void toggle(String taskText) {
- public void delete(String taskText) {
- public void add(String... taskTexts)
- public void assertTasksAre(String... tasks) {
- /*
- а вот и отгадка - почему tasksToDo)
- посмотри на другие методы
- в качестве параметра метода - мы используем текст таски или текстЫ тасок
- и это понятие обозначаем термином taskTexts / taskText
- почему те же тексты тасок
- для метода assertTasksAre
- мы обозначаем другим термином tasks?
- применяй правило
- одно понятие - один термин
- тут тоже имя для параметра метода - taskTexts
- так будет значительно лучше - однозначнее
- */
- *********************************
- public void delete(String taskText)
- public void toggle(String taskText)
- public void clearCompleted()
- public void toggleAllButton()
- /*
- мы в других методах уточнялись до Button & CheckBox?
- а тут зачем?
- лучше методы-действия называть - согласно названию действия
- вот эти уточнения про Button & CheckBox - уже излишни
- тем более, это не Button, а CheckBox)
- тем более, это 10 раз может поменяться от релиза к релизу - Button это или CheckBox
- главное - что мы делаем (какое действие)
- мы же функциональное тестирование реализуем
- и оперируем уже более формальными терминами о логике приложения
- а не кнопками и полями)
- а делаем мы - toggleAll
- посмотри на раздел faq по неймингу
- про имена методов
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.2pvr3ijzfuho
- */
Advertisement
Add Comment
Please, Sign In to add comment