Pull to refresh
8
0
Руслан Тагиев @RuslanTaghiyev

Backend Engineer/Architect

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

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

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

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

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

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

2

Information

Rating
Does not participate
Registered
Activity

Specialization

Backend Developer, Software Architect