如何完美的消除 if...else..或者 switch 这种代码，求大神给个思路。

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

This topic created in 3783 days ago, the information mentioned may be changed or developed.

举个例子，在 iOS 开发中，经常会遇到在设置 tableViewCell 时需要根据 indexPath 来设置不同 cell ，一般代码可能会是下面这个样子：

- (UITableViewCell*)tableView:(UITableView*)tableView cellForRowAtIndexPath:(NSIndexPath*)indexPath {
    
    // 设置第一种 cell
    if (indexPath.section == 3 && indexPath.row != 2) {
        //  cell 代码
        return cell;
    }
    
    // 设置第二种 cell
    if (indexPath.section == 3 && indexPath.row == 2) {
        // cell 代码
        return cell;
    }
    
    // 设置其他 cell
    return cell;
}

当然，举的例子比较简单， if...else...也不多，这种场景在开发中很多，一般使用 if...else...或者 switch 时会使代码非常多，求大神们能不能给个思路，遇到这种场景尽量不要出现这种嵌套代码，最好给个实例膜拜膜拜 ^_^

cell

代码

else

return

86 replies • 2016-07-01 22:17:23 +08:00

wander2008

Jun 30, 2016 via iPhone

不可能

ourcubk

Jun 30, 2016

不可能

chairuosen

Jun 30, 2016

switch 可以打表，简单 if 也可以

zhuangzhuang1988

Jun 30, 2016

1. 表驱动法(看: 代码大全)
2. 使用 match(看: F#, Scala)'
3. 使用类+继承(看: 设计模式)

Jabin

Jun 30, 2016

封装工厂

ipconfiger

Jun 30, 2016

该用则用, if else 是基础逻辑, 如果不是深层次嵌套的话, 最好不要作一些奇技淫巧出来

hyyy

Jun 30, 2016

@wander2008
其实我觉得这是一个有趣的话题^_^

kera0a

Jun 30, 2016

SEL methods[][3] = {
{ @selector(getFirstCell:) },
{ @selector(getSecondCell:) },
{ @selector(getThirdCell:) },
};

SEL sel = methods[indexPath.section][indexPath.row];
[self performSelector:sel withObject:indexPath];

hasbug

Jun 30, 2016

这能避免吗？除非不做判断···

stcasshern

Jun 30, 2016

这个不能吧，印象中这个是最基础得

ma125125t

Jun 30, 2016

目前的 tableView 就是这么艹蛋。但已经有很多开源的方法解决这个问题了（想象一下一百种 cell ，分为数十个 section 和 row 的情况）。去找找吧。

michaelye1988

Jun 30, 2016

我认为这种情况下用 if else 或者 switch 是非常合理的，至少代码可读性还是在的。如果非要套用一些设计模式什么的，维护起来就痛苦了。

chmlai

Jun 30, 2016

这个简单, 用数据驱动就清晰多了

spongebobsun

Jun 30, 2016

@kera0a 学习了

mygirl

Jun 30, 2016

@ma125125t 几十种的 Cell 这应该是属于不合理的设计吧？

chunqiuyiyu

Jun 30, 2016

完美消除是不可能的，不然这几个关键字就不会出现。重要的是理清逻辑，不要嵌套过深。

rekulas

Jun 30, 2016

看不懂两种都是返回 cell 条件为什么不整合在一起

8bit

Jun 30, 2016 via Android

查表，代码大全

subpo

Jun 30, 2016

能只写 if 不写 else 不错了

harry890829

Jun 30, 2016

这种平级的 if...else...已经不错了，我刚接手了一个代码，到手的时候 4-5 层 if 结构嵌套，全都带 else ，因为需求的关系，我现在又在里面加了两层 if..else...结构，我已经恶心死了

woshicixide

Jun 30, 2016 via Android

多态

jason19659

Jun 30, 2016

cell = indexPath.section == 3 ? a : indexPath.row == 2 ? b : c

zhicheng

Jun 30, 2016

静态结构这样写没问题，把 3 和 2 消除就行，用 USER_SECTION 常量定义一下。
动态结构用 model 来判断。

fatedier

Jun 30, 2016

简单工厂模式 + 反射，可以实现根据不同的字符串执行不同业务逻辑，但是不适合用这个替换掉所有的 if-else ，不然就是过度设计了。

之前用 c++ 写过一个例子： http://blog.fatedier.com/2015/03/04/decoupling-by-using-reflect-and-simple-factory-pattern-in-cpp/

lingoerer

Jun 30, 2016

1 ：[CellViewModel] -> 把每一个 Cell 的 ViewModel 定义出来，放进数组
2 ： cellViewModel.identifier, cell.render(cellViewModel) -> 每个 ViewModel 定义自己要用什么 Cell 来展示， dequeue 的时候出不同的 Cell ，然后 Cell 自己对应着绑 UI
3 ： tableView 的回调中直接一行同样的代码把 Cell 弄出来

当然，前面还有个：
0 ：把你要显示的 Model 的内容 map 成 CellViewModel 的数组

ibigbug

Jun 30, 2016

https://zh.wikipedia.org/wiki/图灵机

muller

Jun 30, 2016

策略模式

davisz

Jun 30, 2016

do {
if (false) break;
}while(0);

ilotuo

Jun 30, 2016

这样?
```java
cell = indexPath.section == 3 ? CellFactory.build(...) : indexPath.row == 2 ? CellFactory.build(...): null;
```

xwartz

Jun 30, 2016

map

mdluo

Jun 30, 2016

分支存数组，条件用位运算

ihuotui

Jun 30, 2016 via Android

责任连模式，差不多

hantsy

Jun 30, 2016

State 模式

twoyuan

Jun 30, 2016

首先应该消灭的不是 Magic Number 吗……

sutra

Jun 30, 2016

Map

youyongsong

Jun 30, 2016

pattern match

0x5e

Jun 30, 2016

swift 可能会稍微简化点
let path = (indexPath.section, indexPath.row)
switch(path) {
case (0, 0):
//xxxx
case (0, 1):
//xxxx
}

JasperYanky

Jun 30, 2016

model 决定 view controller 层不关心数据只负责传递

fhefh

Jun 30, 2016

@hyyy https://v2ex.com/t/286497

loveuqian

Jun 30, 2016

之前也是这样判断行数，后来改判断模型 title 了
因为判断行数，一不小心，在第 0 行前面加一行，你这所有判断都要重写啊。。

vincentxue

Jun 30, 2016

看来 iOS 迫切需要一个为 UITableView 或者 UICollectionView 解决数据源封装的库。

sensui7

Jun 30, 2016

OO 中的 tell, don't ask 原则可以消除这种大量的 else ， if else 多说明代码抽象的不好，根据你的逻辑，可以采用相应的设计模式解决

lalalafq

Jun 30, 2016

所有的数据源逻辑（高度，点击，渲染）全部丢到一个 array ；基础的 tableView 委托永远只有大概 10 行左右；
绝壁高聚合，低耦合；产品想怎么改就怎么改；顺序啥的都是小问题。

JohnSmith

Jun 30, 2016

rust 中有个 match

skyoojaa

Jun 30, 2016 via Android

If...else...没法消除，但可以封装判断，让 if...else...里面的代码看起来简洁一些

realpg

Jun 30, 2016

我觉得这种代码很好
瞎搞弄得可维护性差才是大坑

iyeatse

Jun 30, 2016

https://github.com/cocbin/CBTableViewDataSource

dorentus

Jun 30, 2016

@realpg 在最前面加一行的话，要从上到下把数字都改掉，也叫可维护性好么……

so898

Jun 30, 2016

真要很多的话不应该是自定义 Cell ，然后把代码放到 Cell 里面去么？
如果说不能自定义 Cell ，那不是应该通过在外面建立 Cell 内容填充生成的方法来做到精简这一块的代码么？

missdeer

Jun 30, 2016

你看，用 OO 设计模式的时候到了，上面说不行的都不是那个年代过来的人

weirdyu

Jun 30, 2016

你这种按照行数来判断返回何种 cell 的方式不太好

zjyjer

Jun 30, 2016

1. 模式匹配
2. 设计一个状态机每个分支里的代码抽象出一个函数异步调用之

programgou

Jun 30, 2016

我认为我们可以专门提供一个 indexPath 的接口，然后再提供一个 builder ，把实现了 indexPath 接口的类的实例传递给 builder ， builder 就可以返回一个 cell 。这样无论你有多少种 indexPath ，只需要维护 builder ，而不会改变 tableView 的任何代码。当你觉得你现在的 builder 过于复杂的时候，你甚至可以为这个 builder 提供接口，这样你可以在不改变原来代码的基础上，替换掉原来的 builder 。总之，类与类建立关系要通过接口，这样就可以在不改变代码的基础上，换掉一个类。也就是换掉一个功能。

这里面涉及到一些设计模式的东西，不过楼主别怕，看不懂书建议去看源码，我博客上写过一个关于建构者模式的分析，你可以参考一下，不过要解决你的问题一个建构者模式肯定是不够的，还有工厂模式。

sablib

Jun 30, 2016

如果是 swift 的话，就用 enum 了。

billion

Jun 30, 2016

如果语言是 Python 的话，可以使用字典解决：
```
def a(para):
xxx

def b(para):
xxx

def c(para):
xxx

functionDict = {'a':a, 'b': b, 'c': c}

result = functionDict[name](para)
```
这样，当你需要使用 switch 或者 if ...else 来调用函数的时候，可以通过： functionDict('a')(para)这种方式调用函数。

chrisstyle

Jun 30, 2016

同 25 楼～
推荐

i4mszengg

Jun 30, 2016

我想应该不用故意写一些奇葩的逻辑，日后维护很痛苦的吧，如果单个 if 的 condiftions 比较多，可以嵌套 if , 这样更容易懂些，日后好维护。

jasonlz

Jun 30, 2016

用状态机。

ooonme

Jun 30, 2016 via iPhone

@zjyjer 正解

djyde

Jun 30, 2016

程序之所以是程序，因为它能 if else

EAimTY

Jun 30, 2016 via Android

不可能的，不论用什么方法代替，底层肯定还会是 if else

adrianzhang

Jun 30, 2016 via Android

结构化语言之前，都用 goto ，汇编里见 jump 更多。

xwing

Jun 30, 2016

命令链模式。

hrong

Jun 30, 2016 via Android

表驱动，策略模式。

有人说什么反射，想想反射里面有多少个 if/else 就知道用反射有多么的得不偿失

cocoaChina

Jun 30, 2016 via Android

这种判断逻辑必须有，但逻辑内实现却可以瘦身，尽量多用 switch

powergx

Jun 30, 2016 via iPhone

cpu 晶体管可是只有通河短两种情况， lz 竟然说不用 if

rashawn

Jun 30, 2016

最下面都是一样的吧不用这个是不是能跑的快一点

cheng4741

Jun 30, 2016

@EAimTY 并不会，这种情况用表或数组的话，再底层也不会有 if else

aias

Jun 30, 2016

@mygirl Hello ，宁波的同学，又看到你了！招人还顺吗？

peneazy

Jun 30, 2016 via Android

我只知道 js 怎么消除这种东西。利用多态性，比如原型继承，把每一种可能封装在原型里，构造函数直接调用。看看设计模式吧，有至少两种模式能大幅度减少判断语句。

holy_sin

Jun 30, 2016

问题的核心就是区分出不同类型的 cell 要如何渲染。如果在 vc 里做，那么 collectionView 对应的所有 datasource 和 delegate 都需要分别写一套区分方法。所以比较好的方法是在构造数据源的时候，指定好每一种 cell 如何显示。由于 cell 信息可能会比较多，所以就衍生出来用 cellViewModel 来管理这些信息，这就是数据驱动的意思吧。当 cell 的排列关系需要改动的时候，只需要修改构造数据源的地方。其实就是 @lingoerer 所说的做法。