作者:
saladim (殺拉頂)
2024-07-22 23:55:30小魯目前在一家還算大的公司工作 現在有兩三位頂大的junior的同事
寫程式的習慣讓我覺得是不是要趕快跑了 舉兩個例子好了
他們都喜歡if-else combo, 沒巢狀到波動拳那麼深 但就是動不動就if-else
三層 然後三層裡面還會再if-else
第二個例子就是如果有function 90%(50~100行)適合他們想要的用途,
他們就會copy整個function 然後修改一下後"整段"code插到他們需要的地方
光是上面兩個例子已經讓我的頭超大的 原本公司的codebase已經長得像科學怪人
了 然後又有他們持續"加持" 根本就沒辦法理解
更慘的是 跟他們講他們也不當一回事 又有頂大當紅碩論護體 一副你也不敢砍我的底氣
公司也沒人想要當壞人 code review也沒人出聲 而且大家都知道 上市公司每個都喜歡
有學歷的人當門面 反正真正主力有人會扛 XDDDD
每次改到他們經手過的code都很痛苦 若是要幫忙擦屁股根本擦不完阿 因為一直拉....
自己寶貴的時間也都被吃掉了
這樣是不是該走了比較好?
我知道爛code到處都是啦 但是至少不能一直拉吧 又是都講不聽的 更資深的也都能閃就閃
但大家輪流中槍.............
作者: lee457088 2024-07-23 00:10:00
你不如放段 code 舉個例我們才知道嚴重程度有多大呀~
作者:
soga5731 (Letiz)
2024-07-23 00:16:00打不贏就加入,你code寫得再好薪水會拿比較多?
作者:
DrTech (竹科管理處網軍研發人員)
2024-07-23 00:26:00不知道為什麼要強調學歷。你換了更好的工作,只會有更多學歷好的頂大碩。
作者:
wulouise (在線上!=在電腦前)
2024-07-23 00:27:00code review建立在平等尊重,沒有的話的確沒什麼好說的不是你的理由不夠好,就是太minor有更重要的事
作者:
DrTech (竹科管理處網軍研發人員)
2024-07-23 00:29:00貴公司主管有在意這些事嗎? 不要當個小兵,操皇帝的心。
作者:
saladim (殺拉頂)
2024-07-23 00:32:00不是操皇帝的心 是影響到我了
作者:
ko27tye (好滋好滋)
2024-07-23 00:33:00薪水還行的話就練練糞坑裡挑黃金的能力 爛code單位到處都是 你遲早要面對的
作者:
saladim (殺拉頂)
2024-07-23 00:33:00提到學歷+當紅碩論的原因 文中有寫了不是要討論程式or review 哲學, copy 100行的程式插到另外一個地方 很難相信有單位接受每次都這樣做 還是再說一次 這邊不是討論哲學...是在提一件很基本的事情....
作者:
acgotaku (otaku)
2024-07-23 00:47:00如果程式已經肥大老舊到等待重構,舊 method 被大量呼叫我是覺得這種方式不能說不好,你也沒更安全的做法實作要重構不知歷史沒單元測試的舊方法,是我也想動拉
作者:
neo5277 (I am an agent of chaos)
2024-07-23 00:52:00你們單位有code rule嘛?
作者:
acgotaku (otaku)
2024-07-23 00:53:00這種只能等待某個時機點 有人開始重視並重構專案
會提到學歷是不是摳摳不平衡要不找主管談到願意擦屎要不就離職
作者:
neo5277 (I am an agent of chaos)
2024-07-23 00:54:00100行感覺還好阿
主管在會議還默許這樣,是我打不贏就加入,幹嘛因為別的智障換工作
作者:
acgotaku (otaku)
2024-07-23 00:57:00大點軟體公司一年總會有三到五成精力重構不合需求舊專案
你就拉的比他們更兇更超過 改天就換他們求你改寫法了
作者:
neo5277 (I am an agent of chaos)
2024-07-23 01:23:00感覺是銀行
沒有會噹人的主管嗎?這種寫法review都會被噴吧?
作者: c8dog (草莓豬烏梅豬百香狗) 2024-07-23 01:43:00
因為這種原因想走,在別的公司還是會遇到的,如果沒有改變環境的能力,還是別想東想西
作者: superpandal 2024-07-23 01:53:00
我遇到的都是老屁股或有權的在亂搞 有時候會懷疑人生到底這樣有什麼意義不過這篇畢業沒多久的人就像老油條一樣想著保飯碗拖時間 真的有前途 比我傻傻受傷害好多了現在職場就是誰跟你好來好去當好人
作者:
MoonCode (MoonCode)
2024-07-23 02:06:00紅明顯 建議不要內耗 好好刷題跳槽
作者:
Phater (肥特)
2024-07-23 02:20:00junior都吃不下那要你review是橡皮圖章喔
作者:
alihue (wanda wanda)
2024-07-23 06:15:00離職
作者:
knives 2024-07-23 06:36:00新人就是這樣,功能能跑就好了,除非有人叫你重構,不然別動別人的程式
作者:
pot1234 (鍋子)
2024-07-23 07:11:00為什麼要改他們經手過的code,他們不能自己改嗎?話說你們公司不code review嗎
作者:
brucetu (sec)
2024-07-23 07:16:00快跑垃圾公司
作者: ericthree (如果 她這樣動人) 2024-07-23 08:06:00
錢多嗎? 主管不在意的話就大家一起寫爛code啊
爛Code哪裡都有,很在意就面試先直球問這點二的情況我也遇過,真要說優點就是低耦合新人進來第一天就能直接改code,改壞了就那個func壞掉而已,免洗好用而且你幫他們改code幹嘛?最煩新人進公司一知半解在那自以為的改code,有的是歷史問題
作者:
jobintan (Robin Artemstein)
2024-07-23 08:38:00那是人家頂大資深工程師厲害的地方,他們先到處屙屎,等接手的菜雞處理不來,自己再帶菜雞清理屎山,到時候OKR就能說是自己的credit了。
作者: vinex518 2024-07-23 08:51:00
離職
作者:
NDark (溺於黑暗)
2024-07-23 09:06:00if elss 本身並沒有錯 都要看情況
作者:
Lhmstu (lhmstu)
2024-07-23 09:38:00沒差吧,你知道這樣是爛就足夠了,他們早晚有一天會知道
沒設定程式風格的公司 你是主管嗎 不是的話 管很寬喔
有講過就好,不然就看你有沒有心跟主管提或是下去帶人啦啦啦
作者:
ji3g4zo6 (黑人閉嘴歐巴馬)
2024-07-23 10:07:00同58樓 不懂就問 不然就離職就好
我現在寫也都會直覺式寫if else,之後再重構現在有AI就方便多了,只要再code review 就好
作者:
zxzx8059 (zxzx8059)
2024-07-23 10:43:00有些chip原廠出來的sdk塞了滿滿巢狀switch-case +if-else + do-while combo
作者: alex70266 (小眼) 2024-07-23 11:13:00
看主管態度啊,主管覺得沒差的話就…適當的切好自己的interface就好。除非你是mentor,不然真的bug等他們自己處理coding sense這種東西強求不來的
作者: alex70266 (小眼) 2024-07-23 11:19:00
我自己的經驗就是讓他們自己痛幾下才會學乖,那就不要太早出手做些什麼
作者: worf 2024-07-23 11:24:00
你主管都不care了
作者: yamagishi (山岸刑務官) 2024-07-23 11:35:00
純看你文章內容,這麼 junior 的問題用 gpt 尬一下就好ㄌ
作者:
fatb (胖逼=口=)
2024-07-23 11:40:00哈欠 不是人的問題一率都不是問題
作者:
labbat (labbat)
2024-07-23 11:58:00用一堆SETcc MOVcc CMPccXADD會比較高級嗎
code review你還不是連個屁都不敢放 問就是離職
作者:
arhtur945 (AnthonyBennet)
2024-07-23 12:58:00Request changes?
作者:
angusyu (〒△〒)
2024-07-23 13:32:00問就是離職,怕你不敢
作者:
pilor (Formosa)
2024-07-23 13:48:00沒圖沒依據
作者: psychic (過路人) 2024-07-23 13:51:00
自己寫自己的,又沒礙到你 interface切乾淨就好
動不動就if-else三層 然後三層裡面還會再if-else為什麼不直接說四層?
不知能不能找一個寫得好的專門 review 跟整理/清理程式PR 都要過他,他負責整理好再併
當到 senior 還沒辦法在 code review 上管動 junior是怎麼把自己搞得這麼卑屈?2008就在發文 根本超級老 怎麼會連幾個剛畢業的都管不動 到底人緣多差會靠北 if-else 的人 往往是想要搞一些 strategy patern 或依賴反轉之類的東西 但又常不喜歡寫文件 最後就是一大坨沒有文件又過度設計的屎 還不如幾個 if-else
有本事就到有厲害同事的公司啊,沒本事的話你也跟你同事半斤八兩,不要在那邊自我優越了
作者:
knives 2024-07-23 16:39:00已經16年了阿,怎麼還在這噹這種小事
作者:
ma721 (UndeadJ)
2024-07-23 16:50:00是不是傻,他們比你賺的清鬆
作者: hobnob (hobnob) 2024-07-23 18:03:00
第一次看到工作十幾年還壓不過Junior的
作者:
gpctv (gpctv)
2024-07-23 18:26:00本來想分享我的類似經驗,但我看被嘴成這樣,想想還是算了。
作者: cylee (Artisan) 2024-07-23 18:49:00
你覺得寫的不好,那有提出你的解法嗎? Code review 要出聲review 時每次開炮,我就不信不會改
作者: HelloPTT 2024-07-23 19:14:00
問題可能不在if else,有做好分類、單一職責嗎
作者:
Csongs (西歌)
2024-07-23 19:42:00其實真的可以叫他請ai code review
作者:
ma721 (UndeadJ)
2024-07-23 20:16:00半桶水響叮噹,問ai都比你強
作者:
uncle925 (-新潮x樂活x便捷)
2024-07-23 22:14:00問就是:你聽過Copilot嗎
作者:
ECMA 2024-07-23 23:55:00還在手寫程式喔 呵呵
作者:
saladim (殺拉頂)
2024-07-24 00:37:00ㄟ 不是啦 很多人沒看文中有寫 都有跟對方講過了 也有其他相對資深的也有講過 而且大家都是尊重平等理解的狀況下去看程式碼跟討論的 這是很自然的事 是不是大家把自己或是自己的體會帶入才會覺得別人都在亂噹? 現在這時代早就只能用尊重平等理解的方式來講話了 當然也有提出寫法 這都是標準作業流程阿 也不只是我也有其他幾位有同樣感想不是格式問題 遇過得比較能體會 很難用例子去描述 例子很難顯現那種雜亂跟讓人覺得思路很亂的方式傳炸彈阿~~~塊陶阿~~~
作者: boomberm (boom) 2024-07-24 02:32:00
code review不要讓他過就好,有什麼難的
每間公司都有這種問題,跳到其它公司只是又是另一個坑而己。反正要爛大家一起爛,只要沒顯示error就好
作者:
mathrew (Joey)
2024-07-24 05:53:00同意樓上,除非你能力真的夠,去那種管很好的公司
你可以Design Pattern重構啊,但如果你同事們很菜的話高機率他們會看不懂,反而被告狀說改成這樣只有你看得懂影響到整個部門,怕事的主管就會寧可大家都看得懂方便出事情後人灑下去一起幫忙issue tracking,也不要只有你一個人處理
相同階級的人講了沒用,上級主管又不想管所以你操這個心要幹嘛?既然有真正主力在做事那又跟你有什麼關系?你就是想嘴人家寫一手爛Code而已
作者:
pig2014 (Rocking Man)
2024-07-24 10:01:00coding 如果不在同一個等級,就不該在同間公司。如果你跳不走充其量代表你algo或是design 也不夠強不能被更強的公司認可,那在一起人各有長也沒啥好抱怨的
作者: cylee (Artisan) 2024-07-24 12:31:00
如果對方真的是講不聽,跟主管說也不想管的話,那就算了公事公辦,code review 就是嚴格把關,卡到他們改為止
有沒有一種可能你就算去NV Google還是都是這種code居多 不是不想refacotr 而且成本太高
作者:
Abbee (阿比)
2024-07-24 14:44:00code review是資深帶資淺在作的 你帶不動他們那就是你資歷不夠 無法說服別人 都說他們是ju了 你不教怎麼變se
作者:
jily (吉利)
2024-07-24 16:11:00每個人看別人的code都很痛苦XD 無法習慣表示不適合這一行
作者:
acgotaku (otaku)
2024-07-25 01:57:00重構都是成本拉,寫完壓測 e2e 什麼測試都要重新跑一遍
作者:
invidia (莎莉竟然會飛)
2024-07-25 09:16:00copy 的那個 function 你故意請他們改個小東西他們就會懂有多難 maintain 了吧
作者: tsaigi (菜雞) 2024-07-25 16:43:00
沒人在意你自己在那操心幹嘛
作者:
hooll111 (Katsudon)
2024-07-26 12:44:00知道是爛code 又不在review上講 那codereview 幹嘛
作者:
ck237 (白色小雞)
2024-07-26 13:24:00覺得你這樣有點問題,寫if else很正常啊有四層判定式不就是需要這個判定,你要資淺的寫遞迴本來就強人所難
作者: windlll (我要工作阿) 2024-07-26 15:53:00
能不能動?能。有解決問題嗎?有。有人喊效率嗎?沒有。結案….鬼事太多,到後來沒心力優化這些問題,理想很棒,現實賺錢要緊
老實講你遇到的真的還好 還有更多可怕的 例如不懂因地制宜強硬要你按照最潮的做法做那才叫痛苦我寧願遇到解決問題的同事也不要遇到增加問題的同事 給你參考有時候過於匠氣真的會氣死
作者:
acenova (歸零)
2024-07-29 14:44:00程式能動真的沒必要花時間在這上面,除非你沒事做了
不要浪費時間在這些沒kpi的事情大家都是混口飯吃而已 東西掛掉也別太在意
作者:
Ghamu (貓丸)
2024-09-08 23:49:00講不通或許可以考慮分享文章的方式指桑罵槐 也可以說是單純技術分享這樣?會不會比較對事不對人就等於透過另一個權威reference 做擔保 而不是你個人的意見要別人接受