Validacja - gdzie to robić?

Validacja - gdzie to robić?
WA
  • Rejestracja:ponad 9 lat
  • Ostatnio:ponad 5 lat
  • Postów:56
0

Posiadam odrębną klasę implementującą Validator. Chcąc np. dodać nowego użytkownika proces rozpoczyna się od kontrolera, który wywołuje serwis, który z kolei po pewnych operacjach wywołuje repozytorium i użytkownik się dodaje. Chciałbym teraz dodać walidację sprawdzającą, czy wszytkie pola użytkownik wprowadził, walidacja hasła, emailu itd. i uczyniłem to w klasie implementującej Validator. I teraz nasuwa mi się pytanie. W którym miejscu powinienem wykorzystać tą klasę walidującą?
W kontrolerze -> a gdy przejdzie poprawnie dokonywać tylko koniecznych operacji w servicie i wywołać repo.
Czy może w serwisie? -> w kontolerze tylko wywołać service i w nim wszystko validować?
Jaki sposób jest najbardziej preferowany?

OV
  • Rejestracja:ponad 8 lat
  • Ostatnio:2 miesiące
1

W miejscu, gdzie masz logikę biznesową. Ja bym proponował w serwisie.

IH
  • Rejestracja:ponad 6 lat
  • Ostatnio:4 minuty
  • Postów:103
1

W springu możesz to np tak zrobić

Kopiuj
public class ProfileInfoDTO {

	@NotBlank
	@Size(min = 4, max = 25)
	public final String displayName;

	@Email
	@NotBlank
	@Size(min = 4, max = 40)
	public final String communicationEmail;

	@NotBlank
	public final String city;

	@NotBlank
	public final String postCode;
(...)
}

I w kontrolerze @Valid

Kopiuj
	@PostMapping("/profile")
	public void saveSettings(@Valid @RequestBody ProfileInfoDTO profileInfoDTO) {

to tylko próbka.

PS
Walidacje też powinieneś przeprowadzić po stronie frontu(chyba)

edytowany 2x, ostatnio: IHaveHandedInMyResignation
TY
Jeśli dobrze pamiętam, to samo @Valid nie wystarczy. Trzeba jeszcze dodać parametr BindingResult, sprawdzić, czy ma errory i jeśli tak to wykonać swoją logikę odpowiadającą za error handling. Czyli najczęściej rzucić wyjątkiem. Tu jest przykład: https://spring.io/guides/gs/validating-form-input/
Schadoow
@Tyvrel Nie trzeba dodawać parametru BindingResult. Możesz użyć np ExceptionHandlerów. Albo czegokolwiek innego w zależności od potrzeb.
TY
Możesz podać przykład? ExceptionHandler ExceptionHandlerem, ale coś też wyjątek musi rzucić
WA
  • Rejestracja:ponad 9 lat
  • Ostatnio:ponad 5 lat
  • Postów:56
0

Tak, @Valid to dobra opcja :) Tzn. u mnie akurat po głębszej analizie i specyficznych wymogach, w których nie chcę, by na raz były validowane wszystkie pole użyłem @Validate z odpowiednim interfejsem grupującym.
Oczywiście pamiętam, by na froncie też przeprowadzić walidację, ale na front przyjdzie jeszcze czas.

Aleksander Brzozowski
  • Rejestracja:ponad 8 lat
  • Ostatnio:prawie 5 lat
  • Postów:26
0

Tak przy okazji, da się zrobić te walidacje jakoś globalnie, żeby pola, które mają np. typ String, były zawsze wymagane, a np. pola Optional, ew. Option były opcjonalne?
Pisanie dla każdego z wymaganych pól '@NonNull' wydaje mi się kiepskim rozwiązaniem.

Schadoow
  • Rejestracja:ponad 13 lat
  • Ostatnio:36 minut
  • Postów:1067
1

@Aleksander Brzozowski: Własny walidator na klasę ? :). Lecisz refleksją po wszystkich polach i sprawdzasz. Aczkolwiek IMO to mega kiepski pomysł. Robienie generycznej logiki biznesowej to rak.

hcubyc
  • Rejestracja:ponad 12 lat
  • Ostatnio:prawie 3 lata
0

Dlaczego nie pisać własnych klas walidujących? Generalnie jest to trochę więcej roboty niż dodanie kilku adnotacji, ale wciąz nie jest to wielki narzut. Plusy to jeżeli ktoś się jeszcze nie zespawał ze springiem to i tu nie będzie spawania no i będzie mozna testować jednostkowo bez stawiania kontekstu springa


Limitations are limitless > ##### Ola Nordmann napisał(a)
> Moim językiem ojczystym jest C++ i proszę uszanować to, że piszę po polsku.
Schadoow
  • Rejestracja:ponad 13 lat
  • Ostatnio:36 minut
  • Postów:1067
1

@hcubyc:

Po to został stworzony JSR 303 aby nie trzeba było tego klepać ręcznie i nie ma to nic wspólnego ze springiem

edytowany 1x, ostatnio: Schadoow
hcubyc
OK, jeżeli stosujesz to tylko w warstwie blisko kontrolerów/infry i nigdy tego nie będziesz testował jednostkowo, to owszem ma sens.
Schadoow
@hcubyc Nie ma żadnego problemu z testowaniem tego jednostkowo.
Aleksander Brzozowski
  • Rejestracja:ponad 8 lat
  • Ostatnio:prawie 5 lat
  • Postów:26
0

Nie zrozumieliśmy się. Chodzi mi o to, że jak moje RequestBody wygląda tak:

Kopiuj
class CreateUser {
  String firstName;
  String lastName;
  Option<Integer> age;
}

To chciałbym, żeby spring mi zwalidował to bez żadnych adnotacji @NotNull czy tam @NonNull. Tutaj jest jasno pokazane co jest wymagane, a co nie. Może jest jakiś moduł do jacksona, który będzie to potrafił?

Schadoow
Pusty string jest okej ?

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.