Ревью кода из интервью 1

Поддержать канал: https://www.donationalerts.com/r/andd... GitHub: https://github.com/andrei-punko 00:00 — Начало ревью и постановка задачи. 00:23 — Анализ объявления класса и замечания по именованию. 00:32 — Нарушение инкапсуляции: почему поля должны быть private. 00:43 — Проблема хранения Connection в контроллере и использование Connection Pool. 01:46 — Ошибка в конструкторе: правильное использование this.connection. 02:22 — Логирование: почему System.out.println не подходит для Production. 03:09 — Информативность логов и применение Аспектно-ориентированного программирования (AOP). 04:04 — Именование методов: правило "вопрос-ответ" для boolean (из validate в isValid). 04:55 — Архитектурный совет: вынос валидации в отдельные классы или аннотации. 05:37 — Обработка исключений: создание собственных Runtime исключений вместо общих. 06:04 — Нарушение слоистой архитектуры (Persistence layer) и работа с БД в контроллере. 06:37 — Уязвимость к SQL Injection: замена Statement на PreparedStatement. 08:50 — Разбор метода delete и внимательность к опечаткам в запросах. 09:44 — Проблемы многопоточности: почему synchronized на методе не защитит Connection. 11:09 — Итоги ревью. #codereview