小魯目前在一家還算大的公司工作 現在有兩三位頂大的junior的同事 寫程式的習慣讓我覺得是不是要趕快跑了 舉兩個例子好了 他們都喜歡if-else combo, 沒巢狀到波動拳那麼深 但就是動不動就if-else 三層 然後三層裡面還會再if-else 第二個例子就是如果有function 90%(50~100行)適合他們想要的用途, 他們就會copy整個function 然後修改一下後"整段"code插到他們需要的地方 光是上面兩個例子已經讓我的頭超大的 原本公司的codebase已經長得像科學怪人 了 然後又有他們持續"加持" 根本就沒辦法理解 更慘的是 跟他們講他們也不當一回事 又有頂大當紅碩論護體 一副你也不敢砍我的底氣 公司也沒人想要當壞人 code review也沒人出聲 而且大家都知道 上市公司每個都喜歡 有學歷的人當門面 反正真正主力有人會扛 XDDDD 每次改到他們經手過的code都很痛苦 若是要幫忙擦屁股根本擦不完阿 因為一直拉.... 自己寶貴的時間也都被吃掉了 這樣是不是該走了比較好? 我知道爛code到處都是啦 但是至少不能一直拉吧 又是都講不聽的 更資深的也都能閃就閃 但大家輪流中槍............. -- ※ 發信站: 批踢踢實業坊(ptt.org.tw), 來自: 1.164.97.88 (臺灣) ※ 文章網址: https://ptt.org.tw/Soft_Job/M.1721663732.A.097
lee457088: 你不如放段 code 舉個例我們才知道嚴重程度有多大呀~ 07/23 00:10
soga5731: 打不贏就加入,你code寫得再好薪水會拿比較多? 07/23 00:16
DrTech: 不知道為什麼要強調學歷。你換了更好的工作,只會有更多學 07/23 00:26
DrTech: 歷好的頂大碩。 07/23 00:26
wulouise: code review建立在平等尊重,沒有的話的確沒什麼好說的 07/23 00:27
wulouise: 不是你的理由不夠好,就是太minor有更重要的事 07/23 00:27
DrTech: 貴公司主管有在意這些事嗎? 不要當個小兵,操皇帝的心。 07/23 00:29
saladim: 不是操皇帝的心 是影響到我了 07/23 00:32
ko27tye: 薪水還行的話就練練糞坑裡挑黃金的能力 爛code單位到處 07/23 00:33
ko27tye: 都是 你遲早要面對的 07/23 00:33
saladim: 提到學歷+當紅碩論的原因 文中有寫了 07/23 00:33
saladim: 不是要討論程式or review 哲學, copy 100行的程式插到另 07/23 00:36
saladim: 外一個地方 很難相信有單位接受每次都這樣做 還是再說一 07/23 00:37
saladim: 次 這邊不是討論哲學...是在提一件很基本的事情.... 07/23 00:37
hermithsieh: 放心啦,直接AI重新整理就好 07/23 00:44
acgotaku: 如果程式已經肥大老舊到等待重構,舊 method 被大量呼叫 07/23 00:47
acgotaku: 我是覺得這種方式不能說不好,你也沒更安全的做法實作 07/23 00:49
acgotaku: 要重構不知歷史沒單元測試的舊方法,是我也想動拉 07/23 00:52
neo5277: 你們單位有code rule嘛? 07/23 00:52
acgotaku: 這種只能等待某個時機點 有人開始重視並重構專案 07/23 00:53
kissmickey: 會提到學歷是不是摳摳不平衡 07/23 00:53
kissmickey: 要不找主管談到願意擦屎 07/23 00:53
kissmickey: 要不就離職 07/23 00:53
neo5277: 100行感覺還好阿 07/23 00:54
nacy204327: 影響到你也不能如何 只能報主管處理 或離職 07/23 00:54
WaterLengend: 主管在會議還默許這樣,是我打不贏就加入,幹嘛因 07/23 00:54
WaterLengend: 為別的智障換工作 07/23 00:54
abccbaandy: 上面都不在意了你在意幹嘛? 一起拉阿 07/23 00:56
acgotaku: 大點軟體公司一年總會有三到五成精力重構不合需求舊專案 07/23 00:57
SHANGOYANYI: 你就拉的比他們更兇更超過 改天就換他們求你改寫法 07/23 00:58
SHANGOYANYI: 了 07/23 00:58
Bencrie: 公司是不是給你很多錢讓你捨不得離職 07/23 01:02
B0988698088: 干你屁事 07/23 01:20
neo5277: 感覺是銀行 07/23 01:23
ohmylove347: 沒有會噹人的主管嗎?這種寫法review都會被噴吧? 07/23 01:37
c8dog: 因為這種原因想走,在別的公司還是會遇到的,如果沒有改變 07/23 01:43
c8dog: 環境的能力,還是別想東想西 07/23 01:43
superpandal: 我遇到的都是老屁股或有權的在亂搞 有時候會懷疑人生 07/23 01:53
superpandal: 到底這樣有什麼意義 07/23 01:54
superpandal: 不過這篇畢業沒多久的人就像老油條一樣想著保飯碗拖 07/23 01:57
superpandal: 時間 真的有前途 比我傻傻受傷害好多了 07/23 01:59
superpandal: 現在職場就是誰跟你好來好去當好人 07/23 02:00
MoonCode: 紅明顯 建議不要內耗 好好刷題跳槽 07/23 02:06
Phater: junior都吃不下那要你review是橡皮圖章喔 07/23 02:20
alihue: 離職 07/23 06:15
knives: 新人就是這樣,功能能跑就好了,除非有人叫你重構,不然別 07/23 06:36
knives: 動別人的程式 07/23 06:36
pot1234: 為什麼要改他們經手過的code,他們不能自己改嗎? 07/23 07:11
pot1234: 話說你們公司不code review嗎 07/23 07:13
brucetu: 快跑垃圾公司 07/23 07:16
catchfly: 向上反應,無效就跑 07/23 07:55
ericthree: 錢多嗎? 主管不在意的話就大家一起寫爛code啊 07/23 08:06
stepnight: 爛Code哪裡都有,很在意就面試先直球問這點 07/23 08:10
stepnight: 二的情況我也遇過,真要說優點就是低耦合 07/23 08:10
stepnight: 新人進來第一天就能直接改code,改壞了就 07/23 08:10
stepnight: 那個func壞掉而已,免洗好用 07/23 08:10
stepnight: 而且你幫他們改code幹嘛?最煩新人進公司 07/23 08:11
stepnight: 一知半解在那自以為的改code,有的是歷史問題 07/23 08:11
jobintan: 那是人家頂大資深工程師厲害的地方,他們先到處屙屎, 07/23 08:38
jobintan: 等接手的菜雞處理不來,自己再帶菜雞清理屎山,到時候O 07/23 08:38
jobintan: KR就能說是自己的credit了。 07/23 08:38
vinex518: 離職 07/23 08:51
TPdidilong: 問就是快陶 07/23 08:57
NDark: if elss 本身並沒有錯 都要看情況 07/23 09:06
nayeonmywife: 就算你到其他地方也都還是會有爛code 07/23 09:33
Lhmstu: 沒差吧,你知道這樣是爛就足夠了,他們早晚有一天會知道 07/23 09:38
airtsubasa: 沒設定程式風格的公司 你是主管嗎 不是的話 管很寬喔 07/23 09:59
KyuubiKulama: 有講過就好,不然就看你有沒有心跟主管提或是下去 07/23 10:04
KyuubiKulama: 帶人啦啦啦 07/23 10:04
ji3g4zo6: 同58樓 不懂就問 不然就離職就好 07/23 10:07
johnbill: 接受不了就走吧 07/23 10:35
abc0922001: 我現在寫也都會直覺式寫if else,之後再重構 07/23 10:40
abc0922001: 現在有AI就方便多了,只要再code review 就好 07/23 10:41
zxzx8059: 有些chip原廠出來的sdk塞了滿滿巢狀switch-case + 07/23 10:43
zxzx8059: if-else + do-while combo 07/23 10:43
alex70266: 看主管態度啊,主管覺得沒差的話就…適當的切好自己的i 07/23 11:13
alex70266: nterface就好。除非你是mentor,不然真的bug等他們自己 07/23 11:13
alex70266: 處理 07/23 11:13
alex70266: coding sense這種東西強求不來的 07/23 11:15
candy668866: 覺得受不了就走人吧 07/23 11:15
alex70266: 我自己的經驗就是讓他們自己痛幾下才會學乖,那就不要 07/23 11:19
alex70266: 太早出手做些什麼 07/23 11:19
worf: 你主管都不care了 07/23 11:24
yamagishi: 純看你文章內容,這麼 junior 的問題用 gpt 尬一下就好 07/23 11:35
yamagishi: ㄌ 07/23 11:35
fatb: 哈欠 不是人的問題一率都不是問題 07/23 11:40
labbat: 用一堆SETcc MOVcc CMPccXADD會比較高級嗎 07/23 11:58
leo08210917: If else現在丟AI優化方便多了 07/23 12:14
DarkIllusion: code review你還不是連個屁都不敢放 問就是離職 07/23 12:15
arhtur945: Request changes? 07/23 12:58
sososlee: 你學歷一定很爛才一直靠北學歷 07/23 13:18
angusyu: 問就是離職,怕你不敢 07/23 13:32
pilor: 沒圖沒依據 07/23 13:48
psychic: 自己寫自己的,又沒礙到你 interface切乾淨就好 07/23 13:51
doranako: 公司code review 文化的問題, 07/23 14:20
shadow0326: 動不動就if-else三層 然後三層裡面還會再if-else 07/23 14:35
shadow0326: 為什麼不直接說四層? 07/23 14:35
lovdkkkk: 不知能不能找一個寫得好的專門 review 跟整理/清理程式 07/23 15:07
lovdkkkk: PR 都要過他,他負責整理好再併 07/23 15:08
lens82801: 老闆都不介意惹你何必操心呢 07/23 15:16
lturtsamuel: 當到 senior 還沒辦法在 code review 上管動 junior 07/23 15:51
lturtsamuel: 是怎麼把自己搞得這麼卑屈? 07/23 15:51
lturtsamuel: 2008就在發文 根本超級老 怎麼會連幾個剛畢業的都管 07/23 15:52
lturtsamuel: 不動 到底人緣多差 07/23 15:52
lturtsamuel: 會靠北 if-else 的人 往往是想要搞一些 strategy pat 07/23 15:55
lturtsamuel: ern 或依賴反轉之類的東西 但又常不喜歡寫文件 最後 07/23 15:55
lturtsamuel: 就是一大坨沒有文件又過度設計的屎 還不如幾個 if-el 07/23 15:55
lturtsamuel: se 07/23 15:55
kingofage111: 有本事就到有厲害同事的公司啊,沒本事的話你也跟 07/23 16:09
kingofage111: 你同事半斤八兩,不要在那邊自我優越了 07/23 16:09
kingofage111: 看你16年前的文在那邊笑前輩,現在發這種文,看來 07/23 16:26
kingofage111: 你高度就這樣而已 07/23 16:26
kingofage111: 拿出實力,自然別人就會聽有實力的人的話,沒實力 07/23 16:29
kingofage111: 剩張嘴實習生都不會理你啦 07/23 16:29
knives: 已經16年了阿,怎麼還在這噹這種小事 07/23 16:39
ma721: 是不是傻,他們比你賺的清鬆 07/23 16:50
stepnight: 襪靠,工作快二十年還在嘴學歷,是什麼糞職涯 07/23 17:56
hobnob: 第一次看到工作十幾年還壓不過Junior的 07/23 18:03
gpctv: 本來想分享我的類似經驗,但我看被嘴成這樣,想想還是算了 07/23 18:26
gpctv: 。 07/23 18:26
cylee: 你覺得寫的不好,那有提出你的解法嗎? Code review 要出聲 07/23 18:49
cylee: review 時每次開炮,我就不信不會改 07/23 18:50
HelloPTT: 問題可能不在if else,有做好分類、單一職責嗎 07/23 19:14
ikachann: 懶一點 把那些小段落複製丟AI請他幫你簡化就好 07/23 19:39
TSMCfabXX: 以前你可能需要逃 現在你有 AI 輔助 正面上他 07/23 19:40
Csongs: 其實真的可以叫他請ai code review 07/23 19:42
ma721: 半桶水響叮噹,問ai都比你強 07/23 20:16
loliconrd: 你是不是看人家學歷很自卑 07/23 21:55
uncle925: 問就是:你聽過Copilot嗎 07/23 22:14
viper9709: 能找到更好的就走~不然也只能繼續撐... 07/23 23:46
ECMA: 還在手寫程式喔 呵呵 07/23 23:55
saladim: ㄟ 不是啦 很多人沒看文中有寫 都有跟對方講過了 也有其 07/24 00:37
saladim: 他相對資深的也有講過 而且大家都是尊重平等理解的狀況下 07/24 00:38
saladim: 去看程式碼跟討論的 這是很自然的事 是不是大家把自己或 07/24 00:39
saladim: 是自己的體會帶入才會覺得別人都在亂噹? 現在這時代早就 07/24 00:40
saladim: 只能用尊重平等理解的方式來講話了 當然也有提出寫法 這 07/24 00:41
saladim: 都是標準作業流程阿 也不只是我也有其他幾位有同樣感想 07/24 00:42
saladim: 不是格式問題 遇過得比較能體會 很難用例子去描述 例子很 07/24 00:45
saladim: 難顯現那種雜亂跟讓人覺得思路很亂的方式 07/24 00:46
saladim: 傳炸彈阿~~~塊陶阿~~~ 07/24 00:47
boomberm: code review不要讓他過就好,有什麼難的 07/24 02:32
wizozd84070: 每間公司都有這種問題,跳到其它公司只是又是另一個 07/24 04:07
wizozd84070: 坑而己。反正要爛大家一起爛,只要沒顯示error就好 07/24 04:08
wizozd84070: 主管沒該該叫日子照過錢照領不是很好XD? 07/24 04:11
mathrew: 同意樓上,除非你能力真的夠,去那種管很好的公司 07/24 05:53
new122851: 你可以Design Pattern重構啊,但如果你同事們很菜的話 07/24 08:36
new122851: 高機率他們會看不懂,反而被告狀說改成這樣只有你看得 07/24 08:36
new122851: 懂影響到整個部門,怕事的主管就會寧可大家都看得懂方 07/24 08:36
new122851: 便出事情後人灑下去一起幫忙issue tracking,也不要只 07/24 08:36
new122851: 有你一個人處理 07/24 08:36
stepnight: 相同階級的人講了沒用,上級主管又不想管 07/24 08:59
stepnight: 所以你操這個心要幹嘛?既然有真正主力 07/24 08:59
stepnight: 在做事那又跟你有什麼關系? 07/24 08:59
stepnight: 你就是想嘴人家寫一手爛Code而已 07/24 08:59
pig2014: coding 如果不在同一個等級,就不該在同間公司。如果你 07/24 10:01
pig2014: 跳不走充其量代表你algo或是design 也不夠強不能被更強 07/24 10:01
pig2014: 的公司認可,那在一起人各有長也沒啥好抱怨的 07/24 10:01
cylee: 如果對方真的是講不聽,跟主管說也不想管的話,那就算了 07/24 12:31
cylee: 公事公辦,code review 就是嚴格把關,卡到他們改為止 07/24 12:34
zaq851017: 有沒有一種可能你就算去NV Google還是都是這種code 07/24 13:00
zaq851017: 居多 不是不想refacotr 而且成本太高 07/24 13:00
Abbee: code review是資深帶資淺在作的 你帶不動他們那就是你資歷 07/24 14:44
Abbee: 不夠 無法說服別人 都說他們是ju了 你不教怎麼變se 07/24 14:44
jily: 每個人看別人的code都很痛苦XD 無法習慣表示不適合這一行 07/24 16:11
acgotaku: 重構都是成本拉,寫完壓測 e2e 什麼測試都要重新跑一遍 07/25 01:57
invidia: copy 的那個 function 你故意請他們改個小東西 07/25 09:16
invidia: 他們就會懂有多難 maintain 了吧 07/25 09:16
tsaigi: 沒人在意你自己在那操心幹嘛 07/25 16:43
smalldra: 我遇到的code絕對都是ifelse海的 07/26 12:00
hooll111: 知道是爛code 又不在review上講 那codereview 幹嘛 07/26 12:44
ck237: 覺得你這樣有點問題,寫if else很正常啊 07/26 13:24
ck237: 有四層判定式不就是需要這個判定,你要資淺的寫遞迴本來就 07/26 13:31
ck237: 強人所難 07/26 13:31
windlll: 能不能動?能。有解決問題嗎?有。有人喊效率嗎?沒有。 07/26 15:53
windlll: 結案….鬼事太多,到後來沒心力優化這些問題,理想很棒 07/26 15:53
windlll: ,現實賺錢要緊 07/26 15:53
answermangtr: 老實講你遇到的真的還好 還有更多可怕的 例如不懂 07/26 16:22
answermangtr: 因地制宜強硬要你按照最潮的做法做那才叫痛苦 07/26 16:22
answermangtr: 我寧願遇到解決問題的同事也不要遇到增加問題的同 07/26 16:23
answermangtr: 事 給你參考 07/26 16:23
answermangtr: 有時候過於匠氣真的會氣死 07/26 16:24
acenova: 程式能動真的沒必要花時間在這上面,除非你沒事做了 07/29 14:44
ChungLi5566: 不要浪費時間在這些沒kpi的事情 07/31 09:02
ChungLi5566: 大家都是混口飯吃而已 東西掛掉也別太在意 07/31 09:04
minipopy0314: 吃飽太閒 08/06 16:29
EricTao: 換工作 不然換專案 去擦其他屁股 08/06 18:18