作者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/m.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