这是一个创建于 1959 天前的主题，其中的信息可能已经有所发展或是发生改变。
if(act.Trim() == "edit".Trim() || act.Trim()== "detail".Trim()) { ... } 同事写的，也算是工作几年了
实话实说，忍不了
第 1 条附言 2020-07-21 11:20:10 +08:00
我想说的就是 1 楼的意思
我首先跟当事人吐槽后再发出来的吃瓜，不是在背后，从来就是当面
我首先跟当事人吐槽后再发出来的吃瓜，不是在背后，从来就是当面
 | 1 azcvcza 2020-07-20 17:05:33 +08:00 trim 不是去空格吗，那 "edit".trim()这种有啥意义 |
 | 2 pushback 2020-07-20 17:11:28 +08:00 还好，不算屎 |
 | 3 huiyue 2020-07-20 17:11:40 +08:00  26 这个同事疑心病不小。 |
 | 4 del1214 2020-07-20 17:13:13 +08:00 还知道 trim 很厉害了 |
 | 5 chenyj 2020-07-20 17:19:58 +08:00 ==后面画蛇添足 |
 | 6 lloovve 2020-07-20 17:23:42 +08:00 via iPhone Js ？ Java 不能用等号比较字符串 |
 | 7 wangkun025 2020-07-20 17:25:40 +08:00 没有隐藏的 BUG 已经谢天谢地了。 |
 | 8 wangxiaoaer 2020-07-20 17:25:48 +08:00 via iPhone @lloovve 目测 c 艹 |
| 9 wangxiaoaer 2020-07-20 17:26:12 +08:00 via iPhone @wangxiaoaer 打错了，c 井 |
 | 10 WeaPoon 2020-07-20 17:28:13 +08:00 建议你问下他为什么要这样写，总有原因的，听听他的想法。 |
 | 11 ChenFanlin 2020-07-20 17:28:28 +08:00 6 `act.Trim()== "detail".Trim()` 等号前面没空格, 后面有空格 忍不了  |
 | 12 netnr OP 他回复我的：“你觉得不好 你就改成你认为好的” |
 | 13 scsb 2020-07-20 17:30:16 +08:00 via iPhone 后面那个等号两边空格不一样多 swift 会骂人 |
 | 14 None123 2020-07-20 17:30:48 +08:00 @wangxiaoaer # |
| 15 netnr OP 其实我想吐槽的是，现在很多小公司，对代码的质量没要求，也没人管 |
 | 16 yuyu2140 2020-07-20 17:36:12 +08:00 1 我猜测原先不是 "detail" "edit" ，后面改需求图方便直接复制了改名字 |
 | 18 yimity 2020-07-20 20:56:55 +08:00 1 那你觉得应该怎么写？ |
 | 19 ifxo 2020-07-20 20:58:00 +08:00 已经不错了，lz 太偏执 |
 | 20 laoyur 2020-07-20 21:20:00 +08:00 放宽心啦，能稳定工作就行 |
 | 21 ClericPy 2020-07-20 21:28:11 +08:00 楼主怕是没见过真屎吧... 我就曾经写过, 一个函数两百多行, 我自己都不知道自己在写什么了, 然后又加了一百多行日志进去 |
 | 23 fxxwor99LVHTing 2020-07-20 23:15:45 +08:00 代码能工作就行 |
 | 24 996jiucai 2020-07-20 23:31:52 +08:00 3 // 这里可以吗？ if(["edit","detail"].includes(act.Trim())) { } |
 | 25 qdwang 2020-07-20 23:57:29 +08:00 via iPhone 这个只是态度问题，说明他对你们做的项目不在意。除非你们老板会按照态度好坏来给奖金，不然无解 |
 | 26 Daming 2020-07-21 00:08:04 +08:00 1 这是一年的工作经验重复了几年 对代码没有一点洁癖 |
 | 28 LifStge 2020-07-21 00:28:17 +08:00 @netnr 是啊 怎么都说的过去 说代码敷衍也行 说考虑的多 也不是说不过去. 就上面说的 是替换的字符串 单从片面的看 多此一举的代码 但是如果从全局看 比如这俩字符串是是不是需要改动的 如果改动的时候 直接是批量替换的改 那这里这样写 切实也说的过去了 (虽然多次一举的嫌疑非常大) 不过也是能避免批量操作时候的 隐藏的的问题 |
 | nbsp; 29 20015jjw 2020-07-21 03:23:21 +08:00 via Android code review 呢 看吐了 |
 | 30 hoyixi 2020-07-21 06:55:47 +08:00 过一度时间你会发现，可能你比 TA 还屎 |
 | 31 ipadpro4k 2020-07-21 08:09:53 +08:00 via iPhone 老罗：又不是不能用 |
 | 32 liaokylin2v 2020-07-21 08:10:26 +08:00 via Android 避免手抖 |
 | 33 opengps 2020-07-21 08:25:35 +08:00 这是习惯性将 string 进行 trim，是个偷懒的做法，我经常这样（但我不会在已知不包含前后空格的时候多此一举，我也不会将重复使用的多次重新进行 trim ） |
 | 34 sugars PRO 能跑就行 |
 | 35 kemikemian 2020-07-21 08:28:28 +08:00 既然是后面改的,应该问问公司产品为什么改需求的频率比他妈改嫁还勤 |
 | 36 en20 2020-07-21 08:31:24 +08:00 via iPhone 为什么方法名的首字母是大写？ |
 | 37 ciaoly 2020-07-21 08:33:56 +08:00 via Android 我以为楼主是在吐槽这种“字符串硬编码”的写法。。。。。 这种地方难道不是应该定义一个常量类？ |
 | 39 meteor957 2020-07-21 08:53:26 +08:00 为什么你在的公司没有 code review ? |
 | 41 serical 2020-07-21 09:03:15 +08:00 via iPhone 楼主会怎么写呢 |
 | 42 lululau 2020-07-21 09:31:18 +08:00 3 每个人的能力不一样，薪资不一样，年终奖不一样，价值观也不一样；我的价值观是：随意在背后说吐槽别人代码和屎一样的人，人品和屎一样 |
 | 43 deepall 2020-07-21 11:29:08 +08:00 能看懂就万事大吉了 |
 | 44 xianxiaobo 2020-07-21 12:40:52 +08:00 也许你从来没有写过屎一样的代码。 |
45
ZehaiZhang 2020-07-21 13:26:37 +08:00 其他语言写 if(xxx.trim())会报错，我记得 java 是
 | 46 gaoryrt 2020-07-21 13:39:33 +08:00 也有可能是把代码统一替换过的 之前是 `if(act.Trim() == EDIT.Trim() || act.Trim()== DETAIL.Trim())` |
 | 48 lizz666 2020-07-21 14:12:00 +08:00 |
 | 49 Frank9621 2020-07-21 14:37:20 +08:00 act = act.Trim(); if(act == "edit" || act == "detail") { ... } 这样子？ |
 | 50 VictorJing94 2020-07-21 14:47:50 +08:00 @wangxiaoaer C#的话建议用 Equals |
 | 51 jasonding 2020-07-21 15:06:04 +08:00 想起前几天看到个截图，几十行的 for 和 if 开头，层层嵌套 |
 | 52 aino 2020-07-21 15:20:15 +08:00 还好吧，放宽心，不同公司，不同项目，标准都是参差不齐的，抱怨也没用，解决才是真理 |
 | 54 rming 2020-07-21 15:20:51 +08:00 没毛病，代码很整齐，很健壮，即使手残多大一个空格也正常运行 [狗头] |
 | 55 nuk 2020-07-21 15:29:27 +08:00 没毛病，但是约定一下 act 不包含空格更好吧。 这种保护没有任何意义。 |
 | 56 wellhope 2020-07-21 15:40:11 +08:00 via iPhone 至少还能让你读懂，不够的级别 |
 | 57 qiumaoyuan 2020-07-21 15:48:24 +08:00 少年不识屎滋味 |
 | 58 lovecy 2020-07-21 16:15:23 +08:00 这是一条完整光滑的，无虫的屎，能看出完整的屎结构，就是有点臭。 就怕那种拉稀式的代码，一坨一坨看不出结构 |
 | 59 sidegem 2020-07-21 16:19:59 +08:00 能看懂，不算 |
 | 60 zxcslove 2020-07-21 16:42:54 +08:00 @kemikemian 亲，这句话力度有点不足，建议改成“为什么改需求的频率快赶上他妈改嫁了” |
 | 61 dreampet 2020-07-21 16:48:14 +08:00 强迫症写的代码 |
 | 62 terrywater 2020-07-21 16:49:16 +08:00 能看懂，并且无 bug，不影响性能的代码，不算大粪。 |
 | 63 kikyous 2020-07-21 16:56:26 +08:00 这代码真的还好，你是没见过更垃圾的吧 |
| 65 netnr OP 目前做的一个老项目升级改版，整个项目没理由可言，还有比这更恼火的，比如写绝对路径资源 等等 如果是新项目，会要求同事整改 |
 | 66 Acoolda 2020-07-22 09:01:31 +08:00 via Android 不会用代码格式工具吗？也就随手一下的事情 |
Select Language