Spring MVC no default constructor found?

Question

I'm having problems with my Spring controllers - I'm getting no default constructor found - but they do have a constructor which I am trying to created via the applicationContext.xml - heres the relevant bit:

<bean id="PcrfSimulator" class="com.rory.services.pcrf.simulator.PcrfSimulator" init-method="start">  
</bean>

<bean id="CacheHandler" class="com.rory.services.pcrf.simulator.handlers.CacheHandler">
    <constructor-arg index="0" type="com.rory.services.pcrf.simulator.CustomGxSessionIdCacheImpl">
        <bean factory-bean="PcrfSimulator" factory-method="getGxSessionIdCache">
        </bean>
    </constructor-arg>       
</bean>

Ie. I'm creating a bean first, and then trying to pass the result of a method call from that bean into the second bean's (CacheHandler) constructor.

Here'e the start of CacheHandler:

    @Controller
    public class CacheHandler {

    private final CustomGxSessionIdCacheImpl gxSessionIdCache;

    public CacheHandler(CustomGxSessionIdCacheImpl gxSessionIdCache) {
        this.gxSessionIdCache = gxSessionIdCache;
    }

Here's the error I'm getting:

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'cacheHandler' defined in URL [jar:file:/users/rtorney/Documents/apache-tomcat-7.0.25/webapps/PCRFSimulator-4.0/WEB-INF/lib/PCRFSimulator-4.0.jar!/com/rory/services/pcrf/simulator/handlers/CacheHandler.class]: Instantiation of bean failed; nested exception is org.springframework.beans.BeanInstantiationException: Could not instantiate bean class [com.rory.services.pcrf.simulator.handlers.CacheHandler]: No default constructor found; nested exception is java.lang.NoSuchMethodException: com.rory.services.pcrf.simulator.handlers.CacheHandler.<init>()

Any help is much appreciated!

soulcheck · Accepted Answer · 2012-02-15T16:21:48.743

You should either define your beans in xml or annotate them, not both (if only to avoid errors like the one you're getting).

The problem here is that you're not autowiring constructor args, so spring doesn't know what to do with your controller. It knows it has to create a bean (@Controller annotation), but it doesn't know how (no default, nor autowired constructor).

You can try to do something like:

@Controller
public class CacheHandler {

private final CustomGxSessionIdCacheImpl gxSessionIdCache;

@Autowired 
public CacheHandler(CustomGxSessionIdCacheImpl gxSessionIdCache) {
   this.gxSessionIdCache = gxSessionIdCache;
}

and then in xml:

<bean id="gxSessionIdCache"
  factory-bean="PcrfSimulator"
  factory-method="getGxSessionIdCache"/>

So it will autowire constructor parameters.

Another option is to simply create default constructor and autowire gxSessionIdCache property.

great to confirm this, spring need either default constructor, or auto wired constructor. — Jackie, Jan 04 '13 at 08:54

score 2 · Answer 2 · answered Feb 15 '12 at 16:11

You have to add an empty default constructor :

    @Controller
    public class CacheHandler {

    private final CustomGxSessionIdCacheImpl gxSessionIdCache;

    @Autowired
    public CacheHandler(CustomGxSessionIdCacheImpl gxSessionIdCache) {
        this.gxSessionIdCache = gxSessionIdCache;
    }

But be carefull, because it seems that you are mixing annotation based configuration (@Controller) and XML configuration. In the example above, it uses the annotation based config (so please remove the bean declaration from your XML file).

[Does Spring require all beans to have a default constructor?](http://stackoverflow.com/questions/7492652/does-spring-require-all-beans-to-have-a-default-constructor): [**no**](http://stackoverflow.com/a/7492713/545127) — Raedwald, Dec 11 '14 at 17:00

score 0 · Answer 3 · answered Dec 13 '12 at 12:40

0

You can also get this error if you haven't activated Spring's annotation-based config. Include this in your Spring Xml:

<context:annotation-config/>

answered Dec 13 '12 at 12:40

Paul Hilliar

579
6
8

score 0 · Answer 4 · answered Dec 11 '14 at 17:14

Other posters have pointed out that you can get problems if you mix autowiring/component-scanning with explicit instantiation of beans. I had a similar problem with a web application that did that. I was able to fix the problem by telling the component-scanner not to automatically instantiate a bean of the crucial class. Like this:

 <?xml version="1.0" encoding="UTF-8"?>
 <beans xmlns=...>

    <aop:aspectj-autoproxy />

    <import resource="repository.xml" />
    ...

    <context:component-scan base-package="com.example.webserver">
       <context:exclude-filter type="regex" expression="MyRepositoryImpl" />
       <context:exclude-filter type="annotation" expression="org.springframework.stereotype.Repository" />
    </context:component-scan>
 </beans>

where repository.xml included the explicit bean instantiation:

 <beans xmlns=...>
    <bean id="password" class="org.springframework.jndi.JndiObjectFactoryBean">
       <property name="jndiName" value="java:/comp/env/store/clientPassword" />
    </bean>
    <bean id="repository" class="com.example.webserver.datalayer.MyRepositoryImpl">
       <constructor-arg ref="password" />
    </bean>
...
 </beans>

Spring MVC no default constructor found?

4 Answers4