作者saladim (杀拉顶)
看板Soft_Job
标题[请益] 痾 遇到这种事情 是不是需要赶快离职了?
时间Mon Jul 22 23:55:30 2024
小鲁目前在一家还算大的公司工作 现在有两三位顶大的junior的同事
写程式的习惯让我觉得是不是要赶快跑了 举两个例子好了
他们都喜欢if-else combo, 没巢状到波动拳那麽深 但就是动不动就if-else
三层 然後三层里面还会再if-else
第二个例子就是如果有function 90%(50~100行)适合他们想要的用途,
他们就会copy整个function 然後修改一下後"整段"code插到他们需要的地方
光是上面两个例子已经让我的头超大的 原本公司的codebase已经长得像科学怪人
了 然後又有他们持续"加持" 根本就没办法理解
更惨的是 跟他们讲他们也不当一回事 又有顶大当红硕论护体 一副你也不敢砍我的底气
公司也没人想要当坏人 code review也没人出声 而且大家都知道 上市公司每个都喜欢
有学历的人当门面 反正真正主力有人会扛 XDDDD
每次改到他们经手过的code都很痛苦 若是要帮忙擦屁股根本擦不完阿 因为一直拉....
自己宝贵的时间也都被吃掉了
这样是不是该走了比较好?
我知道烂code到处都是啦 但是至少不能一直拉吧 又是都讲不听的 更资深的也都能闪就闪
但大家轮流中枪.............
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 1.164.97.88 (台湾)
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1721663732.A.097.html
1F：推 lee457088: 你不如放段 code 举个例我们才知道严重程度有多大呀～ 07/23 00:10
2F：推 soga5731: 打不赢就加入，你code写得再好薪水会拿比较多？ 07/23 00:16
3F：→ DrTech: 不知道为什麽要强调学历。你换了更好的工作，只会有更多学 07/23 00:26
4F：→ DrTech: 历好的顶大硕。 07/23 00:26
5F：推 wulouise: code review建立在平等尊重，没有的话的确没什麽好说的 07/23 00:27
6F：→ wulouise: 不是你的理由不够好，就是太minor有更重要的事 07/23 00:27
7F：→ DrTech: 贵公司主管有在意这些事吗？ 不要当个小兵，操皇帝的心。 07/23 00:29
8F：→ saladim: 不是操皇帝的心 是影响到我了 07/23 00:32
9F：推 ko27tye: 薪水还行的话就练练粪坑里挑黄金的能力 烂code单位到处 07/23 00:33
10F：→ ko27tye: 都是 你迟早要面对的 07/23 00:33
11F：→ saladim: 提到学历+当红硕论的原因 文中有写了 07/23 00:33
12F：→ saladim: 不是要讨论程式or review 哲学, copy 100行的程式插到另 07/23 00:36
13F：→ saladim: 外一个地方 很难相信有单位接受每次都这样做 还是再说一 07/23 00:37
14F：→ saladim: 次 这边不是讨论哲学...是在提一件很基本的事情.... 07/23 00:37
15F：推 hermithsieh: 放心啦，直接AI重新整理就好 07/23 00:44
16F：→ acgotaku: 如果程式已经肥大老旧到等待重构,旧 method 被大量呼叫 07/23 00:47
17F：→ acgotaku: 我是觉得这种方式不能说不好,你也没更安全的做法实作 07/23 00:49
18F：→ acgotaku: 要重构不知历史没单元测试的旧方法,是我也想动拉 07/23 00:52
19F：推 neo5277: 你们单位有code rule嘛? 07/23 00:52
20F：→ acgotaku: 这种只能等待某个时机点 有人开始重视并重构专案 07/23 00:53
21F：→ kissmickey: 会提到学历是不是抠抠不平衡 07/23 00:53
22F：→ kissmickey: 要不找主管谈到愿意擦屎 07/23 00:53
23F：→ kissmickey: 要不就离职 07/23 00:53
24F：→ neo5277: 100行感觉还好阿 07/23 00:54
25F：推 nacy204327: 影响到你也不能如何 只能报主管处理 或离职 07/23 00:54
26F：推 WaterLengend: 主管在会议还默许这样，是我打不赢就加入，干嘛因 07/23 00:54
27F：→ WaterLengend: 为别的智障换工作 07/23 00:54
28F：推 abccbaandy: 上面都不在意了你在意干嘛？ 一起拉阿 07/23 00:56
29F：→ acgotaku: 大点软体公司一年总会有三到五成精力重构不合需求旧专案 07/23 00:57
30F：推 SHANGOYANYI: 你就拉的比他们更凶更超过 改天就换他们求你改写法 07/23 00:58
31F：→ SHANGOYANYI: 了 07/23 00:58
32F：推 Bencrie: 公司是不是给你很多钱让你舍不得离职 07/23 01:02
33F：嘘 B0988698088: 干你屁事 07/23 01:20
34F：推 neo5277: 感觉是银行 07/23 01:23
35F：推 ohmylove347: 没有会当人的主管吗？这种写法review都会被喷吧？ 07/23 01:37
36F：→ c8dog: 因为这种原因想走，在别的公司还是会遇到的，如果没有改变 07/23 01:43
37F：→ c8dog: 环境的能力，还是别想东想西 07/23 01:43
38F：→ superpandal: 我遇到的都是老屁股或有权的在乱搞 有时候会怀疑人生 07/23 01:53
39F：→ superpandal: 到底这样有什麽意义 07/23 01:54
40F：→ superpandal: 不过这篇毕业没多久的人就像老油条一样想着保饭碗拖 07/23 01:57
41F：→ superpandal: 时间 真的有前途 比我傻傻受伤害好多了 07/23 01:59
42F：→ superpandal: 现在职场就是谁跟你好来好去当好人 07/23 02:00
43F：嘘 MoonCode: 红明显 建议不要内耗 好好刷题跳槽 07/23 02:06
44F：推 Phater: junior都吃不下那要你review是橡皮图章喔 07/23 02:20
45F：嘘 alihue: 离职 07/23 06:15
46F：嘘 knives: 新人就是这样，功能能跑就好了，除非有人叫你重构，不然别 07/23 06:36
47F：→ knives: 动别人的程式 07/23 06:36
48F：推 pot1234: 为什麽要改他们经手过的code，他们不能自己改吗？ 07/23 07:11
49F：→ pot1234: 话说你们公司不code review吗 07/23 07:13
50F：嘘 brucetu: 快跑垃圾公司 07/23 07:16
51F：推 catchfly: 向上反应，无效就跑 07/23 07:55
52F：→ ericthree: 钱多吗? 主管不在意的话就大家一起写烂code啊 07/23 08:06
53F：→ stepnight: 烂Code哪里都有，很在意就面试先直球问这点 07/23 08:10
54F：→ stepnight: 二的情况我也遇过，真要说优点就是低耦合 07/23 08:10
55F：→ stepnight: 新人进来第一天就能直接改code，改坏了就 07/23 08:10
56F：→ stepnight: 那个func坏掉而已，免洗好用 07/23 08:10
57F：→ stepnight: 而且你帮他们改code干嘛？最烦新人进公司 07/23 08:11
58F：→ stepnight: 一知半解在那自以为的改code，有的是历史问题 07/23 08:11
59F：推 jobintan: 那是人家顶大资深工程师厉害的地方，他们先到处屙屎， 07/23 08:38
60F：→ jobintan: 等接手的菜鸡处理不来，自己再带菜鸡清理屎山，到时候O 07/23 08:38
61F：→ jobintan: KR就能说是自己的credit了。 07/23 08:38
62F：推 vinex518: 离职 07/23 08:51
63F：推 TPdidilong: 问就是快陶 07/23 08:57
64F：推 NDark: if elss 本身并没有错 都要看情况 07/23 09:06
65F：→ nayeonmywife: 就算你到其他地方也都还是会有烂code 07/23 09:33
66F：推 Lhmstu: 没差吧，你知道这样是烂就足够了，他们早晚有一天会知道 07/23 09:38
67F：→ airtsubasa: 没设定程式风格的公司 你是主管吗 不是的话 管很宽喔 07/23 09:59
68F：推 KyuubiKulama: 有讲过就好，不然就看你有没有心跟主管提或是下去 07/23 10:04
69F：→ KyuubiKulama: 带人啦啦啦 07/23 10:04
70F：推 ji3g4zo6: 同58楼 不懂就问 不然就离职就好 07/23 10:07
71F：推 johnbill: 接受不了就走吧 07/23 10:35
72F：推 abc0922001: 我现在写也都会直觉式写if else，之後再重构 07/23 10:40
73F：→ abc0922001: 现在有AI就方便多了，只要再code review 就好 07/23 10:41
74F：→ zxzx8059: 有些chip原厂出来的sdk塞了满满巢状switch-case + 07/23 10:43
75F：→ zxzx8059: if-else + do-while combo 07/23 10:43
76F：推 alex70266: 看主管态度啊，主管觉得没差的话就…适当的切好自己的i 07/23 11:13
77F：→ alex70266: nterface就好。除非你是mentor，不然真的bug等他们自己 07/23 11:13
78F：→ alex70266: 处理 07/23 11:13
79F：→ alex70266: coding sense这种东西强求不来的 07/23 11:15
80F：推 candy668866: 觉得受不了就走人吧 07/23 11:15
81F：→ alex70266: 我自己的经验就是让他们自己痛几下才会学乖，那就不要 07/23 11:19
82F：→ alex70266: 太早出手做些什麽 07/23 11:19
83F：嘘 worf: 你主管都不care了 07/23 11:24
84F：→ yamagishi: 纯看你文章内容，这麽 junior 的问题用 gpt 尬一下就好 07/23 11:35
85F：→ yamagishi: ㄌ 07/23 11:35
86F：推 fatb: 哈欠 不是人的问题一率都不是问题 07/23 11:40
87F：→ labbat: 用一堆SETcc MOVcc CMPccXADD会比较高级吗 07/23 11:58
88F：推 leo08210917: If else现在丢AI优化方便多了 07/23 12:14
89F：嘘 DarkIllusion: code review你还不是连个屁都不敢放 问就是离职 07/23 12:15
90F：推 arhtur945: Request changes? 07/23 12:58
91F：→ sososlee: 你学历一定很烂才一直靠北学历 07/23 13:18
92F：→ angusyu: 问就是离职，怕你不敢 07/23 13:32
93F：嘘 pilor: 没图没依据 07/23 13:48
94F：嘘 psychic: 自己写自己的，又没碍到你 interface切乾净就好 07/23 13:51
95F：推 doranako: 公司code review 文化的问题, 07/23 14:20
96F：→ shadow0326: 动不动就if-else三层 然後三层里面还会再if-else 07/23 14:35
97F：→ shadow0326: 为什麽不直接说四层? 07/23 14:35
98F：推 lovdkkkk: 不知能不能找一个写得好的专门 review 跟整理/清理程式 07/23 15:07
99F：→ lovdkkkk: PR 都要过他，他负责整理好再并 07/23 15:08
100F：推 lens82801: 老板都不介意惹你何必操心呢 07/23 15:16
101F：嘘 lturtsamuel: 当到 senior 还没办法在 code review 上管动 junior 07/23 15:51
102F：→ lturtsamuel: 是怎麽把自己搞得这麽卑屈？ 07/23 15:51
103F：→ lturtsamuel: 2008就在发文 根本超级老 怎麽会连几个刚毕业的都管 07/23 15:52
104F：→ lturtsamuel: 不动 到底人缘多差 07/23 15:52
105F：→ lturtsamuel: 会靠北 if-else 的人 往往是想要搞一些 strategy pat 07/23 15:55
106F：→ lturtsamuel: ern 或依赖反转之类的东西 但又常不喜欢写文件 最後 07/23 15:55
107F：→ lturtsamuel: 就是一大坨没有文件又过度设计的屎 还不如几个 if-el 07/23 15:55
108F：→ lturtsamuel: se 07/23 15:55
109F：嘘 kingofage111: 有本事就到有厉害同事的公司啊，没本事的话你也跟 07/23 16:09
110F：→ kingofage111: 你同事半斤八两，不要在那边自我优越了 07/23 16:09
111F：嘘 kingofage111: 看你16年前的文在那边笑前辈，现在发这种文，看来 07/23 16:26
112F：→ kingofage111: 你高度就这样而已 07/23 16:26
113F：嘘 kingofage111: 拿出实力，自然别人就会听有实力的人的话，没实力 07/23 16:29
114F：→ kingofage111: 剩张嘴实习生都不会理你啦 07/23 16:29
115F：→ knives: 已经16年了阿，怎麽还在这当这种小事 07/23 16:39
116F：嘘 ma721: 是不是傻，他们比你赚的清松 07/23 16:50
117F：嘘 stepnight: 袜靠，工作快二十年还在嘴学历，是什麽粪职涯 07/23 17:56
118F：嘘 hobnob: 第一次看到工作十几年还压不过Junior的 07/23 18:03
119F：→ gpctv: 本来想分享我的类似经验，但我看被嘴成这样，想想还是算了 07/23 18:26
120F：→ gpctv: 。 07/23 18:26
121F：→ cylee: 你觉得写的不好，那有提出你的解法吗? Code review 要出声 07/23 18:49
122F：→ cylee: review 时每次开炮，我就不信不会改 07/23 18:50
123F：推 HelloPTT: 问题可能不在if else，有做好分类、单一职责吗 07/23 19:14
124F：推 ikachann: 懒一点 把那些小段落复制丢AI请他帮你简化就好 07/23 19:39
125F：→ TSMCfabXX: 以前你可能需要逃 现在你有 AI 辅助 正面上他 07/23 19:40
126F：推 Csongs: 其实真的可以叫他请ai code review 07/23 19:42
127F：→ ma721: 半桶水响叮当，问ai都比你强 07/23 20:16
128F：嘘 loliconrd: 你是不是看人家学历很自卑 07/23 21:55
129F：→ uncle925: 问就是：你听过Copilot吗 07/23 22:14
130F：推 viper9709: 能找到更好的就走～不然也只能继续撑... 07/23 23:46
131F：→ ECMA: 还在手写程式喔 呵呵 07/23 23:55
132F：→ saladim: ㄟ 不是啦 很多人没看文中有写 都有跟对方讲过了 也有其 07/24 00:37
133F：→ saladim: 他相对资深的也有讲过 而且大家都是尊重平等理解的状况下 07/24 00:38
134F：→ saladim: 去看程式码跟讨论的 这是很自然的事 是不是大家把自己或 07/24 00:39
135F：→ saladim: 是自己的体会带入才会觉得别人都在乱当? 现在这时代早就 07/24 00:40
136F：→ saladim: 只能用尊重平等理解的方式来讲话了 当然也有提出写法 这 07/24 00:41
137F：→ saladim: 都是标准作业流程阿 也不只是我也有其他几位有同样感想 07/24 00:42
138F：→ saladim: 不是格式问题 遇过得比较能体会 很难用例子去描述 例子很 07/24 00:45
139F：→ saladim: 难显现那种杂乱跟让人觉得思路很乱的方式 07/24 00:46
140F：→ saladim: 传炸弹阿~~~块陶阿~~~ 07/24 00:47
141F：→ boomberm: code review不要让他过就好，有什麽难的 07/24 02:32
142F：推 wizozd84070: 每间公司都有这种问题，跳到其它公司只是又是另一个 07/24 04:07
143F：→ wizozd84070: 坑而己。反正要烂大家一起烂，只要没显示error就好 07/24 04:08
144F：推 wizozd84070: 主管没该该叫日子照过钱照领不是很好XD? 07/24 04:11
145F：推 mathrew: 同意楼上，除非你能力真的够，去那种管很好的公司 07/24 05:53
146F：推 new122851: 你可以Design Pattern重构啊，但如果你同事们很菜的话 07/24 08:36
147F：→ new122851: 高机率他们会看不懂，反而被告状说改成这样只有你看得 07/24 08:36
148F：→ new122851: 懂影响到整个部门，怕事的主管就会宁可大家都看得懂方 07/24 08:36
149F：→ new122851: 便出事情後人洒下去一起帮忙issue tracking，也不要只 07/24 08:36
150F：→ new122851: 有你一个人处理 07/24 08:36
151F：嘘 stepnight: 相同阶级的人讲了没用，上级主管又不想管 07/24 08:59
152F：→ stepnight: 所以你操这个心要干嘛？既然有真正主力 07/24 08:59
153F：→ stepnight: 在做事那又跟你有什麽关系？ 07/24 08:59
154F：→ stepnight: 你就是想嘴人家写一手烂Code而已 07/24 08:59
155F：嘘 pig2014: coding 如果不在同一个等级，就不该在同间公司。如果你 07/24 10:01
156F：→ pig2014: 跳不走充其量代表你algo或是design 也不够强不能被更强 07/24 10:01
157F：→ pig2014: 的公司认可，那在一起人各有长也没啥好抱怨的 07/24 10:01
158F：→ cylee: 如果对方真的是讲不听，跟主管说也不想管的话，那就算了 07/24 12:31
159F：→ cylee: 公事公办，code review 就是严格把关，卡到他们改为止 07/24 12:34
160F：推 zaq851017: 有没有一种可能你就算去NV Google还是都是这种code 07/24 13:00
161F：→ zaq851017: 居多 不是不想refacotr 而且成本太高 07/24 13:00
162F：→ Abbee: code review是资深带资浅在作的 你带不动他们那就是你资历 07/24 14:44
163F：→ Abbee: 不够 无法说服别人 都说他们是ju了 你不教怎麽变se 07/24 14:44
164F：→ jily: 每个人看别人的code都很痛苦XD 无法习惯表示不适合这一行 07/24 16:11
165F：→ acgotaku: 重构都是成本拉,写完压测 e2e 什麽测试都要重新跑一遍 07/25 01:57
166F：→ invidia: copy 的那个 function 你故意请他们改个小东西 07/25 09:16
167F：→ invidia: 他们就会懂有多难 maintain 了吧 07/25 09:16
168F：推 tsaigi: 没人在意你自己在那操心干嘛 07/25 16:43
169F：推 smalldra: 我遇到的code绝对都是ifelse海的 07/26 12:00
170F：推 hooll111: 知道是烂code 又不在review上讲 那codereview 干嘛 07/26 12:44
171F：推 ck237: 觉得你这样有点问题，写if else很正常啊 07/26 13:24
172F：推 ck237: 有四层判定式不就是需要这个判定，你要资浅的写递回本来就 07/26 13:31
173F：→ ck237: 强人所难 07/26 13:31
174F：→ windlll: 能不能动？能。有解决问题吗？有。有人喊效率吗？没有。 07/26 15:53
175F：→ windlll: 结案….鬼事太多，到後来没心力优化这些问题，理想很棒 07/26 15:53
176F：→ windlll: ，现实赚钱要紧 07/26 15:53
177F：推 answermangtr: 老实讲你遇到的真的还好 还有更多可怕的 例如不懂 07/26 16:22
178F：→ answermangtr: 因地制宜强硬要你按照最潮的做法做那才叫痛苦 07/26 16:22
179F：→ answermangtr: 我宁愿遇到解决问题的同事也不要遇到增加问题的同 07/26 16:23
180F：→ answermangtr: 事 给你参考 07/26 16:23
181F：→ answermangtr: 有时候过於匠气真的会气死 07/26 16:24
182F：推 acenova: 程式能动真的没必要花时间在这上面，除非你没事做了 07/29 14:44
183F：→ ChungLi5566: 不要浪费时间在这些没kpi的事情 07/31 09:02
184F：→ ChungLi5566: 大家都是混口饭吃而已 东西挂掉也别太在意 07/31 09:04
185F：嘘 minipopy0314: 吃饱太闲 08/06 16:29
186F：推 EricTao: 换工作 不然换专案 去擦其他屁股 08/06 18:18