Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public class TodoMvcTest {
- @Test
- public void testFunctionalityTODOList() {
- /*
- все же мне кажется вариант с применением термина Tasks - корректнее
- во-первых - ниже используем термин tasks
- а без надобности - лучше новых терминов не добавлять
- особенно - если речь идет про какое-то одно понятие
- (лучше придерживаться принципа - одно понятие = один термин, так код будет однозначнее)
- testTasksFunctionality
- testTasksLifeCycle
- testTasksCommonFlow
- testTasksFlow
- вот эти варианты - из таких соображений - будут лучше
- следи за порядком слов (см в faq про это
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.2pvr3ijzfuho )
- и придерживайся conventions - CamelCase - https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
- */
- *******************************************
- delete ("2");
- assertTasksAre("1", "3", "4");
- /*
- пропуска одной строки - достаточно для форматирования кода
- https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
- Multiple consecutive blank lines are permitted, but never required (or encouraged).
- */
- *************************************************************
- tasks.shouldBe(empty);
- /*
- реализуй и для этой проверки - свой метод
- не забудь учесть
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.tezjc2whgbl7
- */
- ***********************************************
- Condition completed = cssClass("completed");
- /*
- Ты уверен - что это поле нужно?
- не оставляй в коде не нужного
- если есть сомнения в том, что это не нужно - пересмотри второе видео, примерно с 58ой минуты, минут 5
- */
- ***************************************************
- private void add (String... taskText){
- private void assertTasksAre(String... taskText) {
- /*
- http://joxi.ru/krDOZldFE0dXkA
- 1 - точек с запятой после реализации метода - не нужно
- это учти во всем коде -
- 2 - пропуски строк - применяй с умом и единообразно - если они ничего не дают для понимания и наглядности -
- значит в данном случае они и не нужны
- уточни в именах параметров - ты же ждешь не текст таскИ, а текстЫ тасОК
- если бы мы в этих методах оперировали только одной таской - да, имя taskText - было бы ок
- а так - нам нужно множественное число
- раз мы можем оперировать несколькими текстами
- */
- *****************************************************
Advertisement
Add Comment
Please, Sign In to add comment