Выполненно задание "Пишем консольное приложение №2" - #1

По коду заметил, что ссылки на эти объекты не изменяются, поэтому можно заменить var на val

Перед преобразованием строки в число, стоит удалить лишние пробелы, используя функцию trim:

val inputString = Scanner(System.in).nextLine().trim()

Таким образом твоя программа будет корректно обрабатывать следующий ввод:
" 2"
"2 "
" 2 "

Также можно воспользоваться функцией-расширением toIntOrNull(), которая вернет null в случае невозможности преобразования строки в число, вместо выброса исключения.

Здесь return уже не нужен, так как после выхода из блока else метод selectMenu завершит свою работу.

Логику проверки стоит вынести в отдельный метод или класс, так как она дублируется. Также стоит проверить строку на то, что она не состоит из одних пробелов. Для этого ты можешь воспользоваться функцией-расширением isBlank():

if (name.isBlank()) {
   ...
   return
}

С применением некоторых приятных особенностей Kotlin можно переписать данный метод так:

1. Вынести return за when, либо же написать вообще одним выражением
2. Вместо проверки на null можно воспользоваться оператором элвис

private fun getSize() = when (screenType) {
   ScreenType.ARCHIVES -> archives.size
   ScreenType.NOTES -> notes[currentArchive]?.size ?: 0
}

Можно воспользоваться функцией-расширением ifBlank в данном случае следующим образом:

    val str = Scanner(System.`in`).nextLine()
    return str.ifBlank {
        println("Ошибка: Введенна пустая строка\n")
        null
    }

Если строка пустая или состоит только из пробелов, то ifBlank возвращает результат, переданный в качестве параметра. Если же строка содержит какие-либо другие символы, то возвращается сама строка.

Предлагаю использовать более осмысленные названия методов для улучшения читаемости кода:

addData -> addArchive или addNoteByArchive: это позволит сразу понять, что происходит при добавлении данных, а именно, добавление архива или заметки в архив.
getData -> getAllArchives или getNotesByArchive: такое название явно указывает на то, что возвращается, а именно, все архивы или заметки, относящиеся к конкретному архиву.

С названиями действительно можно поработать, но в данном случае мне было интересно воспользоваться перегрузкой функций, поэтому предложенные названия мне бы не подошли

Вместо forEach можно использовать функцию map следующим образом:

fun getData(): ArrayList<Archive> {
   return notes.map { entry -> entry.key }.let{ archives -> ArrayList(archives) }
}

Это позволяет избежать лишней инициализации переменной и делает код более компактным

Вместо явной проверки на null в блоке when можно сразу использовать вариант null, что позволит избежать лишней вложенности в функции.

when (input) {
            null -> Unit
            0 -> create()
            getSize() + 1 -> back()
            in 1..getSize() -> select(input - 1)
            else -> {
                println("Ошибка: Выбранный элемент отсутствует\n")
            }
}

Если нужно выполнить какое-то выражение с переменной, только если она не равна null, то можно использовать функцию области видимости let с проверкой на null:

inputString()?.let { str -> MenuData.addData(str) }