Обновить
8
0
Руслан Тагиев@RuslanTaghiyev

Backend Engineer/Architect

Отправить сообщение
Спасибо, исправил
Добавил в описание, спасибо)
Исправил. Очень конструктивно, спасибо)
Спасибо. Исправил первые два замечания. NPE не будет, посмотрите на метод в репозитории

Многие пишут, что StringBuffer тут не нужен, хотя @Scheduled использует многопоточность. Я не являюсь экспертом в области потоков, поэтому может ли кто-то из высказавшихся привести аргументы, и я исправлю его StringBuilder?

Вообще то нужен, это одна из спецификаций

Нам не нужен @EqualsAndHashcode и @AllArgsConstructor, которые есть в Data. Но нужен конструктор без аргументов.

  1. Ну, шедулер завязан на многопоточности
  2. Последнее предложение перед заключением. Поменял на то, чтобы раз в день была отправка.

Последнее предложение перед заключением. Поменял на то, чтобы раз в день была отправка. Проект не нужно перезапускать, т к по идее нужно было изначально задавать это значение. 10 секунд я сделал для примера, чтобы видно было логи

2

Информация

В рейтинге
Не участвует
Зарегистрирован
Активность

Специализация

Бэкенд разработчик, Архитектор программного обеспечения