Validacja - gdzie to robić?

Odpowiedz Nowy wątek
2018-09-13 20:17
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?

Pozostało 580 znaków

2018-09-13 20:48
1

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

Pozostało 580 znaków

2018-09-14 00:01

W springu możesz to np tak zrobić

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

    @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, 2018-09-14 00:02
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/ - Tyvrel 2018-09-14 16:12
@Tyvrel Nie trzeba dodawać parametru BindingResult. Możesz użyć np ExceptionHandlerów. Albo czegokolwiek innego w zależności od potrzeb. - Schadoow 2018-09-14 17:09
Możesz podać przykład? ExceptionHandler ExceptionHandlerem, ale coś też wyjątek musi rzucić - Tyvrel 2018-09-17 09:18

Pozostało 580 znaków

2018-09-14 08:00
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.

Pozostało 580 znaków

2018-09-14 18:29
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.

Pozostało 580 znaków

2018-09-17 10:54
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.

Pozostało 580 znaków

2018-09-17 18:42
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.

Pozostało 580 znaków

2018-09-17 18:51
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, 2018-09-17 18:52
OK, jeżeli stosujesz to tylko w warstwie blisko kontrolerów/infry i nigdy tego nie będziesz testował jednostkowo, to owszem ma sens. - hcubyc 2018-09-17 23:52
@hcubyc Nie ma żadnego problemu z testowaniem tego jednostkowo. - Schadoow 2018-09-18 10:50

Pozostało 580 znaków

2018-09-17 19:19
0

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

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ł?

Pusty string jest okej ? - Schadoow 2018-09-18 00:36

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0