Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public class CommonActions {
- /*
- учти мои предыдущие комментарии - по поводу имени этого класса
- из первого ревью
- если ты таки решил оставить класс - давай назовем его правильно)
- по сути - если ты учтешь мои пожелания выше - у тебя получится пейдж-объект
- таким объектам дают имена, заканчивающиеся на Page
- (TodoMVCPage - будет ОК)
- и размешают в пекедже pages
- из второго ревью
- TodoMVCPage tasks = new TodoMVCPage();
- обрати внимание на ...
- имя класса для вспомогательных методов - писала тебе про это в прошлый раз
- переименуй класс)
- как это сделать правильно - см faq
- https://docs.google.com/document/d/10qSwWTQ6pGfVZSwOes-1QSmdflMiGD2U_y53VHq2m20/edit#heading=h.vwuqi54t6fyg
- про пейджи - будет на курсе
- правда, не сразу)
- но раз забежал вперед - надо в любом случае делать правильно)
- */
- public void add(String value)
- public void delete(String elementPath)
- public void toggle(String elementPath)
- /*
- еще важный момент
- на первый взгляд кажущийся занудством с моей стороны
- для одного понятия - будем использовать один термин
- сейчас я об имени параметра методов
- по сути - все методы оперируют текстом таски
- вот пусть имя параметра для каждого из методов будет taskText
- во-первых - это точно
- во-вторых - однозначно
- когда будет порядок в используемой терминологии - и код легче воспринимать
- ранее приведенная линка - пригодится и для оптимального переименования параметра метода
- */
Advertisement
Add Comment
Please, Sign In to add comment