吐槽一下这屎一样的代码

[netnr]

if(act.Trim() == "edit".Trim() || act.Trim()== "detail".Trim())
{
  ...
}

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

实话实说，忍不了

[azcvcza]

trim 不是去空格吗，那 "edit".trim()这种有啥意义

[pushback]

还好，不算屎

[huiyue]

这个同事疑心病不小。

[del1214]

还知道 trim 很厉害了

[chenyj]

==后面画蛇添足

[lloovve]

Js ？ Java 不能用等号比较字符串

[wangkun025]

没有隐藏的 BUG 已经谢天谢地了。

[wangxiaoaer]

@lloovve 目测 c 艹

[wangxiaoaer]

@wangxiaoaer 打错了，c 井

[WeaPoon]

建议你问下他为什么要这样写，总有原因的，听听他的想法。

[ChenFanlin]

`act.Trim()== "detail".Trim()` 等号前面没空格, 后面有空格 忍不了

[netnr]

他回复我的：“你觉得不好 你就改成你认为好的”

[leschans]

后面那个等号两边空格不一样多 swift 会骂人

[None123]

@wangxiaoaer #

[netnr]

其实我想吐槽的是，现在很多小公司，对代码的质量没要求，也没人管

[yuyu2140]

我猜测原先不是 "detail" "edit" ，后面改需求图方便直接复制了改名字

[netnr]

@yuyu2140 你猜得√

[yimity]

那你觉得应该怎么写？

[ifxo]

已经不错了，lz 太偏执

[laoyur]

放宽心啦，能稳定工作就行