好的提高代码质量的方法有哪些?
转载自:https://www.zhihu.com/question/20017545/answer/1364963057
从一个故事说起 —— 王国的工匠师
在一个古老的王国里,有两个建筑工匠——风师与土师。就实操经验而言,他们的技艺相差无几。差别在于,风师以“效率”闻名,若你要修一栋房子,找他仅需一月时间便可完成;土师以“稳定”闻名,他总是将东西归置整齐后,方才破土动工,风师只需一月完成的工程,土师需要三个月。
风师有一个大麻袋,这是他的百宝箱,他所有的工具都在里面。他把锤子命名为 1 号,斧子命名为 2 号。这样他工作时,只需叫出简短的名字:1 号拿来固定,2 号拿来伐木……用完之后,将所有工具收回麻袋。风师总是背着麻袋来,背着麻袋去。不拘泥于形式,不管哪里都潇洒走一回。风师常说:这就是“效率”的秘诀。
土师有非常多的工具箱,每种工具都放在自己的类别里。他从不给工具起别名或简称,锤子便叫锤子,斧子便叫斧子,甚至更为详细。向下又细分为羊角锤、检验锤、开山斧、板斧等等。土师开工前总是兴师动众,但他的工作从未出过大的纰漏,倒也令人钦佩不已。
风师每天总是业务繁忙,一个接一个的工程甚至排到了明年。百姓修房子,就图一个“快”字。相比之下,土师的生意就显得门可罗雀了。经常是在风师的活接不过来时,才有人请土师来帮忙建造。久而久之,人们已经默认风师的技艺高过土师,大户人家动工时,都以请到风师为荣,又因为风师总是背着他的大麻袋,坊间便尊称其为“金麻袋”。风师赚得盆满钵满,已经成为王国的小富人家了。但土师仍然是不慌不忙,细致地做着自己的工作。
王国几十年来风调雨顺,国库日渐充盈。国王大喜,这日,国王决定扩建皇宫。百官向国王举荐了风师与土师。
国王请来两位工匠,命其各修建一座凉亭,作为考验。
风师扛来他的大麻袋,准备好建材后,风风火火开始修建。10 号和砂石,15 号打桩……风师操作行云流水,但由于他的工具编号只有自己才熟悉,其他人很少能帮上忙。国王看着风师的操作,渐渐皱起了眉头。
土师将其工具箱一一打开,当他的工具都准备好时,风师已经开始打地基了。然后土师才开始建造凉亭。他指挥着建筑工用砂浆机混合砂浆,龙门架起吊……看着土师有条不紊的指挥,国王眉头方才渐渐舒展开,眼中露出赞许之色。
最终土师仍然使用了三倍于风师的时间方才完成凉亭的建造,但国王决定,任命土师为皇宫建造的御用工程师。
百官不解,只听国王说到:风师、土师的技艺巧夺天工,都是王国的能工巧匠。但风师做事随性,工具杂乱。皇宫扩建工程浩大,当人员逐渐增加,此法必乱。相比之下,风师只可偏居一隅,土师方可担此大任。
后来土师的表现恰好印证了国王的话,在土师的指挥下,皇宫的扩展只用了一年的时间。新建的皇宫恢弘大气,安全可靠。百姓在震惊之余,方才想起:由土师经手的项目,无论大小,似乎从未超过一年。风师虽以快闻名,但负责一个中等建筑时,常常会延期至两年左右。负责小型建筑尚可,当格局扩大后,土师才是最有“效率”的人。
故事的寓意很简单,工程质量与效率离不开好的建筑习惯。当我们负责一个小型项目时,我们追求速度,力求快速出成果,这时可以率性而为。当项目逐渐扩大,规范就会逐步显出它的重要性。
在软件开发中也是一样,干净的地板能减少事故发生,归置到位的工具能提升生产力。软件质量,不但依赖于架构及项目管理,而且跟代码质量息息相关。代码质量与其整洁度成正比。干净的代码,既在质量上较为可靠,也为后期维护、升级奠定了良好的基础。
破窗理论与童子军军规
犯罪心理学中有一个破窗理论,以一幢有少许破窗的建筑为例,如果那些窗不被及时修理好,可能会有破坏者破坏更多的窗户。最终他们甚至会闯入建筑内,如果发现无人居住,也许就在那里定居或者纵火。一面墙,如果出现一些涂鸦没有被清洗掉,很快地,墙上就布满了乱七八糟、不堪入目的东西;一条人行道有些许纸屑,不久后就会有更多垃圾,最终人们会理所当然地将垃圾顺手丢弃在地上。
在编程中也经常出现这样的问题,当我们接手一份混乱的代码后,如果不及时加以整理,最终将会导致代码越来越混乱。这是一种基础的价值谜题,之前的混乱与期限的压力让开发者继续制造混乱。当我们 review 代码时,听到最多的辩解就是:前一个人就是这么写的。事实上,问题的症结在于:他们没有花时间让自己做得更快。
美国童子军有一条简单的军规:让营地比你来时更干净。
代码整洁之道
清理代码也许只是改好一个变量名,拆分一个有点过长的函数,消除一点点重复代码,清理一个嵌套 if 语句。当梳理代码时,坚守此军规:每次 review 代码,让代码比你发现它时更整洁。
一、谨慎命名
给函数和变量取个好名字是优秀程序员的基本功,取名的基本要求是 名副其实,见文知意。如果名称需要注释来补充,那就不算是个好名字。
取名的第二个要求是避免误导。比如数据本身不是一个 list,那就别用 list 来命名,因为 list 一词对程序员有特殊的含义。
取名的第三个要求是去掉冗余。Variable一词永远不应当出现在变量名中,Table一词永远不应当出现在表名中。nameString 会比 name 好吗?ProductInfo 、 ProductData 和 Product 有什么区别?更糟糕的是,如果代码中同时存在 Article 和 ArticleInfo 类,程序员怎么知道该调用哪个类呢?多个意义含混的冗余词汇只会让阅读者困惑,要区分名称,就要以读者能鉴别不同之处的方式来区分。
例如:开始你用了 address 变量表示用户居住地,如上海、北京。后来又要求更详细地描述用户的居住地。如上海黄浦区、浦东区,北京海淀区、朝阳区。用什么来命名这个详细地址呢?detailAdress?smallAddress?还是 anotherAddress?这些统统都不是好的做法,牢记上述法则:以读者能鉴别不同之处的方式来区分,这时比较好的做法是修改之前的 address 变量名字为 city,再将区域的地址命名为 district。
二、函数和类
函数和类应该坚持 单一权责原则。保持高内聚,低耦合。隔离会让系统每个元素的理解变得容易。
过长的函数会造成不易理解,如果某天这个函数需要修改的话,一个长长的函数会大大增加理解成本。并且,小函数也能更好地复用。
如果一个函数做了多件事,一个明显的标志是无法为它起一个精准的名字。你会觉得需要函数名需要使用 and 连接,比如 calculateAndPrintPrice,这时候最佳做法是将其拆分为 calculatePrice 和 printPrice 两个小函数。
函数的第二个规范是 尽量不要在参数中传递状态值,状态值是函数做了多件事的明显标志。例如:
fun setLoading(status: Boolean) {if (status) {loading.VISIBILITY = View.VISIBLE} else {loading.VISIBILITY = View.GONE}
}
修改为:
fun showLoading() {loading.VISIBILITY = View.VISIBLE
}fun hideLoading() {loading.VISIBILITY = View.GONE
}
函数的第三个规范是 同一个函数中的代码应该属于同一层级。良好的软件设计要求分离位于不同层级的概念,较低层级概念和较高层级概念不应混杂在一起。
fun print() {printTitle()// 打印详情System.out.println(“details:”)System.out.println(“It’s a simple sample”)
}
修改为:
fun print() {printTitle()printDetails()
}fun printDetails() {System.out.println(“details:”)System.out.println(“It’s a simple sample”)
}
三、坏注释与好注释
注释并不是越多越好,有的注释纯属无意义的废话。但,最好的注释是没有注释,若代码足够有表达力,用代码来展示意图往往会更好。注释总是一种失败。当我们无法找到不用注释就能表达自我的方法时,我们写了注释,这并不值得庆贺。因为注释常常会撒谎。原因很简单:程序员不能坚持维护注释,尤其是别人写的注释。当另一个人修改了代码后,往往不会去阅读上一个人写的注释,再修改注释。所以注释常常会与其所描述的代码分割开来,孑然飘零,越来越不准确。
写注释的常见动机之一是原有代码混乱。当我们阅读代码时,发现已有的代码令人困扰、乱七八糟。这时我们也许会告诉自己:“等我阅读清楚后,给它写点注释!”别那样做!最好的做法是把代码整理干净。
四、良好的格式
在我们阅读报纸时,在顶部,你期望有个头条,告诉你故事的主题。然后第一段是整个故事的大纲,给出粗线条概述,但隐藏了故事细节。接着读下去,细节渐次增加,直至你了解所有的细节。
代码的格式也要像新闻文章一样,最顶部给出高层次概念,向下渐次展开细节。函数应该紧跟调用处,保证垂直方向上的靠近。如果格式混乱,读者在阅读时总会滑上滑下,导致思维跳跃,增加不必要的理解难度。
影响格式的第二个要素是 缩进与间隔,现代化的 IDE 都有格式化代码快捷键,你也可以在设置中搜索"Reformat Code",自定义格式化代码快捷键。随时格式化,并去掉多余的空行,让我们的代码保持清爽、整洁。
五、数据结构
在有的源代码中,作者采用长长的链式调用,甚至会鼓吹自己只写了一行代码便实现了此功能。事实上,绝不应该为了节省变量,写过长的链式调用,否则容易造成"火车失事"。正确的做法是遵守“得墨忒定律”:适当拆解链式调用,只和朋友谈话,不和朋友的朋友谈话,使得代码阅读和调试都更方便。
六、错误处理
在处理程序异常时,我们常常会用到 try / catch 代码块,而 try / catch 代码块丑陋不堪,他们搞乱了代码结构,把错误处理与正常流程混为一谈。最好的做法是 把 try 和 catch 代码块的主体部分抽离出来,另外形成函数。错误处理就是一件事。
现代化的语言都有异常机制,异常情况如果不及时加以处理,可能会导致程序崩溃。所以有的程序员对于程序的异常情况会选择返回 0 或者 -1 等错误码,以保持程序不崩溃。
别这样做,正确的做法是将错误码替换为抛出异常,只有这样才能保证出现错误时立马就可以发现,而不是让程序在错误的状态下继续执行,将来造成更加迷惑的错误。
要讨论错误处理,就一定要提及那些容易引发错误的做法。第一项就是返回 null 值。我不想去计算曾经见过多少几乎每行代码都在检查 null 值的应用程序。返回 null 值基本上就是在给自己增加工作量,也是在给调用者添乱。只要有一处没有检查 null 值,应用程序就会失控。返回 null 不如抛出 NullPointerException ,或是替换为一个 空对象。让调用者不再需要检查 null,代码也就更整洁了。
提高代码质量的具体经验
永远不要复制代码
不惜任何代价避免重复的代码。如果一个常用的代码片段出现在了程序中的几个不同地方,重构它,把它放到一个自己的函数里。重复的代码会导致你的同事 在读你的代码时产生困惑。而重复的代码如果在一个地方修改,在另外一个地方忘记修改,就会产生到处是bug,它还会使你的代码体积变得臃肿。测试你完成的代码
你知道你的代码能做什么,而且试了一下,它确实好用,但你实际上需要充分的验证它。分析所有可能的边界情况,测试在所有可能的条件下它都能如期的工作。如果有参数,传递一些预期范围外的值。传递一个null值。如果可能,让同事看看你的代码,问他们能否弄坏它。单元测试是到达这种目的的常规方法。代码审查
提交你的代码之前,找个同事一起坐下来,向他解释你做了哪些修改。通常,这样做的过程中你就能发现代码中的错误,而不需要同事说一句话。这比自己审查自己的代码要有效的多得多。编写不言自明的代码
勿庸置疑,注释是编程中很重要的一部分,但能够不言自明的代码跟胜一筹,因为它能让你在看代码时就能理解它。函数名变量名要慎重选择,好的变量/方法名字放到语言语义环境中时,不懂编程的人都能看懂。不要使用纯数字
直接把数字嵌入代码中是一种恶习,因为无法说明它们是代表什么的。当有重复时更糟糕——相同的数字在代码的多个地方出现。如果只修改了一个,而忘记了其它的。这就导致bug。一定要用一个命名常量来代表你要表达的数字,即使它在代码里只出现一次。不要做手工劳动
当做一系列动作时,人类总是喜欢犯错误。如果你在做部署工作,并且不是一步能完成的,那你就是在做错事。尽量的让工作能自动化的完成,减少人为错误。当做工作量很大的任务时,这尤其重要。
7、不要试图死磕代码加快速度,找个更加有效的算法可能更加有效。
8、代码要先做对,在弄快。先使其可靠,再让其更快。先把代码弄干净,再让它变快
9、当发现一个函数具有以下特征时,需要考虑抽取函数
(1)、过长
(2)、嵌套层数过深。
(3)、自然分块,需要使用注释描述该程序块
(4)、判断条件过于复杂
(5)、函数的某些判断分支不断变化
(6)、参数过于复杂
(7)、逻辑重复
10、局部变量应当用途单一
11、程序员应当将整洁的代码风格作为一种习惯,时刻意识到整洁代码的重要性并不断地提高重构技巧
用Sonarqube可以改善代码风格,也容易找出一些简单的bug。
好的提高代码质量的方法有哪些?相关推荐
- Python心得--如何提高代码质量
前些日子用python基于prometheus开发了一个vsphere volume卷监控的exporter,于是跟vsphere的api(pyvmomi)接口打上了交道,开发的过程中你会发现pyvm ...
- Java基础学习总结(148)——如何提高代码质量
前言 人跟人的能力千差万别,所以写出来的代码质量,肯定是不同的.有的人,写一个小逻辑,可能需要100行,而有的人,可能仅仅需要10行.代码永远会有Bug,在这方面没有最好只有更好.模块化与面向对象是实 ...
- 良好的编码习惯 —— 5 个提高代码质量的技巧
原文地址:Good Coding Practices – Five Tips to Enhance Code Quality 原文作者:Jay 译文出自:掘金翻译计划 本文永久链接:github.co ...
- 提高代码质量:如何编写函数
原文地址:http://luopq.com/2016/02/21/write-good-function/ 函数是实现程序功能的最基本单位,每一个程序都是由一个个最基本的函数构成的.写好一个函数是提 ...
- 一堂如何提高代码质量的培训课【转】
今天这堂培训课讲什么呢?我既不讲Spring,也不讲Hibernate,更不讲Ext,我不讲任何一个具体的技术.我们抛开任何具体的技术,来谈谈如何提高代码质量.如何提高代码质量,相信不仅是在座所有人苦 ...
- 提高代码质量 CheckStyle FindBugs PMD
提高代码质量-工具篇 注:这是一篇翻译文章,原文:How to improve quality and syntax of your Android code,为了理解连贯,翻译过程中我修改了一些陈述 ...
- 使用Lint检查提高代码质量
使用Lint检查提高代码质量 1.概述 2.代码中使用标记 2.1 概述 2.2 在工程中使用标记 2.3 一些标记的使用 2.3.1 Nullness标记 2.3.2 资源标记 2.3.3 线程标记 ...
- 如何提高代码质量,或者说高质量代码的特征是什么
高质量代码的三大要素: 可读性.可维护性和可变更性!! 做好代码规范.提高代码质量,能显著增强代码的可读性.可维护性和可变更性.努力提高代码的读写可维护性,是做好代码规范的必要非充分条件.代码规范和架 ...
- 使用GitHub Actions通过CI提高代码质量
不论是开发.暂存还是生产环境,无时无刻都有代码不间断地被推送到 Git 上. 我们总是想要确保我们投入大量时间设计和编写的代码是具备可读性与安全性的,并且没有漏洞,能够平稳地运行. 使用自动化可以节省 ...
- FindBugs,第 1 部分: 提高代码质量
http://www.ibm.com/developerworks/cn/java/j-findbug1/ http://www.ibm.com/developerworks/cn/java/j-fi ...
最新文章
- android uinput 按键_linux 虚拟输入设备(uinput)模拟鼠标和键盘的使用方法
- ORA-600[4194]/[4193]解决
- CentOS6.5下搭建SVN服务器
- 挑个很吉利的日子,开通我的博客!
- Spring系列(十):@Autowired 和@Resource注解用法介绍
- 嘿!你的“苹果”已经被盯上啦
- 跨域解决方案CROS最简单演示——JSP演示示例
- 测绘——AutoCAD教育版打印戳去除
- 黑苹果驱动_兼容黑苹果macOS Catalina系统的USB无线网卡型号及驱动下载地址
- 大数据十大核心原理(互联网上整理)
- linux 添加raid0驱动,网众linux添加新raid驱动.doc
- 深入理解加载FBX模型文件
- 做引流的方法:真实案例短视频如何涨粉的秘密
- 两个jquery 类似igoogle的portlets插件
- 涨握在线|欧或重启QE;英市港市合并!
- Quirks(怪癖)模式是什么?它和Standards(标准)模式有什么区别
- 一种绝对提高开发水平的方法(转)
- 通配符的匹配很全面, 但无法找到元素 ‘aop:aspectj-autoproxy‘ 的声明
- 好文:练习一万小时成天才?(by同人于野)
- c语言第三章程序设计实训
热门文章
- PowerDesigner如何自定义报表模板
- 零基础21天搞定Python分布式爬虫_分布式网络爬虫入门进阶视频教程
- Python+Selenium程序执行完,浏览器自动关闭问题
- java 仙剑奇侠传_仙剑奇侠传-繁体版
- 小米手机安装欧洲版系统(MIUI12) 详细安装教程
- win10怎么添加计算机共享的打印机,win10如何添加打印机共享?
- 台式计算机cpu扣不下去,台式电脑CPU反应太慢了!是怎么回事导致?有什么方法解决?...
- 融新聚力,筑梦畅行|云畅科技“融云计划”第一期集训营圆满结营
- python学习笔记--3.函数
- IDEA 一直Updating indexes问题解决