java

У джунов есть какая-то любовь к примитивам, и они пытаются использовать их везде, где возможно. Но не всегда это хорошая идея.

@Entity
public class Like {
    
     @Id
     private long id;

}

В данном случае, когда к нам приходит новый объект, мы не сможем точно сказать, сохранен ли он в БД. Может к нам попал объек с идентификатором 0.

В случае идентификатора лучше использовать обертку, то есть Long.

Часто у новичков вижу подобное форматирование:

public class NewReviewDto {
     @NotNull
     private Long event;
     @Size(max = 7000)
     private String review;
     @NotNull
     private Boolean positive;
 }

Эту кашу довольно сложно читать. Если над полями класса присутствуют аннотации, то я делаю несколько переносов строк, вот так:

public class NewReviewDto {

     @NotNull
     private Long event;
     
     @Size(max = 7000)
     private String review;
     
     @NotNull
     private Boolean positive;
     
 }

Если ваша программа может вернуть коллекцию, которая не содержит никаких значений, убедитесь, что возвращается пустая коллекция, а не Null. Это сэкономит вам время на различные проверки и избавит от многих ошибок.

public List<Event> getAllEventByUserId(int userId) {
    if (userId == 0) {
        return Collections.emptyList();
    }
    return eventRepository.findAllByUserId(userId);
}