吐槽一下这屎一样的代码

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

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

• 資深大佬 : 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

• 資深大佬 : 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 亲，这句话力度有点不足，建议改成“为什么改需求的频率快赶上他妈改嫁了”

• 資深大佬 : dreampet

  强迫症写的代码

• 資深大佬 : terrywater

  能看懂，并且无 bug，不影响性能的代码，不算大粪。

• 資深大佬 : kikyous

  这代码真的还好，你是没见过更垃圾的吧

• 資深大佬 : rigortek

  @netnr 明显讨厌你了，觉得你找事

• 主 資深大佬 : netnr

  目前做的一个老项目升级改版，整个项目没理由可言，还有比这更恼火的，比如写绝对路径资源 等等
  如果是新项目，会要求同事整改

• 資深大佬 : Acoolda

  不会用代码格式工具吗？也就随手一下的事情