一个值得记住的教训啊

昨天下午实现一段代码,如果一说的话非常简单,非常低级的问题,但当时测试好几次,包括两个人review了,都没有发现问题.

结果集成到应用发布的时候出现BUG了,只好回滚.好在我们的应用都有预发而机制,没有影响线上的真实应用.

场景是这样的:

switch(x)｛

case 1:

case 2:

case 3: // nothing

break;

case 4:

dosomething;break;

case 5:

dosomething;break;

default:

dosomething;break;

｝

昨天要实现case 3的逻辑,是一个URL编码.结果改成:

switch(x)｛

case 1:

case 2:

case 3: // nothing

urlencode(url);

break;

case 4:

dosomething;break;

case 5:

dosomething;break;

default:

dosomething;break;

｝

当时测试的时候只写了个URL传进来,正确输出了.

没想到case 1和case 2的情况穿越到 case 3了,结果是case 1和case 2的字符串都进入了urlencode的逻辑.

这么低级有错误,说出来谁都明白,或者如果整个代码是自己一次性写的,绝对不会犯这种低级错误,但在修改别人原有代码时

却忘记了在上面加个break来阻止穿越.

这样的代码如果考虑到以后别人可能修改和维护,其实不如费点事每个case直接加上break;否则别人很容易在增加处理状态时忘记

上面的穿越.

小心啊,小心................................