Guardar OneToMany Hibernate

[detective_jd]

Hola a todos, quería hacerles la siguiente consulta: Estoy usando hibernate y hasta ahora no he tenido problemas excepto para guardar una entidad que tiene una colección onetomany, y no he podido guardar, pongo el código:

Código Java:

public class Subasta implements Serializable
{
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "suId")
    private int id;
    @Column(name = "suFecha")
    private GregorianCalendar fecha;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "dirId")
    private Direccion direccion;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "subId")
    private Subastador subastador;
    //aqui empiezan mis problemas
    @OneToMany(mappedBy = "subasta",cascade = CascadeType.ALL)
    private List<Venta>ventas;
    ..........................
    
    public void addVenta(Venta v){
        v.setSubasta(this);
        ventas.add(v);        
    }
    public void delVenta(Venta v){
        ventas.remove(v);
    }
}
 

Código Java:

package app.model.entity;
import java.io.Serializable;
import java.util.List;
import javax.persistence.*;
@Entity
@Table(name = "ventas")
public class Venta implements Serializable
{
    @EmbeddedId
    private VentaKey key;
    @JoinColumns({
        @JoinColumn(name = "SuId",referencedColumnName = "suId"), 
    })
    @ManyToOne(cascade = CascadeType.ALL)
    private Subasta subasta;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "estId")
    private Estado obje;
    @OneToMany(cascade = CascadeType.ALL, mappedBy = "venta")    
    private List<Registro>registros;
    public Subasta getSubasta() {
        return subasta;
    }
    public void setSubasta(Subasta subasta) {
        this.subasta = subasta;
    }
    public VentaKey getKey() {
        return key;
    }
    public void setKey(VentaKey key) {
        this.key = key;
    }    
    public Estado getObje() {
        return obje;
    }
    public void setObje(Estado obje) {
        this.obje = obje;
    }
    public List<Registro> getRegistros() {
        return registros;
    }
    public void setRegistros(List<Registro> registros) {
        this.registros = registros;
    }    
    public Venta() { }        
    public Venta(Subasta xobjs, Artefacto xobja, Estado obje) {
        this.key = new VentaKey(xobjs,xobja);
        this.obje = obje;
    }     
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Venta){
            Venta v  =(Venta)obj;
            return v.key.getObja() == this.key.getObja() && v.obje.equals(this.obje);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.key.getObja().hashCode();
        hash = 31 * hash + this.obje.hashCode();
        return hash;
    }
    @Override
    public String toString() {
        return this.key.getObja().getNombre() + " - estado: " + this.obje.getNombre();
    }   
}
 

Pero en el formulario lo tengo así:

Código Java:

private void btnaceptarActionPerformed(java.awt.event.ActionEvent evt) {                                           
        if(this.dtfecsubasta.getCalendar() == null || this.txtdirsubasta.getText().isEmpty() || this.txtsubsubasta.getText().isEmpty()){
            Mensajes.MostrarAdvertencia("Ingrese los datos obligatorios (*)"); 
        } else {
            if(accion.equals("crear")){
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(0, this.getDateTime(), d, s);                    
                    sub.setVentas(((ModeloVenta)mv.getModelo()).getDatos().toList());                    
                    FacadeContr.getInstance().manSubastas().add(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Creada");                    
                    btncancelarActionPerformed(evt);
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            } else {
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(Conversiones.CInt(this.txtnrosubasta.getText()), this.getDateTime(), d, s);
                    sub.setVentas(((ModeloVenta)mv.getModelo()).getDatos().toList());
                    FacadeContr.getInstance().manSubastas().edit(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Editada");
                    btncancelarActionPerformed(evt);
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            }            
        }
    }
 

Pero no me guarda las ventas de la subasta y no se que rayos hacer...

Espero sus respuestas y saludos.

[Fuzzylog]

Comprueba primero que las ventas se setean correctamente y luego asegúrate de que llegan al manager/dao.

[detective_jd]

Estuve mirando y si se setean sólo que cuando voy a guardar la subasta con las ventas me da este error:

ERROR: HHH000346: Error during managed flush [org.hibernate.exception.SQLGrammarException: could not execute statement]

Por otro lado cuando seteo las ventas en la subasta, el atributo subasta de la venta esta null, ahí tengo la duda de sí primero tengo guardar la subasta y después setear y llegar al manager.

Tengo una clase abstracta en la cual hago el CRUD:

Código Java:

package app.contr.util;
import app.model.service.HibernateUtil;
import org.hibernate.Session;
import org.hibernate.query.Query;
import up.max.proyectpages.structs.PagesList;
public abstract class Controlador<T> 
{
    Session ss = null;
    public Controlador() {
        ss = HibernateUtil.getInstance().getSession();
    }
    public void add(T entity) throws Exception{
        if(checkUnique(entity)){
            throw new Exception(this.checkUniqueMessage());
        } else {
            ss.beginTransaction();
            ss.save(entity);
            ss.getTransaction().commit();
        }
    }
    ..............
    protected abstract boolean checkUnique(T entity);
}
 

y cuando heredo en una clase y redefino

Código Java:

@Override
    protected boolean checkUnique(Rol entity) {
        return findByX(entity.getNombre())!= null;
    }
    @Override
    protected String checkUniqueMessage() {
        return "El rol ya existe";
    }
 

cuando no es necesario el unique le redefino con false.....

Espero sus respuestas y saludos

[Fuzzylog]

Según entiendo, subasta debería ser obligatorio en cada venta y mapearía la id de la subasta al atributo subasta de la venta, y ventas sería opcional en una subasta de forma que sí, primero tendrías que crear la subasta sin ventas y luego rellenar el atributo subasta de las ventas y actualizar la subasta con las ventas.

[detective_jd]

Hey FuzzyLog verás intenté hacer unos pequeños cambios:

En el formulario, cuando creo por primera vez la subasta:

Código Java:

if(accion.equals("crear")){
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(Conversiones.CInt(this.txtnrosubasta.getText()), this.getDateTime(), d, s);                    
                    sub.setVentas(((ModeloVenta)mv.getModelo()).getDatos().toList());
                    sub.getVentas().toList().forEach((v) -> {
                        v.setSubasta(sub);
                    });
                    FacadeContr.getInstance().manSubastas().add(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Creada");                    
                    btncancelarActionPerformed(evt);
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            }
 

y en la clase Venta:

Código Java:

public void setSubasta(Subasta subasta) {
        this.subasta = subasta;
        key.setObjs(subasta);
    }
 

Pero me sigue dando este error:

ERROR: HHH000346: Error during managed flush [org.hibernate.exception.SQLGrammarException: could not execute statement]

Te pongo una imagen para que veas cómo tengo diseñado mi formulario, en este link:

http://imgur.com/a/KKSPe

Espero sus respuestas y saludos

[Fuzzylog]

Trata de sacar la traza completa del error. Lo más seguro es que te dé más información.

Por otra parte es posible que necesites ejecutar todo en transacciones separadas, para que hibernate haga commit del guardado de la subasta antes de insertar las ventas.

[detective_jd]

Hey FuzzyLog depurando el sistema ahora no muestra elementos nulos en la clase Venta pero me da el mismo error, estoy comenzando a pensar que el problema está en la clase Venta o VentaKey (clase que uso para la relación N a N)

Código Java:

package app.model.entity;
import java.io.Serializable;
import java.util.List;
import javax.persistence.*;
@Entity
@Table(name = "ventas")
public class Venta implements Serializable
{
    @EmbeddedId
    private VentaKey key;
    @JoinColumns({
        @JoinColumn(name = "SuId",referencedColumnName = "suId"), 
    })
    @ManyToOne(cascade = CascadeType.ALL)
    private Subasta subasta;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "estId")
    private Estado obje;
    @OneToMany(cascade = CascadeType.ALL, mappedBy = "venta")    
    private List<Registro>registros;
    public Subasta getSubasta() {
        return subasta;
    }
    public void setSubasta(Subasta subasta) {
        this.subasta = subasta;
    }
    public VentaKey getKey() {
        return key;
    }
    public void setKey(VentaKey key) {
        this.key = key;
    }    
    public Estado getObje() {
        return obje;
    }
    public void setObje(Estado obje) {
        this.obje = obje;
    }
    public List<Registro> getRegistros() {
        return registros;
    }
    public void setRegistros(List<Registro> registros) {
        this.registros = registros;
    }    
    public Venta() { }        
    public Venta(Subasta xobjs, Artefacto xobja, Estado obje) {
        this.key = new VentaKey(xobjs,xobja);
        this.subasta = xobjs;
        this.obje = obje;
    }     
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Venta){
            Venta v  =(Venta)obj;
            return v.key.getObja() == this.key.getObja() && v.obje.equals(this.obje);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.key.getObja().hashCode();
        hash = 31 * hash + this.obje.hashCode();
        return hash;
    }
    @Override
    public String toString() {
        return this.key.getObja().getNombre() + " - estado: " + this.obje.getNombre();
    }   
}
 

Código Java:

package app.model.entity;
import java.io.Serializable;
import javax.persistence.*;
@Embeddable
public class VentaKey implements Serializable
{
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "suId")
    private Subasta objs;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "artId")
    private Artefacto obja;
    public Subasta getObjs(){
        return objs;
    }
    public void setObjs(Subasta objs) {
        this.objs = objs;
    }
    public Artefacto getObja() {
        return obja;
    }    
    public void setObja(Artefacto obja) {
        this.obja = obja;
    }    
    public VentaKey(){}
    public VentaKey(Subasta xobjs, Artefacto xobja) {
        this.objs = xobjs;
        this.obja = xobja;
    }
    @Override
    public boolean equals(Object obj){
        if(obj instanceof VentaKey){
            VentaKey id =(VentaKey)obj;
            return id.objs.equals(this.objs) && id.obja.equals(this.obja);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode(){
        int hash = 17;
        hash = 31 * hash + this.objs.hashCode();
        hash = 31 * hash + this.obja.hashCode();
        return hash;
    }
}
 

Por otro lado hice lo que me dijiste quedandome el formulario así:

Código Java:

private void btnaceptarActionPerformed(java.awt.event.ActionEvent evt) {                                           
        if(this.dtfecsubasta.getCalendar() == null || this.txtdirsubasta.getText().isEmpty() || this.txtsubsubasta.getText().isEmpty()){
            Mensajes.MostrarAdvertencia("Ingrese los datos obligatorios (*)"); 
        } else {
            if(accion.equals("crear")){
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(Conversiones.CInt(this.txtnrosubasta.getText()), this.getDateTime(), d, s);
                    FacadeContr.getInstance().manSubastas().add(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Creada");
                    ventana.setTitle("Editar Subasta");
                    accion = "editar";
                    objs = FacadeContr.getInstance().manSubastas().findById(sub.getId());
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            } else {
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(Conversiones.CInt(this.txtnrosubasta.getText()), this.getDateTime(), d, s);
                    sub.setVentas(objs.getVentas().toList());
                    sub.getVentas().toList().forEach((v) -> {
                        v.setSubasta(sub);
                    });
                    FacadeContr.getInstance().manSubastas().edit(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Editada");
                    btncancelarActionPerformed(evt);
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            }            
        }
    }
 

Así tengo la parte de ediciones de las entidades:

Código Java:

public void edit(T entity) throws Exception {
        if(this.checkUpdate(entity)){
            ss.beginTransaction();
            ss.merge(entity);
            ss.getTransaction().commit();
        }
    }
 

Espero sus respuestas y saludos.

[Fuzzylog]

Creo que es un problema de la anotación OneToMany con el cascade type.

Revisa esto:

http://www.rndblog.com/hibernate-how...en-by-cascade/

[Tipdar]

Hola detective_jd y Fuzzylog,
Verán, en el hibernate.cfg.xml si especificamos que nos muestre en consola el código SQL que está generando Hibernate muchas veces nos ayuda a entender qué estamos haciendo mal. Intentalo:

Código XML:

<property name="show_sql">true</property>
 

Y si quieres formatear el código para que sea más entendible agregas:

Código XML:

<property name="format_sql">true</property>
 

Esto me ha tirado un cable muchas veces...

[detective_jd]

Hola Tipdar y FuzzyLog en base a lo que me dijeron me salió esto:

Hibernate:
insert
into
ventas
(estId, SuId, artId, suId)
values
(?, ?, ?, ?)
abr 22, 2017 11:57:06 PM org.hibernate.engine.jdbc.spi.SqlExceptionHelper logExceptions
WARN: SQL Error: 1110, SQLState: 42000
abr 22, 2017 11:57:06 PM org.hibernate.engine.jdbc.spi.SqlExceptionHelper logExceptions
ERROR: Column 'suId' specified twice
abr 22, 2017 11:57:06 PM org.hibernate.engine.jdbc.batch.internal.AbstractB atchImpl release
INFO: HHH000010: On release of batch it still contained JDBC statements
abr 22, 2017 11:57:06 PM org.hibernate.internal.ExceptionMapperStandardImpl mapManagedFlushFailure
ERROR: HHH000346: Error during managed flush [org.hibernate.exception.SQLGrammarException: could not execute statement]

Con eso descubrí el gran error porque mi tabla ventas tiene 2 claves primarias que a la vez son foráneas y otra foránea y no 4.

ventas(suId,artId,estId)

pero no sé cómo rayos solucionar esto xq había visto algo sobre clase embebidas para este tipo de tablas pero ahora eso es lo que me está complicando.

Código Java:

package app.model.entity;
import app.contr.util.Conversiones;
import java.io.Serializable;
import java.util.*;
import javax.persistence.*;
import up.max.proyectpages.structs.PagesList;
@Entity
@Table(name = "subastas")
@NamedQueries({
    @NamedQuery(name = "subasta.findAll",query = "select s from Subasta s"),
    @NamedQuery(name = "subasta.maxId",query = "select max(s.id)+1 from Subasta s"),
    @NamedQuery(name = "subasta.search",query="select s from Subasta s "
            + "where s.direccion.nombre like :criterio or s.subastador.ape1 like :criterio")
})
public class Subasta implements Serializable
{
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "suId")
    private int id;
    @Column(name = "suFecha")
    private GregorianCalendar fecha;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "dirId")
    private Direccion direccion;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "subId")
    private Subastador subastador;
    @OneToMany(mappedBy = "subasta",cascade = CascadeType.ALL)
    private List<Venta>ventas;
    public int getId() {
        return id;
    }
    public void setId(int id) {
        this.id = id;
    }
    public GregorianCalendar getFecha() {
        return fecha;
    }
    public void setFecha(GregorianCalendar fecha) {
        this.fecha = fecha;
    }
    public Direccion getDireccion() {
        return direccion;
    }
    public void setDireccion(Direccion direccion) {
        this.direccion = direccion;
    }
    public Subastador getSubastador() {
        return subastador;
    }
    public void setSubastador(Subastador subastador) {
        this.subastador = subastador;
    }    
    public void setVentas(List<Venta> ventas) {
        this.ventas = ventas;
    }
    public PagesList<Venta>getVentas(){
        PagesList<Venta>p = new PagesList(ventas);
        return p;
    }    
    public void addVenta(Venta v){
        ventas.add(v);
    }
    public void delVenta(Venta v){
        ventas.remove(v);
    }
    public Subasta() { }
    public Subasta(int id, GregorianCalendar fecha, Direccion xdireccion, Subastador xsubastador) {
        this.id = id;
        this.fecha = fecha;
        this.direccion = xdireccion;
        this.subastador = xsubastador;
    }
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Subasta){
            Subasta objs = (Subasta)obj;
            return objs.subastador.equals(this.subastador) && objs.fecha.equals(this.fecha);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.fecha.hashCode();
        hash = 31 * hash + this.subastador.hashCode();
        return hash;
    }    
    @Override
    public String toString() {
        return Conversiones.MostrarFechaYHora(fecha) + " -> " + direccion.toString();
    }    
}
 

Código Java:

package app.model.entity;
import java.io.Serializable;
import java.util.List;
import javax.persistence.*;
@Entity
@Table(name = "ventas")
public class Venta implements Serializable
{
    @EmbeddedId
    private VentaKey key;
    @JoinColumns({
        @JoinColumn(name = "SuId",referencedColumnName = "suId"), 
    })
    @ManyToOne(cascade = CascadeType.ALL)
    private Subasta subasta;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "estId")
    private Estado obje;
    @OneToMany(cascade = CascadeType.ALL, mappedBy = "venta")    
    private List<Registro>registros;
    public Subasta getSubasta() {
        return subasta;
    }
    public void setSubasta(Subasta subasta) {
        this.subasta = subasta;
    }
    public VentaKey getKey() {
        return key;
    }
    public void setKey(VentaKey key) {
        this.key = key;
    }    
    public Estado getObje() {
        return obje;
    }
    public void setObje(Estado obje) {
        this.obje = obje;
    }
    public List<Registro> getRegistros() {
        return registros;
    }
    public void setRegistros(List<Registro> registros) {
        this.registros = registros;
    }    
    public Venta() { }        
    public Venta(Subasta xobjs, Artefacto xobja, Estado obje) {
        this.key = new VentaKey(xobjs,xobja);
        this.subasta = xobjs;
        this.obje = obje;
    }     
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Venta){
            Venta v  =(Venta)obj;
            return v.key.getObja() == this.key.getObja() && v.obje.equals(this.obje);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.key.getObja().hashCode();
        hash = 31 * hash + this.obje.hashCode();
        return hash;
    }
    @Override
    public String toString() {
        return this.key.getObja().getNombre() + " - estado: " + this.obje.getNombre();
    }   
}
 

Código Java:

package app.model.entity;
import java.io.Serializable;
import javax.persistence.*;
@Embeddable
public class VentaKey implements Serializable
{
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "suId")
    private Subasta subasta;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "artId")
    private Artefacto obja;
    public Subasta getObjs(){
        return subasta;
    }
    public void setObjs(Subasta objs) {
        this.subasta = objs;
    }
    public Artefacto getObja() {
        return obja;
    }    
    public void setObja(Artefacto obja) {
        this.obja = obja;
    }    
    public VentaKey(){}
    public VentaKey(Subasta xobjs, Artefacto xobja) {
        this.subasta = xobjs;
        this.obja = xobja;
    }
    @Override
    public boolean equals(Object obj){
        if(obj instanceof VentaKey){
            VentaKey id =(VentaKey)obj;
            return id.subasta.equals(this.subasta) && id.obja.equals(this.obja);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode(){
        int hash = 17;
        hash = 31 * hash + this.subasta.hashCode();
        hash = 31 * hash + this.obja.hashCode();
        return hash;
    }
}
 

Lo peor de todo es que esto me hace replantear todo lo hecho hasta ahora, cómo resolverían esto??

Espero sus respuestas y Saludos.

[detective_jd]

Hola Tipdar y FuzzyLog verán descubrí gran parte de mi problema y cómo resolverlo:

En la clase Subasta tenía que borrar el atributo subasta que estaba redundante y hacer lo que puse ahora

Código Java:

@OneToMany(mappedBy = "key.subasta",cascade = CascadeType.ALL)
    private List<Venta>ventas = new ArrayList();
 

Pero ahora estoy en que cuando guardo la subasta está queda almacenada en la base de datos, pero cuando la edito con las ventas almacenadas me da este error, cuando guardo la subasta nueva:

abr 24, 2017 11:54:51 PM org.hibernate.internal.ExceptionMapperStandardImpl mapManagedFlushFailure
ERROR: HHH000346: Error during managed flush [A different object with the same identifier value was already associated with the session : [app.model.entity.Venta#app.model.entity.VentaKey@8 d078dc4]]

Cuando la edito más tarde no me da ese problema que raro ¿no?, pondré el código del botón aceptar

Código Java:

private void btnaceptarActionPerformed(java.awt.event.ActionEvent evt) {                                           
        if(this.dtfecsubasta.getCalendar() == null || this.txtdirsubasta.getText().isEmpty() || this.txtsubsubasta.getText().isEmpty()){
            Mensajes.MostrarAdvertencia("Ingrese los datos obligatorios (*)"); 
        } else {
            if(accion.equals("crear")){
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(Conversiones.CInt(this.txtnrosubasta.getText()), this.getDateTime(), d, s);
                    FacadeContr.getInstance().manSubastas().add(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Creada");
                    ventana.setTitle("Editar Subasta");
                    accion = "editar";
                    objs = FacadeContr.getInstance().manSubastas().findById(FacadeContr.getInstance().manSubastas().maxId()-1);
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            } else {
                try {
                    Direccion d = FacadeContr.getInstance().manDirecciones().findByX(this.txtdirsubasta.getText());
                    Subastador s = FacadeContr.getInstance().manSubastadores().findByX(this.txtsubsubasta.getText());
                    Subasta sub = new Subasta(Conversiones.CInt(this.txtnrosubasta.getText()), this.getDateTime(), d, s);
                    sub.setVentas(objs.getVentas().toList());
                    sub.getVentas().toList().forEach((v) -> {
                        v.getKey().setObjs(sub);
                    });
                    FacadeContr.getInstance().manSubastas().edit(sub);                
                    Mensajes.MostrarConfirmacion("Subasta Editada");
                    btncancelarActionPerformed(evt);
                } catch (Exception ex) {
                    Mensajes.MostrarAdvertencia(ex.getMessage());
                }
            }            
        }
    }
 

Espero sus respuestas y Saludos.

[Fuzzylog]

Prueba con:

@OneToMany(mappedBy = "key.subasta",cascade = CascadeType.MERGE)
private List<Venta>ventas = new ArrayList();

[detective_jd]

Hola FuzzyLog y para agregar en los OneToMany funcionó, pero ahora para borrar una venta de la subasta no sirvió, que tengo que cambiar??

Código Java:

public void delVenta(Venta v){
        ventas.remove(v);
    }
 

Espero sus respuestas y Saludos.

[Fuzzylog]

Igual el problema ahí no tiene que ver con el CascadeType

¿Te sale algo en los logs?

[detective_jd]

Hola Fuzzylog el error que me da es este:

may 03, 2017 12:37:58 AM org.hibernate.internal.ExceptionMapperStandardImpl mapManagedFlushFailure
ERROR: HHH000346: Error during managed flush [deleted object would be re-saved by cascade (remove deleted object from associations): [app.model.entity.Estado#1]]

y el pequeño cambio al código es este:

Código Java:

@OneToMany(mappedBy = "key.subasta",cascade = CascadeType.MERGE,orphanRemoval = true)
    private List<Venta>ventas = new ArrayList();
 

Espero sus respuestas y saludos.

[Fuzzylog]

Tienes que quitar antes el objeto a borrar de cualquier asociación (obj,set,list,map...) que lo tenga referenciado.

[detective_jd]

Hola Fuzzylog Intenté lo que me djisiste y me dio este error:

ERROR: HHH000346: Error during managed flush [org.hibernate.HibernateException: identifier of an instance of app.model.entity.Venta was altered from app.model.entity.VentaKey@476435fd to null]

¿La clase del composite key no será el problema?

Código Java:

package app.model.entity;
import app.contr.util.Conversiones;
import java.io.Serializable;
import java.util.*;
import javax.persistence.*;
import up.max.proyectpages.structs.PagesList;
@Entity
@Table(name = "subastas")
..............
public class Subasta implements Serializable
{
    .................
    @OneToMany(mappedBy = "key.subasta",cascade = CascadeType.MERGE)   
    private List<Venta>ventas = new ArrayList();
    .................
}
 

Código Java:

package app.model.entity;
import java.io.Serializable;
import java.util.List;
import javax.persistence.*;
@Entity
@Table(name = "ventas")
public class Venta implements Serializable
{
    @EmbeddedId
    private VentaKey key;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "estId")
    private Estado obje;
    @OneToMany(cascade = CascadeType.ALL, mappedBy = "venta")    
    private List<Registro>registros;
    public VentaKey getKey() {
        return key;
    }
    public void setKey(VentaKey key) {
        this.key = key;
    }    
    public Estado getObje() {
        return obje;
    }
    public void setObje(Estado obje) {
        this.obje = obje;
    }
    public List<Registro> getRegistros() {
        return registros;
    }
    public void setRegistros(List<Registro> registros) {
        this.registros = registros;
    }    
    public Venta() { }        
    public Venta(Subasta xobjs, Artefacto xobja, Estado obje) {
        this.key = new VentaKey(xobjs,xobja);
        this.obje = obje;
    }     
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Venta){
            Venta v  =(Venta)obj;
            return v.key.getObja() == this.key.getObja() && v.obje.equals(this.obje);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.key.getObja().hashCode();
        hash = 31 * hash + this.obje.hashCode();
        return hash;
    }
    @Override
    public String toString() {
        return this.key.getObja().getNombre() + " - estado: " + this.obje.getNombre();
    }   
}
 

Código Java:

package app.model.entity;
import java.io.Serializable;
import javax.persistence.*;
@Embeddable
public class VentaKey implements Serializable
{
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "suId")
    private Subasta subasta;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "artId")
    private Artefacto obja;
    public Subasta getObjs(){
        return subasta;
    }
    public void setObjs(Subasta objs) {
        this.subasta = objs;
    }
    public Artefacto getObja() {
        return obja;
    }    
    public void setObja(Artefacto obja) {
        this.obja = obja;
    }    
    public VentaKey(){}
    public VentaKey(Subasta xobjs, Artefacto xobja) {
        this.subasta = xobjs;
        this.obja = xobja;
    }
    @Override
    public boolean equals(Object obj){
        if(obj instanceof VentaKey){
            VentaKey id =(VentaKey)obj;
            return id.subasta.equals(this.subasta) && id.obja.equals(this.obja);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode(){
        int hash = 17;
        hash = 31 * hash + this.subasta.hashCode();
        hash = 31 * hash + this.obja.hashCode();
        return hash;
    }
}
 

Subasta tiene una colección onetomany de ventas, pero ventas tiene una variable de tipo ventakey la cual tiene la subasta y artefacto (que son las claves primarias de la tabla ventas en mi bd) como atributo.
Que rebuscado es esto.

Espero sus respuestas y saludos.

[detective_jd]

Hola a todos estuve probando y viendo un poco al hacer esto:

Código Java:

public void delVenta(Venta v){
        ventas.remove(v);
        v.setKey(null);
    }
 

Me da este error:

ERROR: HHH000346: Error during managed flush [org.hibernate.HibernateException: identifier of an instance of app.model.entity.Venta was altered from app.model.entity.VentaKey@476435fd to null]

pero si hago esto:

Código Java:

public void delVenta(Venta v){
        v.setKey(null);
        ventas.remove(v);        
    }
 

me da este error:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at app.vistas.modelos.ModeloVenta.getValueAt(ModeloVe nta.java:64)
at javax.swing.JTable.getValueAt(JTable.java:2717)
at javax.swing.JTable.prepareRenderer(JTable.java:570 6)

Sé que estoy fallando en algo, pero comienzo a pensar que los frameworks de Java me están rechazando xq con JPA no me andaba, y con hibernate que estoy intentando aprender a usarlo pasa esto.

Espero sus respuestas y saludos.

[detective_jd]

Hola a todos, resulta que todavía no resolví mi problema, no me sale el error que mencioné anteriormente, pero no me borra la venta de la subasta, así tengo el código:

Código Java:

package app.model.entity;
import app.contr.util.Conversiones;
import java.io.Serializable;
import java.util.*;
import javax.persistence.*;
import up.max.proyectpages.structs.PagesList;
@Entity
@Table(name = "subastas")
@NamedQueries({
    @NamedQuery(name = "subasta.findAll",query = "select s from Subasta s"),
    @NamedQuery(name = "subasta.maxId",query = "select max(s.id)+1 from Subasta s"),
    @NamedQuery(name = "subasta.search",query="select s from Subasta s "
            + "where s.direccion.nombre like :criterio or s.subastador.ape1 like :criterio")
})
public class Subasta implements Serializable
{
   ........
   @OneToMany(mappedBy = "key.subasta",cascade = {CascadeType.PERSIST, CascadeType.REMOVE}, fetch = FetchType.LAZY,orphanRemoval=true)
    private List<Venta>ventas = new ArrayList();
   ........
   public void addVenta(Venta v) throws Exception{
        if(ventas.contains(v)){
            throw new Exception("La venta ya fue registrada");
        } else {
            ventas.add(v);
        }
    }
    public void delVenta(Venta v){
        v.getKey().setObjs(null);
        ventas.remove(v);
    }
 

el delVenta no me funciona.

Código Java:

package app.model.entity;
import java.io.Serializable;
import java.util.List;
import javax.persistence.*;
@Entity
@Table(name = "ventas")
public class Venta implements Serializable
{
    @EmbeddedId
    private VentaKey key;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "estId")
    private Estado obje;
    @OneToMany(cascade = CascadeType.ALL, mappedBy = "venta")    
    private List<Registro>registros;
    .........
}
 

Me borra en el jtable pero no me borra en la base de datos.

Código Java:

package app.model.entity;
import java.io.Serializable;
import javax.persistence.*;
@Embeddable
public class VentaKey implements Serializable
{
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "suId")
    private Subasta subasta;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "artId")
    private Artefacto obja;
    .........
}
 

No sé que solución me dan pero quiero terminar con este onetomany.

Espero sus respuestas y Saludos.

[Fuzzylog]

No faltará algún commit a la bd?

[detective_jd]

Ante todo FuzzyLog gracias por responder, verás ahí es que me agarro la confusión, porque para agregar ventas a la subasta primero agrego la venta a la subasta y luego le doy click en Aceptar para que me edite la subasta agregando la venta, que eso funciona.
Y lo peor de todo es que en addVenta así cómo lo ves está y funciona, algo me falta pero no sé bien que.

Espero sus respuestas y saludos.

[detective_jd]

Hola a todos estaba medio ausente, pero voy al punto es que cambié un poquito el código:

Código Java:

package app.model.entity;
import java.io.Serializable;
import javax.persistence.*;
@Entity
@Table(name = "ventas")
public class Venta implements Serializable
{
    @Id
    @Column(name = "id")
    private int id;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "suId")
    private Subasta objs;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "artId")
    private Artefacto obja;
    @ManyToOne(cascade = CascadeType.ALL)
    @JoinColumn(name = "estId")
    private Estado obje;
    //@OneToMany(cascade = CascadeType.ALL, mappedBy = "venta")    
    //private List<Registro>registros;
 
    public int getId() {
        return id;
    }
 
    public void setId(int id) {
        this.id = id;
    }
    
    public Subasta getObjs() {
        return objs;
    }
 
    public void setObjs(Subasta objs) {
        this.objs = objs;
    }
 
    public Artefacto getObja() {
        return obja;
    }
 
    public void setObja(Artefacto obja) {
        this.obja = obja;
    }
        
    public Estado getObje() {
        return obje;
    }
    public void setObje(Estado obje) {
        this.obje = obje;
    } 
    public Venta() { }        
    public Venta(Subasta xobjs, Artefacto xobja, Estado obje) {
        this.objs = xobjs;
        this.obja = xobja;
        this.obje = obje;
    }     
    @Override
    public boolean equals(Object obj){
        if(obj instanceof Venta){
            Venta v =(Venta)obj;
            return v.objs.equals(this.objs) && v.obja.equals(this.obja);
        } else {
            return false;
        }
    }
    @Override
    public int hashCode(){
        int hash = 17;
        hash = 31 * hash + this.objs.hashCode();
        hash = 31 * hash + this.obja.hashCode();
        return hash;
    }   
}
 

Código Java:

import ....;
@Entity
@Table(name = "subastas")
@NamedQueries({
    @NamedQuery(name = "subasta.findAll",query = "select s from Subasta s"),
    @NamedQuery(name = "subasta.maxId",query = "select max(s.id)+1 from Subasta s"),
    @NamedQuery(name = "subasta.search",query="select s from Subasta s "
            + "where s.direccion.nombre like :criterio or s.subastador.ape1 like :criterio")
})
public class Subasta implements Serializable
{
@OneToMany(mappedBy = "objs",cascade = CascadeType.ALL, fetch = FetchType.EAGER)
    private List<Venta>ventas = new ArrayList();
.........
public void delVenta(Venta v){
        //v.getKey().setObjs(null);
        ventas.remove(v);
    }
}
 

No sé si tengo que agregar un controlador o que porque para agregar las ventas de la subasta lo hace así de fácil, pero para borrar no me borra las ventas de la subasta.

Espero sus respuestas y saludos.