Repository navigation
Pull Request итоговый - #2
Iskanderius wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Всю папку .idea лучше сразу добавлять в gitignore - в ней хранятся локальные настройки разработчика, специфичные для проекта. После добавления важно не забыть удалить папку с репозитория
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
Повторяющиеся пустые строки в коде лучше удалять, чтобы код лучше смотрелся и читался проще. Логически делимые блоки можно разделять одной пустой строкой
| fun <T> toInput(obj: T, MaxChoose: Int): Int | ||
| } | ||
|
|
||
| interface select{ |
There was a problem hiding this comment.
Интерфейсы и классы в Котлин принято называть с большой буквы
| @@ -0,0 +1,165 @@ | |||
| import java.util.Scanner | |||
| open class Note() { | |||
There was a problem hiding this comment.
Не стоит все классы делать open - для описания моделей данных (архивов и заметок) лучше воспользоваться data class
| } | ||
|
|
||
| override fun <T> toInput(obj: T, MaxChoose: Int): Int { | ||
| while (true) { |
There was a problem hiding this comment.
Не рекомендую писать бесконечные циклы через while (true) - лучше всегда явно прописывать условие выхода из цикла, чтобы уменьшить вероятность ошибиться и повысить читабельность кода
| if (obj is DataBaze) { | ||
| println("Введите название архива:") | ||
| val nameInput = scanner.nextLine() | ||
| if (nameInput.isNotBlank()) { |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
2 participants
Итоговый Pull Request