吐槽一下这屎一样的代码

[netnr]

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

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

实话实说，忍不了

[serical]

楼主会怎么写呢

[lululau]

每个人的能力不一样，薪资不一样，年终奖不一样，价值观也不一样；我的价值观是：随意在背后说吐槽别人代码和屎一样的人，人品和屎一样

[deepall]

能看懂就万事大吉了

[xianxiaobo]

也许你从来没有写过屎一样的代码。

[ZehaiZhang]

其他语言写 if(xxx.trim())会报错，我记得 java 是

[gaoryrt]

也有可能是把代码统一替换过的
之前是 `if(act.Trim() == EDIT.Trim() || act.Trim()== DETAIL.Trim())`

[serge001]

@996jiucai 我日常就是这么写,看到楼主贴的代码,我也会想打人

[lizz666]

@996jiucai
用 includes 的话还要内部遍历一次是不是没必要呢？
不过我并没有研究过这些 API 内部遍历效率与短路运算的效率对比

[Frank9621]

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

这样子？

[VictorJing94]

@wangxiaoaer C#的话建议用 Equals

[jasonding]

想起前几天看到个截图，几十行的 for 和 if 开头，层层嵌套

[aino]

还好吧，放宽心，不同公司，不同项目，标准都是参差不齐的，抱怨也没用，解决才是真理

[onecode]

@lloovve 加上方法首字母大写，应该是 C#

[rming]

没毛病，代码很整齐，很健壮，即使手残多大一个空格也正常运行 [狗头]

[nuk]

没毛病，但是约定一下 act 不包含空格更好吧。
这种保护没有任何意义。

[wellhope]

😂至少还能让你读懂，不够💩的级别

[qiumaoyuan]

少年不识屎滋味

[lovecy]

这是一条完整光滑的，无虫的屎，能看出完整的屎结构，就是有点臭。
就怕那种拉稀式的代码，一坨一坨看不出结构

[sidegem]

能看懂，不算💩

[zxcslove]

@kemikemian 亲，这句话力度有点不足，建议改成“为什么改需求的频率快赶上他妈改嫁了”