代码审核

JustYY.com 小赖子的英国生活和资讯

10 years ago

再NB的程序员也有可能写出很垃圾的代码,特别是没睡醒的情况下.所以代码审核就显得尤其的重要. 当一个开发项目即将完工或者一个BUG修复时,尽管已经通过了测试,但是还是需要有一些人专门针对上传的代码进行审核 (Review).

有些代码是一般不会造成问题, 但是却有可能会效率低下,比如:

有意思的代码很有意思的想法

很有意思,有一同事写了这样的代码,其实本意是想把一个角度限制在 0 到 360 度之间,但是是浮点角度,也许他认为不知道怎么对浮点数取余,所以就这样设计代码,不停的减,不停的加360度, 至到在范围内.

且不说这样做的效率,浮点数不断多次的运算会造成误差.如果这个角度极大或极小,则这两个循环则非常的费时.这是有可能测试人员无法测出来的,不经过代码审核是不会发现问题的.

C#里是支持浮点取余的(不过得注意符号,负数的取余也是负数), 即使不支持,也可以通过:

double fmod(double a, double b) {
   return a - b * (int)(a / b);
}

想起了儿子很喜欢的一首英文儿歌: “The wheels on the bus go round and round”

微信公众号: 小赖子的英国生活和资讯 JustYYUK