Подтвердить что ты не робот

AspectJ - почему "совет, определенный в XYZ, не применялся"?

Я только начал играть с AspectJ (1.6.11). Я отправляю электронные письма через commons-email libary, и я хотел бы знать, сколько времени потребуется, чтобы отправить сообщение. Итак, это мой код отправки электронной почты:

import org.apache.commons.mail.Email;
import org.apache.commons.mail.EmailException;
import org.apache.commons.mail.SimpleEmail;

public class EmailTest
{
    public static void main(String[] args) throws EmailException
    {
        Email e = new SimpleEmail();
        e.setHostName("localhost");
        e.setFrom("[email protected]");
        e.addTo("[email protected]");
        e.setSubject("Test " + System.currentTimeMillis());
        e.setMsg("Message");
        e.send();
    }
}

и это мой аспект:

public aspect EmailAspect
{
    private long start;

    pointcut conn() : call(* javax.mail.Transport.send(..));

    void around() : conn() {
        start = System.currentTimeMillis();
        Throwable t = null;
        try {
            proceed();
        }
        catch(Throwable _t) {
            t = _t;
        }
        long spent = System.currentTimeMillis() - start;
        System.out.println("Send time: " + spent);
        if (t != null) {
            throw new RuntimeException(t);
        }
    }
}

Однако, когда я скомпилирую его с помощью

java -jar ../lib/aspectjtools-1.6.11.jar -cp "$CLASSPATH" -source 6 Email*{java,aj}

Я получаю следующее предупреждение:

/home/batto/work/ajtest/test/EmailAspect.aj:8 [warning] advice defined in EmailAspect has not been applied [Xlint:adviceDidNotMatch]


1 warning

И, конечно, аспект не работает. Я попробовал до()/after() советы, разные шаблоны в call(), но все это получило одно и то же предупреждение.

Я отлаживал программу в Eclipse (с источниками commons-email), и я знаю, что Transport.send() выполняется.

В чем проблема?

Спасибо.

UPDATE

Итак, я только что понял, что мне нужен исходный код javax.mail.Transport для используемого метода ткачества (сплетение источника). Поэтому я использовал бинарное ткачество, и это сработало. Однако я передумал, и теперь хочу измерить время java.net.Socket.connect(..) (org.apache.commons.mail.Email.send(..) вызывает javax.mail.Transport.send(..), который вызывает java.net.Socket.connect(..), я проверил его путем отладки в Eclipse). Но я не хочу включать весь JDK в свой скомпилированный код. Поэтому я попробовал ткачество во время загрузки. Это мой модифицированный аспект:

public aspect EmailAspect {
    // Some day I'll learn what the difference between call() and execution()
    pointcut conn() :
        call(* java.net.Socket.connect(..)) ||
        execution(* java.net.Socket.connect(..));

    before() : conn() {
        System.out.println("It works");
    }
}

И эти шаги я сделал для компиляции и запуска программы:

$ mkdir META-INF
$ cat >META-INF/aop.xml
<aspectj>
  <aspects>
    <aspect name="EmailAspect"/>
    <!-- I think this is not neccessary -->
    <include within="java..*"/>
    <include within="javax..*"/>
  </aspects>
  <weaver>
    <include within="java..*"/>
    <include within="javax..*"/>
  </weaver>
</aspectj>
$ # add jars from ../lib to CLASSPATH
$ javac -source 6 EmailTest.java
$ java -jar ../lib/aspectjtools-1.6.11.jar -cp ".:$CLASSPATH" -source 6 EmailAspect.aj
$ java -cp ".:$CLASSPATH" -javaagent:../lib/aspectjweaver-1.6.11.jar EmailTest

Но это не работает: (.

4b9b3361

Ответ 1

Итак, я только что рассмотрел проблему с ткачеством с загрузкой. Я изменил aop.xml на:

<aspectj>
  <aspects>
    <aspect name="EmailAspect"/>
  </aspects>
  <weaver options="-verbose -Xset:weaveJavaxPackages=true -Xset:weaveJavaPackages=true">
  </weaver>
</aspectj>

Я нашел ответ здесь https://bugs.eclipse.org/bugs/show_bug.cgi?id=149261#c11 (а также среда выполнения AspectJ сообщила, что мне нужно предоставить -Xset: weaveJavaxPackages = true вариант все время, я был слишком тупым:().

Ответ 2

Быстрая проверка документации apache, кажется, показывает, что ни Email, ни SimpleEmail не являются экземплярами Transport. Измените pointcut на:

pointcut conn() : call(* SimpleEmail.send(..));