そんなわけで (どんなわけで？)，クローズ．

そう説明するので、クローズでいいです。

15:38 のコメントを見ないで 16:00 のコメント書いちゃいました．きりのいい時間になるまでリロードしないで置いておいたもので．

> AOPの実装によっては、FooにtoString()を追加する可能性もあるので、微妙な話になりそうな気が・・・

だから，最初のコメントで「AOP をどのように実現しているかによって getTargetClass() の実装を差し替え可能にする必要が」あると書いたわけ．対象のクラスを求める方法は AOP の実装次第ですよね？ そして AbstractInterceptor の実装は S2AOP が前提になっているわけです．

> も、イヤですか？

いやです．あまりにアドホックすぎるし意味不明．
だいたい，

public class Foo {}
public class Bar extends Foo {}

Bar の toString() メソッドに AOP を適用した場合，サブクラス化しない AOP 実装だと getTargetClass() が Foo.class を返すことになりませんか？
不都合なケース (別の AOP 実装 + static メソッド) のことだけ考えて，他のケース (S2AOP やインスタンスメソッド) が考慮されてないように見えます．

> S2Javelinを使いたかった

そちらで AbstractInterceptor がロードされるときに getTargetClass() をそちらの AOP 実装に合わせて置き換えればいいのでは？

一個前のソースは、無かったことに。。(^^;
クラス構造によって違う取れ方が出来てしまうので。

AOPの実装によっては、FooにtoString()を追加する可能性もあるので、微妙な話になりそうな気が・・・

protected Class getTargetClass(MethodInvocation invocation) {
if (invocation instanceof S2MethodInvocation)

try{
Class thisClass = invocation.getThis().getClass();
Class superClass = thisClass.getSuperclass();
if (superClass == Object.class)

return superClass;
}catch(RuntimeException e) {
return invocation.getMethod().getDeclaringClass();
}
}
も、イヤですか？

> return method.getDeclaringClass();
> としてもらうだけでも助かります。

できません．

public class Foo {}

の toString() メソッドにインターセプタを適用した場合，getTargetClass() は Foo.class を返すべきですが，Method#getDeclaringClass() は Object.class を返します．

S2だけだと、
protected Class getTargetClass(MethodInvocation invocation) {
if (invocation instanceof S2MethodInvocation)

Class thisClass = invocation.getThis().getClass();
Class superClass = thisClass.getSuperclass();
if (superClass == Object.class)

return superClass;
}
の最初のifが処理されると思うので、S2以外も考慮しているのかと思いました。
以下は、考慮されているとしての話、
例外が出て困っているので、インターフェースうんぬんはおいといて、
return method.getDeclaringClass();
としてもらうだけでも助かります。

> S2 以外でも利用したいインターセプタは，AbstractInterceptor を継承しなければいいと思います．
S2Javelinを使いたかったので、対応していただけなければ、別途方法を考えます。

このメソッドが親クラスを取得しているのは，サブクラス化によって AOP を実現する S2AOP に依存しているからであり，S2 以外での動作は想定されていません．
S2AOP 以外もサポートするということだと，AOP をどのように実現しているかによって getTargetClass() の実装を差し替え可能にする必要がありますが，それはあまりに過剰だと思います．

よって対応せずにクローズするに +1．

S2 以外でも利用したいインターセプタは，AbstractInterceptor を継承しなければいいと思います．

AbstractInterceptorで(^^;