julia_v_iluhina

Untitled

Dec 15th, 2016
97
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.50 KB | None | 0 0
  1. public class TodoMvcTest {
  2.  
  3.     @Test
  4.     public void testFunctionalityTODOList() {
  5.     /*
  6.         все же мне кажется вариант с применением термина Tasks - корректнее
  7.         во-первых - ниже используем термин tasks
  8.        
  9.         а без надобности - лучше новых терминов не добавлять
  10.         особенно - если речь идет про какое-то одно понятие
  11.         (лучше придерживаться принципа - одно понятие = один термин, так код будет однозначнее)
  12.  
  13.         testTasksFunctionality
  14.         testTasksLifeCycle
  15.         testTasksCommonFlow
  16.         testTasksFlow
  17.  
  18.         вот эти варианты - из таких соображений - будут лучше
  19.         следи за порядком слов (см в faq про это
  20.         https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.2pvr3ijzfuho )
  21.         и придерживайся conventions - CamelCase - https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
  22.     */
  23. *******************************************
  24.         delete ("2");
  25.         assertTasksAre("1", "3", "4");
  26.  
  27.  
  28.         /*
  29.             пропуска одной строки - достаточно для форматирования кода
  30.             https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
  31.             Multiple consecutive blank lines are permitted, but never required (or encouraged).
  32.         */
  33. *************************************************************
  34.         tasks.shouldBe(empty);
  35. /*
  36.     реализуй и для этой проверки - свой метод
  37.     не забудь учесть
  38.     https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.tezjc2whgbl7
  39. */
  40. ***********************************************
  41.     Condition completed = cssClass("completed");
  42. /*
  43.     Ты уверен - что это поле нужно?
  44.     не оставляй в коде не нужного
  45.     если есть сомнения в том, что это не нужно - пересмотри второе видео, примерно с 58ой минуты, минут 5    
  46. */
  47.  
  48. ***************************************************
  49.     private void add (String... taskText){
  50.     private void assertTasksAre(String... taskText) {
  51. /*
  52.     http://joxi.ru/krDOZldFE0dXkA
  53.  
  54.         1 - точек с запятой после реализации метода - не нужно
  55.         это учти во всем коде -
  56.  
  57.         2 - пропуски строк - применяй с умом и единообразно - если они ничего не дают для понимания и наглядности -
  58.         значит в данном случае они и не нужны
  59.  
  60.     уточни в именах параметров - ты же ждешь не текст таскИ, а текстЫ тасОК
  61.     если бы мы в этих методах оперировали только одной таской - да, имя taskText - было бы ок
  62.  
  63.     а так - нам нужно множественное число
  64.     раз мы можем оперировать несколькими текстами
  65. */
  66. *****************************************************
Advertisement
Add Comment
Please, Sign In to add comment