吐槽一下这屎一样的代码

2020 年 7 月 20 日
 netnr
if(act.Trim() == "edit".Trim() || act.Trim()== "detail".Trim())
{
  ...
}

同事写的，也算是工作几年了

实话实说，忍不了

8251 次点击
所在节点    编程
66 条回复
azcvcza
2020 年 7 月 20 日
trim 不是去空格吗，那 "edit".trim()这种有啥意义
pushback
2020 年 7 月 20 日
还好，不算屎
huiyue
2020 年 7 月 20 日
这个同事疑心病不小。
del1214
2020 年 7 月 20 日
还知道 trim 很厉害了
chenyj
2020 年 7 月 20 日
==后面画蛇添足
lloovve
2020 年 7 月 20 日
Js ？ Java 不能用等号比较字符串
wangkun025
2020 年 7 月 20 日
没有隐藏的 BUG 已经谢天谢地了。
wangxiaoaer
2020 年 7 月 20 日
@lloovve 目测 c 艹
wangxiaoaer
2020 年 7 月 20 日
@wangxiaoaer 打错了，c 井
WeaPoon
2020 年 7 月 20 日
建议你问下他为什么要这样写，总有原因的，听听他的想法。
ChenFanlin
2020 年 7 月 20 日
`act.Trim()== "detail".Trim()` 等号前面没空格, 后面有空格 忍不了
netnr
2020 年 7 月 20 日
他回复我的：“你觉得不好 你就改成你认为好的”
scsb
2020 年 7 月 20 日
后面那个等号两边空格不一样多 swift 会骂人
None123
2020 年 7 月 20 日
@wangxiaoaer #
netnr
2020 年 7 月 20 日
其实我想吐槽的是，现在很多小公司，对代码的质量没要求，也没人管
yuyu2140
2020 年 7 月 20 日
我猜测原先不是 "detail" "edit" ，后面改需求图方便直接复制了改名字
netnr
2020 年 7 月 20 日
@yuyu2140 你猜得√
yimity
2020 年 7 月 20 日
那你觉得应该怎么写？
ifxo
2020 年 7 月 20 日
已经不错了，lz 太偏执
laoyur
2020 年 7 月 20 日
放宽心啦，能稳定工作就行

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

https://study.congcong.us/t/691642

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

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

© 2021 V2EX