julia_v_iluhina

Untitled

Oct 6th, 2016
122
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.54 KB | None | 0 0
  1. public class CommonActions {
  2. /*
  3.     учти мои предыдущие комментарии - по поводу имени этого класса
  4.         из первого ревью
  5.             если ты таки решил оставить класс - давай назовем его правильно)
  6.                 по сути - если ты учтешь мои пожелания выше - у тебя получится пейдж-объект
  7.                 таким объектам дают имена, заканчивающиеся на Page
  8.                 (TodoMVCPage - будет ОК)
  9.                 и размешают в пекедже pages
  10.         из второго ревью
  11.             TodoMVCPage tasks = new TodoMVCPage();
  12.             обрати внимание на ...
  13.             имя класса для вспомогательных методов - писала тебе про это в прошлый раз
  14.  
  15.     переименуй класс)
  16.     как это сделать правильно - см faq
  17.     https://docs.google.com/document/d/10qSwWTQ6pGfVZSwOes-1QSmdflMiGD2U_y53VHq2m20/edit#heading=h.vwuqi54t6fyg
  18.            
  19.     про пейджи - будет на курсе
  20.     правда, не сразу)
  21.     но раз забежал вперед - надо в любом случае делать правильно)        
  22. */
  23.     public void add(String value)
  24.     public void delete(String elementPath)
  25.     public void toggle(String elementPath)
  26.      
  27.     /*
  28.         еще важный момент
  29.         на первый взгляд кажущийся занудством с моей стороны
  30.        
  31.         для одного понятия - будем использовать один термин
  32.        
  33.         сейчас я об имени параметра методов
  34.        
  35.         по сути - все методы оперируют текстом таски
  36.        
  37.         вот пусть имя параметра для каждого из методов будет taskText
  38.        
  39.         во-первых - это точно
  40.         во-вторых - однозначно
  41.        
  42.         когда будет порядок в используемой терминологии - и код легче воспринимать
  43.        
  44.         ранее приведенная линка - пригодится и для оптимального переименования параметра метода
  45.     */
Advertisement
Add Comment
Please, Sign In to add comment