Conversation
| val archives = mutableListOf<Archive>() | ||
| var isRunnig = true | ||
| while (isRunnig) { | ||
| val mainMenuOptions = listOf("Создать архив", "Выбрать архив", "Выход") |
There was a problem hiding this comment.
Это лучше вынести за цикл while, т.к. сейчас на каждую итерацию цикла будет создан новый список, а если вынести на строчку выше, то список будет создан один на всю программу, таким образом будет меньше операций с памятью
| if (selectedArchive != null) { | ||
| var isRunningNoteMenu = true | ||
| while (isRunningNoteMenu) { | ||
| val noteMenuOption = listOf("Создать заметку", "Выбрать заметку", "Выход") |
There was a problem hiding this comment.
Здесь аналогично создание списка можно вынести на строчку выше, чтобы было меньше операций с памятью
| import modules.Archive | ||
| import utils.readNonBlankString | ||
|
|
||
| fun createArchive(archives: MutableList<Archive>) { |
There was a problem hiding this comment.
Лучше функции делать чистыми(https://ru.wikipedia.org/wiki/%D0%A7%D0%B8%D1%81%D1%82%D0%BE%D1%82%D0%B0_%D1%84%D1%83%D0%BD%D0%BA%D1%86%D0%B8%D0%B8), чтобы их было проще переиспользовать и покрывать тестами. Чтобы данную функцию сделать чистой, можно убрать аргумент функции и сделать ей возвращаемое значение - объект класса Archive, чтобы метод добавления созданного архива в список была на вызывающей стороне
There was a problem hiding this comment.
В этом файле очень хорошие утилитные функции, молодец. Файл правда суперский, но нейминг ему лучше подбирать более говорящий, например InputUtils
2 participants
Студент 23 когорты
Данилов Аркадий