[討論] 產量高品質差的工程師

作者: yestheway (LKK)   2024-08-01 11:15:38
大家有沒有遇過這樣工程師…
我們公司最近在開發新的專案,找了一位新來的工程師幫忙一起做。這個人Coding速度真
的很快,交給他的功能很快就能做出來。每個sprint下來,他也一直不停的接新ticket和
開發新東西。
最近這個新專案終於要上線了,結果QA卻測出了一大堆bug!!由於數量真的太多了,但
又為了承諾客戶如期上線,所以只好把我和其他2個工程師也叫來,一起昴下去幫忙解bug

結果不去看還好,一下去看他裡面的code,真的是非常可怕…又臭又長像流水帳一樣,結
構也是亂七八糟,很多邏輯明顯沒有想過或設計過硬幹去寫出來,沒有任何彈性和維護性
,大家花了非常多時間再改他的程式,真的改的非常辛苦...
(對…我們為了趕這個專案,完全skip code review、skip unit tests 等等。二來 這
新專案相對獨立,不影響現有系統。所以他commit 什麼 就merge什麼,鬧得今天這下場
。我們的例子,正好回應前幾篇某些人質疑為何要code review......)
最後產品雖然如期上線,但這下好了,老闆和PM現在超喜歡這個工程師,後面很多v2 要
衍生的新功能,都要叫這位工程師來主導開發…
我們幾個幫忙「收爛攤子」的人,聽到真的有種不好的預感…一來害怕又有更多有問題的
程式被他寫出來,後面又要花更多時間來修改;二來有種功勞你在接,爛攤子我們在收的
感覺…
我們原本找主管說這些問題,但目前公司大老闆想正積極開發這項產品,他們只希望快點
見到結果,似乎也不太在乎原有的開發流程了,只想先快點把東西生出來,給客戶demo…
各位如果面對這種情況,和這樣的工程師該怎麼辦?公司想快速看到成品,找了一個產出
快的人,雖然短期快速看得到成果,但卻後患無窮…
作者: mercurycgt68 (發芽的吉它手)   2024-08-01 11:18:00
離職
作者: miyazakisun2 (hidetaka)   2024-08-01 11:24:00
屎擦太乾淨,以後負責專門擦 笑了
作者: andy810306 (小寶)   2024-08-01 11:30:00
有bug當然叫commit的人處理吧,你去擦屎幹嘛
作者: teddy (這是個好問題)   2024-08-01 11:31:00
叫他一人負責就好 其他人去忙其他專案
作者: ssccg (23)   2024-08-01 11:47:00
不是測過才算產量嗎? 不會動的也能算產量?
作者: MOONY135 (談無慾)   2024-08-01 11:47:00
驅動生產 反正你們會如期修好
作者: ssccg (23)   2024-08-01 11:48:00
怎麼會很難推,除非你真的很想待下去那你活該
作者: nh60211as   2024-08-01 11:49:00
你們自己沒有驗收條件就算結案很奇怪餒
作者: seanhiroshi (hiroshi)   2024-08-01 11:52:00
想辦法把他捧起來 之後好處理
作者: B0988698088 (廢文少女小円♥)   2024-08-01 11:55:00
之前不是有啟智說不用code review review是要看什麼怎麼現在那種人又消失了
作者: airtsubasa (偽學姊)   2024-08-01 11:56:00
影響正常運作的bug還能算量?
作者: wuyiulin (龍破壞劍士-巴斯達布雷達)   2024-08-01 11:58:00
直接講不聽就投到他直屬主管那邊啊,他主管不喬是他的事情,出問題就把他們兩個推出去扛刀。
作者: tsaigi (菜雞)   2024-08-01 12:01:00
就不要修啊 大不了一起爛
作者: hobnob (hobnob)   2024-08-01 12:05:00
接案出身?感覺你團隊問題也蠻大的欸..
作者: lazarus1121 (...)   2024-08-01 12:22:00
沒有維運過自己開發的工程師,很大機率是垃圾量產機
作者: Tenno081 (空勁大歸還)   2024-08-01 12:35:00
我好奇他寫好的時候不用先跑一下看看有無問題嗎?
作者: ssccg (23)   2024-08-01 12:42:00
開規格、測案開嚴一點,然後叫他自己全測過
作者: pilor (Formosa)   2024-08-01 12:51:00
好像我們公司
作者: sowulo ( )   2024-08-01 13:14:00
看起來是驗收機制有問題吧 而且主管不會覺得一個人開發要三個人維護哪裡怪怪的嗎
作者: talkmyself (音容苑在)   2024-08-01 13:15:00
離職阿,那mo簡單
作者: lilicoco520 (哩哩)   2024-08-01 13:20:00
出個包讓老闆痛
作者: ma721 (UndeadJ)   2024-08-01 13:38:00
需求定義不明確吧,怎麼會有因環境程式出錯
作者: rereterry (rereterry)   2024-08-01 13:43:00
如果要你們幫忙,那功能設計初期,你們應該可以加入吧!就要求流程,把merge的權力拿走,既然老闆要你們幫忙,我是不認為你們無法要求啦!這樣也可以讓這個工程師知道你們的要求,不急著他可以一步到位的改寫成符合你們的想像,但至少你們的工作也可以隨著紀錄一起被看到
作者: lwecloud (CloudEX)   2024-08-01 13:47:00
樓上把老闆想得太美好了,老闆都覺得這樣的人超讚
作者: j112233446 (MeJump)   2024-08-01 13:47:00
他現在credit 太高了 出個問題就會讓你老闆好好考慮這麼問題了
作者: hakama99 (雜醬麵)   2024-08-01 13:54:00
qa問題吧 你們的嚴謹只有rd?
作者: littlebroken (小柏肯)   2024-08-01 13:58:00
第五行怪怪的 正常QA驗出issue修就是了 為啥搞得像
作者: wuyiulin (龍破壞劍士-巴斯達布雷達)   2024-08-01 13:58:00
等等 corner case 沒測出來你找RD不找QA?還是你司 RD有加領 QA薪水?
作者: littlebroken (小柏肯)   2024-08-01 13:59:00
在救火? 他的東西在進dev branch前應該就要完整了
作者: hakama99 (雜醬麵)   2024-08-01 14:01:00
寫很糟糕被qa退太多次就要review電他啊 你們根本沒有你說的嚴謹吧
作者: chses910372 (loveseeker)   2024-08-01 14:01:00
快點逃,我說真的,這個是制度問題,今天解決了下次還是重蹈覆徹
作者: littlebroken (小柏肯)   2024-08-01 14:02:00
啊還有你們沒peer review跟1:1? code亂寫就提出來講
作者: sjr500 (萬年淺水號)   2024-08-01 14:03:00
這種品質還沒速度的話....
作者: starburs (星爆氣流斬)   2024-08-01 14:04:00
他很爛但是很聰明,你們很強但是很笨
作者: Lordaeron (Terry)   2024-08-01 14:08:00
產量高就CHATGPT 就好了,但測不出來就SIT 做什麼的?
作者: stepnight (桃卡武康)   2024-08-01 14:21:00
你們團隊有問題,他也有問題看內文,照你們團隊正常的流程三個人下去做,案子可能趕不出來?他一人做你們兩人擦屎,結果是趕得出來這就是老闆要的,老闆只在乎這個要先領悟到這點,其他的對老闆都不是問題
作者: ilede (黑狗妹)   2024-08-01 14:49:00
這也很神奇阿...還要別人幫忙擦屎反而做得出來?
作者: s06yji3 (阿南)   2024-08-01 15:00:00
主管在幹嘛...沒救了。不離職就自己吞了。
作者: descent (「雄辯是銀,沉默是金」)   2024-08-01 15:11:00
老闆不知道你們修 bug 的貢獻嗎?
作者: arcade0425 (天海)   2024-08-01 15:30:00
木已成舟,不想單純繼續擦屎就兩條路 1. 把他 merge權限拔掉,每次commit 都有人 互相 review,讓這份專案回歸正常開發流程,就算有屎也會變成大家一起拉的屎2. 離職
作者: x20165 (八風吹不動)   2024-08-01 15:48:00
先工作流程紀錄給老闆看,後面再離職
作者: NTUTM04 (TM終號機)   2024-08-01 16:03:00
恭喜變成專業擦屎工
作者: recorriendo (孟新)   2024-08-01 16:33:00
好爽 本篇簡直軟工如何飛黃騰達的教學文 有收藏有推
作者: atst2 (atst2)   2024-08-01 16:33:00
你們去幫他處理後續的bug,付出的成本有估算過嗎? 老闆知道
作者: tw11509 (John-117)   2024-08-01 16:34:00
公司就這樣啊,我曾經支援一個專案,程式一看就是一坨會動的大便,但聽說他主管對他評價很高,後來他離職接他工作的人也離職了XD
作者: atst2 (atst2)   2024-08-01 16:34:00
嗎? 有沒有加班? 如果有,加班費有拿嗎?個人是覺得,你看來短期內走不了,那最少該拿的拿一拿.其他案子如何,後續有沒有坑,不是你們要負責的。
作者: v86861062 (數字人:3)   2024-08-01 16:38:00
真慘快逃;)
作者: abc0922001 (中士abc)   2024-08-01 16:45:00
表示老闆覺得你們平常開發方式不好
作者: fatb (胖逼=口=)   2024-08-01 16:56:00
其實他在開發過程中應該就要一直在測試了其實這種工程師就走專案的 以完成為第一目標 不能說對錯
作者: newuser (newuser)   2024-08-01 17:01:00
我亂猜 這位工程師是不是印度人 以我遇過同樣的 是印度人
作者: pot1234 (鍋子)   2024-08-01 17:14:00
你們原本開發時間是他的兩倍以上嗎,是的話可以叫他多花一倍時間寫test,然後你們去找別的工作
作者: ku72 (ku72)   2024-08-01 18:06:00
你們沒救了 老闆都是看結果的 老闆喜歡他 你以為是不知道你們幫他擦屁股? 老闆喜歡他 是因為他可以讓你們有幾倍的產出啊
作者: ab4daa (nooooooooooooooooooo)   2024-08-01 18:43:00
我也好想專門拉屎別人擦QQ
作者: brunchbug   2024-08-01 19:21:00
離職 或 讓他爆
作者: eugene31431 (eugene31431)   2024-08-01 19:30:00
原來是bug的產量高,建議可以改一下標題
作者: lchcoding   2024-08-01 19:49:00
不試一下 pair programming 嗎?反正會吵,何不一開始就吵...當他要拉的時候,就電到他不敢拉
作者: codehard   2024-08-01 19:55:00
讓他爆,公司不會一個案子就死掉,如果是也是遲早。
作者: hsiang1049 (pipi)   2024-08-01 20:32:00
要上線才在測?
作者: xephon   2024-08-01 20:42:00
你們驗收條件,只驗Happy Path,那放他過也是正常的你們就是鼓勵這樣產出的工程師
作者: NDark (溺於黑暗)   2024-08-01 20:50:00
錢若不花在教育 就會花在監獄如果不能提早測試 那就要Allocate足夠的測試期預測時程通常都是不準的原本預估拿來測試的時間都不夠所以有經驗的人都會叫你早點測試 這樣早點反應時程規格問題因為有時候不符規格不是開發的錯是規格就開得不準確
作者: benben994 (晨宇)   2024-08-01 22:43:00
環境制度問題 你們也知道很趕時間產品品質三角都要兼顧
作者: jlhc (H)   2024-08-01 23:03:00
通常這種 promote 也很快, 因為老闆喜歡 impacted 也大說真的擦擦屁股就能跟著一起 promote 不好嗎?
作者: Baternest (忍蛋)   2024-08-01 23:32:00
幫忙的人沒有原本負責的項目嗎?多少要推脫一點 不要全力去處理別人造成的問題 (是說大概已經來不及了...)
作者: AoShenFengYu (分類看板)   2024-08-02 00:21:00
這種人我遇過兩位,一位被開除了,另一位正在 pip
作者: viper9709 (阿達)   2024-08-02 00:28:00
覺得是制度的問題+1
作者: internetms52 (Oaide)   2024-08-02 01:50:00
公司衡量產出的方式有問題,這要tech lead或cto層級去講,反應沒有得到改善,唯一能做的就是想辦法將驗收條件納入衡量而且驗收條件要先訂這件事pm也不一定願意,吃力不討好應該是蠻複雜的多個位置的偷懶結果
作者: sachung28 (00)   2024-08-02 05:53:00
說不定他有意的 他一人在忙接PM一堆要求壓榨 你們這些老人在納涼也不幫忙或講話 那時間不夠當然先求會動 快做完你們以為能拿成果想進去分功勞 踩到屎剛好而已換個角度說不定變成接到屎缺辛苦完成後受到老闆賞識結果老人嫉妒開始搞排擠和貶低能力還把程式改得面目全非很難維護
作者: abraxas (Abr.)   2024-08-02 07:35:00
每次都趕,每次都這樣,還不快跑?
作者: Csongs (西歌)   2024-08-02 09:14:00
他紀錄會很漂亮 大老闆會升他
作者: flysonics (飛音)   2024-08-02 09:30:00
勸你快跑吧 這人搞不好之後就變你老闆了他已經紅了 你現在再跟老闆提什麼要求 在老闆眼裡都像扯他後腿
作者: stepnight (桃卡武康)   2024-08-02 09:32:00
看原Po的回應,只想把鍋甩在新人身上把事情塑造成老人們天神下凡拯救專案而老闆卻只看到新人好棒棒搞不好老闆也看透不說破你們在搞啥啦
作者: flysonics (飛音)   2024-08-02 09:35:00
你現在最多就是盡量自保 抓好自己手裡的開發工作 避免被assign去幫他擦屁股 不然就是跟他打好關係 看看有沒有機會他被premote時你能分點羹
作者: lturtsamuel (港都都教授)   2024-08-02 09:55:00
不就是自食惡果的故事嗎== 找個新人來幫忙 舊的人偷懶一行扣都沒看過 到品管才被抓出來 說你們沒問題誰會信==假如產品跟你無關 為什麼叫你來救火?假如有關 為什麼你之前敢放著都讓他弄?
作者: ericthree (如果 她這樣動人)   2024-08-02 10:45:00
能力不行啊 讓他做雜事吧如果老闆敢讓他主導那就陪他做垃圾
作者: flash5408   2024-08-02 11:07:00
Ptt真多大開眼界的事 沒有code review 沒test 隨便merge 沒有dev infra嗎
作者: wuwaiter (掛~~有事打我手機)   2024-08-02 11:20:00
離職啊, 公司有這樣的人 也不會有好發展你的能力這麼強的話 到那邊都會發光
作者: LiebeLion (IchLiebeDich)   2024-08-02 11:42:00
真正問題是qa沒做好QA要負責edge case再怎麼趕也不可能不Q就進
作者: s06yji3 (阿南)   2024-08-02 11:53:00
原po要一直解釋了,網路上超多腦補怪XD
作者: LiebeLion (IchLiebeDich)   2024-08-02 12:12:00
demo就遇到的case不能算corner吧
作者: jeffreyup1 (jeffrey)   2024-08-02 12:36:00
我單純在想為啥沒有pr code review
作者: MonkeyCL (猴總召)   2024-08-02 12:39:00
他以後可能是你主管
作者: GoalBased (Artificail Intelligence)   2024-08-02 13:16:00
你反映給主管,主管不往上反應,某種程度主管覺得他那樣是比較符合公司要的,或者老闆認為那是公司要的
作者: sachung28 (00)   2024-08-02 13:26:00
講完整點別人也不用腦補
作者: zenuo (堅持到底永不放棄)   2024-08-02 14:15:00
推回給原開發的維護,都測出問題還要別人改?
作者: worf   2024-08-02 16:54:00
改得很辛苦 他怎麼可以置身事外 這不是很奇怪嗎你上面的人又不知道你改得很辛苦 他的BUG很多
作者: mathrew (Joey)   2024-08-02 18:00:00
誰叫你們要那麼認真 debug,當然是挖洞給他跳啊
作者: A0091127 (A009)   2024-08-02 19:00:00
建議直接把這篇貼給主管看
作者: SuperTaco (TO-FU OYAKO)   2024-08-02 19:13:00
推mathrew
作者: luke72 (ccc)   2024-08-02 20:09:00
這種反應沒用,別傻了,上面一堆code review的也是幻想code review是有政治權力的人review賤民的code,你有權嗎這種人就是政治滿分,直接跟老闆說他是專家,你不懂他的code快爆炸前就會拖你去當墊背,他置身事外你能做的有兩個,一個是當練功,未來這些都是你的履歷一個是提高政治敏感度,務必在他起手式就阻絕掉讓他自爆當然他可能會找你主管下手,把鍋全丟你身上,這就沒辦法
作者: joke0709   2024-08-02 20:47:00
收藏了 之後朝著方向練習
作者: agra (一審有罪就下台!)   2024-08-02 23:02:00
「每個sprint都不斷接新ticket卻最後QA不過」,聽起來敏捷的部分就很歪啦
作者: imhaha (嘿嘿)   2024-08-02 23:08:00
不要放出來拜託
作者: Ghamu (貓丸)   2024-08-03 01:25:00
這好像有看過那個漫畫 這個人最後會榮譽跳巢漲薪水 後面公司會多請更多工程師來擦屁股 順便抱怨說怎麼這些新來的工程師比不上之前那位做超快的說不定還會說你看不懂他程式是因為你程度不足勒 呵呵這種事情通常是要反映給一個老闆信得過的資深工程師或是主管 讓他們去講才有用 你們平行的工程師去跟不懂技術的老闆講可能還會被視為眼紅要中傷人勒
作者: prag222 (prag)   2024-08-03 04:10:00
看你有沒天天加班在擦屁股能上來抱怨感覺還能接受?
作者: crustancean (要你命三千)   2024-08-03 10:48:00
都Q過一堆問題了,還在QA的問題
作者: wade52 (wade52)   2024-08-03 11:37:00
跟老闆談條件,讓你做專案,換你亂寫,他來debug
作者: eulbos (反串魔人)   2024-08-03 13:14:00
老闆會覺得新人很有潛力 願意栽培
作者: henrylin8086 (小木)   2024-08-03 18:14:00
你們問題在開發流程,你不即時Review誰剛來會知道公司的coding style,除非你們有linting tool在管控,但顯然沒有。再來是CI,你們Merge沒做test的話,工程師們大多都是自己測爽了就上code,畢竟有太多隱性的Bug在一開始是很難發現的,而工程師又很清楚自己設計的程式邏輯,通常會無意識避開沒被考慮進去的邏輯。
作者: yoyoyolin (吳志和)   2024-08-03 21:21:00
接案就是要快啊。接案通常沒在code review很正常,套句sa 講的話,功能可以跑就好。你能做的就是跟對方溝通,請對方要測過再commit。不然這真的無解
作者: NDark (溺於黑暗)   2024-08-03 22:51:00
外包或支援就是看單子辦事 單子沒提到的當然就不算這就是自研開發與外包支援的差異這才是為什麼開單/開規格的人本應該把規格/測試寫清楚所以才會有找外援反而更浪費人力的情況發生因為原本可能對自己人可以模糊一點的需求必須改寫很清楚規格裡面沒有限制效能資源的話做的人浪費資源記憶體也只能默默自己事後改
作者: rogerlarger (宅)   2024-08-04 18:27:00
這樣產一堆垃圾也算產量高嗎
作者: stosto (樹多)   2024-08-05 06:42:00
不就你們做太慢?不然找他幹嘛
作者: DrizztMon   2024-08-05 08:01:00
ku72板友說出答案了,老闆不懂的是穩定運作的東西不要去搞它 不然開發team崩潰還要找人喔
作者: jackhsien (jack)   2024-08-05 16:43:00
把修 bug trace code等等工時統計出來 和其他專案結果做比較 然後向上反應開發必需做更多審核考量 盡量反應事實 不要針對人
作者: deeeplove (想不到ID)   2024-08-05 19:33:00
艾倫
作者: ck237 (白色小雞)   2024-08-06 12:58:00
怎辦,我看到反而是你們開發速度太慢也
作者: purin88 (原來我是憤怒的鄉民)   2024-08-08 21:12:00
code review是要看流程邏輯,以後交接才知道他在寫什麼而不是針對有沒有空格,斷行怎麼斷,小細節在執著其實老闆喜歡他是因為東西要先有,之後的再說每個老闆個性不同,我就遇過要寫的快沒bug要有unit test,要有test case報告然後對空格排版又特別要求
作者: asadman1523 (黑炭)   2024-08-09 09:48:00
這就是老闆要的吧,快速產出你們沒辦法快速產出,他可以
作者: mepowerlmay (用心,找對人)   2024-08-10 00:30:00
這正常吧 先有畫面......?
作者: erowii (保保)   2024-08-10 12:15:00
說不定你以為有問題的情況才能為公司帶來最大的利益
作者: kattte (誠實面對自己吧!)   2024-08-14 10:34:00
他應該很擅長搞垮一間公司,先查查是不是商業間諜XD

Links booklink

Contact Us: admin [ a t ] ucptt.com