stawros/konta-walutowe
1
0
Fork 0
Code Issues Pull requests Releases Packages Activity

fix: SonarLint fixes

Browse source
This commit is contained in:
Piotr Dec 2024-05-25 00:47:12 +02:00
parent d0e21e9eff
commit ec0ee860ee
Signed by: stawros
GPG key ID: F89F27AD8F881A91
3 changed files with 13 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/main/java/eu/ztsh/wymiana/exception/UserNotFoundException.java
View file

@ -5,7 +5,11 @@ import eu.ztsh.wymiana.web.model.CurrencyExchangeRequest;
public class UserNotFoundException extends RuntimeException {
public UserNotFoundException(CurrencyExchangeRequest entity) {
super("User with PESEL %s not found".formatted(entity.pesel()));
this(entity.pesel());
}
public UserNotFoundException(String pesel) {
super("User with PESEL %s not found".formatted(pesel));
}
}

3
src/main/java/eu/ztsh/wymiana/web/controller/ExchangeController.java
View file

@ -1,14 +1,11 @@
package eu.ztsh.wymiana.web.controller;
import eu.ztsh.wymiana.exception.InsufficientFundsException;
import eu.ztsh.wymiana.exception.UserAlreadyExistsException;
import eu.ztsh.wymiana.exception.UserNotFoundException;
import eu.ztsh.wymiana.service.CurrencyService;
import eu.ztsh.wymiana.validation.ValidationFailedException;
import eu.ztsh.wymiana.web.model.CurrencyExchangeRequest;
import jakarta.validation.Valid;
import lombok.RequiredArgsConstructor;
import org.springframework.http.HttpEntity;
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity;
import org.springframework.validation.annotation.Validated;

11
src/main/java/eu/ztsh/wymiana/web/controller/UserController.java
View file

@ -1,6 +1,7 @@
package eu.ztsh.wymiana.web.controller;
import eu.ztsh.wymiana.exception.UserAlreadyExistsException;
import eu.ztsh.wymiana.exception.UserNotFoundException;
import eu.ztsh.wymiana.model.User;
import eu.ztsh.wymiana.service.UserService;
import eu.ztsh.wymiana.validation.ValidationFailedException;
@ -31,9 +32,13 @@ public class UserController {
try {
return userService.get(pesel)
.map(ResponseEntity::ok)
.orElse(ResponseEntity.notFound().build());
} catch (ValidationException e) {
return ResponseEntity.badRequest().build();
.orElseThrow(() -> new UserNotFoundException(pesel));
} catch (Exception e) {
return ResponseEntity.status(switch (e) {
case ValidationException ignore -> 400;
case UserNotFoundException ignore -> 404;
default -> 500;
}).build();
}
}