Vraptor + problema chato

Aproveitando, qual é a melhor formar de fazer um relacionamento entre a entidade filha com a pai.
To tendo que fazer um loop nas entidades filhas colocando a entidade pai em cada uma.

Valeu.

o seu controller receber muitos daos é um forte sinal de que essa classe esteja fazendo coisas demais…

agrupe as funcionalidades parecidas em outras classes (sejam elas controllers, ou outro componente), daí resolve esse problema.

sobre a segunda dúvida, se essas filhas estão vindo da tela, vc pode já setar o pai nelas:

<input type="hidden" name="filhas[${index}].pai.id" value="${pai.id}"/>

Eu gostaria de aproveitar o gancho e tirar uma dúvida:
É legal eu criar um converter do VRaptor para uma classe minha, exemplo:

public class Cidade {
   private long id;
   private String nome;
// getters e setters
}

Essa classe é uma entidade e eu tenho um DAO que consegue buscar pelo ID da cidade.
Na minha jsp eu tenho o seguinte caso:

<select name="cidade">
   <option value="1">São Paulo</option>
   <option value="2"> Rio de Janeiro</option>
</select>

No meu controller, eu tenho uma action que recebe a cidade, e não um long:

@Get("/path/xpto")
public void pegaCidade(Cidade c){
// faz algo.
}

Para que isso funcione, eu preciso cadastrar um converter para Cidade:

@Convert(Cidade.class)
public class CidadeConverter implements Converter<Cidade> {
   private CidadeDAO dao;
   public Cidade (CidadeDAO dao){
      this.dao = dao;
   }

   public Cidade convert(String value, Class<? extends Cidade> type,
			ResourceBundle bundle){
      return dao.get(Long.parseLong(value));
   }
}

Minha dúvida é: isso é uma boa prática? Se não, o que eu devo fazer ao invés disso?

se isso vai facilitar o código, não vejo pq não fazer

só lembre de usar as buscas lazy (tipo o em.getReference ou session.load) assim vc pode fazer isso pros relacionamentos tb…

e já existe a anotação @Load que faz exatamente isso

Mas a anotação @Load retorna um erro quando o objeto não é encontrado. Correto?

E se eu quisesse receber null?

O @Load funciona para os atributos de uma classe minha?
Exemplo:

public class Estado {
   private long id;
   private String uf;
   private String nome;
   private Cidade cidade;
// getters e setters
}
@Get("/abcde/")
public void fazAlgo(@Load Estado estado) {// Assim, se eu tiver um estado.cidade e passando somente o ID da cidade, ela será populada também?
// faz algo
}

nao… o load eh soh pra propria classe mesmo…

o @Load retorna 404 se o cara nao eh encontrado, nao?

Existe alguma forma de sobrescrever o @Load?
Assim eu faria ele retornar null, e não retornar o 404… às vezes a gente pode querer que o objeto venha nulo…

é que esse @Load foi feito pra esse tipo de método:

@Get("/produto/{produto.id}")
public void mostra(@Load Produto produto) {...}

então se o produto não existe tem que dar 404.

mas vc pode usar essa mesma idéia e criar seu próprio @Load… ou usar o seu converter mesmo.

Lucas vlw pelas respostas! Outra coisa… E se o id for gerado automaticamente na hora do save da entidade pai? Vai funcionar?

vc vai adicionar o pai e os filhos ao mesmo tempo?

nesse caso essa estratégia não vai funcionar, e vc vai ter que fazer o loop mesmo, ou mudar o mapeamento pros dois lados do relacionamento serem fortes (cuidado com isso)…

talvez tenha que usar cascade=PERSIST tb

Marcio89, vc pode trocar todos esses DAOs por uma classe que cria os DAOs.