AOP does not works!

[samarthya saurabh]

I am trying may hands on AOP.

and this is what my simple XML metadata looks like.

<bean id="ErrorTracer" class="com.ErrorTracer">
	</bean>

	
	<aop:config>
		<aop:pointcut id="PCErrorTracer"
			expression="execution(* com.Worker.execute(*)) and args(time)" />

		<aop:aspect id="AspectErrorTracer" ref="ErrorTracer">
			<aop:before pointcut-ref="PCErrorTracer" method="beginTracer"
				arg-names="time" />
			<aop:after pointcut-ref="PCErrorTracer" method="endTracer" />
		</aop:aspect>
	</aop:config>

I have defined the aspect and the point cut expression. But I guess I am missing something coz my after and before advice functions never get called.

I am not using ApplicationContext in my main program to execute this simply creating the bean for which aspect has been advised.

Is there something extra that needs to be done?

public class ErrorTracer
{
   
   /**
    * Log4j
    */
   private final static Logger logger = Logger.getLogger( ErrorTracer.class );
   
   /**
    * @param time
    *           - Time passed to the function after.
    * @brief Begin tracing information
    */
   public void beginTracer( Date time )
   {
      if (time != null)
      {
         logger.info( " Last executed at "
               + time);
      }
      else
      {
         logger.info( " Executing first time ");
      }
      logger
            .info( " ------------------------Tracing begins---------------------" );
      logger.info( "Done ---" );
   }
   
   /**
    * @param time
    *           - Time to function called before
    * @brief Ends Tracing information
    */
   public void endTracer( Date time )
   {
      logger
            .info( " ------------------------Tracing Ends---------------------" );
      logger.info( "Allz Done...." );
   }
}

public class Worker
{
 public boolean execute( Date lastExecutionTime )
   {
      System.out.println(" I am the main ");
   }
}

[Christophe Verré]

I don't know how Worker#execute even compiles...

[Christophe Verré]

I am not using ApplicationContext in my main program to execute this simply creating the bean for which aspect has been advised.

How are you creating the com.Worker bean ?

[samarthya saurabh]

Christophe Verré wrote:I don't know how Worker#execute even compiles...

Is there any syntactical issue here? My code compiles fine. I do include all the imports.... or am i missing something?

[samarthya saurabh]

Christophe Verré wrote:
How are you creating the com.Worker bean ?

Using the new operator to create the object of Worker class and calling execute of the function.

[Christophe Verré]

This is not going to work. If Spring is not managing your bean, how is it going to apply AOP ? Declare your worker in your context file, and get your bean via the application context or via a bean factory.

Is there any syntactical issue here? My code compiles fine. I do include all the imports.... or am i missing something?

A method returning a boolean must return something. Your code doesn't return anything.

[samarthya saurabh]

Christophe Verré wrote:This is not going to work. If Spring is not managing your bean, how is it going to apply AOP ? Declare your worker in your context file, and get your bean via the application context or via a bean factory.

Thanks! I will try to do it that way!!

A method returning a boolean must return something. Your code doesn't return anything.

Yep!! i agree i removed the logic of the function, just to avoid un-necessary bloating and forgot the return.

Thank you!