一次代码Review的总结

引子:

手上的虽然是一个小项目,但由于关联性很多,而且是系统服务,如果代码出了问题可能会影响整个系统,所以压力还是挺大,也专门找了老外来帮着Review代码。

虽然做了很长时间的C#,但代码是完全的 C + Windows API,对自己来说,还完全是个新手,也顺便弥补了自己在这方面的空白。


记录一下吧,以备后看:

  1. 每一个系统函数的调用都应该检查返回值,因为理论上都有失败的可能,包括CreateEvent是否失败,CloseHandle的对象是否为空,并做相应的处理。
  2. InitialzieCriticalSection必须要和DeleteCriteicalSection配对使用,并且要确保执行顺序和时机。最好放在程序的开头。
  3. 注册表键值建议用const声明。
  4. 常量值建议用const声明。
  5. 大小写BOOL和bool要区别开。
  6. 大小写TURE和ture也要区别开。
  7. 内核事件内不要写业务逻辑,只发送消息或改变信号。
  8. 如果是手动的事件,在WaitForSingleObject以后要记得手动重置。
  9. 如果Event的等待时间是INFINITE要记得如果不触发事件,会永远等待。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值