Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public void create(String... taskNames) {
- public void toggle(String taskName) {
- public void delete(String taskName) {
- /*
- в этих методах - ты оперируешь термином имя таски
- ок, годный вариант
- */
- ***************
- public void assertTasksAre(String... tasksForCheck){
- /*
- а тут - почему не taskNames ?
- для одного понятия - используй один термин
- тогда твой код будет понятен и однозначен
- в имени метода - все сказано - и что делаем = проверяем, и что проверяем = таски
- все)
- параметр - смело можно называть уже ранее употребляемым термином taskNames
- */
- *******************
- public void assertEmpty(){
- /*
- а что должно быть пустым?
- из имени метода не понятно )
- assertNoTasks
- assertTaskListIsEmpty
- assertTasksEmpty
- может - и еще есть годные варианты)
- если в методах - действиях - мы слово Tasks - намеренно опускали из имени метода
- т к ясно - над чем действие производится
- то в именах методов - проверок - надо быть максимально точным
- даже тут, в этом не сложном приложении - вариантов проверок столько, что
- если мы начнем что-то не писать в именах методов - запутаться будет очень просто
- */
Advertisement
Add Comment
Please, Sign In to add comment