[Resolvido] vraptor - gravando relacionamento onetomany com dados extrasna join table

25 de maio de 2011 9 respostas

kindurys 25 de maio de 2011

Estou utilizando Vraptor com Hibernate e gostaria de uma dica para melhorar minha implantação:

Tenho que um cadastro de contas que tem um relacionamento NxN com Usuarios, utilizando uma tabela do meio com colunas extras.
ContasxContasUsuariosxUsuarios

No form do cadastro de contas, eu listo todos os usuários que podem ser associados aquela conta com uma checkbox ao lado de cada, e
um inputText para adicionar observações.
para que ao cadastrar uma conta o usuário possa escolher quais usuários serão associados com aquela conta.

Eu implementei desta forma:
Ao submeter o form de cadastro da conta, também envio a lista dos ids dos usuários a serem associados para esta conta.

no form de conta:

<c:forEach items="${usuarios}" var="usuario" varStatus="s">
<li> <input type="checkbox"  value ="${usuario.id}" name="usuariosSelecionados[${s.index}]" />${usuario.nome}
<input type="text" name="observacaoUsuario[${s.index}]"/>
</li>
</c:forEach>

no controller:

public void add(Conta conta, List<String> usuariosSelecionados, List<Integer> observacaoUsuario) {

E no controller eu busco cada usuário pelo id e associo a esta conta e no final gravo a conta.

Está funcionando, mas não gostei muito da forma que ficou, queria dicas de implementar isso de acordo boas práticas.

9 Respostas

Lucas_Cavalcanti 25 de mai. de 2011

você não precisa buscar cada usuário do banco, só dele ter o id o hibernate já consegue relacionar…

então vc poderia usar os parâmetros:

conta.usuarios[0].id = 2
conta.usuarios[1].id = 5
conta.usuarios[3].id = 7
....

kindurys 27 de mai. de 2011

Não consegui pois minhas entidades Conta e Usuarios foram mapeados seguindo isso aqui: http://www.mkyong.com/hibernate/hibernate-many-to-many-example-join-table-extra-column-annotation/

Então dentro de Conta eu tenho um:

private Set<ContaUsuario> usuarios =  new HashSet<ContaUsuario>(0);

e a classe ContaUsuario ficou assim:

@Entity
@Table(name = "conta_usuario")
@AssociationOverrides({
	@AssociationOverride(name = "pk.conta", 
		joinColumns = @JoinColumn(name = "conta_id")),
	@AssociationOverride(name = "pk.usuario", 
		joinColumns = @JoinColumn(name = "usuario_id")) })
public class ContaUsuario implements java.io.Serializable {

	/**
	 * 
	 */
	private static final long serialVersionUID = -5716086006460889389L;

	 private ContaUsuarioId pk = new ContaUsuarioId();
	

	 private Double valor;
	 

	 public ContaUsuario() {
	 }

	 
	@EmbeddedId
	public ContaUsuarioId getPk() {
		return pk;
	}
	
	public void setPk(ContaUsuarioId pk) {
		this.pk = pk;
	}


	
	@Transient
	public Conta getConta() {
		return getPk().getConta();
	}
 
	public void setConta(Conta stock) {
		getPk().setConta(stock);
	}
 
	@Transient
	public Usuario getUsuario() {
		return getPk().getUsuario();
	}
 
	public void setUsuario(Usuario usuario) {
		getPk().setUsuario(usuario);
	}
	
	
	public boolean equals(Object o) {
		if (this == o)
			return true;
		if (o == null || getClass() != o.getClass())
			return false;
 
		ContaUsuario that = (ContaUsuario) o;
 
		if (getPk() != null ? !getPk().equals(that.getPk())
				: that.getPk() != null)
			return false;
 
		return true;
	}
 
	public int hashCode() {
		return (getPk() != null ? getPk().hashCode() : 0);
	}
	


	public void setValor(Double valor) {
		this.valor = valor;
	}

	public Double getValor() {
		return valor;
	}



}

Então tem alguma outra forma para o hibernate cuidar disso para mim, para melhorar minha implementação?

Lucas_Cavalcanti 27 de mai. de 2011

se o getter e o setter usarem List ao inves de Set vai funcionar…

Lucas_Cavalcanti 27 de mai. de 2011

na verdade, o VRaptor usa os getters e setters, então se vc tiver um getUsuarios() e um setUsuarios() que retorna os usuários de acordo com o que está no ContaUsuario deve funcionar

kindurys 31 de mai. de 2011

Mudei de Set para List e mudei a view para

<c:forEach items="${usuarios}" var="usuario" varStatus="s">  
<li> <input type="checkbox"  value ="${usuario.id}" name="usuariosSelecionados[${s.index}].usuario.id" />${usuario.nome}  
<input type="text" name="observacaoUsuario[${s.index}]"/>  
</li>  
</c:forEach>

O que acontece agora é:

Ele esta salvando o usuario_id e valor na tabela conta_usuario sem gravar o id da nova conta(conta_id) além de que grava uma linha a mais na tabela conta_usuario, com um usuario_id que não existe(está parecendo algum incremento) e null em conta_id e valor.

O que deve ser?

Lucas_Cavalcanti 31 de mai. de 2011

vc ainda tá usando um parâmetro adicional…

troque para usar a conta:

<input ... name="conta.usuarios[${s.index}].id"

ou algo do tipo

kindurys 1 de jun. de 2011

ops...copiei o código errado...
na verdade já esta assim:

contas\form.jsp

<c:forEach items="${usuarios}" var="usuario" varStatus="s">  
<li> <input type="checkbox"  value ="${usuario.id}" name="conta.usuariosSelecionados[${s.index}].usuario.id" />${usuario.nome}  
<input type="text" name="conta.usuariosSelecionados[${s.index}].valor"/>  
</li>  
</c:forEach>

O que eu percebi que acontece é que ele sempre vai tentar gravar todos usuariosSelecionados, independente de marcar ou não checkbox, que no caso só está servindo para enviar o id do usuário se marcado.

Acho que vou criar uma função javascript que seja acionada no onclick do checkbox e insira um hidden com o valor que quero gravar.

Lucas_Cavalcanti 1 de jun. de 2011

hum… é acho que isso talvez não funcione mesmo…

o que vc precisa fazer é pegar a lista dos usuários, buscar os ContaUsuario correspondentes no banco, e fazer a alteração

kindurys 27 de jun. de 2011

No fim usei jquery, fiz algo parecido com isso aqui: http://jsbin.com/itugug/edit
dessa forma eu adiciono/removo input hidden com o id do usuário que quero associar a nova conta.
Lucas, Valeu pela ajuda!

Criado 25 de maio de 2011

Ultima resposta 27 de jun. de 2011

Respostas 9

Participantes 2

9 Respostas

Topicos relacionados