need help here

[Hussein Baghdadi]

I have two entity beans: PlayerBean and TeamBean.
the realtionship between these beans is : one to many (many players to one team).
I am using CMP 2.0 model and the relationship is bidirectional.
the fields of Team table in the database are :
teamname, teamcity
the fields of Player table in the database are:
pid, pname, psalary, pteam (pteam is a string and it serves as a FK).
in the PlayerLocal interface I wrote :
All the codes are inspired from Mastering EJB

// set and get mehtods
public void setPlayerTeam(TeamLocal tl);
public TeamLocal getPlayerTeam( );  

in the PlayerBean I wrote:

// abstract set and get methods including setPlayerTeam and getPlayerTeam
public String ejbCreate(String id, String name, TeamLocal tl, int s) throws CreateException {
  setId(id);
  setName(name);
  setSalary(s);
  return null;
}
public String ejbCreate(String id, String name, String team, int s) throws CreateException {
  Team t = teamHome.findByPrimaryKey(team);
  return this.ejbCreate(id, name, t, s);
}
public void ejbPostCreate(String, String, TeamLocal tl, int) throws ... {
  setPlayerTeam(tl);
}  

using J2EE RI, I don't persist the pteam..
are there any things wrong with this design ?
if there is a better way to perform this relationship please tell me.
if I wrote in my client:
player.setTeam(teamloca);
will the container fills the pteam field in the Player table with the right value ?

[Mapraputa Is]

Redirect to the twin thread in "EJB and Other J2EE Technologies". Please, post your answers there. Thanks...