有人提出了这样的问题：
代码1：

if ([responseDict[@"result"] boolValue] == false) {
    [CQHud showToastWithMessage:responseDict[@"msg"]];
    return;
}
/// code

代码2：

if ([responseDict[@"result"] boolValue] == false) {
    [CQHud showToastWithMessage:responseDict[@"msg"]];
} else {
    /// code
}

你们更倾向于哪种写法？
我的做法是如果if的代码块中的代码比较少就用if return，想问下你们是怎么选择的。

我认同 J_Knight 的说法：“如果这两组代码是互斥关系，建议使用 if & else; 如果是要首先排除某些特殊的错误情况，建议使用return”。

if-else 用来表示逻辑上的分支处理，但是在程序中有一种情况是其实我们更关心的主流程。

比如来看下面的一个简单例子

if object is Pseron {
  if ((Person *)object).age > 18 {
    // is adult
  }else {
    error("not allowed")
  }
}else {
  error("not allowed")
}

其实程序只关注两个部分：

• 判断是否合法
• 合法后进行对应操作

在这种场景下，上面的这种写法就显得不够清晰，尤其是当判断超过3层之后。提前return就会好的多：

if object isNot Pseron {
    error("not allowed")
    return
}
if object.age < 18 {
    error("not allowed")
    return
}

if ((Person *)object).age > 18 {
    // is adult
 }

这也是为什么swift要单独推出guard关键字的原因。

如果从更高的抽象来看，本质上是因为前面的那些return是一个filter，换做FRP的写法是这样的：

let response: Observal<Object>
response.filter { object in
  return object is Person
}.filter { object in
  return object.age > 18
}.subscribNext {
  // is  adult
}

如果这么写就可以明显的看出，如果是分支的逻辑就会写在操作数据这层。前面用于提前return的部分在这个场景里不是一个逻辑分支。

相比是不是该用return，这个代码有个更严重的问题

很容易看出这是处理api返回数据的代码。这样写犯了一个非常严重且简单的错误：代码没有复用。

用更高一点目光看所暴露的问题是抽象层次低。

如果这样直接处理服务端的数据，等于每一个请求的地方都要写这种很傻的代码。有50个api，就要写50次[responseDict[@"result"] boolValue]，如果有一天后端架构调整，把标志为请求是否成功的字段从result字符串改为状态码表示呢？比如200表示成功，其他都是失败。你的代码要怎么维护？

每个请求的代码都要取一次“msg”，弹出来烦不烦？所有的api请求做一层处理，如果是错误的统一处理一下不就好了？如果所有错误的请求要记日志，这种写法不是又要改很多地方？

抽象层次低是你只看到了原始数据，没有看到这些数据复合后的抽象。可以理解为服务端返回的是一个Response对象，里面有返回的数据和错误信息。写起来大概是这样：

struct Response<T: JSONMappedObject>: {
  var data: T?
  var rawData: [String: Any]
  var msg: String?
  
  init(json: [String: Any]) {
    
  }
  
  init(json: [Any]){
  }
}

如果以后返回的数据要做整体变更，只会是Response这个结构发生改变，外层的业务处理不会受影响，不用关心我要的data是怎么取出来的。

代码请至少要做到三件事

正确的命名，不要有重复的代码，每个函数里的逻辑步骤不要超过7步。
最开始写代码，先严格达到上面的3个目标再想其他的吧。

不要扫二维码

欢迎关注我的微博：@没故事的卓同学
里面没有任何有益的内容。