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

[lilei2023]

[iOCZ]

能一眼看懂的都是好代码

[Vegetable]

这个有很多改进的方法，但这种代码并不值得嘲笑。

这个代码足够简单，虽然写的不优雅，但是也没带来什么问题，甚至可以非常简单的改写成 computed 。写多了业务代码就会意识到，庞大的项目中，类似这种缺乏设计的细节肯定会非常多，代码最关键的是可靠而不是漂亮。如果一段不太好的代码并没有带来理解上的困难和性能上的问题，他就是标准的，合格的代码。

[caqiko]

@taotaodaddy #14 这个比 33 楼更直观

[fisherman0459]

@angrylid 只是 watch 变动, 不能 computed 依赖吧

[wulili]

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

[lincanbin]

可能考核代码行数，我这边就是这样，代码都成屎山了。
还在坚持少写没必要代码的人可能只有我了。

[zjw7sky]

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

[zsc8917zsc]

写成#33 楼的好处是什么呢？

[zj9495]

this.isSms = channelType == 2 ? true : false

你怕是没有见过这种写法的 🐶

[ksco]

这段代码暴露出的最大的问题，是贵司根本没有任何实际意义上的 code review 。一个允许将这样的代码提交到主线的项目，我有理由详细其他的地方也是屎山，为了身心健康，快跑吧。

[oatw]

楼主是不是想说用 computed ？

[343]

这么写很好，甚至推荐所有开发这样写。
因为不知道后面产品会增加什么需求，可能在这个判断条件下，还会增加非常多的代码。

[wei2629]

楼主能不能贴个标准答案，我想学习下怎么写。

[loveumozart]

程序员不应该抠这些细节，这种思维方式不好

[MangoCloud]

程序员就是喜欢喷别人的代码，这代码不好吗，小小年纪以为懂一点代码技巧就可以纵横江湖了

[Narcissu5]

评论区居然这么多人没有看出问题在哪儿。。。

[superedlimited]

@iapplebear 这是半前辈

[westoy]

这代码挺好的啊

等增加需求出来一个 chanelType = 3/4/5/6 ， 处理的东西可能不止一个 this.isSms

这段代码只需要根据情况增加块

用三元或者 map 就需要大改了

[evil0harry]

我觉得是删了 else

[cwWqjBJJRPak]

@zsc8917zsc 好处就是传进来一个'2'直接死逼