代码审查 - 循环避免

2个帖子/ 0新
最后一篇
oren.
离线
最后一次露面:1年7个月前
专家
加入:2014-06-28 22:03
代码审查 - 循环避免

这不是错误,但更多的代码审查建议。
有许多循环迭代Cust_prf_funcs数组,如prf_utils.c:


_ uint8_t i = 0;
_ while(cust_prf_funcs [i] .task_id!= task_none)
_ {
_____如果(cust_prf_funcs [i] .init_func!= null)
_____ {
_________ cust_prf_funcs [i ++]。init_func();
_____}
_____ else i ++;
_}

相同的代码可以以更漂亮的方式编写:


_ for(uint8_t i = 0; cust_prf_funcs [i] .task_id!= task_none; i ++){
_____如果(cust_prf_funcs [i] .init_func!= null)
_____ {
_________ cust_prf_funcs [i] .init_func();
_____}
_}

循环条件不一定是“我<...”!
对于循环的循环不太容易出错,而不是循环。

关键词:
设备:
mt_dialog.
离线
最后一次露面:2个月2周前
职员
加入:2015-06-08 11:34
嗨oren,

嗨oren,

感谢代表SDK团队的代码审查,您的指示将在下一个SDK发布中考虑。

谢谢mt_dialog.