Готов к критике, уже совсем запутался. Буду рад если подскажите куда …#1
Готов к критике, уже совсем запутался. Буду рад если подскажите куда …#1MaxMaceman wants to merge 2 commits intomainfrom
Conversation
| fun showMenu() { | ||
| while (true) { | ||
| println("\nАрхив: ${archive.name}") | ||
| println("0. Создать заметку") |
There was a problem hiding this comment.
Может быть имеет смысл во всех местах начинать счёт с "1", а не "0"? Понятно что в программировании "0" является точкой отсчёта в списках и т.п., но программа в первую очередь для людей и привычнее информация воспринимается когда 1 это 1
|
|
||
| val choice = notesApp.readInput("Выберите действие: ") | ||
|
|
||
| when { |
There was a problem hiding this comment.
Этот весь блок с when в целом можно сделать чуть компактнее и понятнее, если мы choice добавим в условие when, например
when (choice) {
null -> println("Ошибка: Вы должны ввести число.\n")
0 -> notesApp.createArchive()
in 1..notesApp.archives.size -> {
val selectedArchive = notesApp.archives[choice - 1]
ArchiveMenu(notesApp, selectedArchive).showMenu()
}
notesApp.archives.size + 1 -> notesApp.exit()
else -> println("Ошибка: Выбраный пункт отсутсвует.\n")
}
Из этого кода будет понятно, что мы здесь работаем только с choice и такой код будет труднее сломать.
| printMenu() | ||
| val choice = notesApp.readInput("Выберите действие: ") | ||
|
|
||
| when { |
There was a problem hiding this comment.
Тут с when аналогично тому, что написал выше
| println(note.content) | ||
| println("1. Вернуться") | ||
| val choice = readInput("Введите 1, чтобы вернуться: ") | ||
| if (choice == 1) return |
There was a problem hiding this comment.
Такую запись лучше сделать либо в 1 строку
if (choice == 1) return else println("Ошибка: Выбраный пункт отсутствует.\n")
либо полная запись с block body
if (choice == 1) {
return
} else {
println("Ошибка: Выбраный пункт отсутствует.\n")
}
Общее правило такое, что если мы не можем записать в 1 строку, то используем полную запись
| @@ -0,0 +1,129 @@ | |||
| import java.util.Scanner | |||
There was a problem hiding this comment.
Про этот класс уже написал в критических замечаниях на портале, его надо удалить
Готов к критике, уже совсем запутался. Буду рад если подскажите куда смотреть.