吐槽一下这屎一样的代码

[netnr]

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

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

实话实说，忍不了

[ClericPy]

楼主怕是没见过真屎吧... 我就曾经写过, 一个函数两百多行, 我自己都不知道自己在写什么了, 然后又加了一百多行日志进去

[wisunny]

@netnr 我觉得这个回复很好

[fxxwor99LVHTing]

代码能工作就行

[996jiucai]

// 这里可以吗？
if(["edit","detail"].includes(act.Trim())) {

}

[qdwang]

这个只是态度问题，说明他对你们做的项目不在意。除非你们老板会按照态度好坏来给奖金，不然无解

[Daming]

这是一年的工作经验重复了几年
对代码没有一点洁癖

[miv]

@huiyue 哈哈哈哈 笑死了

[LifStge]

@netnr 是啊 怎么都说的过去 说代码敷衍也行 说考虑的多 也不是说不过去.
就上面说的 是替换的字符串 单从片面的看 多此一举的代码 但是如果从全局看 比如这俩字符串是是不是需要改动的 如果改动的时候 直接是批量替换的改 那这里这样写 切实也说的过去了 (虽然多次一举的嫌疑非常大) 不过也是能避免批量操作时候的 隐藏的的问题

[20015jjw]

code review 呢
看吐了

[hoyixi]

过一度时间你会发现，可能你比 TA 还屎

[ipadpro4k]

老罗：又不是不能用

[liaokylin2v]

避免手抖

[opengps]

这是习惯性将 string 进行 trim，是个偷懒的做法，我经常这样（但我不会在已知不包含前后空格的时候多此一举，我也不会将重复使用的多次重新进行 trim ）

[sugars]

能跑就行

[kemikemian]

既然是后面改的,应该问问公司产品为什么改需求的频率比他妈改嫁还勤

[en20]

为什么方法名的首字母是大写？

[ciaoly]

我以为楼主是在吐槽这种“字符串硬编码”的写法。。。。。
这种地方难道不是应该定义一个常量类🐴？

[optional]

@en20 c#和 go 不都是大小写开头吗。

[meteor957]

为什么你在的公司没有 code review ?

[codingadog]

@en20 因为不是 Java （ doge