分享一段我司前辈的代码，哈哈！

[lilei2023]

[MuSeCanYang]

简单易懂没毛病

[dudubaba]

以前觉得这种多余，直接等于 channelType === 2 这种多好，但是这个条件下还有其他赋值，代码就得改成 if else 那还不如最早就用这种。

[danhua]

感觉没什么毛病，简单易懂。或者说可以写简单点但是难懂增加不可替代性。

[LaGeNanRen]

我约了个 meeting ，下班你待一下我们聊聊
@lilei2023

[xyjincan]

改个变量名称 this.sms 咋样

[rivercherdeeeeee]

写的没有问题，记住工作和打工二码事

[hideonwhere]

弄不好原本还有别的业务然后删掉了
但是这种也不影响就是了

[kuaner]

不明白 OP 在笑啥

[fresco]

op 刚上班可能

[twofox]

我朋友在的公司，他们的后端提供的接口是中文的
炸裂

[HFX3389]

笑完记得改代码，别只顾着笑

[sl0000]

用 switch 效率高点

[LawlietZ]

你在嘲笑啥呢 我觉得是你年轻了。。。

[F281M6Dh8DXpD1g2]

以前不只有 channelType == 2 这一个条件吧

[Aviciii]

虽然一行代码能搞定
但是这种写法其实更好，不明白 OP 在笑啥。。。。。

[GzhiYi]

多写一些，可能存在的问题：
1. == 符号尽量少用，这会让类型变得模糊难定，尽可能用全等。
2. channelType 应该使用常量替代，让 2 这个值更具有可读性（这个重要）。

至于楼上说了那么多的拿三元运算符替换，我觉得是不应该的。强行将逻辑 if else 变更为三元运算符，是个坏主意。

再看这个是 vue 的 watch ，除开 channelType 不一定为 2 之外，还可能在后续判断 channelType 后做一些其余的在 channelType 变更后的逻辑。

所以我的看法是，尽管有优化，但写法 if else 没毛病。

[lilei2023]

刚才在捋 n 个前辈的代码，还是继续堆💩山把，跑起来就行，不管那么多了

[fao931013]

可以改 但没有必要

[newmlp]

简单易懂

[finab]

菜鸟程序员会这样写
中等程序员会直接赋值
经验丰富的程序员会回到这种写法，这样是理解成本最低阅读障碍最小甚至不用思考就能看懂的写法

这段代码里魔法数字的问题，比这个要严重的多，最应该改的不是 if else 而是这个