博客链接:https://www.bughui.com/2017/08/21/how-to-write-code/ 作者:Bug辉
写代码就是学一门语言然后开始撸代码吗?看完了我的《GoF设计模式》系列文章的同学或者本身已经就是老鸟的同学显然不会这么认为。编程是一项非常严谨的工作!虽然我们自嘲为码农,但是这工作毕竟不是真正的搬砖,我们是软件工程师。编程需要关注的问题太多,不仅仅有语言,还有算法、数据结构、编程技巧、编码风格、设计、架构、工程化、开发工具、团队协作等方方面面,涉及到很多层面的问题。本文将分享一下根据我这几年来的编程经验总结出的一些关于如何写代码的个人见解。
由于“跟我混”的一些小伙伴编程功底相对来说比较薄弱,所以在此总结一篇“编程内功心法”帮助他们渡过职业生涯的第一个瓶颈期。顺便,也造福一下路过的有缘的同学!于是有了此文。
前言
首先,思考一个问题,何谓编程?编程就是写代码吗?
所谓的编程,其实就是不断的对这个现实世界中的问题建立模型并将其固化为代码自动化执行的过程。
~ Bug辉 《GoF设计模式 - 解释器模式》
在对问题建立模型的过程中,我们会遇到非常多不同层面的问题,所以我们需要很多领域的知识去解决这些问题。
- 我们需要管理被操作的数据,因为数据与数据之前是相互有关联的。将数据结构化,通常是编程的第一步。关于结构化数据的相关理论以及实践,需要有一个专门的学科分支或者说课题去研究——数据结构。
- 我们需要解决一个具体的问题,这个具体的问题如何一步步去解决,过程是怎么样子的——算法。
- 我们需要将解决方案进行自动化,并以代码的形式进行交付——编程语言。
- 如果将一个抽象的模型进行编码实现,如何实现“这个功能”,如何实现“那个功能”——编程技巧。
- 问题的规模大了,众多代码糅合在一起,连程序员自己都看不懂了!怎么来拆分、模块化这些代码——设计。
- 代码量已经到了一个人无法完成的地步了,需要团队分工合作才能完成了——工程化。
- 你写的代码我看不懂,没法调用或者很难调用,我写的代码你也看不懂,或者很难看懂。还怎么愉快的玩耍——编码风格/编码规范。
- 问题的规模继续扩大,到了系统工程的规模了,之前学的套路已经不管用了!怎么来构建这个系统才能实现正确、安全、高性能、高可用——架构。
然而这些也只是一个系统工程中的冰山一角!这是一个庞大的体系。也正是因为软件开发需要考虑到的问题太多且团队成员水品参差不齐,所以团队开发中并不是每个程序员做的事情都是一样的。每个人都有自己的角色、初级工程师、中级工程师、高级工程师、架构师、CTO。。。
所以编程不仅仅只是堆砌代码!
说到这里,我想起来了一件事情——为啥业界普遍鄙视培训出来半道出家的新人?人与人的区别是很大的!我见过培训出来也很牛的。其实,说到底,被鄙视的并不是所有人。而是那些培训了几个月之后发现随便找个工作也能拿“高薪”然后还自认为编程很简单的新人。因为这种经历给了他们一种错觉——编程如此简单,我培训几个月也会嘛!有点像刚学会开车的新司机,很嚣张的对老司机说“开车很简单嘛!你看我也会啊!”。语言和开发工具只是招式,这是外功。而编程思想、经验是内功。这些内功并不是靠短短几个月的培训能够掌握的,这一点有点像中国制造业和日本制造业的区别。动不动赶英超美可不好。。。
编程并不简单!这是一件很严肃的事情。不过今天,我没有办法介绍完所有的方面!或者说,到今天为止,我也并没能掌握所有领域的知识。所以今天我只是分享一些关于编码本身的一些经验。
另外,本文主要分享如何写代码,并不是如何用Java写代码。所以文章中各种语言都有可能出现。
编码风格
先来一个圈内的段子。
大部分程序员在工作中都很讨厌这四件事情:
- 写注释
- 写文档
- 别人不写注释
- 别人不写文档
o(∩_∩)o 哈哈。。中枪了没!
这个段子其实反映出来一个问题,即大部分代码都需要通过大量注释和文档来说明才能将意图传达给维护这些代码的程序员!然而,就像上面的段子说的那样,写大量的注释和文档其实是一件很麻烦的事情。所以很多时候,由于嫌麻烦,注释和文档就没写,导致维护代码的人相当的痛苦。这个苦同学们肯定都是体会过的!相当于给你个精密仪器要你维护还不给说明书。
其实,打破上面那个段子描述的那个怪圈的一个很有效的手段就是统一编码风格。优秀的代码可以实现代码即注释,代码本身就可以非常清晰的体现出它的意图来,让别人可以很容易读懂。这就是所谓的可读性!
命名
计算机科学领域中最难的两件事是命名和缓存失效!命名并不简单,很复杂。好的名字可以见名知意,非常容易理解。之所以说命名难是因为命名的过程同时也是概念提取的过程!对问题建立模型,需要提取概念并赋予其“术语”。这个过程其实是“万里长征”中最难的一步。毕竟,设计也好,架构也罢,都有成熟的套路可以参考。唯独这个过程,是需要程序设计者自己进行充分的思考的创造性工作!
以下是总结出来的一些命名经验:
- 一个类是某物、某事、某人的抽象,是数据与行为的集合体。这恰好符合名词的定义,因此 类名 是一个名词!
- 方法名 或者说 函数名 是某操作或者某过程的抽象,是一个动作。这恰好符合动词的定义,因此函数名通常是一个动词。
- 变量名宁可长一些说明清楚用途也不要用
a
、b
、c
之类的无意义的名称,除非是循环计数器中用i
、j
、k
等约定俗成的一些变量名。比如pageIndex
和pageSize
就要比取名成i
和s
好!取成这种和用混淆器混淆过后的代码一样的名称没有什么好处,如果算法比较复杂的话,过一段时间恐怕自己都会看不懂。 - 变量名最好包含变量本身的业务含义。比如
List<Student> studentList = new ArrayList<>();
就比List<Student> list = new ArrayList<>();
好很多。如果同一段代码里再出现一个List<Teacher>
的话,这样就可以很方便的取名为teacherList
或者teachers
而不是list1
和list2
这样的毫无意义的名称!
英文不好怎么办
这个问题怎么说呢。。
作为一名程序员吧,基础的英文还是要懂的。要不然发展也容易遇到天花板,学不好编程的。毕竟,最新的技术、解决方案、工具都是从国外传过来的。如果是解决一些基础性的问题,每天只做做CRUD,好像英文确实不怎么用得上。但是一旦遇到一些实质性问题,恐怕只能到英文网站上找喽!ㄟ(▔ ,▔)ㄏ 不要跟我说你编程可以不需要Stack Overflow。copying and pasting from stackoverflow 可是终极编程大法!o(∩_∩)o 这句话可是编程的真谛啊!(如果你看不懂这个梗那你有可能是伪程序员)
其实,话说回来,实在不方便用英文的时候,我认为也可以用拼音命名。这个问题上可以务实一点,量力而行。但是,拼音和英语混用的做法就不太好了。最好别这样!逼格不高。
注释
怎么添加代码注释
关于注释,我们需要解决的第一个问题是如何添加代码注释。
对于Java、C#之类的语言,有专用的文档注释语法,很好处理。对于C/C++,则按约定的格式说明一下类和函数、代码片段的作用和意图即可,至少编译器会进行静态检查。在Python中,有更牛逼的文档字符串这样的语言级特性支持,看注释用help()
很方便。不过对于Lua这样的弱类型解释型语言,注释就比较难处理了。这里以Lua为例给出一种注释的解决方案。
借用Java语言文档注释的风格。
文件注释,或者说类/模块注释。
--[[
Object-oriented helper functions for Lua
@author: Elvin Zeng
@date: 2017-8-21
--]]
函数注释
--[[
create a class with specified super class.
if number of parameters is zero, derived class will extends from {}.
@param superClass super class of target class
@return derived class
--]]
local function createClass(superClass)
local derivedClass = {}
-- 省略一堆代码
return derivedClass
end
--[[
register a new account
@param user
{
username = "username",
password = "password"
}
@return registered user
--]]
local function register(user)
end
tips: Lua中可以通过metatable机制实现类和继承,这一点与Javascript通过原型机制来实现类和继承有点类似。
注释里该写些什么
我们首先来看个反例。
/**
* 查询
*/
public List<Article> queryPage(int pageSize, int pageIndex) throws PageIndexOutOfBoundsException {
// 定义一个整型变量
int offset;
// 省略一堆代码
}
首先这个方法名本身就取得不好,这个暂且不说,先说注释问题。这里的注释犯了几个错:
- 方法注释为“查询”,这简直就是废话!方法名已经告诉别人这是查询方法了,还在这个注释里写这两个字有什么意义呢?而且到底查询些什么这里也没说!
- 参数没有注释。没有描述每一个参数的意义以及取值范围等!
- 什么情况下会抛出
PageIndexOutOfBoundsException
没有描述清楚。 - “定义一个整型变量”这种垃圾注释就不要写了,这么简单的语句谁看不懂啊!如果要注释,也是写上这个变量的含义。
这里我们先不考虑设计问题(分页索引号最好做成可以自己调整成合理值),下面再来看改善注释之后的代码。
/**
* 列出指定分页的文章
* @param pageSize 分页大小。如果等于0则表示查询出所有文章。
* @param pageIndex 分页索引号。必须为一个大于0的整数,第一页索引为1。
* @return 指定分页的文章列表
* @throws PageIndexOutOfBoundsException 当分页索引号超出正常范围时抛出,即pageIndex小于0或大于最大页索引时。
*/
public List<Article> listArticle(int pageSize, int pageIndex) throws PageIndexOutOfBoundsException{
// 第一条文章记录在MySql数据库中的偏移量
int offset;
// 省略一堆代码
}
改完之后的注释有没有感觉信息更全很多!虽然说代码本身就是最好的注释,但是必要的注释还是得写上去,毕竟调用的时候别人没法猜测你的索引号到底从0还是从1开始。另外,如果函数内算法比较复杂,可以在代码块内注释,也可以在函数注释上直接写清楚这个函数内部的大概算法/逻辑。代码写出来就是给别人调用的,如果没有基本的注释信息,那么每次调用你的代码的时候,都得去看一下你的函数内具体逻辑才能知道怎么调用。这显然是非常低效的!
命名与注释这两个基本方面没做好的话,会影响到整个团队的运作。也就是说,你封装的东西并没有给队友节省什么时间,别人用到你的代码的时候,又需要花上一些时间去读你的代码。如果团队里每个人都这样,那整个团队都会极其低效。我个人是非常不愿意与这种代码风格恶劣的人合作的。
参考规范
关于编码风格的问题,本文只说命名和注释这两个方面。关于缩进、空格、断行、空行等其他方面的问题,可以参考本节给出的参考规范。
不同的企业会有不同的编码规范,所以这里没有办法给出一个符合所有公司的规范。不过制定自己团队的规范的时候,可以参考一些大企业的做法。以下是世界上最大的互联网公司谷歌的编码规范,同学们可以参考这个。
- Google Java Style Guide
- Google C++ Style Guide
- Google Python Style Guide
- Google HTML/CSS Style Guide
- Google JavaScript Style Guide
异常处理
异常与返回值有什么不同
在C语言中,我们的函数通常会返回一个整型值作为状态码用于通知客户端调用的结果。比如0表示成功,非0表示失败。并且可以通过不同的数值来表示不同原因导致的失败。然而在Java、C#、C++一类面向对象语言中,一般不会用返回值来表示状态。返回值一般用于表示返回的业务值,而异常用于通知客户端程序运行状态改变了。
什么时候需要抛出异常
关于这个问题,我想到了一句极其精炼的话:当函数无法完成其宣称的任务的时候抛出异常!
比如上面的那个日子,当listArticle
方法由于种种原因无法查询出文章列表的时候,则抛出异常。
抛出异常在这种场景下是非常有必要的,因为这样其他人调用你的代码时可以非常放心的去调用,只要调用了你的方法,就会返回文章列表。如果无法返回文章列表,则会抛出异常。完全不用在调用这个函数的时候去怀疑是否执行成功了。
再来一句至理名言:
宁愿终止程序也不要带错运行下去。
也就是说,遇到错误的时候,宁愿抛出异常终止程序,也不要带着错运行下去。这是在掩耳盗铃!
异常需要携带什么信息
首先,异常的类型本身会带有异常种类信息。其次,异常的message
属性可以带上更详细一些的信息。这里需要注意,千万不要像下面这么做。
throw new PageIndexOutOfBoundsException("失败!");
抛出异常了肯定是执行失败了呀!带上这种信息有什么用,不是带了一句废话嘛!
应该是下面这样
throw new PageIndexOutOfBoundsException("参数分页索引号pageIndex不能大于分页总数");
此外,异常堆栈也会携带很多信息。
日志
谈到日志,首先要搞清楚一个问题,日志是干嘛用的?
用来记录运行时的错误信息啊!
是啊。好像大家都知道日志是干什么用的,但是为什么写起代码来就会忘记初衷呢!
来看看代码:
/**
* 异步发送通知邮件。
* @param templateFile 邮件模板文件路径,相对于classpath。
* @param modelMap 模型对象
*/
public void sendEmail(String templateFile, Map<String, String> modelMap){
// 这里省略一些代码
System.out.println("1");
// 这里省略一些代码
System.out.println("2");
// 这里省略一些代码
}
这里的代码是什么意思呢?程序员们应该都能明白的!很显然,这位程序员是想借助这些标记来调试,想知道代码到底执行到哪一行了。但是,这里很明显地犯了两个错。
- 为什么是
System.out.println("");
而不是logger.debug("");
? - 为什么是
1
、2
而不是一些更明确的文字信息呢?
在这里,合理的方式是下面这样。
/**
* 异步发送通知邮件。
* @param templateFile 邮件模板文件路径,相对于classpath。
* @param modelMap 模型对象
* @throws ServiceException 当邮件模板文件不存在或者modelMap中缺少必须的字段。
*/
public void sendEmail(String templateFile, Map<String, String> modelMap) throws ServiceException{
// 这里省略一些代码
if (isTemplateExists){
logger.debug("模板文件存在");
// 这里省略一些代码
logger.debug("邮件发送任务成功入队。任务id:" + taskId);
// 这里省略一些代码
}else{
logger.error("指定的模板文件[" + templateFile + "]不存在,邮件发送失败。");
// 抛出异常
}
}
我想给正在犯上面的错的同学提个醒:
-
使用日志框架,并用合适的级别输出日志非常重要。
好多程序员从来不负责也不参与运维相关的工作,甚至是做了好几年的Web都从来没有自己发布过网站。所以压根没有后期维护的意识!
如果没有这些日志,当项目上线之后,运维的背锅侠兄弟发现网站挂了之后只能直接重启,然后当作什么也没看到。因为没有排错的线索。 -
输出有效信息。
不要去输出一些像1
、2
、3
、成功
、失败
、hello
这样的毫无意义的日志,要输出logger.debug("邮件发送任务成功入队。任务id:" + taskId);
这样的有效信息。
也许当时你调试的时候,在你看来这些奇怪的字符串是有意义的,但是在其他人看来,这些就是天书。运维的背锅侠会提刀过来砍你的!另外像"-------开始执行--------"
这种对运行期间定位问题没有半点好处的日志就不要输出了!自己用可以,提交代码前一定要删掉。 -
日志中带上上下文信息。
孤立的一句错误日志通常没有什么实际作用。比如上面的例子中,如果在找不到指定的模板文件的时候未将发送邮件时指定的模板文件名输出,那么排错的时候无法知道到底是少了哪个模板文件。 -
不要在日志中输出用户的敏感信息。
千万不要在日志中输出像用户密码、邮件内容之类的涉及用户隐私的敏感信息,也不要去输出像验证码的值之类的敏感信息。
参数校验
在你对外公开的方法前先插入一些检查参数的代码,以确保方法被“正确的姿势”调用。比如:
/**
* 列出指定分页的文章
* @param pageSize 分页大小。如果等于0则表示查询出所有文章。
* @param pageIndex 分页索引号。必须为一个大于0的整数,第一页索引为1。
* @return 指定分页的文章列表
* @throws PageIndexOutOfBoundsException 当分页索引号超出正常范围时抛出,即pageIndex小于0或大于最大页索引时。
*/
public List<Article> listArticle(int pageSize, int pageIndex) throws PageIndexOutOfBoundsException{
if (pageSize < 0){
throw new IllegalArgumentException("pageSize不能小于0");
}
if (pageIndex < 1){
throw new IllegalArgumentException("pageIndex不能小于1");
}
// 第一条文章记录在MySql数据库中的偏移量
int offset;
throw new PageIndexOutOfBoundsException("");
// 省略一堆代码
}
参数校验的作用
如果在对外公开的重要方法开始的位置不插入校验参数的代码,有时恐怕方法需要运行到方法内部比较深的位置才会抛出一个异常来。而且那种情况下,抛出的异常可能就会有各种各样的了。比如空指针、除零异常等。
这种情况下,很难一眼看出引发这个异常的根源是参数传错了。需要对你的代码进行一番调试才行!如果一开始就在代码的入口插入了校验参数的代码,那么调用的时候,一眼就能看出来是参数传错了导致了一个异常。这样其他程序员看到这个异常之后就会去看一下你的方法注释。他一看,哦!原来分页索引号是从1开始计数的,那么这个问题就会就此打住,给团队节省了时间。
参数校验问题是会影响团队运行效率的一个很关键的因素。所以,请同学们重视起这个问题来。我们都是工程师,团队作战的,自己写代码快不叫快,整个团队快起来才叫真的快!用好断言,可以让你的代码更健壮。
tips: Java中默认断言是不开启的,所以建议无视Java语言的断言,自己处理。
什么时候需要进行参数校验
我认为一个方法或者函数在满足以下条件时有必要进行参数校验:
- 方法或者函数是对外公开的,不是私有的。
- 参数有可能为空指针的时候。
- 参数的合理值无法通过方法名、参数名、参数类型一眼看出来的时候!比如上面那个pageIndex是从1开始计数的,但别人并不知道你是从1开始计数的。
如果对每一方法都进行校验的话,其实挺麻烦的。程序员的时间是很宝贵的,没这么多闲工夫。不过在满足上面条件的情况下,最好还是校验一下。因为做了这个校验,你自己是会稍微浪费几分钟的时间,不过从团队整体来看,总的调试损耗的时间却降下来了。要记住方法/函数写出来就是给别人调用的!
参数校验需要做到什么程度
我有一个标准,就是把自己当成调用这些代码的那个人,把自己想象成有可能以任何“姿势”调用的菜鸟(实际上也有可能是不了解你的代码的大牛)。如果这个时候自己也有可能会犯某些错(比如没注意边界值,没注意是否可空),那么这个时候是必须要做校验的。对于一些已经在其他层做过处理不太可能有错误的值的情况,可以不做校验。比如你的UserService
中有一个签名为public void register(User user)
的方法,用于注册一个用户。这种情况下,可以只校验一下user
参数是否为空,而不用对user
的username
、password
属性进行校验(用户名密码长度是否合法等)。因为你在上一层控制器层模型绑定的时候已经做过非常严谨的校验了。当然,这里如果你有充足的时间,也可以校验一下。具体做到什么程度,还需要你根据情况去自己把握。