作者:HW-ZWT
作为committer,看代码就是做日常。最大的感触,就是代码其实可以更简单。
编码也需要练习基本功,就像学拳需要扎马步一样。希望通过一些实际的案例,讲讲我认为简单的代码是什么样的。
需求
由于芯片限制,只支持 4 个mac地址容限模板。0 和 512 作为默认模板必须支持,用户可自由配置的只有 2 个,需要在配置流程中校验mac地址容限模板的数量。
原始代码
添加和删除函数暂且不说,总体还算简单。
但CheckElanMaxSizeTemplate,我第一感觉就是晕了,看不懂。
我傻傻的看了五分钟,老是走神,静不下心来,担心判断不完备。
没办法,给开发同事打了一个电话,更细致的了解场景。通过线下交流,我建议她重写这段代码,使用更简单的方法。
重构后的代码
虽然这个PR修改点不止一个,但这里重点是展示CheckElanMaxSizeTemplate的重构。
现在这个函数,即使没有那段注释,各位看官应该也很容易理解这段代码了吧。






































苏公安网备 32011402010933号