<servlet>
    <servlet-name>some</servlet-name>
    <servlet-class>SomeServlet</servlet-class>
  </servlet>

   <bean id="someBean" class="SomeBean">
         <property name="servlet">
            <ref bean="servlet"/>
         </property>
   </bean>

floater wrote:
sounds like a wrong way to go.

public class SomeInterceptor implements MethodBeforeAdvice {

  public void before(Method method, Object[] objects, Object o)
      throws Throwable {
      
    ActionMapping mapping=(ActionMapping)objects[0];
    ActionForm form=(ActionForm)objects[1];
    HttpServletRequest request=(HttpServletRequest)objects[2];
    HttpServletResponse response=(HttpServletResponse)objects[3];
     
    DispatchAction action=(DispatchAction)o;
    ActionServlet servlet=(ActionServlet)action.getServlet();
    ...

  }  
}

public class SomeAction extends DispatchAction {

  ...

  public ActionForward somefunction(ActionMapping mapping, ActionForm form,
      HttpServletRequest request, HttpServletResponse response)
      throws Exception {
    ...
  }

  ...
}

<bean name="targetA" class="TargetA"/>
<bean name="targetB" class="TargetB"/>

<bean name="interceptorA"  class="InterceptorA"/> 
<bean name="interceptorB"  class="InterceptorB"/> 


<bean name="autoProxyA" 
          class="org.springframework.aop.framework.autoproxy.BeanNameAutoProxyCreator">

    <property name="beanNames">
      <list>
    <value>targetA</value>
  </list>
    </property>
    <property name="interceptorNames">
        <list>
          <value>interceptorA</value>
        </list>
    </property>
</bean>


<bean name="autoProxyB" 
          class="org.springframework.aop.framework.autoproxy.BeanNameAutoProxyCreator">

    <property name="beanNames">
      <list>
    <value>targetB</value>
  </list>
    </property>
    <property name="interceptorNames">
        <list>
          <value>interceptorB</value>
        </list>
    </property>
</bean>

floater wrote:
try give them different ids

<bean id="autoProxyB" ...

floater wrote:
try setting the proxyTargetClass property to true for the BeanNameAutoProxyCreator definition or google for this, I've never seen this before.

Dynamic Proxy in java is the best way for AOP, but only suitable for interface not class.
If you want to use class, Spring AOP will use CGLib to genearte the subclass of the target class to implement AOP.
In this way, we should set the propery "proxyTargetClass" to be true in the Proxy class.

floater wrote:
one thing I notice you always use bean name attribute, did you replace these with id attribute everywhere?

name is just an alias, not the identifier. we should use id always at least.