作者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