julia_v_iluhina

Untitled

Sep 21st, 2016
91
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 1.78 KB | None | 0 0
  1. public void create(String... taskNames) {
  2. public void toggle(String taskName) {
  3. public void delete(String taskName) {
  4. /*
  5.     в этих методах - ты оперируешь термином имя таски
  6.     ок, годный вариант
  7. */
  8. ***************
  9.     public void assertTasksAre(String... tasksForCheck){
  10. /*
  11.     а тут - почему не taskNames ?
  12.    
  13.     для одного понятия - используй один термин
  14.     тогда твой код будет понятен и однозначен
  15.    
  16.     в имени метода - все сказано - и что делаем = проверяем, и что проверяем = таски
  17.     все)
  18.     параметр - смело можно называть уже ранее употребляемым термином taskNames
  19. */
  20. *******************
  21.     public void assertEmpty(){
  22. /*
  23.      а что должно быть пустым?
  24.      из имени метода не понятно )
  25.      
  26.      assertNoTasks
  27.      assertTaskListIsEmpty
  28.      assertTasksEmpty
  29.      может  - и еще есть годные варианты)
  30.      
  31.      если в методах - действиях - мы слово Tasks - намеренно опускали из имени метода
  32.      т к ясно - над чем действие производится
  33.      
  34.      то в именах методов - проверок - надо быть максимально точным
  35.      даже тут, в этом не сложном приложении - вариантов проверок столько, что
  36.      если мы начнем что-то не писать в именах методов - запутаться будет очень просто
  37. */
Advertisement
Add Comment
Please, Sign In to add comment