一次群体code review

博客园的Jeffrey Zhao的一个帖子里列出了一段有问题的代码。

群众的眼睛是雪亮的。区区半天，就发现了这么多问题。

1. 注释和代码不匹配
2. 参数未检查
3. 浮点数精度问题
4. 返回对象可能为原对象也可能为新建对象的不确定性
5. 极端情况下可能产生图像宽高为零的异常。

这里是老赵的总结

也许这就能体现出开源和闭源的区别吧。闭源虽然是全职人员开发，但质量不一定有开源高，毕竟一个popular的开源项目的代码经过很多人审阅，而一段商业项目的代码能看到的人毕竟有限。仅凭有限的dev/test,要考虑这么多问题也不是很容易。