文档开发者中心
我的

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

作者:HW-ZWT


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

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


需求

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


原始代码



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

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

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

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


重构后的代码


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

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


点赞
收藏
回复
2
分享
举报
浏览932 发布于2019-11-28 06:04未知归属地
全部评论
最多点赞
最新发布
最早发布

这么好的帖子顶起来呀

2楼回复于2019-11-28 08:43 未知归属地

赞赞赞赞

1楼回复于2019-11-28 08:43 未知归属地
写回答
新增插入模板功能
一键使用模板,快速填写内容,轻松发帖~
知道了
  • 为了保障您的信息安全,请勿上传您的敏感个人信息(如您的密码等信息)和您的敏感资产信息(如关键源代码、签名私钥、调试安装包、业务日志等信息),且您需自行承担由此产生的信息泄露等安全风险。
  • 如您发布的内容为转载内容,请注明内容来源。

我要发帖子

目录 收起
需求
原始代码
重构后的代码
了解社区公约,与您携手共创和谐专业的开发者社区。