search

sikor272 / GoBooks

1 stars 0 forks source link

Użycie prymitywów zamiast klas opakowujących w repozytorium #6

Closed DuDiiC closed 3 years ago

DuDiiC commented 3 years ago

W metodach zwracających wartości logiczne z repozytorium użytkowników, można zamienić użycie Boolean na typ prymitywny boolean. W przypadku tych metod nie ma ryzyka zwrócenia nulla, zatem Boolean jest nadmiarowy.

https://github.com/sikor272/GoBooks/blob/5852fa2c34a23bb0798d3d28a5d817b6fc090ff6/src/main/java/pl/umk/mat/gobooks/users/UserRepository.java#L12-L14

Pozwoli to na uproszczenie warunków wewnątrz instrukcji if:

https://github.com/sikor272/GoBooks/blob/5852fa2c34a23bb0798d3d28a5d817b6fc090ff6/src/main/java/pl/umk/mat/gobooks/auth/AuthService.java#L50

https://github.com/sikor272/GoBooks/blob/5852fa2c34a23bb0798d3d28a5d817b6fc090ff6/src/main/java/pl/umk/mat/gobooks/auth/AuthService.java#L53

sikor272 commented 3 years ago

Naprawione