Hibernate: could not set a field value by reflection setter

[Jozo Stan]

Hello everybody!

I have a problem with hibernate exception when i want to insert data to db. This is a part of STS trace.

SEVERE: Servlet.service() for servlet [springDispatcher] in context with path
 [/SportInfo] threw exception [Request processing failed; nested exception is org.springframework.orm.hibernate3.HibernateSystemException:
 could not set a field value by reflection setter of hr.tvz.java.web.aplikacije.spring.info.Sport.sport_id; nested exception is org.hibernate.PropertyAccessException: 
could not set a field value by reflection setter of hr.tvz.java.web.aplikacije.spring.info.Sport.sport_id] with root cause
java.lang.IllegalArgumentException: Can not set int field hr.tvz.java.web.aplikacije.spring.info.Sport.sport_id to org.hibernate.id.IdentifierGeneratorHelper$2
	at sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(Unknown Source)
	at sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(Unknown Source)
	at sun.reflect.UnsafeIntegerFieldAccessorImpl.set(Unknown Source)
	at java.lang.reflect.Field.set(Unknown Source)
	at org.hibernate.property.DirectPropertyAccessor$DirectSetter.set(DirectPropertyAccessor.java:139)
	at org.hibernate.mapping.Component$ValueGenerationPlan.execute(Component.java:439)
	at org.hibernate.id.CompositeNestedGeneratedValueGenerator.generate(CompositeNestedGeneratedValueGenerator.java:122)
	at org.hibernate.event.def.AbstractSaveEventListener.saveWithGeneratedId(AbstractSaveEventListener.java:121)
	at org.hibernate.event.def.DefaultSaveOrUpdateEventListener.saveWithGeneratedOrRequestedId(DefaultSaveOrUpdateEventListener.java:210)
	at org.hibernate.event.def.DefaultSaveEventListener.saveWithGeneratedOrRequestedId(DefaultSaveEventListener.java:56)
	at org.hibernate.event.def.DefaultSaveOrUpdateEventListener.entityIsTransient(DefaultSaveOrUpdateEventListener.java:195)
	at org.hibernate.event.def.DefaultSaveEventListener.performSaveOrUpdate(DefaultSaveEventListener.java:50)
	at org.hibernate.event.def.DefaultSaveOrUpdateEventListener.onSaveOrUpdate(DefaultSaveOrUpdateEventListener.java:93)
	at org.hibernate.impl.SessionImpl.fireSave(SessionImpl.java:713)
	at org.hibernate.impl.SessionImpl.save(SessionImpl.java:701)
	at org.hibernate.impl.SessionImpl.save(SessionImpl.java:697)
	at org.springframework.orm.hibernate3.HibernateTemplate$12.doInHibernate(HibernateTemplate.java:686)
	at org.springframework.orm.hibernate3.HibernateTemplate$12.doInHibernate(HibernateTemplate.java:1)
	at org.springframework.orm.hibernate3.HibernateTemplate.doExecute(HibernateTemplate.java:406)
	at org.springframework.orm.hibernate3.HibernateTemplate.executeWithNativeSession(HibernateTemplate.java:374)
	at org.springframework.orm.hibernate3.HibernateTemplate.save(HibernateTemplate.java:683)
	at hr.tvz.java.web.aplikacije.spring.dao.SportRepositoryDAOHibernate.insertNewSport(SportRepositoryDAOHibernate.java:48)
	at hr.tvz.java.web.aplikacije.spring.service.SportServiceImpl.insertNewSport(SportServiceImpl.java:36)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:318)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
                  .
                  .
                  .

Java code:

@Controller
public class NoviSport {
	
		
	@Autowired
	public SportService sportService;
	
	
	@RequestMapping(value = "/noviSport.html", method = RequestMethod.GET)
	public String redirect (Model model){	
		model.addAttribute("sport", new Sport());
		return "noviSport";
	}
	
	
	@RequestMapping(value="/noviSport.html", method=RequestMethod.POST)
	public String adminInsert(@Valid @ModelAttribute Sport sport, BindingResult results, Model model) {
			
		/*if(results.hasErrors()) {
			return "noviSport";
			}*/
		sportService.insertNewSport(sport);
		model.addAttribute("sport", new Sport());
		model.addAttribute("uneseno", "Podaci su uspješno uneseni.");
	
		return "index";
	}
}

Sport.java

@Entity
@Table(name="sportBazaPodataka.SPORT")

public class Sport implements Serializable{
	
	
	/**
	 * 
	 */
	private static final long serialVersionUID = 7753003120604201087L;
 

	public Sport() {}
	
	@Id
	@Column(name = "SPORT_ID")
	@GeneratedValue
	private int sport_id;
	
	
	@Column(name = "SPORT_NAZIV")
	@NotNull
	@Size(min=3, max=30, message="Duljina teksta mora biti između 3 i 20.")
	private String nazivSporta;
	
	public int getSport_id() {
		return sport_id;
	}

	public void setSport_id(int sport_id) {
		this.sport_id = sport_id;
	}

	public String getNazivSporta() {
		return nazivSporta;
	}

	public void setNazivSporta(String nazivSporta) {
		this.nazivSporta = nazivSporta;
	}


	@Id
	@ManyToOne
	@JoinColumn(name = "DRZAVA_ID") 
	private Drzava drzava;

                public Drzava getDrzava() {
		return drzava;
	}

	public void setDrzava(Drzava drzava) {
		this.drzava = drzava;
	}

Drzava means Country.


And the part of jsp

<form:form id="sport" action="noviSport.html?locale=${param.locale}" method="POST" commandName="sport" >

<table cellpadding="5"  border="3" width="20%" style="border-color: blue;">
       <thead>
       <tr>
       <th>
							<font style="font-family: Arial; color: blue;"><fmt:message key="odabirSport.drzava"/></font>
							
							</th>
							<td><%
							Class.forName("com.mysql.jdbc.Driver").newInstance();
							Connection con=DriverManager.getConnection("jdbc:mysql://localhost:3306/sportBazaPodataka","root","admin");
							Statement st=con.createStatement();
          					ResultSet rs=st.executeQuery("select * from Drzava");%>
       
       <form:select path="drzava">  <%while(rs.next()){%>
            					<form:option value='<%=rs.getInt("drzava_id")%>' label='<%=rs.getString("drzava")%>'/>
            						<%} %>
								</form:select>
							
							</td>
						</tr>	
       		<tr>
       		<th><font style="font-family: Arial; color: blue;"><fmt:message key="noviSp.naziv"/> </font></th>
       		
       		<td>
				<form:input path="nazivSporta" type="text" SIZE="20" maxlength="30" />
				<br />
							
				<form:errors path="nazivSporta" style="color:red" />
			</td>
			</tr>
      </thead>
   </table> <br/>      		
		<input type="submit" name="submit" value="<fmt:message key="noviSp.spremi" />">
			</form:form>
   

When I start the web application and open the page for inserting new Sport for a country, I choose a country and then i write for example basketball. Then I press submit and the exception is there. In db the sport ID is not NULL and I don't know where is the mistake. I've also found this http://blogs.warwick.ac.uk/colinyates/entry/i_hate_hibernate/ but I don't have such a mistake.

Please help.
Thanks in advance!
Jozo

Edited line length on the stack trace to prevent horizontal scroll bars. - Bill

[Bill Gorder]

Well things were looking pretty good until I got to your JSP

Rule #1 - Don't put any persistence/DB stuff in your JSP/view layer ever
Rule #2 - Don't use scriptlets in your JSP


Your Spring controller looks pretty normal so we won't focus to much on that yet. You apparently have a service what is in there? You should also have a Repository (persistence layer) which is called by your service. This is where ALL of the data access stuff belongs (not in your jsp). Hibernate can be used to retrieve all your objects too. this list of objects can than be added to your model and displayed in your option select.

All of your db configuration user name password etc belongs in your Spring config.

I think I would try a tutorial project. As far as I can see the blog you listed is not really relevant to your problem.

Try this tutorial: Note this one is a little over simplified in that the hibernate stuff really should be in the persistence layer but it gives you an idea for the controller to JSP part.

http://krams915.blogspot.com/2011/01/spring-mvc-3-hibernate-annotations.html

For the separation between the service and repository layer (which appears to be non-existent in the blog I linked) take a look at my suggestions in this thread:

http://www.coderanch.com/t/590071/ORM/databases/org-hibernate-SessionException-Session-closed#2687293

[Bill Gorder]

Oh and welcome to Java Ranch

[Jozo Stan]

Thanks for your advices .

This are parts of my code files:

SportRepositoryDAO.java

public interface SportRepositoryDAO {

public void insertNewSport(Sport sport);
	
public void insertNewDrzava(Drzava drzava);
	
}

SportRepositoryDAOHibernate.java

public class SportRepositoryDAOHibernate extends HibernateDaoSupport implements SportRepositoryDAO{

@Override
	public void insertNewSport(Sport sport) {
		 getHibernateTemplate().save(sport);

	}

@Override
	public void insertNewDrzava(Drzava drzava) {
		getHibernateTemplate().save(drzava);
		
	}

}

SportService.java

public interface SportService {

@Secured(value = "ROLE_ADMIN")
	public void insertNewSport(Sport sport);

@Secured(value = "ROLE_ADMIN")
	public void insertNewDrzava(Drzava drzava);
	
}

SportServiceImpl.java

@Service
public class SportServiceImpl implements SportService{

@Override
	public void insertNewSport(Sport sport) {
		sportRepositoryDAO.insertNewSport(sport);
	}

@Override
	public void insertNewDrzava(Drzava drzava) {
		 sportRepositoryDAO.insertNewDrzava(drzava);
		
	}
}

About the persistence/DB stuff in my JSP. Unfortunately, I didn't know if there exists another way to populate my drop down box. I found this on the web.

[Bill Gorder]

It looks like you are on the right track. I would use the @Repository on your DAO for the reason I mentioned in the second link I provided. Also In you DAO don't use hibernateTemplate unless you have opted to use a very old version of hibernate (not recommended). With newer versions this is no longer considered to be the best practice instead follow the examples in the links I provided.

I would also use the newest version of Spring as many things are improved as time goes on. Currently that is 3.1.2-RELEASE

About the persistence/DB stuff in my JSP. Unfortunately, I didn't know if there exists another way to populate my drop down box. I found this on the web.

That web page should be purged from the Internet!

Once again read through the tutorial I linked and the post I made in that other thread. Implement what you learn from it and post back here or in our Spring forums with any specific problems you might encounter.