BT

如何利用碎片时间提升技术认知与能力? 点击获取答案

从“用断言检查null”到“设计单元测试”

| 作者 李剑 关注 1 他的粉丝 发布于 2009年3月14日. 估计阅读时间: 8 分钟 | GMTC大前端的下一站,PWA、Web框架、Node等最新最热的大前端话题邀你一起共同探讨。

防御式编程是保护自己的程序不受外部侵害的一种有效方式。代码大全在“防御式编程”这一章中讲到,可以用断言(assert)来检查真实情况是否满足自己的预期,例如指针非空。

上个月Miško Hevery写了一篇文章:断言,还是不断言,描述了在构造器中编写断言给单元测试所造成的种种不便。

他先举了一个实际例子:

class House {
  Door door;
  Window window;
  Roof roof;
  Kitchen kitchen;
  LivingRoom livingRoom;
  BedRoom bedRoom;

  House(Door door, Window window,
            Roof roof, Kitchen kitchen,
            LivingRoom livingRoom,
            BedRoom bedRoom){
    this.door = Assert.notNull(door);
    this.window = Assert.notNull(window);
    this.roof = Assert.notNull(roof);
    this.kitchen = Assert.notNull(kitchen);
    this.livingRoom = Assert.notNull(livingRoom);
    this.bedRoom = Assert.notNull(bedRoom);
  }

  void secure() {
    door.lock();
    window.close();
  }
}

然后说到,因为secure方法只需要door和window两个对象,所以理所当然的是,在测试这个方法的时候只需要初始化door和window,其他参数用null代替,如: 、

House house = new House(door, window,null, null, null, null);

这样的方法才能够让人看明白意图,但是因为构造器中做了参数非空的断言,所以就不得不这样初始化House对象:

House house = new House(door, window,
    new Roof(),
    new Kitchen(),
    new LivingRoom(),
    new BedRoom());

这样就让人不太能看得懂到底是哪些对象真正在测试方法中被用到了。而当构造器中的参数增多,代码的可读性就会更差。Miško最后提到:

我不是反对断言,我也在自己的代码中也常用,不过我的大多数断言只是用来检查对象的内部状态,而不是是不是传入了一个null值。检查是不是null往往会影响到代码的测试,如果要我在完好测试过的代码和有断言但是没测试的代码之间做选择,结果是不言而喻的。

有不少人跟帖反对Miško的观点,他们认为这根本不是断言的问题,而是来自于设计本身,例如

Rasmus Kromann-Larsen就提出,

你为啥不用mock框架而非要自己初始化那么一堆东西呢?在.NET世界里面有个新词,叫做automocker——你可以把它看作是mock框架和ioc容器的混合体……

看看这个链接吧:http://blog.eleutian.com/CommentView,guid,762249da-e25a-4503-8f20-c6d59b1a69bc.aspx

Howie则说:

不管是我写的代码,还是我看到的代码,都是在调用对象的时候去做断言,而不是在保存对象的时候。这样合理不?所以你应该在 secure()方法里面写Assert.notNull(door);Assert.notNull(window)。用到的时候再去检查,而不是提前 就把一切都检查好,以备过几天以后需要用。

zdsbs提出用Builder来取代构造器:

Door door = new Door();
Window window = new Window();
House house = HouseBuilder.new().with(door).with(window);

house.secure();

assertTrue(door.isLocked());
assertTrue(window.isClosed());
这样问题就很好的解决了,其他变量都可以在HouseBuilder里面提供一些缺省值,你也不会创建出具有非法状态的对象来。

earlNameless对Miško的设计思路直接抨击说:

我不同意你的看法,因为你这个例子里面,人们得先知道你所测试的方法只用到了door和window,没用其他任何东西。

所以你的测试人员就得了解方法的具体实现,而这是不应该的……我如果有个可以滑开的房顶,我就得还给它上锁,可现在房顶是null。

后面有几个人同意earlNameLess的看法:测试代码需要清楚的了解实现代码的内部实现方式实在不是什么好做法。建议根据具体情况提取出接口,或者用一下Ioc容器。

不过也有些人对earlNameless的看法表示反对,例如 Adam Tybor说到:

为了写测试,不就是得知道实现方式么。这是TDD,不是TAD,不是么?如果房顶还需要加锁,那就让测试失败,然后改实现,直到所有测试变绿。

Christian Gruber说:

上面有很多讨论都是针对Misko的测试代码不得不知道实现细节这个做法的。虽然,测试代码了解内部细节是有点不太稳定,但这是单元测试,单元测试是白盒 测试,不是黑盒。如果你用mock,那你肯定得知道内部细节……再说的清楚一点,你是在测试基于给定的条件,你的代码能够得到预期的结果。你必须得知道这 些条件是啥,把那些不满足的条件排除掉。

如果有人来改了实现细节,那测试就会失败。不过这也暴露出一个问题来,你必须得有一整套单元测试可以做回顾。如果我改了行为,我也就修改了测试的意图。然后我就得修改测试,让它对新的假设起作用。

所以这要看你是想测试行为还是输入输出,这已经是设计测试的哲学范畴了……

跑题跑了一阵子以后,rnaufal回到原来话题上,直接抛出了JDK文档:

JDK文档中有一篇的标题是“用断言编程”,链接在这里:“http://java.sun.com/j2se/1.4.2/docs/guide/lang/assert.html#usage”

里面写着:不要在public方法中用断言做变量检查。

读者朋友,你在自己的代码中使用防御式编程么?用过assert来检查null与否么?你的单元测试是在检查行为,还是检查输入输出呢?欢迎分享你们的意见。

评价本文

专业度
风格

您好,朋友!

您需要 注册一个InfoQ账号 或者 才能进行评论。在您完成注册后还需要进行一些设置。

获得来自InfoQ的更多体验。

告诉我们您的想法

允许的HTML标签: a,b,br,blockquote,i,li,pre,u,ul,p

当有人回复此评论时请E-mail通知我

assert有点被滥用了 by lu robbie

在我们的项目中,发现几乎到处都用assert,不管是public的service方法,还是private的方法
个人的建议是
在public的方法中,不要用assert来检查输入参数,在私有方法可以
public的方法,本来就是要面对外部的各种情况,应该给出明确的exception,更好一点.

Re: assert有点被滥用了 by Jacky Li

嗯,JDK的文档也是这么写的

assert无错,关键是用法 by Yang Bob

在一个完整的模块内部的方法调用之间,可以使用断言

断言是很有用,滥用就是不对了 by lai danny

断言是很有用,滥用就是不对了!

Re: assert有点被滥用了 by Andy Yao

如果给出明确的异常,可能会造成异常泛滥。

我一般会封装一个Assert的util类,对输入参数进行判断,然后给出运行期异常。

允许的HTML标签: a,b,br,blockquote,i,li,pre,u,ul,p

当有人回复此评论时请E-mail通知我

允许的HTML标签: a,b,br,blockquote,i,li,pre,u,ul,p

当有人回复此评论时请E-mail通知我

5 讨论

登陆InfoQ,与你最关心的话题互动。


找回密码....

Follow

关注你最喜爱的话题和作者

快速浏览网站内你所感兴趣话题的精选内容。

Like

内容自由定制

选择想要阅读的主题和喜爱的作者定制自己的新闻源。

Notifications

获取更新

设置通知机制以获取内容更新对您而言是否重要

BT