julia_v_iluhina

Untitled

Sep 29th, 2016
74
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.58 KB | None | 0 0
  1. /*
  2.     практически идеально
  3.     уточнить селектор + добавить одну проверку
  4.    
  5.     и моя придирка)
  6.    
  7. */
  8. *****************
  9.  
  10. $("strong")
  11.  
  12. /*
  13.     лучше уточни селектор
  14.     идентифицировать элемент только по типу элемента  - слишком рискованно
  15.     добавится еще элемент такого же типа - и все... хатка поломается)
  16.  
  17.     http://joxi.ru/DrlQ5oLh4aOVnm
  18.     у родительского элемента есть id
  19.     это будет хорошим надежным уточнением
  20. */
  21. ********************************************
  22.         ...
  23.         page.filterCompleted();
  24.         page.assertVisibleTasks("a edited", "b");
  25.  
  26.         page.toggle("b");
  27.         /*
  28.             вот тут после переоткрытия таски - нужна проверка
  29.  
  30.             следующее действие не проверяет переоткрытие таски
  31.         */
  32.         page.clearCompleted();
  33.         ...
  34. ************************************
  35. http://joxi.ru/1A5zNxjuKeyGJr
  36. /*
  37.     это придирка)
  38.     на самом деле - лучше послушайся
  39.     хорошо оформленный код - сильно облегчает работу в команде)
  40.     а если он еще и просто хороший - вообще цены тебе не будет)
  41.     тем более - что правила оформлени не сложные и многое можно подправлять очень быстро
  42.  
  43.     см рисунок,
  44.         1 - я бы в одно-двустрочных методах такие пропуски не делала
  45.    
  46.         а вот между методами - стоит пропускать строку - облегчает чтение кода
  47.         выделила прямоугольниками слева это
  48.  
  49.     а в многострочных методах пропуски строк между блоками кода - тоже облегчают
  50.     ты это очень хорошо используешь как раз)
  51.  
  52.     про пропуски строк  - тоже есть конвеншенсы
  53.     https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
  54.  
  55.     про это надо отдельно самому заботиться
  56.     реформатирование кода тут уже не поможет)
  57. */
Advertisement
Add Comment
Please, Sign In to add comment