All issues

Search results

такие вещи минимум надо выносить в константы, а максимум (предпочтительнее) в конфигурационные файлы

Status: Open.

#12 In belovlaska/itmo;

· Vasliska1 opened on May 25, 2023

а что у вас с форматированием везде? ctrl alt L в IDEA используете?

Status: Open.

#10 In belovlaska/itmo;

· Vasliska1 opened on May 25, 2023

вот эту строчку лучше соединить с предыдущей и тернарный оператор добавить

Status: Open.

#9 In belovlaska/itmo;

· Vasliska1 opened on May 25, 2023

Желательно такие вещи выносить в конфигурационный файл ( можете не исправлять, но имейте в виду, и на 7 лабе лучше сделайте)

Status: Open.

#8 In belovlaska/itmo;

· Vasliska1 opened on May 25, 2023

На будущее, по конвенции есть такой стандарт, для лучшей читаемости -- всегда используют скобки, даже если в блоке 1 строчка и их можно опустить

Status: Open.

#6 In belovlaska/itmo;

· Vasliska1 opened on Apr 10, 2023

я думаю эту строчку можно в файнал блок закинуть, чтобы ее не дублировать 3 раза

Status: Open.

#5 In belovlaska/itmo;

· Vasliska1 opened on Apr 10, 2023

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

Status: Open.

#4 In belovlaska/itmo;

· Vasliska1 opened on Apr 10, 2023

пустых кетч блоков не должно быть. И сделайте форматирование ctr+alt+L если IDEA пользуетесь

Status: Open.

#3 In belovlaska/itmo;

· Vasliska1 opened on Apr 10, 2023

Скорее совет, чем придирка - лучше такие вещи назвать не контроллерами, а handler либо manager. Особенно, когда будете пилить проекты связанные с вебом. Там контроллеры отвечают немного за другие действия.

Status: Open.

#2 In belovlaska/itmo;

· Vasliska1 opened on Apr 10, 2023

Скорее совет, чем придирка - лучше такие вещи назвать не контроллерами, а handler либо manager

Status: Open.

#1 In belovlaska/itmo;

· Vasliska1 opened on Apr 10, 2023