Como melhorar essa implementação JSF?

Oi,

Bom, eu geralmente faço assim:

Na classe Pergunta:

/*
 * @hibernate.one-to-one  
 *  class = "meupacote.Categoria"
 */
 private Categoria categoria;

Mas isso é um exemplo de um-pra-um, nos idos do hibernate 2.

Depende muito do que você quer fazer, qual será a visibilidade de um objeto para com outro.

Tem como você postar alguns trechos de código e explicar melhor o que pretendes ?

Quando faz essa anotação.

Como fica a relação entre as tabelas Pergunta e Categoria no banco de dados?

É uma página simples que estou tentando gravar os dados.

Tem a descrição da pergunta e um listbox com as categorias.

Acho que não teria a necessidade de ter uma relação entre Categoria e Pergunta teria?

Então, como o banco de dados é relacional é necessário ter a relação entre Categoria e Pergunta.

Como tem um listbox com as categorias, imagino que a relação é de que uma categoria pode ter várias perguntas. Assim, a tabela pergunta deve ter uma chave estrangeira que identifica a categoria a que pertence. Por isso, no banco de dados, precisamos do campo categoria_id na tabela pergunta.

Isso não quer dizer que na classe Pergunta esse atributo deva existir. Você pode indicá-lo somente no mapeamento (Pergunta.hbm) :

<many-to-one
            name="categoria"
            class="meupacote.Categoria"
            cascade="none"
            outer-join="auto"
            update="true"
            insert="true"
            access="property"
            column="categoria_id"
        />

Na classe Pergunta teríamos:

/* 
   * @hibernate.many-to-one
   *  class = "meupacote.Categoria" 
   */  
   private Categoria categoria;

Espero ter ajudado. Boa sorte.