julia_v_iluhina

Untitled

Sep 13th, 2016
82
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.49 KB | None | 0 0
  1.  private void complete(String taskText) {
  2.    tasks.filterBy(cssClass("active")).findBy(exactText(taskText)).$(".toggle").click();
  3.  }
  4.  
  5.  private void activate(String taskText){
  6.     tasks.filterBy(cssClass("completed")).findBy(exactText(taskText)).$(".toggle").click();
  7.  }
  8.  
  9. /*
  10.     посмотри - код этих методов очень похож
  11.     и уже - не сказать, что он очень прост
  12.  
  13.     потому - чтобы код был более DRY лучше резализовать метод
  14.     toggle(String taskClass, String taskText)
  15.  
  16.     и уже - переиспользовать этот метод внутри complete и activate
  17.  
  18.     впоследствии - когда мы вынесем из тест-класса вспомогательные методы в другой класс
  19.     такой метод - toggle(String taskClass, String taskText)
  20.     можно будет объявить как private
  21.     чтобы он не был видим со стороны тест-класса
  22.     такой - внутренний вспомогательный метод, чтоб код более DRY сделать
  23. */
  24. ************************************************
  25.  private void completeAll() {
  26.     $("#toggle-all").click();
  27.  }
  28. /*
  29.     А вот тут уже вряд ли такой номер пройдет
  30.  
  31.     вызови такой метод  - для всех закомпличеных тасок
  32.     и выполнится не complete all
  33.     а - activate all
  34.  
  35.     следовательно - имя метода не отражает в точности то, что он делает
  36.  
  37.     при такой реализации метода - это вариант переключить, а не закомптитить/активировать
  38.  
  39.     так что - в данном случае - имя метода = toggleAll
  40. */
  41. ******************************************************
  42.  private SelenideElement setNewTaskText(String oldTaskText, String newTaskText){
  43. /*
  44.     и твое имя метода - точное и использует уже применяемые термины
  45.     значит - хорошее)
  46.    
  47.     можно чуть лаконичнее - startEdit
  48.     в данном случае - меньше подробностей, зато есть отсыл - какую операцию начали делать
  49.        
  50.     на изменениях не настаиваю    
  51. */
Advertisement
Add Comment
Please, Sign In to add comment