我的酒窝

场景一：

个人喜欢state-based的单元测试。（定义见mock is not stub）。

可是有些时候，比如在测试一些使用java.sql.Connection, ibatis SqlMapClient等接口的类的时候，明显写stub很不好。（1，接口很大，有很多不相干的东西。2，版本一旦变化，这些接口可能跟着变化，如果写stub的话，就意味着stub要跟着这些第三方接口变化）

于是，只好mock。只好interaction based。只好每回内部实现一变就死盯着一坨坨的expectAndReturn找不再有效的expectation。

场景二：

一个遗留系统用自己的连接池，使用ConnectionManager.checkIn(Connection)来释放连接(而不是Connection.close)

为了重构，希望能够重载Connection.close()，让它调用checkIn()。

跟场景一的提到的原因一样，不希望直接写一个MyConnection implements Connection。于是只好自己写dynamic proxy，形成如下的代码：

  private Connection conn;
  public Object invoke(Object proxy, Method method, Object[] args){
    if((args==null || args.length==0) && method.getName().equals("close")){
      ConnectionManager.checkIn((Connection)conn);
    }
    else {
      try {
        return method.invoke(conn, args);
      }
      catch(InvocationTargetException e){
        throw e.getTargetException();
      }
    }
  }

后面又发现这个变态的一流系统居然用自己的startMyTransaction, endMyTransaction(), rollbackMyTransaction()来搞事务处理！
于是又要继续判断method.getName().equals("startTransaction"), method.getName().equals("commitTransaction")等等等等。

dynamic proxy的代码难看的要死。



最近终于受不了了。愤然写了一个叫做酒窝的东西(dimple)。

所谓dimple，就是dynamic implementor的意思。我要用纯java语法来实现我想实现的方法（不是expectAndReturn，也不是if(method.getName().equals("method1"))），但是同时我又不愿意写“implements TheInterface”，以避免被迫处理那些我不关心的method。

用法如下：

场景一：

这样实现我的stub：

MyParameter myparam = ...;
SqlMapClient client = (SqlMapClient) Implementor.proxy(SqlMapClient.class, new Object(){
  public Object insert(String id, Object param) {
    assertEquals("test id", id);
    assertSame(myparam, param);
    return null;
  }
});
assertNull(new SomeClassUsingSqlMapClient(client).runSomeInsert(myparam));

没有讨厌的expectAndReturn，不用担心parameter matcher。


场景二：

final Connection realConn = ...;
Connection conn = (Connection) Implementor.proxy(Connection.class, new Object(){
  public void close() {
    ConnectionManager.checkIn(realConn);
  }
  public void startTransaction(){
    ConnectionManager.startMyTransaction(realConn);
  }
  public void commitTransaction(){
    ConnectionManager.commitTransaction(realConn);
  }
}, realConn);

于是，对close(), startTransaction, commitTransaction()，我们都直接调用ConnectionManager, 而对其它的method则委托给realConn。




这个Implementor类是纯java代码，除了dynamic proxy没有用任何其它技术。（没有enhancement，aop之类的）



个人感觉还是挺有用的。大家点评一下？

评论

51 楼 shaucle 2006-12-18  

俺还是很喜欢ajoo的这个，好用极了．．．而且符合30-70原则（还没到20-80),呵呵

50 楼 ajoo 2006-12-16  

场景一：
做stub。只实现一百个方法中的一个或者两个

场景二：
动态地仅仅intercept一个对象，而不是全局地针对一个类。


aspectj怎么做？既然就几行代码，写一下不是很容易？

49 楼 partech 2006-12-16  

俺还就是只想说说。
在interface上定义切点，用Proxy得到代理类，around想要处理的方法。

48 楼 floating 2006-12-16  

aspectJ支持在class loading的时候织入代码，但是还不支持在runtime时纯动态地织入。不过，我觉得ajoo举的这个例子，即使需要点动态的色彩，在class loading时能织入代码也应该可以满足需要了吧？

47 楼 ajoo 2006-12-16  

好啊。show一下aspectj怎么做这两个场景？

46 楼 partech 2006-12-16  

aspectJ也可以动态的，测试本来就是构件的一个方面（aspect），我看不出有什么充分的理由不能用。

45 楼 ajoo 2006-12-16  

aspectj和这个做的不同。它是静态的，而dimple是动态的。
另外你做测试的stub也用aspectj么？

44 楼 floating 2006-12-16  

问一下楼主为什么觉得采用aop的方式不好？用AspectJ来实现这个功能，就很简单的几句话啊。

43 楼 ajoo 2006-12-16  

项目搞定了。

http://dimple.codehaus.org

42 楼 ajoo 2006-12-14  

shaucle 写道

公司里不能上传

主要还是楼主的核心代码，

不过看起来稍有点层次感，提供一起扩展性，不会侵权吧，呵呵

我会用apache 2.0 license。你只要代码里保留我的注释，怎么改都行的。

41 楼 shaucle 2006-12-14  

公司里不能上传

主要还是楼主的核心代码，

不过看起来稍有点层次感，提供一起扩展性，不会侵权吧，呵呵

40 楼 shaucle 2006-12-14  

俺重写了一下

1 提出专门的对外接口.
public class Proxy {
   
    public static Object proxy(Object target, Object interceptor)

    public static Object proxy(Class asType, Object target, Object interceptor)

    public static Object proxy(Class asType, Object interceptor)

2 对于缓存中的Method新用一个对象(MethodKey)作为Key

3 对于dock优先级问题用strategy模式提出来,并提供默认实现(现提供3种实现)

4 添加参数, 可选择是否intercept Object对象中的方法(在上面3个方法中另加3个方法).


俺重写后,共8个类,4个关于dock优先级的strategy类加起来驶200行,4个核心类200行(不过俺没写注释)-_-

39 楼 ajoo 2006-12-14  

直接用getDeclaredMethods不好。这样继承就不能用了，只能用最下面的那个类里面的方法。

我改了一下，现在缺省直接delegate这三个hashCode(), equals(), toString()调用了（如果这个类没有定义这三个方法）。


equals()地实现相当麻烦，需要考虑unwrap dynamic proxy。对cglib尤其困难，我不知道怎么unwrap一个cglib enhanced的对象实例

38 楼 shaucle 2006-12-14  

还有这个，
final Class[] itfs = c.getInterfaces();
for (int i = 0; i < itfs.length; i++) {
int itfDepth = 1 + getHierarchyDepth(itfs[i]);
//这里调用itfs[i].getSuperclass()==null
if (itfDepth > depth) {
depth = itfDepth;
}
}

37 楼 ajoo 2006-12-14  

getDeclaredMethods不好。by-design我就是要拦截equals(), hashCode(), toString()的。

用户如果想delegate很简单，就实现一下delegate就是了。（equals的delegate还要写点if instanceof代码的）

我就在我们的项目中自己实现了一个DelegateObject来负责delegate。
具体类继承一下就是了。

36 楼 shaucle 2006-12-14  

ajoo 写道

shaucle 写道

D d=(D)Implementor.proxy(D.class, new Object());
如果D中实现了如hashCode之类的方法,那你的with(也就是那个new Object())会intercept D的hashCode么?　（cls.getMethods())
不知道你的CglibUtils怎么实现的?
我这边测的会.

会的。
cglib没有大区别，就是允许asType是一个有default constructor的non-final类

getMethods　－＞　getDeclaredMethods？？就行了

35 楼 ajoo 2006-12-13  

shaucle 写道

final Class[] itfs = c.getInterfaces();
        for (int i = 0; i < itfs.length; i++) {
            int itfDepth = 1 + getHierarchyDepth(itfs[i]);
//这里调用itfs[i].getSuperclass()==null
            if (itfDepth > depth) {
                depth = itfDepth;
            }
        }

另外,如果
对于:f(A a, B b)
有如下intercept方法:
f(A2 a, B1 b)//A2中的2代表有2层继承
f(A1 a, B2 b)
那应该intercept哪一个?

没有试图对这个给出定义。估计实际上是第一个f。使用者应该避免这种情况发生。

34 楼 ajoo 2006-12-13  

shaucle 写道

D d=(D)Implementor.proxy(D.class, new Object());
如果D中实现了如hashCode之类的方法,那你的with(也就是那个new Object())会intercept D的hashCode么?　（cls.getMethods())
不知道你的CglibUtils怎么实现的?
我这边测的会.

会的。
cglib没有大区别，就是允许asType是一个有default constructor的non-final类

33 楼 shaucle 2006-12-13  

final Class[] itfs = c.getInterfaces();
        for (int i = 0; i < itfs.length; i++) {
            int itfDepth = 1 + getHierarchyDepth(itfs[i]);
//这里调用itfs[i].getSuperclass()==null
            if (itfDepth > depth) {
                depth = itfDepth;
            }
        }

另外,如果
对于:f(A a, B b)
有如下intercept方法:
f(A2 a, B1 b)//A2中的2代表有2层继承
f(A1 a, B2 b)
那应该intercept哪一个?

32 楼 shaucle 2006-12-13  

getMethods　－＞　getDeclaredMethods？？