Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- ElementsCollection tasks = $$("#todo-list li");
- /*
- Есть селекторо поточнее - "#todo-list>li"
- тут можно и так оставить
- в некоторых случаях это может быть важным
- */
- ***************************************************
- private void assertTasksAre(String... task){
- /*
- на самом деле мы не таскУ передаем в качестве параметров
- а таскИ
- или еще точнее - текстЫ тасок
- taskTexts - тут более уместный термин
- то же самое касается и других методов, принимающих текст/таксты тасок в качестве параметра
- taskText / taskTexts
- кстати, для метода add - ты как таз такой термин и применила)
- старайся придерживаться правила
- для одного понятия использовать один термин
- тогда код будет понятнее/однозначнее
- как переименовать параметр / метод/ класс / переменную
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.uaiq15o3yjor
- */
- *******************************
- private void deleteTask(String task){
- /*
- в именах большинства вспомогательных методов-действий - ты не уточняла - что работаем с тасками -
- а тут - уточняешь
- тут за счет того - что мы работаем только с тасками - можно из имен
- вспомогательных методов-действий убрать слово Task
- получишь
- add
- delete
- markCompleted
- toggleAll
- clearCompleted
- заметь, мы тут не говорим о методах-проверках
- с ними - другая история )
- даже в таком простом приложении - проверок будет несколько
- и потому важно в имени метода-мпроверки - точно описать - что проверяем
- */
- ***************************************************
- private void markCompleted(String task){
- tasks.find(exactText(task)).find(".toggle").click();
- }
- /*
- что касается термина markCompleted - можно лаконичнее - complete
- но и этого нам будет мало
- примени этот метод к уже запомпличеной таске
- разве метод в таком случае выполнит complete?
- а имя метода должно четко отражать - что делает метод
- вне зависимости от контекста вызова
- вот посмотри про однозначность имен методов
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.tmkuktenqqdi
- подумай, какое имя для метода с такой реализацией - будет однозначным
- */
- *************************************************
- private void shouldBeEmpty(){
- /*
- вот это почитай)
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.sswzyp7qkm9o
- и прими окончательное решение по именам методов-проверок
- */
Advertisement
Add Comment
Please, Sign In to add comment