关于代码中的 if else 重构

使用的是 php 语言....目前用的是 tp5 框架

你需要的是 validation 组件

重构 重要的是程序的结构 /流程，这种 if else 可有可无，在项目中无伤大雅

$error = 'success';

do{

if(!$user){
$error = 'need login';
break;

}

if(!$a){
$error = 'a not set' ;
break;
}

$uid = $user->id;

}while(false)

validation

@LevineChen 这啥玩意，简直看不懂。你是来黑 php 的？

assert?

复杂的 if else 可以重构为 dispatch table，相当于把逻辑抽象成数据另外保存，原本的逻辑本身就变成了遍历这个数据

5 楼+1，有个叫做 validation 的东西，可以参考一下。

我不知道是不是因为我的水平低的问题.用其他组件经常被别人鄙视.你们有没有这样的情况?

有些 if else 的作用可以用哈希表代替。

策略模式了解一下

switch case

策略模式了解一下

@yangfch3 谢谢

看一遍
Javascript 设计模式与开发实践

看一遍自己的代码

改

然后-----
看一遍
Javascript 设计模式与开发实践

看一遍自己的代码

改

@powergx 但是这样会导致后面的迭代更多 if else 吧...

尽早返回

@torbrowserbridge #6 do{...}while(false) 简化嵌套 if

@xzpjerry731 方便给个 demo 吗？我不太理解

不能避免多层 if 的话，及早处理错误情况然后返回
https://coolshell.cn/articles/17757.html

@MaZiT
@xzpjerry731 的意思应该就是使用策略模式，因为策略模式就是用一个哈希表来存储不同的策略，然后策略管理器根据不同的策略 key 来调用不同的策略属性。

下面是一个简单的策略模式用例：
```
function logicCall(type, ...args) {
// 其他逻辑

registeredTypeHandler[type] && registeredTypeHandler[type](...args); // 这部分还可以新增一个策略控制器进行管理
}

// 策略哈希表
var registeredTypeHandler = {
foo: _foo,
bar: _bar
};

function _foo(...args) {
...
}

function _bar(..args) {
...
}
```

状态机了解一下

Pattern Matching 了解一下：

```
x match {
case 1 => "one"
case 2 => "two"
case _ => "many" // case _ will trigger if all other cases fail.
}
```

http://mp.weixin.qq.com/s?__biz=MzAxMTI4MTkwNQ==&mid=2650825067&idx=1&sn=19d7658944f3e497851783ff8e5f0b35&chksm=80b7b5f5b7c03ce3be84ff637642076423f22760108e0e719b130967d65d1591e20dd6157f62&mpshare=1&scene=23&srcid=0227P9czfDyNbO6G2kOz1RPQ#rd


这个篇文章说了 if else 的问题

弱弱地提一下 职责链模式。。。不知道有没有啥问题

@torbrowserbridge 老铁我只能说你理解能力比较差.

策略模式不能避免 if else，如果你真的需要很多判断，无论用什么模式都无法避免这些判断
只是说策略模式更符合开放封闭原则，但是代价是你需要更多地代码去写接口以及更多类
与其套用模式，不如重新审视你的流程是否可以分解，从而让代码更清晰

只是为了看起来更优雅给自己找麻烦完全没必要，把一堆判断分散到各个角落，一段时间看回来肯定会骂自己傻逼。老老实实写就好。

逻辑上相关的 if else 我现在倾向于拆分成多个 function，逻辑不相关的参数类型检查，就用类似 table 的东西。以前就爱搞些多态。。结果多写了 n 多代码。

没必要还是不要搞复杂了。

LZ 我告诉你一个笨但是有用的办法:画真值表硬怼 :)

不要写 else

参数校验可以看下
Yii 的 Form Model
Laravel 的 Validation
之类的实现

可以实现参数的默认值、过滤、以及校验

还有 symfony 的 entity，TP 也有 Model 的 validate

公司代码服务器配个 SonarQube, 这种 if else 多几层就过不去，一段时间就治好了你的症状

@LevineChen 老铁我说的是，你代码有问题。set 判断都是错的

四楼的写法一般常见在 C 中，关注点是 do while 和 break 的使用

这种东西有两种解决方案
一个是尽量用函数式的方法
另一个就是把脏东西藏起来 用可靠的 validation 的库 或者自己写相关的库 把 if..else 藏起来

@windygoose 我也觉得是 C/C++转 php 的老司机，这个写法很常见，包括一些宏为了避免展开出问题也会用这样的写法

form validator 很多的把 vue react 自带的好多啊， 后台框架的 form 类也有好多抽离验证的，我用的 spring flask django 以及前端的 vue 都有。

if 不就是判断用的么，有什么不优雅的，不要为了设计而设计，而且如果判断数量不多的话，改用其他会带来性能问题，虽然不多

@torbrowserbridge do while 这不是？？

责任链模式吧，专门用来避免超多 if-else 的情况

《代码大全》里面对多层 if else 的建议是，if 里面总是处理预期情况，else 处理其他情况。 看起来不太优雅，但是逻辑很清楚（这一句我说的）。

不要一开始就通过设计去避免 if-else，等将来分支确实多起来了再考虑也不迟。