这是一个创建于 2439 天前的主题，其中的信息可能已经有所发展或是发生改变。
写接口的时候，经常需要验证参数的数据类型、参数的值。还有一些选填 /必选之类的判断.
如果是用普通的 if esle 去判断的话，导致代码不优雅，后期迭代困难. 在知乎上看的 “如何优雅地解决项目中可能存在的很多的 if/else ” 也不太理解设计模式， 大家平常是用什么方式可以解决呢或者是大家有什么建议应该看哪些书、方法之类的？ 求指点迷津，谢谢
 |
1
MaZiT OP 使用的是 php 语言....目前用的是 tp5 框架
|
 |
2
torbrowserbridge 2018-03-19 09:22:21 +08:00 via Android
你需要的是 validation 组件
|
 |
3
powergx 2018-03-19 09:26:28 +08:00 via iPhone
重构 重要的是程序的结构 /流程，这种 if else 可有可无，在项目中无伤大雅
|
 |
4
LevineChen 2018-03-19 09:29:44 +08:00  1
$error = 'success';
do{ if(!$user){ $error = 'need login'; break; } if(!$a){ $error = 'a not set' ; break; } $uid = $user->id; }while(false) |
 |
5
graysheeep 2018-03-19 09:32:56 +08:00
validation
|
|
6
torbrowserbridge 2018-03-19 09:33:11 +08:00
@LevineChen 这啥玩意，简直看不懂。你是来黑 php 的？
|
 |
7
param 2018-03-19 09:35:22 +08:00
assert?
|
 |
8
irenicus 2018-03-19 09:35:40 +08:00 via Android
复杂的 if else 可以重构为 dispatch table，相当于把逻辑抽象成数据另外保存，原本的逻辑本身就变成了遍历这个数据
|
 |
9
nickel123666 2018-03-19 09:38:21 +08:00
5 楼+1，有个叫做 validation 的东西，可以参考一下。
|
 |
10
daydaydayup 2018-03-19 09:41:28 +08:00
我不知道是不是因为我的水平低的问题.用其他组件经常被别人鄙视.你们有没有这样的情况?
|
 |
11
xzpjerry731 2018-03-19 09:44:58 +08:00
有些 if else 的作用可以用哈希表代替。
|
 |
12
zenxds 2018-03-19 09:45:45 +08:00
策略模式了解一下
|
 |
13
songz 2018-03-19 09:46:56 +08:00
switch case
|
 |
14
yangfch3 2018-03-19 09:47:47 +08:00
策略模式了解一下
|
 |
16
wwdyy 2018-03-19 09:49:46 +08:00
看一遍
JavaScript 设计模式与开发实践 看一遍自己的代码 改 然后----- 看一遍 JavaScript 设计模式与开发实践 看一遍自己的代码 改 |
 |
18
laobaozi 2018-03-19 09:50:11 +08:00
尽早返回
|
 |
19
suikator 2018-03-19 09:50:41 +08:00 via Android
@torbrowserbridge #6 do{...}while(false) 简化嵌套 if
|
|
20
MaZiT OP @xzpjerry731 方便给个 demo 吗？我不太理解
|
 |
21
zjp 2018-03-19 09:52:56 +08:00 via Android
不能避免多层 if 的话，及早处理错误情况然后返回
https://coolshell.cn/articles/17757.html |
|
22
yangfch3 2018-03-19 10:02:02 +08:00
@MaZiT
@xzpjerry731 的意思应该就是使用策略模式，因为策略模式就是用一个哈希表来存储不同的策略，然后策略管理器根据不同的策略 key 来调用不同的策略属性。 下面是一个简单的策略模式用例： ``` function logicCall(type, ...args) { // 其他逻辑 registeredTypeHandler[type] && registeredTypeHandler[type](...args); // 这部分还可以新增一个策略控制器进行管理 } // 策略哈希表 var registeredTypeHandler = { foo: _foo, bar: _bar }; function _foo(...args) { ... } function _bar(..args) { ... } ``` |
 |
23
lurenw 2018-03-19 10:09:40 +08:00
状态机了解一下
|
 |
24
Todd_Leo 2018-03-19 10:13:01 +08:00
Pattern Matching 了解一下：
``` x match { case 1 => "one" case 2 => "two" case _ => "many" // case _ will trigger if all other cases fail. } ``` |
 |
25
ahmcscx 2018-03-19 10:24:41 +08:00
|
 |
26
daisyxdx 2018-03-19 10:32:29 +08:00
弱弱地提一下 职责链模式。。。不知道有没有啥问题
|
|
27
LevineChen 2018-03-19 10:39:01 +08:00
@torbrowserbridge 老铁我只能说你理解能力比较差.
|
 |
28
CFMY 2018-03-19 10:40:31 +08:00
策略模式不能避免 if else，如果你真的需要很多判断，无论用什么模式都无法避免这些判断
只是说策略模式更符合开放封闭原则，但是代价是你需要更多地代码去写接口以及更多类 与其套用模式，不如重新审视你的流程是否可以分解，从而让代码更清晰 |
 |
29
Building 2018-03-19 10:57:58 +08:00 via iPhone
只是为了看起来更优雅给自己找麻烦完全没必要，把一堆判断分散到各个角落，一段时间看回来肯定会骂自己傻逼。老老实实写就好。
|
 |
30
crysislinux 2018-03-19 11:03:18 +08:00
逻辑上相关的 if else 我现在倾向于拆分成多个 function，逻辑不相关的参数类型检查，就用类似 table 的东西。以前就爱搞些多态。。结果多写了 n 多代码。
没必要还是不要搞复杂了。 |
 |
31
gnaggnoyil 2018-03-19 11:03:19 +08:00
LZ 我告诉你一个笨但是有用的办法:画真值表硬怼 :)
|
 |
32
realityone 2018-03-19 11:06:57 +08:00 via iPhone
不要写 else
|
 |
33
qiyon 2018-03-19 11:11:32 +08:00
参数校验可以看下
Yii 的 Form Model Laravel 的 Validation 之类的实现 可以实现参数的默认值、过滤、以及校验 |
|
34
qiyon 2018-03-19 11:15:34 +08:00
还有 symfony 的 entity，TP 也有 Model 的 validate
|
 |
35
nicevar 2018-03-19 11:27:15 +08:00
公司代码服务器配个 SonarQube, 这种 if else 多几层就过不去，一段时间就治好了你的症状
|
|
36
torbrowserbridge 2018-03-19 11:29:38 +08:00 via Android
@LevineChen 老铁我说的是，你代码有问题。set 判断都是错的
|
 |
37
windygoose 2018-03-19 11:39:09 +08:00
四楼的写法一般常见在 C 中，关注点是 do while 和 break 的使用
|
 |
38
TaoSama 2018-03-19 13:58:43 +08:00 via Android
这种东西有两种解决方案
一个是尽量用函数式的方法 另一个就是把脏东西藏起来 用可靠的 validation 的库 或者自己写相关的库 把 if..else 藏起来 |
 |
39
koalli 2018-03-19 14:02:18 +08:00
@windygoose 我也觉得是 C/C++转 php 的老司机，这个写法很常见，包括一些宏为了避免展开出问题也会用这样的写法
|
 |
40
AwayEM 2018-03-19 14:07:34 +08:00
form validator 很多的把 vue react 自带的好多啊， 后台框架的 form 类也有好多抽离验证的，我用的 spring flask django 以及前端的 vue 都有。
|
 |
41
newmlp 2018-03-19 14:38:16 +08:00
if 不就是判断用的么，有什么不优雅的，不要为了设计而设计，而且如果判断数量不多的话，改用其他会带来性能问题，虽然不多
|
 |
42
r7st 2018-03-19 14:41:30 +08:00
@torbrowserbridge do while 这不是？？
|
 |
43
zjsxwc 2018-03-19 14:45:19 +08:00
责任链模式吧，专门用来避免超多 if-else 的情况
|
 |
44
cljnnn 2018-03-19 14:49:00 +08:00 via Android
《代码大全》里面对多层 if else 的建议是，if 里面总是处理预期情况，else 处理其他情况。 看起来不太优雅，但是逻辑很清楚（这一句我说的）。
|
 |
45
yidinghe 2018-03-19 17:30:24 +08:00
不要一开始就通过设计去避免 if-else，等将来分支确实多起来了再考虑也不迟。
|
Select Language