spring aop not supporing aspectJ annotation

[sidhartha pani]

@Aspect
public class Beforeaspect {

@Before("com.beans.Student.addCustomer()")
public void logBefore(JoinPoint jp)
{
System.out.println("logbefore");
System.out.println("method "+jp.getSignature().getName());
}
}
public class Student implements Studentimpl {

@Override
public void addCustomer() {
// TODO Auto-generated method stub
System.out.println("addcustomer");

}

@Override
public String addCustomername(String stud) {
// TODO Auto-generated method stub
System.out.println("addcustomername");
return "hello";
}
<beans xmlns="http://www.springframework.org/schema/beans"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns:aop="http://www.springframework.org/schema/aop"
xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans-2.5.xsd

http://www.springframework.org/schema/aop http://www.springframework.org/schema/aop/spring-aop-2.5.xsd">

<aop:aspectj-autoproxy/>

<bean id="stud" class="com.beans.Student"/>
<bean  class="com.beans.Beforeaspect"/>

</beans>
public class Test {

/**
* @param args
*/
public static void main(String[] args) {
// TODO Auto-generated method stub


@SuppressWarnings("resource")
ApplicationContext context = new ClassPathXmlApplicationContext("com/xmlfiles/beans.xml");
Studentimpl studs=(Studentimpl) context.getBean("stud");

// System.out.println(studs.addCustomers());
//studs.addCustomername("aaa");
studs.addCustomer();



}

}

I am using java 8 and spring 4.3

[Sergej Smoljanov]

I change location of file. (pay note of package, and i change Student to be interface and Studentimpl become class.

package com;

public interface Student {
	public void addCustomer();
	public String addCustomername(String stud);
}

package com;

public class Studentimpl implements Student {

		@Override
		public void addCustomer() {
		// TODO Auto-generated method stub
		System.out.println("addcustomer");

		}

		@Override
		public String addCustomername(String stud) {
		// TODO Auto-generated method stub
		System.out.println("addcustomername");
		return "hello";
		} 
}

@Before("execution( ** com.Student.addCustomer(..))")
or
@Before("execution(** com.Student.addCustomer())")

package com;

import org.aspectj.lang.JoinPoint;
import org.aspectj.lang.annotation.Aspect;
import org.aspectj.lang.annotation.Before;

@Aspect
public class BeforeAspect {
	@Before("execution( ** com.Student.addCustomer(..))") 
	public void logBefore(JoinPoint jp)	{
	System.out.println("logbefore");
	System.out.println("method "+jp.getSignature().getName());
	}
}
	

package com;

import org.springframework.aop.framework.ProxyFactory;
import org.springframework.context.ApplicationContext;
import org.springframework.context.support.ClassPathXmlApplicationContext;

public class Demo {
	public static void main(String [] arg){
		ApplicationContext context = new ClassPathXmlApplicationContext("com/xmlfiles/beans.xml");
		Student studs=(Student) context.getBean("stud");

		// System.out.println(studs.addCustomers());
		//studs.addCustomername("aaa");
		studs.addCustomer(); 
		
	}
}

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns = "http://www.springframework.org/schema/beans"
	xmlns:xsi = "http://www.w3.org/2001/XMLSchema-instance"
	xmlns:context = "http://www.springframework.org/schema/context"
	xmlns:aop = "http://www.springframework.org/schema/aop"
	xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans.xsd
						http://www.springframework.org/schema/aop http://www.springframework.org/schema/aop/spring-aop.xsd
						http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context.xsd">
	<aop:aspectj-autoproxy />
	<bean id="stud" class="com.Studentimpl"/>
	<bean  class="com.BeforeAspect"/> 

</beans>

[Sergej Smoljanov]

Also you can change BeforeAspect (your custom aspect class like this)

package com;

import org.aspectj.lang.JoinPoint;
import org.aspectj.lang.annotation.Aspect;
import org.aspectj.lang.annotation.Before;
import org.aspectj.lang.annotation.Pointcut;

@Aspect
public class BeforeAspect {
	@Pointcut("execution (* com.Student.addCustomer())")
	public void m(){}
	
	@Before(value = "com.BeforeAspect.m()") 
	public void logBefore(JoinPoint jp)	{
	System.out.println("logbefore");
	System.out.println("method "+jp.getSignature().getName());
	}
}

note:
@Pointcut("execution (* com.Student.addCustomer())") - you place your pointcut outside @Before and now you can use it as below
and this
@Before(value = "com.BeforeAspect.m()")

[sidhartha pani]

Sergej Smoljanov wrote:Also you can change BeforeAspect (your custom aspect class like this)

package com;

import org.aspectj.lang.JoinPoint;
import org.aspectj.lang.annotation.Aspect;
import org.aspectj.lang.annotation.Before;
import org.aspectj.lang.annotation.Pointcut;

@Aspect
public class BeforeAspect {
	@Pointcut("execution (* com.Student.addCustomer())")
	public void m(){}
	
	@Before(value = "com.BeforeAspect.m()") 
	public void logBefore(JoinPoint jp)	{
	System.out.println("logbefore");
	System.out.println("method "+jp.getSignature().getName());
	}
}

note:
@Pointcut("execution (* com.Student.addCustomer())") - you place your pointcut outside @Before and now you can use it as below
and this
@Before(value = "com.BeforeAspect.m()")