分享一段我司前辈的代码,哈哈!

2023-06-16 10:09:05 +08:00
 lilei2023

29782 次点击
所在节点    程序员
310 条回复
BlueBing
2023-06-16 10:10:52 +08:00
写的蛮好,一眼懂。
LeegoYih
2023-06-16 10:12:13 +08:00
嘲讽前辈,理解前辈,成为前辈。
Moyyyyyyyyyyye
2023-06-16 10:12:42 +08:00
简单逻辑简单写,还整个设计模式吗
Jxnujason
2023-06-16 10:14:19 +08:00
如果 webstorm 估计会提示,还能一键转掉
751327
2023-06-16 10:14:20 +08:00
所有这个应该怎么写?
yazinnnn
2023-06-16 10:15:54 +08:00
@LeegoYih
超越前辈.exe
iyiluo
2023-06-16 10:16:19 +08:00
没毛病,鬼知道 type 有多少种
iapplebear
2023-06-16 10:17:38 +08:00
没啥毛病,逻辑清晰,用三元逼格会高点?
qwertyzzz
2023-06-16 10:18:07 +08:00
又不是不能跑
tsohgdivil
2023-06-16 10:18:08 +08:00
有啥好笑的
sockball07
2023-06-16 10:18:58 +08:00
好奇 前端都不定义常量吗 也没注释这个 2 是什么东西
meteor957
2023-06-16 10:20:24 +08:00
以前我还会看看,现在感觉好无聊... 可能是我岁数大了
lowett
2023-06-16 10:20:32 +08:00
代码配色挺好啊,看着很舒服
taotaodaddy
2023-06-16 10:20:39 +08:00
三元倒不至于,无非是
this.isSms = (channelType == 2);
说实话如果贵司考核代码行数,宁愿像你前辈那样写...
Xbluer
2023-06-16 10:21:20 +08:00
挺好的。多了几行,把逻辑运算和赋值分开,简单明了。
Helsing
2023-06-16 10:21:22 +08:00
有什么问题吗,除了没有把 2 定义为不可变常量
lilei2023
2023-06-16 10:21:29 +08:00
主要是想起来自己刚学的时候,也是这样写的,猜测应该是刚入门的前辈的代码
taotaodaddy
2023-06-16 10:21:48 +08:00
魔法数字确实应该重构
vdrapb
2023-06-16 10:22:28 +08:00
整挺好,一眼就知道他要干嘛
xujinkai
2023-06-16 10:23:11 +08:00
说不定是以前有好几个 if else 现在删的就剩这个了 每一段历史代码都有一段故事

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/949195

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX