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

[lilei2023]

[cwWqjBJJRPak]

难道你还想让他用 channelType===2 吗？！

[jsrunner]

自从自己写了烂代码太多了，现在小弟不喷其他人了

[thorneLiu]

@iapplebear 用得着三元？😂😂😂😂

[still97]

@xujinkai 也有可能一开始写完发现只有一行，考虑到代码行数的考核指标，苦思冥想，改成这样了

[gosansam]

应该是删了别的 else 吧

[isno]

真没必要笑话人家，也没必要在这种小技巧上争强好胜！

[lilei2023]

@lowett vscode monokia

[lyxeno]

成为前辈!

[javen73]

绩效按代码行数吗？😂

[ixixi]

并不是几行所谓的不优雅代码 就能看出别人水平

写一堆自认为风骚的代码 对下个人来说可能就是屎山

多写几行没问题

[shyrock]

OP 把自己的版本发出来，让 v2er 对两者打分看看。

[Ricardoo]

其实挺好的。
如果是自己的私有代码，随便使用利用语言特性，一行代码搞定.
但是如果是项目代码，一眼能让人看懂的代码就是好代码，而且这种代码方便在分支中增加逻辑，想想善变的需求，你以为前辈在初学层级，实际人家是懂业务懂代码的=。=！

[angrylid]

那么比较优雅的写法是下面这样吗？
isSms = computed( () => channelType === 2 )

[lilei2023]

嘲笑谈不上，谁不是从 0-1 开始的，只是感慨，项目这么久了也没人加注释，没人优化

[ksco]

楼上都给我看傻了，这都能洗？😨

[Yukiteru]

简单易懂，没毛病。

[lilei2023]

@angrylid 要是我现在写，肯定写成你这样了
@shyrock 看看#33

[miaotaizi]

代码写再好, 跟业务不匹配有什么用?

[Huelse]

硬要说的话也就判等和枚举有问题，而且我记得会被编译成 this.isSms = ...

[erwin985211]

现在公司好多这样的代码 主要搞得一个组件就有 4 、500 行，看着很难受