JSR-303 @Valid annotation not working for list of child objects

95

Minha aula principal é

public class UserAddressesForm {

    @NotEmpty
    private String firstName;

    @NotEmpty
    private String lastName;

    private List<AddressForm> addresses;

...
setters and getters 

public class AddressForm {

    @NotEmpty
    private String customName;
    @NotEmpty
    private String city;
    @NotEmpty
    private String streetAn;
    @NotEmpty
    private String streetHn;
    @NotEmpty
    private String addressCountry;
    @NotEmpty
    private String postCode;
...
setters and getters

Um controlador

@RequestMapping(value = "/up", method = RequestMethod.POST)
    public String completeForm(@Valid @ModelAttribute("userAddressesForm") UserAddressesForm userAddressesForm,  
            BindingResult result, HttpServletRequest req)
...

Uma página JSP

<form:form commandName="userAddressesForm" action="registered">
    <table>

        <tr>
            <td class="formLabels"><form:label path="firstName">
                <spring:message code="label.name" />
            </form:label></td>
            <td><form:input path="firstName" /></td>
            <td><form:errors path="firstName" cssClass="error" /></td>
        </tr>
        <tr>
            <td class="formLabels"><form:label path="lastName">
                <spring:message code="label.surname" />
            </form:label></td>
            <td><form:input path="lastName" /></td>
            <td><form:errors path="lastName" cssClass="error" /></td>
        </tr>
    </table>

    <c:forEach items="${userAddressesForm.addresses}" varStatus="gridRow">  
        <div id="main_address" class="address_data_form">
            <fieldset>
                <legend><spring:message code="label.stepThreeMainAddressInfo" /></legend>
                <a href="#" class="deleteItem"></a>
                <table>
                    <tr>            
                        <td class="formLabels">
                            <spring:message code="label.address.custom.name" />
                        </td>
                        <td>
                            <spring:bind path="addresses[${gridRow.index}].customName">
                                <input type="input" name="<c:out value="${status.expression}"/>"
                                    id="<c:out value="${status.expression}"/>"
                                    value="<c:out value="${status.value}"/>" />
                                    <form:errors path="${status.expression}"/>
                            </spring:bind>
                        </td>   
                    </tr>               
                    <tr>            
                        <td class="formLabels">
                            <spring:message code="label.streetAnStreetHn" />
                        </td>
                        <td>
                            <spring:bind path="addresses[${gridRow.index}].streetAn">
                                <input type="input" name="<c:out value="${status.expression}"/>"
                                    id="<c:out value="${status.expression}"/>"
                                    value="<c:out value="${status.value}"/>" />
                            </spring:bind>
                            <spring:bind path="addresses[${gridRow.index}].streetHn">
                            <input type="input" name="<c:out value="${status.expression}"/>"
                                id="<c:out value="${status.expression}"/>"
                                value="<c:out value="${status.value}"/>" >
                            <form:errors path="addresses[${gridRow.index}].streetHn"/>
                            </spring:bind>

                        </td>
                    </tr>
                    <tr>                        
                        <td class="formLabels">
                            <spring:message code="label.postCode" />
                        </td>
                        <td>
                            <spring:bind path="addresses[${gridRow.index}].postCode">
                                <input type="input" name="<c:out value="${status.expression}"/>"
                                    id="<c:out value="${status.expression}"/>"
                                    value="<c:out value="${status.value}"/>" />
                            </spring:bind>
                        </td>                   
                    </tr>
                    <tr>                
                        <td class="formLabels">
                            <spring:message code="label.city" />
                        </td>
                        <td>
                            <spring:bind path="addresses[${gridRow.index}].city">
                                <input type="input" name="<c:out value="${status.expression}"/>"
                                    id="<c:out value="${status.expression}"/>"
                                    value="<c:out value="${status.value}"/>" />
                                <form:errors path="addresses[${gridRow.index}].city" cssClass="error" />
                            </spring:bind>
                        </td>
                    </tr>       
                </table>    
            </fieldset>
        </div>
    </c:forEach>

Por que os campos do objeto não são validados como AddressForm?

Por favor ajude.

user6778654
fonte

Respostas:

164

Você precisa decorar addressesmembro de UserAddressesFormcom @Validanotação. Consulte a seção 3.1.3 e 3.5.1 de JSR 303: Validação de feijão . Como expliquei em minha resposta à pergunta Existe uma maneira padrão de habilitar a Validação do Bean JSR 303 usando o método anotado , este é o uso real da @Validanotação de acordo com o JSR 303.

Editar código de exemplo: Hibernate Validator- Object Graph . (A lista de passageiros no carro)

Editar do Hibernate Validator 6 Documento de referência:

Em versões anteriores a 6, o Hibernate Validator suportava validação em cascata para um subconjunto de elementos de contêiner e era implementado no nível do contêiner (por exemplo, você usaria @Valid private List<Person>para habilitar a validação em cascata para Person).

Isso ainda é compatível, mas não é recomendado. Use @Validanotações de nível de elemento de contêiner , pois são mais expressivas.

Exemplo:

public class Car {

        private List<@NotNull @Valid Person> passengers = new ArrayList<Person>();

        private Map<@Valid Part, List<@Valid Manufacturer>> partManufacturers = new HashMap<>();

       //...
   }

Veja também o que há de novo no Bean Validation 2.0 / Jakarta Bean Validation .

Ritesh
fonte
Que tal agrupamento? Quero que o endereço seja validado apenas quando o grupo A for validado. @Validignora o grupo.
Akshay
1
A @Validanotação @Akshay funciona no nível inferior. Acho que @GroupSequence pode ser usado para especificar a ordem de validação.
Ritesh
Minha pergunta detalhada stackoverflow.com/q/41457404/1534925 . Não pense que a sequência funcionaria no meu caso.
Akshay
27

Adicionando à resposta @Ritesh, a @Validrestrição instruirá o Bean Validator a se aprofundar no tipo de sua propriedade aplicada e validar todas as restrições encontradas lá. Responda com o código à sua pergunta, o validador, ao ver uma @Validrestrição na addressespropriedade, irá explorar a AddressFormclasse e validar todas as JSR 303restrições encontradas dentro, da seguinte maneira:

public class UserAddressesForm {

    @NotEmpty
    private String firstName;

    @NotEmpty
    private String lastName;

    @Valid
    private List<AddressForm> addresses;

...
setters and getters 

public class AddressForm {

    @NotEmpty
    private String customName;
    @NotEmpty
    private String city;
    @NotEmpty
    private String streetAn;
    @NotEmpty
    private String streetHn;
    @NotEmpty
    private String addressCountry;
    @NotEmpty
    private String postCode;
...
setters and getters
Arpit Aggarwal
fonte
1
Que tal agrupamento? Quero que o endereço seja validado apenas quando o grupo A for validado. @Valid ignora o grupo
Akshay
Eu nem li sua resposta, mas só de olhar setters and gettersresolveu meu problema.
Shubham Khandare
8

Na classe UserAddressesForm adicione as seguintes linhas

@Valid
private List<AddressForm> addresses;
Tapan Banker
fonte
4

Solução de trabalho.

public class UserAddressesForm {

    @NotEmpty(message="firstName is required")
    private String firstName;

    @NotEmpty(message="lastNameis required")
    private String lastName;

    @NotNull(message="addresses attributes are required")
    @Valid
    private List<AddressForm> addresses;

...
setters and getters 

public class AddressForm {

    @NotEmpty(message="customNameis required")
    private String customName;
    @NotEmpty
    private String city;
    @NotEmpty
    private String streetAn;
    @NotEmpty
    private String streetHn;
    @NotEmpty
    private String addressCountry;
    @NotEmpty
    private String postCode;
...
setters and getters
Waqas Ahmed
fonte