一次群体code review
博客园的Jeffrey Zhao的一个帖子里列出了一段有问题的代码。
群众的眼睛是雪亮的。区区半天,就发现了这么多问题。
- 注释和代码不匹配
- 参数未检查
- 浮点数精度问题
- 返回对象可能为原对象也可能为新建对象的不确定性
- 极端情况下可能产生图像宽高为零的异常。
这里是老赵的总结
也许这就能体现出开源和闭源的区别吧。闭源虽然是全职人员开发,但质量不一定有开源高,毕竟一个popular的开源项目的代码经过很多人审阅,而一段商业项目的代码能看到的人毕竟有限。仅凭有限的dev/test,要考虑这么多问题也不是很容易。
posted on 2009-11-24 19:47 Michael Peng 阅读(307) 评论(1) 编辑 收藏 举报