楼主

6572发表于 2019-11-28 14:04:05
只看该作者楼主

【综合】代码可以再简单点吗(1) -- 校验mac容限模板数量 [复制链接]

作者:HW-ZWT


作为committer,看代码就是做日常。最大的感触,就是代码其实可以更简单。

编码也需要练习基本功,就像学拳需要扎马步一样。希望通过一些实际的案例,讲讲我认为简单的代码是什么样的。


需求

由于芯片限制,只支持 4 个mac地址容限模板。0512 作为默认模板必须支持,用户可自由配置的只有 2 个,需要在配置流程中校验mac地址容限模板的数量。


原始代码

1.png


2.png


3.png

添加和删除函数暂且不说,总体还算简单。

但CheckElanMaxSizeTemplate,我第一感觉就是晕了,看不懂。

我傻傻的看了五分钟,老是走神,静不下心来,担心判断不完备。

没办法,给开发同事打了一个电话,更细致的了解场景。通过线下交流,我建议她重写这段代码,使用更简单的方法。


重构后的代码

4.png

虽然这个PR修改点不止一个,但这里重点是展示CheckElanMaxSizeTemplate的重构。

现在这个函数,即使没有那段注释,各位看官应该也很容易理解这段代码了吧。


举报
发表于 2019-11-28 16:43:24
只看该作者沙发

赞赞赞赞

举报
发表于 2019-11-28 16:43:32
只看该作者板凳

这么好的帖子顶起来呀

举报

您需要登录后才可以回帖

登录注册
发表回复