posts - 68, comments - 8, trackbacks - 0, articles - 0
  IT博客 :: 首页 :: 新随笔 :: 联系 :: 聚合  :: 管理

新工作学习11:代码Review问题

Posted on 2010-06-25 16:16 Richmond 阅读(166) 评论(0)  编辑 收藏 引用

1、因为这种是特殊逻辑,执行到之后需要输出error信息,以便跟踪问题   
2、
如果发上来的包格式有问题,没有判断.安全的写法是

     Buffer >> info;

     If (!buffer ) return; 或者 if( buffer.good()==false) return;或者判断 wDataLen>sizeof(包结构)

3、处理一个消息码,不需要新增一个接口,因为这个不是对象的主要行为,是边缘的逻辑。每一个消息都新增一个接口那接口不是庞杂无比,并且极其不稳定?因为消息一遍,增加删减接口都要变化.而且像其他的状态也继承了这个接口,但根本没有这个逻辑,也被迫需要实现这个接口,这就是典型的:接口被污染了(仅仅为了实现这个空接口,修改了6个文件)。
只有注册用户登录后才能发表评论。