8

注釈付きフィールドでのパブリックメソッドの実行に一致するポイントカットを記述したいと思います。しかし、これはうまくいかないようです。get(@Important)は(それ自体で)期待どおりに機能しますが、もちろん、フィールドへのすべてのアクセスと一致します。これをパブリックメソッドの実行のみに制限したいと思います。

これは可能ですか?コンパイルエラーは発生しませんが、動作しないようです。


public class Counter {
  private int count = 0;

  public void add(int value) {
    count = count + value;
  }
}

public class Visitors {
  @Important
  Counter counter = new Counter()

  public void increaseCounter() {
    counter.add(1);
  }
}

作品:

@Pointcut(value = "get(@Important * *)")
void testPointCut() {
}

動作しません:

@Pointcut(value = "get(@Important * *) && execution(public * *(..))")
void testPointCut() {
}

4

2 に答える 2

2

オブジェクトのメソッド実行をインターセプトすると、それらのオブジェクトを指す可能性のある注釈付きフィールドへの接続がないため、すぐに使用できる AspectJ ソリューションはありません。注釈付きクラスまたは注釈付きメソッドのメソッド実行をインターセプトする方が簡単ですが、これはあなたがしたいことではありません。

以下は、回避策とその制限を示す小さなコード例です。

import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;

@Retention(RetentionPolicy.RUNTIME)
public @interface Important {}
public class Counter {
    private int count = 0;

    public void add(int value) {
        count = count + value;
    }

    @Override
    public String toString() {
        return super.toString() + "[count=" + count + "]";
    }
}
public class Visitors {
    @Important
    Counter counter = new Counter();

    public void increaseCounter() {
        counter.add(1);
    }

    public static void main(String[] args) {
        Visitors visitors = new Visitors();
        visitors.increaseCounter();
        visitors.counter.add(3);
        System.out.println("visitors.counter = " + visitors.counter);
        System.out.println("--------------------");

        Counter unimportantCounter = new Counter();
        unimportantCounter.add(11);
        unimportantCounter.add(22);
        System.out.println("unimportantCounter = " + unimportantCounter);
        System.out.println("--------------------");

        unimportantCounter = visitors.counter;
        unimportantCounter.add(5);
        System.out.println("visitors.counter = " + visitors.counter);
        System.out.println("unimportantCounter = " + unimportantCounter);
        System.out.println("--------------------");

        visitors.counter = new Counter();
        visitors.increaseCounter();
        visitors.counter.add(3);
        unimportantCounter.add(100);
        System.out.println("visitors.counter = " + visitors.counter);
        System.out.println("unimportantCounter = " + unimportantCounter);
        System.out.println("--------------------");

        Visitors otherVisitors = new Visitors();
        otherVisitors.increaseCounter();
        otherVisitors.counter.add(50);
        System.out.println("otherVisitors.counter = " + otherVisitors.counter);
        System.out.println("--------------------");

        otherVisitors.counter = visitors.counter;
        System.out.println("visitors.counter = " + visitors.counter);
        System.out.println("otherVisitors.counter = " + otherVisitors.counter);
        System.out.println("--------------------");

        otherVisitors.counter = new Counter();
        visitors.increaseCounter();
        otherVisitors.increaseCounter();
        System.out.println("visitors.counter = " + visitors.counter);
        System.out.println("otherVisitors.counter = " + otherVisitors.counter);
    }
}
import java.lang.reflect.Field;
import java.util.HashMap;
import java.util.HashSet;
import java.util.Map;
import java.util.Set;

import org.aspectj.lang.Signature;
import org.aspectj.lang.SoftException;

public aspect ImportantMethodInterceptor {
    Map<Object, Set<Object>> importantObjects = new HashMap<Object, Set<Object>>(); 

    pointcut importantSetter(Object newValue, Object target) :
        set(@Important * *) && args(newValue) && target(target);
    pointcut unimportantSetter(Object newValue, Object target) :
        !set(@Important * *) && set(* *) && !withincode(*.new(..)) && args(newValue) && target(target);
    pointcut publicMethod(Object target) :
        execution(public * *(..)) && target(target) && !execution(public String *..toString());

    before(Object newValue, Object target) : importantSetter(newValue, target) {
        Object oldValue = getFieldValue(thisJoinPoint.getSignature(), target);
        System.out.println("Important object for target " + target + ": " + oldValue + " -> " + newValue);
        synchronized (importantObjects) {
            Set<Object> referrers;
            if (oldValue != null) {
                referrers = importantObjects.get(oldValue);
                if (referrers != null) {
                    referrers.remove(target);
                    if (referrers.size() == 0)
                        importantObjects.remove(oldValue);
                }
            }
            if (newValue != null) {
                referrers = importantObjects.get(newValue);
                if (referrers == null) {
                    referrers = new HashSet<Object>();
                    importantObjects.put(newValue, referrers);
                }
                referrers.add(target);
            }
        }
    }

//  before(Object newValue, Object target) : unimportantSetter(newValue, target) {
//      Object oldValue = getFieldValue(thisJoinPoint.getSignature(), target);
//      System.out.println("Unimportant object for target " + target + ": " + oldValue + " -> " + newValue);
//  }

    before(Object target) : publicMethod(target) {
        synchronized (importantObjects) {
            if (importantObjects.get(target) != null)
                System.out.println("Important method on " + target + ": " + thisJoinPointStaticPart);
            else
                System.out.println("Unimportant method on " + target + ": " + thisJoinPointStaticPart);
        }
    }

    private Object getFieldValue(Signature signature, Object target) {
        try {
            Field field = signature.getDeclaringType().getDeclaredField(signature.getName());
            field.setAccessible(true);
            return field.get(target);
        }
        catch (Exception e) { throw new SoftException(e); }
    } 
}

ご覧のとおり、私のアスペクトは一連の「重要なオブジェクト」を保持しています。より正確にはMap、キーが「重要なオブジェクト」であり、値がリファラーのセットである です。これが必要なのは、理論的には複数のリファラー (Visitorsオブジェクトなど) が同一の「重要なオブジェクト」 (特定の など) を指す可能性があるためですCounter。「重要なオブジェクト」を単純なセットに記録したばかりの以前のバージョンのサンプル コードでは、以前の「重要なオブジェクト」が参照されなくなった場合でもセットから決して削除しないか、常に削除するかのいずれかを選択できました。 2 番目のリファラーがまだ「重要なオブジェクト」を指している場合。マップ アプローチにより、「重要なオブジェクト」ごとに複数のリファラーを記録できます。

を実行Visitors.main(String[])すると、次の出力が表示されます (before ... : unimportantSetter ...ログ出力をさらに表示したい場合は、アドバイスのコメントを解除してください)。

Important object for target Visitors@1404536: null -> Counter@7fdcde[count=0]
Unimportant method on Visitors@1404536: execution(void Visitors.increaseCounter())
Important method on Counter@7fdcde[count=0]: execution(void Counter.add(int))
Important method on Counter@7fdcde[count=1]: execution(void Counter.add(int))
visitors.counter = Counter@7fdcde[count=4]
--------------------
Unimportant method on Counter@18ac738[count=0]: execution(void Counter.add(int))
Unimportant method on Counter@18ac738[count=11]: execution(void Counter.add(int))
unimportantCounter = Counter@18ac738[count=33]
--------------------
Important method on Counter@7fdcde[count=4]: execution(void Counter.add(int))
visitors.counter = Counter@7fdcde[count=9]
unimportantCounter = Counter@7fdcde[count=9]
--------------------
Important object for target Visitors@1404536: Counter@7fdcde[count=9] -> Counter@1d6096[count=0]
Unimportant method on Visitors@1404536: execution(void Visitors.increaseCounter())
Important method on Counter@1d6096[count=0]: execution(void Counter.add(int))
Important method on Counter@1d6096[count=1]: execution(void Counter.add(int))
Unimportant method on Counter@7fdcde[count=9]: execution(void Counter.add(int))
visitors.counter = Counter@1d6096[count=4]
unimportantCounter = Counter@7fdcde[count=109]
--------------------
Important object for target Visitors@b02e7a: null -> Counter@bb6ab6[count=0]
Unimportant method on Visitors@b02e7a: execution(void Visitors.increaseCounter())
Important method on Counter@bb6ab6[count=0]: execution(void Counter.add(int))
Important method on Counter@bb6ab6[count=1]: execution(void Counter.add(int))
otherVisitors.counter = Counter@bb6ab6[count=51]
--------------------
Important object for target Visitors@b02e7a: Counter@bb6ab6[count=51] -> Counter@1d6096[count=4]
visitors.counter = Counter@1d6096[count=4]
otherVisitors.counter = Counter@1d6096[count=4]
--------------------
Important object for target Visitors@b02e7a: Counter@1d6096[count=4] -> Counter@5afd29[count=0]
Unimportant method on Visitors@1404536: execution(void Visitors.increaseCounter())
Important method on Counter@1d6096[count=4]: execution(void Counter.add(int))
Unimportant method on Visitors@b02e7a: execution(void Visitors.increaseCounter())
Important method on Counter@5afd29[count=0]: execution(void Counter.add(int))
visitors.counter = Counter@1d6096[count=5]
otherVisitors.counter = Counter@5afd29[count=1]

main私がテストした通常のケースと特別なケースを確認するために、コードをログ出力と注意深く比較してください。

前述したように、このアプローチには次のような制限があります。

  • いくつかの無関係な重要なメンバーが同等のオブジェクトを作成するため、重要なフィールドが「重要なオブジェクト」として理論的に複数回発生する可能性のある のintようなプリミティブ型を持っている場合に何が起こるかをテストしていません。Stringまた、オート (アン) ボクシングに関して何が起こるかはテストしていません。自分で試してみてください。
  • アスペクト コードはいくぶん複雑で、おそらく驚くほど高速ではありません。
  • 私がまだ考えていない他の問題がないかもしれないとは保証できません。

しかし、境界条件とユース ケースを管理している場合は、十分な情報に基づいて決定を下し、コードをそのまま使用することも、その変形を使用して、必要なものを達成することもできます。コードにはおそらく改善の可能性があります。私は興味があり、概念実証をハックしたかっただけです。

于 2013-03-17T17:50:30.720 に答える
0

withinCode次のように、ポイントカットを使用します。

@Pointcut(value = "get(@Important * *) && withinCode(public * *(..))")
void testPointCut() {
}

AspectJ プログラミング ガイドをご覧ください。

于 2013-03-01T05:04:47.773 に答える