Validar seleção de autor no combobox

Fala Marcio, tudo bem?

tenta colocar esses atributos no componente do selectOneMenu:

<h:selectOneMenu value="#{livroBean.autorId}" id="autor" required="true"   requiredMessage="Por favor selecione um Autor" >
    <f:selectItem itemLabel="Selecione..."  itemValue="#{null}" noSelectionOption="true"/>
    <f:selectItems value="#{livroBean.autores}" var="autor" itemLabel="#{autor.nome}" itemValue="#{autor.id}"/>
</h:selectOneMenu>

Depois conta pra gente se funcionou =)

Abraço

Boa tarde Samir, obrigado pela resposta. Não deu certo, não está mostrando a mensagem. Mas não entendi por que colocar o trecho requiredMessage="Por favor selecione um Autor" uma vez que foi feita a validação no bean que cria um FacesMessage com a mensagem a ser renderizada. segue como ficou o código da página, o bean ficou da forma como estava no inicio do post.

            <legend>Dados do Autor</legend>
            <h:outputLabel value="Selecione o Autor: " for="autor"/>
            <h:selectOneMenu value="#{livroBean.autorId}" id="autor" required="true" requiredMessage="Por favor selecione um Autor">
                <f:selectItem itemLabel="Selecione..." itemValue="#{null}" noSelectionOption="true"/>
                <f:selectItems value="#{livroBean.autores}" var="autor" itemLabel="#{autor.nome}" itemValue="#{autor.id}"/>
            </h:selectOneMenu>
            <h:commandButton value="Gravar Autor" action="#{livroBean.gravarAutor}">
                <f:ajax execute="autor" render="tabelaAutores"/> <!-- o atributo execute executa a tag com o id="autor" e o render renderiza após a execução a tabela com o id="tabelaAutores" -->
            </h:commandButton>

Veja o log no console

Fase: RESTORE_VIEW 1
Fase: APPLY_REQUEST_VALUES 2
Fase: PROCESS_VALIDATIONS 3
Fase: RENDER_RESPONSE 6

Fala Marcio!A ideia de colocar o requiredMessage é fazer a validação pela view e não pelo Bean, dessa forma vc já adiciona a mensagem na view e não precisaria da validação no método gravar. Lembrando que precisa ter o componente dentro do form. Mas sem problema fazer sem o requiredMessage.

Tenta tirar a tag ajax e veja se funciona.

Caso não funcione, manda o código da página toda e do bean tb, caso funcione da um toque tb que a gnt faz funcionar com Ajax.

Abraço

Oi Samir, retirando a tag ajax funcionou usando o atributo requiredMessage, mas eu fiz pelo bean pra testar o funcionamento, da mesma forma que o campo isbn que valida pelo método comecaComDigitoUm. Mas ao retirar a tag ajax perde-se a funcionalidade e todo o formulário é enviado, ao invés de enviar somente o combobox. Quero entender porque não funcionou com a validação no bean. Segue o código do da view e do bean

<h:body>
    <h1>Novo Livro</h1>
    <h:form>
        <h:messages id="messages"/>
        <fieldset>
            <legend>Dados do Livro</legend>
            <h:panelGrid columns="2">

                <h:outputLabel value="Titulo:" for="titulo" />
                <h:inputText id="titulo" value="#{livroBean.livro.titulo}" required="true" requiredMessage="Título obrigatório" validatorMessage="Valor máximo de 40 caracteres">
                    <f:validateLength maximum="40"/>
                    <f:ajax event="blur" render="messages"/>
                </h:inputText>

                <h:outputLabel value="ISBN:" for="isbn" />
                <h:inputText id="isbn" value="#{livroBean.livro.isbn}" validator="#{livroBean.comecaComDigitoUm}">
                    <f:ajax event="blur" render="messages"/>
                </h:inputText>

                <h:outputLabel value="Preço:" for="preco" />
                <h:inputText id="preco" value="#{livroBean.livro.preco}" validatorMessage="Preço deve ser entre 1 e 1000" >
                    <f:validateDoubleRange minimum="1.0" maximum="1000.00"/>
                </h:inputText>

                <h:outputLabel value="Data de Lançamento:" for="dataLancamento" />
                <h:inputText id="dataLancamento" value="#{livroBean.livro.dataLancamento.time}" >
                    <f:convertDateTime pattern="dd/MM/yyyy HH:mm" />
                </h:inputText>

            </h:panelGrid>
        </fieldset>

        <fieldset>
            <legend>Dados do Autor</legend>
            <h:outputLabel value="Selecione o Autor: " for="autor"/>
            <h:selectOneMenu value="#{livroBean.autorId}" id="autor" required="true" requiredMessage="Por favor selecione um Autor">
                <f:selectItem itemLabel="Selecione..." itemValue="#{null}" noSelectionOption="true"/>
                <f:selectItems value="#{livroBean.autores}" var="autor" itemLabel="#{autor.nome}" itemValue="#{autor.id}"/>
            </h:selectOneMenu>
            <h:commandButton value="Gravar Autor" action="#{livroBean.gravarAutor}">
                <!-- <f:ajax execute="autor" render="tabelaAutores"/> --> <!-- o atributo execute executa a tag com o id="autor" e o render renderiza após a execução a tabela com o id="tabelaAutores" -->
            </h:commandButton>
            <br />
            <h:commandLink value="Cadastrar novo autor" action="#{livroBean.formAutor}" immediate="true"/>

            <h:dataTable value="#{livroBean.autoresDoLivro}" var="autor" id="tabelaAutores">
                <h:column>
                    <h:outputText value="#{autor.nome}"/>
                </h:column>
            </h:dataTable>
        </fieldset>

        <h:commandButton value="Gravar" action="#{livroBean.gravar}">
            <f:ajax execute="@form" render="@form :tabelaLivros" />
        </h:commandButton>
    </h:form>    

</h:body>

</html>


@ManagedBean
//@RequestScoped // --> É o default. O bean dura apenas durante a requisição.
@ViewScoped  // o bean dura enquanto a tela existir
public class LivroBean {

    private Livro livro = new Livro();
    private Integer autorId;

    public Integer getAutorId() {
        return autorId;
    }

    public void setAutorId(Integer autorId) {
        this.autorId = autorId;
    }

    public Livro getLivro() {
        return livro;
    }

    public List<Autor> getAutores(){
        return new DAO<Autor>(Autor.class).listaTodos();
    }

    public List<Autor> getAutoresDoLivro(){
        return this.livro.getAutores();
    }

    public List<Livro> getLivros() {
        return new DAO<Livro>(Livro.class).listaTodos();
    }    

    public void gravarAutor(){
        Autor autor = new DAO<Autor>(Autor.class).buscaPorId(this.autorId);
        if(autor == null){
            FacesContext.getCurrentInstance().addMessage("autor", new FacesMessage("Selecione um Autor"));
        }else{
            this.livro.adicionaAutor(autor);
            System.out.println("Livro escrito por: " + autor.getNome());
        }
    }
    public void gravar() {
        System.out.println("Gravando livro " + this.livro.getTitulo());

        if (livro.getAutores().isEmpty()) {
            //throw new RuntimeException("Livro deve ter pelo menos um Autor.");
            FacesContext.getCurrentInstance().addMessage("autor", new FacesMessage("Livro deve ter pelo menos um Autor."));
        }else{
            new DAO<Livro>(Livro.class).adiciona(this.livro);
            this.livro = new Livro();            
        }
    }

    public String formAutor(){
        System.out.println("Chamando o formulário do Autor");
        return "autor?faces-redirect=true";
    }

    public void comecaComDigitoUm(FacesContext fc, UIComponent component, Object value) throws ValidatorException{
        String valor = value.toString();
        if(!valor.startsWith("1")){
            throw new ValidatorException(new FacesMessage("ISBN deve começar com 1"));
        }
    }
}

Isso significa que o problema está no ajax! Tenta colocar : (o caractere de dois pontos) antes do id nos atributos render e execute.

No caso do ISBN, vc criou um método que joga uma exception e recebe os parametros necessários para que o validator do JSF consiga lançar a mensagem. Se vc fizer outro método nesse formato e jogar no validator do SelectOneMenu, provavelmente funcionará também.

Ixi Samir, agora to confuso. A tag ajax está funcionando normalmente, ou seja, enviando somente o combobox. O commandButton "Gravar Autor" através da tag ajax executa o selectOneMenu de id="autor" e renderiza a tabela de id="tabelaAutores" Isso tudo funciona de boa. Mas resolvi validar se o autor foi selecionado no combobox. Fiz um método semelhante ao comecaComDigitoUm, mas não está sendo chamado. É que o commandButton tem o action que está sendo chamado no bean, por isso que resolvi fazer a validação no método chamado pelo action.

Não entendi o que vc quis com isso "Tenta colocar : (o caractere de dois pontos) antes do id nos atributos render e execute."

    public void verificaAutor(FacesContext fc, UIComponent component, Object value) throws ValidatorException{
        System.out.println("Varificando combobox Autor");
        String autor = value.toString();
        if(autor == null){
            throw new ValidatorException(new FacesMessage("Selecione um Autor"));
        }
    }

            <legend>Dados do Autor</legend>
            <h:outputLabel value="Selecione o Autor: " for="autor"/>
            <h:selectOneMenu value="#{livroBean.autorId}" id="autor" validator="#{livroBean.verificaAutor}">
                <f:selectItem itemLabel="Selecione..." itemValue="#{null}" noSelectionOption="true"/>
                <f:selectItems value="#{livroBean.autores}" var="autor" itemLabel="#{autor.nome}" itemValue="#{autor.id}"/>
            </h:selectOneMenu>
            <h:commandButton value="Gravar Autor" action="#{livroBean.gravarAutor}">
                <f:ajax execute="autor" render="tabelaAutores"/> <!-- o atributo execute executa a tag com o id="autor" e o render renderiza após a execução a tabela com o id="tabelaAutores" -->
            </h:commandButton>

Descobri o que estava errado. Faltou colocar na tag ajax pra renderizar o messages. Agora funciona com a validação que eu havia feito no início

Obrigado pela ajuda.

<f:ajax execute="autor" render="tabelaAutores messages"/>

    public void gravarAutor(){
        Autor autor = new DAO<Autor>(Autor.class).buscaPorId(this.autorId);
        if(autor == null){
            FacesContext.getCurrentInstance().addMessage("autor", new FacesMessage("Selecione um Autor"));
        }else{
            this.livro.adicionaAutor(autor);
            System.out.println("Livro escrito por: " + autor.getNome());
        }
    }

Boa! Qualquer dúvida estamos aí!