Резюме зустрічі 02.03.16

Видео

На встрече было проведено code review задачи №53.

• в тесте нужно проверять, что возвращается пустой список; [01:53]
• ReminderAboutEventService - неудачное название, лучше - EventReminder; [03:30]
• метод execute() - неудачное название, лучше - remind(); [06:20]
• поле MailService sendGridMailService лучше назвать mailService; [07:25]
• квалификаторы в аннотациях (в данном случае - @Service("reminderAboutEventService")) лучше не использовать, так как нужно следить, что-бы название квалификатора соответствовало имени класса; [08:30]
• нужно изменить значение в аннотации @Value над полем private int daysTillEvent; [15:50]
• LogManager.getLogger() - лучше использовать с классом в качестве параметра; [20:35]
• новое правило в требованиях к коду: запрещено создание статических методов; [25:00]
  • OOP Alternative to Utility Classes;
• у MailSenderScheduler нет интерфейса; [30:50]
  • Евгений Борисов — Spring-потрошитель (часть 1, часть 2);
• желательно проверить scheduler в реальной работе; [33:50]
• добавить логирование в MailSenderScheduler; [36:20]
• плохо, что MailSenderScheduler объявлен в двух контекстах; [37:00]
• нужно научиться работать с коммитами (переименование, откат); [42:05]
• MailReminderService - нужно обработать исключение BuildMailException из метода MailReminderService.buildMail() [43:50]
  • Top 20 Java Exception Handling Best Practices;
  • можно, например, вместе с записью ошибки в лог отсылать письмо администратору;