让自己的代码更容易让别人读懂

优秀的程序员会用尽可能简单的方式来解释他们的代码,即使是物理学家都可以用一张白纸和一只铅笔来解释虫洞,我们又何尝不可?

我会尽可能让代码写地简单、易读,包括选择合适的变量名、使用编码规范(code conventions)等等,但还是缺了点东西,理解代码不应该是去理解“如何”实现的,而是要理解想要“达成”什么。

甚至可以说要让读代码像读小说一样,而不是一大堆代码。

下面讨论三大主题:

问题

阅读其他人的代码可能会非常折磨,如果不提供合适的上下文,我们会迷失在寻找某个函数或属性的意义中。

建议

不论是二进制语言、低级语言还是高级语言,语法都在变得越来越友好,以便吸引更多开发者。而随着语法变得更接近英语,我们的代码也应该简明扼要、不言自明。

结果

写出良好的代码,读起来像小说一样,容易阅读和理解(即使没有给出上下文)。

函数命名

正确的方式是:

我们写函数时都会假设阅读它的人拥有足够的上下文,能够理解函数想实现什么。用模糊的含义来命名函数,例如“handleRedView()”会引起很多问题,”RedView”是啥?这个函数主要是想干啥?

所以在某些情况下函数的用途会很模糊,如果没有提供足够的上下文,阅读起来会非常困难

我们可以把函数的用途分为四类:

通知(Informer)函数

管理(Management)函数

路由(Router)函数

执行(Execution)函数

1. 通知函数

通常会触发路由/管理函数,例子如下:

delegate.dataHasUpdated()funcdataHasUpdated(){  //通知某事的发生 }// 通知函数override funcengineStarted(){  super.engineStarted()  handleCarStarted() }

回调函数,通知某事已经/即将发生,并给机会进行响应。

大部分情况下用于被代理(delegate)触发的操作,或是通知(notification)处理函数。

2. 管理函数

用于联合多个函数以实现更高级的用途,不需要依赖参数,block 中的所有代码都会执行。

// 管理函数funchandleCarStarted(){  turnLights(on:true)  turnAC(on:true)}

上面的函数包含所有需要的信息,汽车启动时执行这些函数,此时我们不关注这是“如何”实现的,而是关注它做了“什么”。

3. 路由函数

用于联合多个函数以实现更高级的用途,需要依赖一些参数,只在我们想执行的时候才执行。

// 路由函数private func turnLights(on shouldTurnLightsOn: Bool){ifshouldTurnLightsOn {  turnExteriorLightsOn()  checkForBurnedBulbs() }else{ turnExteriorLightsOff() } 如果if语句只执行一件事,我喜欢把"if""else"和执行语句写在同一行,这样代码读起来会更流畅。

路由函数通常同于指向执行函数,但在某些情况下,如果逻辑代码不超过一行,也可以包含自己的逻辑。

4. 执行函数

函数名字的具体实现

// 执行函数private functurnExteriorLightsOn(){  leftFrontLight.isOn =truerightFrontLight.isOn =trueleftBackLight.isOn =truerightBackLight.isOn =true} private funccheckForBurnedBulbs() {forlightBulbinbulbswhere!lightBulb.isUseable  {    Dashboard.(errorType: .lights)break} }

这样就能写出一个干净、简短的类,可读性强,容易维护。

* 谨记一项原则,每个函数都应该只有一个责任。

避免在函数名称里使用”and“:playAndMinimize()loadAndPlay()这个坏习惯会打破单一责任原则,写出能够适应两种情况的代码。

避免在函数名称里进行猜测:moveRedViewIfNeeded()上面的例子会导致后面的程序员必须深入此函数,才能理解触发移动 Red View 的时机,这样不够清晰。

不,layoutIfNeeded() 并不属于这种情况,因为我们知道如果某个 view 的 setNeedDisplay 为 true,就应该重新布局。这种情况在 Swift 语言里很普遍,因为函数基本上都是应用私有的。

谈及代码可读性,我首先会想到编码规范(code convention),它们被普遍接受、应用广泛,但使用编码规范并不一定会提升代码质量,虽然有跨应用性但可读性更差。

”is“前缀应该用于布尔型变量和方法,以便解释返回值是布尔类型的。#编码规范

既然”if“语句总是用于布尔值,那还有必要给每个布尔属性都加上”is“吗?为什么苹果要把 Swift 语法从view.hidden改成view.isHidden?我只能想到一种答案……因为**“if view.isHidden”**看起来更自然。

尝试以以下原则使用“is”前缀:

如果类的某个布尔属性/方法用于该类(公开)的实例,就有正当理由使用“is”前缀。

public var isHidden: Bool{returnalpha == 0.0 }ifcontainerView.isHidden

如果布尔属性在类内部(私有)使用,前缀就是多余的。

private var positionedVerticaly: Bool {returnview.frame.width/2 == centerX }ifpositionedVerticalyifpositionedVerticaly && positionedHorizontally VSifisPositionedVerticalyifisPositionedVerticaly && isPositionedHorizontally

如果布尔型属性/方法同时被私有公开使用,那么应该用计算属性来返回该私有属性值。

public var isPositionedVerticaly: Bool{returnpositionedVerticaly }ifcontainerView.isPositionedVerticaly

虽然用私有 set 并公开使用该属性也是可以的,但上面这种方式可以实现封装(encapsulation)。

封装用于隐藏类中结构化数据对象的值或状态,防止未授权方直接访问。

en.wikipedia.org/wiki/Encaps…

那如果不是自己直接处理的布尔型,如果如何命名呢?

private var playerIsPlaying: Bool

private var gridConstraintIsEnabled()

“is” 需要指向某个东西:view.isHidden, “is”指向 view。上面的例子里使用了此原则,playerIsPlaying,“is”指向 player。

谨记:开发者通常会在阅读属性声明之前先阅读函数内部的代码,尝试搞明白这些属性的用途。

/ifplayerIsPlaying { }/ 对比 /ifisPlayerIsPlaying {}/


转载:链接:https://juejin.im/post/5add93866fb9a07aa76744d9

©著作权归作者所有,转载或内容合作请联系作者
【社区内容提示】社区部分内容疑似由AI辅助生成,浏览时请结合常识与多方信息审慎甄别。
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

相关阅读更多精彩内容

友情链接更多精彩内容