@EmbeddedId with getter/setter ?

[nimo frey]

When I use something like:

@EmbeddedId
private CompositeID compositeID = new CompositeID();

Do I need getter or setter for an EmbeddedID-Field?


Look at my class:

public class MyEntity implements Serializable {
 
// here I do not provide getter/setter and do field-annotation
@EmbeddedId
private CompositeID compositeID = new CompositeID();
 
// here I do provide getter/setter and do getter-annotation
private Integer fieldA;
@Column(name = "FIELD_A", nullable = false)
public Integer getFieldA() { return fieldA; }
public void setFieldA(Integer fieldA) { this.fieldA= fieldA;}
 
..
 
}

Is that correct? Or should I do this:

private CompositeID compositeID = new CompositeID ();
// annotate getter
@EmbeddedId
public CompositeID getCompositeID () { return compositeID ; }
public void setCompositeID (CompositeID compositeID ) { this.compositeID = compositeID ;}

[ December 18, 2008: Message edited by: nimo frey ]

[Angel Taveras]

Hello Nimo, it depends on how the JPA Implementation access the properties if it is defined to access via a setter/getter then you should add them, otherwise it will access the properties with reflection.

In your example you should add the setter and getter because your composite id it's not generated or you will have to also use reflection to change it's value

Regards,

[nimo frey]

okay.

I use hibernate and found out, that I do not need to have getter/setter for my EmbeddedID (all works fine).

However, for all my other fields I have getter/setter and annotate the @Column to my getter (instead of field-annotation).


I have a minor question, I do not know, when do I need to instantiate an Object in my Entity-Bean(do I have to? should I?)


=new CompositeID

@EmbeddedId
//here I instantiate it directly
private CompositeID compositeID [B]= new CompositeID()[/B];
//getter/setter

=new HashSet<Entity>();

private Set<Entity> e= [B]new HashSet<Entity>()[/B];
@OneToMany(mappedBy = "refEntity")
//getter/setter

[ December 18, 2008: Message edited by: nimo frey ]

[Angel Taveras]

In my particular case i like to instantiate every collection that i want to use inside the entity, but also, you could add a few helper methods too. For instance,

 
@OneToMany
List<Object> objects=new ArrayList<Object>();
 
//getters/setters
 
//helper methods
 
public void addObject(Object o)
{
  if(o==null) return;
  object.add(o);
}
 
public void removeObject(Object o)
{
    objects.remove(o);
}
 
public void removeObject(int index)
{
   if(objects.size()<index&&index>0)
     objects.remove(index);
}
 

[ December 18, 2008: Message edited by: Angel Taveras ]