Lab 1-6

[AlekseiMaslakov]

Здравствуйте

Lab1.A: Раз уж и тут решили делать обработку исключений, то делали бы до конца))

В остальном всё хорошо, единственное общее замечание, т.к. пишете в идее, то не игнорируйте её подсказки и замечания:

• модификаторы доступа (во многих местах видимость шире, чем необходимо)
• избыточные операторы (continue в Lab5, ненужный боксинг)
• неочищенные импорты (Analyze -> code cleanup и она сама везде всё подчистит, чтобы самому руками не делать)

[akkalomator]

Первое замечание пофиксил, скоро залью.

По поводу модификаторов доступа - если вы имеете ввиду, что для большинства классов случше бы использовать package-private, а не public, то это делалось с той целью, чтобы в будущем когда-нибудь где-нибудь кем-нибудь эти классы юзались. Разумеется, в пределах лабораторной работы это лишнее, но мне кажется так правильнее. Я еще раз пересмотрю модификаторы, если что-то действительно нужно будет сузить, пофиксим.

Про continue в 5й лабе - просто привычка уже, уж лучше постаить ненужный continue, чем если в будушем будешь писать что-то дальше, забудешь добавить continue, а потом -30мин дебага на такую тупую ошибку. Про боксинг - да действительно, новая идея начала подсвечивать, раньше такого не было) Пофиксил

Импорты почищу

[akkalomator]

Пофиксил, новая версия всего в ветке master. Также немного переделал команды в StringBuilder, так как то, что я там увидел, вызвало фейспалм

[akkalomator]

Все лабы сделаны, жду фидбека

[AlekseiMaslakov]

Здравствуйте
По 7 лабе:

1. я бы добавил очистку полей для ввода, ибо неудобно
2. Посмотрите типы ошибок, которые обрабатываете, например при операции insert в 3 лабе, в случае неверного ввода будет диалоговое окно, но если ничего не ввести, то NumberFormatException уже никто не обработает)
3. если захочется, можете пройтись встроенным анализатором, чтобы почистить код
   

[akkalomator]

1. Что вы имеете ввиду под "очисткой полей"? Когда именно их нужно (и нужно ли) очищать?
2. Пофиксил, спасибо
3. Все ворнинги по джаве - излишни, можно было бы поставить @SupressWarnings, но это замусорит код. Ворнинги по проперти - намеренно сделанные ошибки, чтобы проверить работу парсера в Lab5. Ворнинги по спеллингу - оно ругается на abcde и т.п. (ну и на мою фамилию лол), думаю, их исправлять вообще смысла нет

Жду ответа))

[AlekseiMaslakov]

Сразу после ввода, иначе приходится сначала стирать старое, чтобы написать что-то ещё (например, всё в той же Lab3), либо делать выделение текста (как при дабл клике), чтобы он сам стирался при наборе нового.

[akkalomator]

Ладно, сейчас допилю. Не совсем понял про дабл клик, у меня все работает, а при навигации tab-ом, текст вообще автоматически выделяется

[akkalomator]

Fixed. В первых двух лабах теперь есть автоочистка полей после нажатия на кнопки. В остальных решил не делать, так как считаю, что это лишнее.

[akkalomator]

@AlekseiMaslakov