julia_v_iluhina

Untitled

Aug 24th, 2016
71
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 4.33 KB | None | 0 0
  1.     public SelenideElement startEdit(String oldTask, String newTask) {
  2. /*
  3.     http://pastebin.com/kEaDwPg0 строки 158-166
  4. */
  5. **********************************
  6.     public static void filterAll() {
  7.         $$("#filters>li").find(exactText("All")).click();
  8.     }
  9. /*
  10.     что реализовал 3 метода без параметров - это ОК
  11.     мысль уловил
  12.  
  13.     сам код $$("#filters>li").find(exactText("All")) - можно упростить
  14.     до $(By.linkText(...))
  15.     и наглядно, и лаконично
  16.     мы ничего не потеряем, обращаясь к этим линкам напрямую, минуя $$("#filters>li")
  17.     если бы нам было нужно еще что-то от $$("#filters>li") - тогда было бы правильно
  18.     использовать переменную для $$("#filters>li")
  19.     но в данном  случае - это излишне
  20.  
  21.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.pk1xvngaj4rk
  22.  
  23.     касается всех 3-х методов - filter...()
  24. */
  25. ***********************************
  26. /*
  27.     еще - в рамках е2е сценария для smoke покрытия стоило покрыть единожды -
  28.     проверку счетчика активных тасок (items left counter)
  29.    
  30.     да, штука это не высокоприоритетная (если только это не будет работать - приложение все еще функционально),
  31.    
  32.     но - реагирует на все действия над тасками
  33.     (т е - если бы стояла задача проверить логику полностью -
  34.     то надо было бы это проверять после каждой операции, как и тексты тасок в списке)
  35.    
  36.     перегружать этим несущественным код е2е-а - не стоит
  37.     но - единожды проверить в рамках е2е сценария - хорошая идея
  38. */
  39. ************************************
  40. /*
  41.     оформление кода
  42.  
  43.     https://www.jetbrains.com/help/idea/2016.2/reformatting-source-code.html
  44.     используй реформатирование кода
  45.     это приведет все отступы - к наиболее стандартному виду
  46.     код для большинства - будет нагляднее
  47.  
  48.     https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
  49.     используй пропуски строк как средство оформления кода
  50.     пропускать несколько строк полряд - смысла не имеет
  51.  
  52.     делить пропусками строк - код на блоки, разделять методы друг от друга -
  53.     очень разумно
  54.     код легче воспринимается
  55.  
  56.     ну и напоследок)
  57.     технически - порядок методов может быть любым
  58.     но
  59.     если методы будут в неком логическом порядке - их легче будет понять/запомнить
  60.     и вообще - ориентироваться и анализировать
  61.     правила - простые
  62.         идем от общего к частному, от базового к детальному
  63.         рядом держим - похожее или противоположное
  64.  
  65.     сначала - тест-метод
  66.     потом - переменные используемые
  67.     потом - базовые методы (CRUD)
  68.     рядом - toggle & toggleAll
  69.     рядом - startEdit & edit
  70.     рядом - ассерты
  71.  
  72.     читать и понимать так организованный код - проще
  73.     что нам и нужно
  74.  
  75.     еще приятный бонус - так проще увидеть все нелогичности в именах методов и параметров
  76.     все duplications - такого рода шероховатости
  77.  
  78.     старайся по финишу - это всегда делать
  79. */
Advertisement
Add Comment
Please, Sign In to add comment