用这些手法让你的代码更易读

好的开发者能够竭尽他们所能去简单的描述他们的意图。就像物理学家会通过用一根铅笔插入一张对折的纸张来描述虫洞一样,简洁而直观。是什么让我们如此不同?

我经常尽我所能让我的代码更加易读,很多时候依据代码的约定为我们的变量选取一个合适的名字很有用,但是还有一些东西仍然时缺失的,我们缺乏一种编写代码的方式,让我们能够直接理解代码的意思而不是迷惑的尝试去理解这段代码为什么要这样?我正在尝试去实现它。

你可能会说这种能力就是让代码读起来像是一个故事,而不是一堆单词

接下来要介绍3个主要的规则

问题

阅读别人的代码真的是一件让人头大的事情。如果没有程序的上下文,我们很容易迷失在寻找函数或参数含义的途中,无法自拔。

建议

从二进制到低级语言再到高级语言,很容易意识到编码变的越来越友好了,因此有了更多的开发者加入程序员的世界里。就像编码方式正在变得更简洁易读,我们写代码也应该如此,简单的说,我们的代码要直中要义而切能自我解释。

结果

一个好的编码方法可以让我们的代码变得像一个故事一样,更加易读也更易被别人所理解。

方法命令

如何进行一个正确的命名:

当我们创建一个方法时,我们总是假设人们在读这个方法时是知道上下文的,知道我们想要达到什么样的效果。例如我们给方法一个模糊的命名handleRedView()会导致很多的问题出现,RedView代表的是什么东西?这个方法的主要目的是什么呢?

可以看出,在一些情况下,我们的方法功能可能太过模糊或太过复杂,我们很难来脱离上下文的情况下理解它的作用。

现在,我们要把所有的方法分为4个类别:

  • Informer functions
  • Management functions
  • Router functions
  • Execution functions

1. Informer function

经常用来触发 router/management functions

例如:

//dataHasUpdated.swift

delegate.dataHasUpdated()

func dataHasUpdated {
    //Someone is informing you that something has happed.
}
// viewDidLoad.swift

// Informer Functions
override func engineStarted() {
    super.engineStarted()
    handleCarStarted()
}

回调方法,通知一些方法调用或状态改变将要或正在发生,给你提供选择反应的机会。
多数情况下,是用来触发delgate方法,或是发送通知。

2. Management function

通常用来集合多个相互之间没有依赖的方法用来实现一个更高级的目的。代码块中所有的方法都会被调用

// Management Function

func handleCarStarted() {
    trunLights(on: true)
    turnAC(on: true)
}

阅读这个方法,我们能够知道所有我们需要进行的操作, 执行这个方法汽车就会启动, 在这一个时刻,我们只关心调用的方法是什么意思,而不用管他们时如何实现的。

3. Router function

用来组合多个方法实现一个更高级的目的,这些方法之间有着一定的选择和排序关系, 只有被需要的代码才会被执行。

//Router Function

private func turnLights(on shouldTurnLightsOn: Bool) {
    if shouldTurnLightsOn {
        turnExteriorLightsOn()
        checkForBurnedBulbs()
    } else {
        turnExteriorLightOff()
    }
}

Router functions 通常直接调用 execution functions,但是偶尔也可以包含一些逻辑,如果这些代码没有超过一行的话。

4. Execution function

核心的方法实现,实现了方法名所描述的内容。

// Execution Function

private func trunExteriorLightsOn() {
    leftFrontLight.isOn  = true
    rightFrontLight.isOn = true
    leftBackLight.isOn   = true
    rightBackLight.isOn  = true    
}

private func checkForBurnedBulbs() {
    for lightBulb in bulbs where !lightBulb.isUseable {
        Dashboard.display(errorType: .lights)
        break
    }
}

这个方法具体的逻辑我们可能很难完全去理解,但是我们早已经把它要做的事情写道了方法名里面。这个方法打开了灯光,并查看了是否有灯泡烧毁。理解这些将有助于我们定位bug,也更容易在不改变这些方法名的情况下,为其添加新的逻辑和特性。

最终, 当你的在项目当中实现了这个结构以后,你只需要在你的class实现中管理 informer, management, router 方法。

class Car: Vehicle
{
// Informer Functions
   override func engineStarted()
   {
     super.engineStarted()
     handleCarStarted()
   }
// Management Functions
   private func handleCarStarted()
   {
     turnLights(on: true)
     turnAC(on: true)
   }
// Router Functions
   private func turnLights(on shouldTurnLightsOn: Bool)
   {
     if shouldTurnLightsOn
     {
       turnExteriorLightsOn()
       checkForBurnedBulbs()
     }
     else { turnExteriorLightsOff() }
     // When an if statment has only 1 thing to execute i like to write it
     // at the same line with the "if" "else" word, it makes reading your code more fluent.
   }
   private func handleTurnAC(on shouldTurnACOn: Bool)
   {
     if shouldTurnACOn
     {
       let tempeture = currentTemptureSelected()
       let toggleFocus = currentACToggleFocus()
       startAC(with: tempeture, aimedTo: toggleFocus)
     }
     else { turnACOff() }
   }
}

所有的 execution/logic 方法都可以放在class的扩展当中,在同一个文件内部。

extension Car
{
// Execution Functions
  private func turnExteriorLightsOn()
  {
    leftFrontLight .isOn = true
    rightFrontLight.isOn = true
    leftBackLight  .isOn = true
    rightBackLight .isOn = true
  }
  
  private func checkForBurnedBulbs()
  {
     for lightBulb in bulbs where !lightBulb.isUseable
     {
       Dashboard.display(errorType: .lights)
       break
     } 
  }
}

结果就是你将得到一个简洁的、易读的、益于理解的类实现。

记住任何方法一定要遵守单一责任愿者

避免使用and在你的方法名当中:

  • playAndMinimize()
  • loadAndPlay()

这些错误的使用方法,打破了单一责任原则, 这样使你的代码看起来要适用于所有的场景。

避免在取方法名的时候玩猜一猜的游戏:

  • moveRedViewIfNeeded()

这个例子保证了,将来的每一个开发者都必须要亲自查看方法的内部实现,才能明确的知道哪种情况下会移动red view. 这样模棱两可的方法会增加阅读代码的难度。

但是 layoutIfNeeded 不是同样的情况, 我们都知道只有在 setNeedsLayout 是 true 的情况下,视图才会刷新布局。 类似的这种方法,通常情况下,最好保持只使用在你项目的私有方法里面。

小讨论

闪现在脑海中的第一件事情就是关于代码可读性的约定, 这些约定大家都知道,也经常被使用,但是使用这些约定不一定让你的代码更好了,它们会让代码俱备跨跨应用/跨平台的的能力,但是对可读性没有什么提升。

is 前缀应该被用在布尔属性,或是表示一个方法要返回布尔值 #代码约定

if总是被来表示布尔值, 为什么我们需要在使用布尔值时,在前面加上is呢?为什么苹果会修改 swift 的语法,从 view.hidden ~> view.isHidden? 我能想到的唯一答案就是 if view.isHidden 读起来感觉更加的地道。

让我们尝试着用以下规则来实现is前缀:

  • 如果一个类的布尔值属性或方法是公开的,那添加 is 就很合适
public var isHidden: Bool
{
  return alpha == 0.0
}
if containerView.isHidden
  • 如果一个类的布尔值属性或方法是私有的,那添加 is 就看起来很多余
private var positionedVerticaly: Bool
{
  return view.frame.width/2 == centerX
}
if positionedVerticaly
if positionedVerticaly && positionedHorizontally
VS
if isPositionedVerticaly
if isPositionedVerticaly && isPositionedHorizontally
  • 如果一个布尔值属性或方法私有的,但是可能要提供一个公开的取值方法,我们可以使用一个计算属性返回这个私有的值。
{
  return positionedVerticaly
}
if containerView.isPositionedVerticaly

我们也可以使用 private(set) 然后公开使用其属性。但是考虑到产生的副作用,我们决定使用这样的封装。

封装被用来隐藏来自于一个类当中的结构化数据或状态, 防止不确定的调用直接方法这些数据。封装

你可能会问,如果一个布尔值不是直接表示自己名字所代表的含义呢,在这种情况看下,你可以写的属性名类似于这样:

  • private var playerIsPlaying: Bool
  • private var gridConstraintIsEnabled()

is需要一些指代对象,view.isHidden 中的is指向 view。 在其它的一些例子当中也表达着同样的意思, playerIsPlayingis 指向的是那个 player

通常,相比去查看属性的声明,开发者更喜欢直接通过阅读代码中的变量而去猜测他们所代表的含义。

/if playerIsPlaying { }/ **opposing of** /if isPlayerIsPlaying {}/

以上那个看起来更舒服呢?由你来决定。

翻译于:Making your iOS application easy to read with these simple steps.

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 216,125评论 6 498
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 92,293评论 3 392
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 162,054评论 0 351
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 58,077评论 1 291
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 67,096评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 51,062评论 1 295
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,988评论 3 417
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,817评论 0 273
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 45,266评论 1 310
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,486评论 2 331
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,646评论 1 347
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,375评论 5 342
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,974评论 3 325
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,621评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,796评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,642评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,538评论 2 352

推荐阅读更多精彩内容

  • 这两幅画是国画大师程十发先生画的昆曲人物画,上面那幅画的是《牡丹亭·惊梦》中的杜丽娘和柳梦梅,下面这幅画的是《钟馗...
    东丰林波阅读 316评论 0 0
  • 晒一下我粗放的工位,整体如下。 笔记本电脑屏太小了,但优势在于超薄超轻超好携带,考虑接个大屏。小屏还有个坏处就是需...
    扣舷阅读 849评论 14 29
  • 很想找一个未曾一起出行过的小伙伴,然后选一个都曾去过的城市,我想一定会有不少的惊喜吧…… 吴哥之行让各种突如其来的...
    蛋白艾力克阅读 287评论 0 0
  • 今天的题目是“虽然我只是一个两个月大的小婴鹅,可是阻挡不了我想吃癞蛤蟆的决心。”
    水沐正年华阅读 111评论 0 2