PTT評價

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

看板Soft_Job標題[討論] 產量高品質差的工程師作者
yestheway
(LKK)
時間推噓59 推:66 噓:7 →:138

大家有沒有遇過這樣工程師…

我們公司最近在開發新的專案,找了一位新來的工程師幫忙一起做。這個人Coding速度真的很快,交給他的功能很快就能做出來。每個sprint下來,他也一直不停的接新ticket和開發新東西。

最近這個新專案終於要上線了,結果QA卻測出了一大堆bug!!由於數量真的太多了,但又為了承諾客戶如期上線,所以只好把我和其他2個工程師也叫來,一起昴下去幫忙解bug…

結果不去看還好,一下去看他裡面的code,真的是非常可怕…又臭又長像流水帳一樣,結構也是亂七八糟,很多邏輯明顯沒有想過或設計過硬幹去寫出來,沒有任何彈性和維護性,大家花了非常多時間再改他的程式,真的改的非常辛苦...
(對…我們為了趕這個專案,完全skip code review、skip unit tests 等等。二來 這
新專案相對獨立,不影響現有系統。所以他commit 什麼 就merge什麼,鬧得今天這下場。我們的例子,正好回應前幾篇某些人質疑為何要code review......)

最後產品雖然如期上線,但這下好了,老闆和PM現在超喜歡這個工程師,後面很多v2 要衍生的新功能,都要叫這位工程師來主導開發…

我們幾個幫忙「收爛攤子」的人,聽到真的有種不好的預感…一來害怕又有更多有問題的程式被他寫出來,後面又要花更多時間來修改;二來有種功勞你在接,爛攤子我們在收的感覺…

我們原本找主管說這些問題,但目前公司大老闆想正積極開發這項產品,他們只希望快點見到結果,似乎也不太在乎原有的開發流程了,只想先快點把東西生出來,給客戶demo…

各位如果面對這種情況,和這樣的工程師該怎麼辦?公司想快速看到成品,找了一個產出快的人,雖然短期快速看得到成果,但卻後患無窮…





--

※ PTT 留言評論
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 1.145.105.198 (澳大利亞)
PTT 網址

mercurycgt6808/01 11:18離職

miyazakisun208/01 11:24屎擦太乾淨,以後負責專門擦 笑了

形容的太貼切了!所以我們現在超怕他繼續接下去做後續的功能..

andy81030608/01 11:30有bug當然叫commit的人處理吧,你去擦屎幹嘛

teddy08/01 11:31叫他一人負責就好 其他人去忙其他專案

沒辦法…我們不是老闆…這案子是公司目前最重要的任務,主管要我們其他人補上去修, 也很難退掉吧..

※ 編輯: yestheway (1.145.105.198 澳大利亞), 08/01/2024 11:42:57

ssccg08/01 11:47不是測過才算產量嗎? 不會動的也能算產量?

很好的問題!因為這些新功能都沒有unit test, integration test 等等,完全依賴QA手 動測試,在90%的情況程式都能動,但像是換個環境 改個context後,某些情況就掛了…

MOONY13508/01 11:47驅動生產 反正你們會如期修好

ssccg08/01 11:48怎麼會很難推,除非你真的很想待下去那你活該

其它同事我不知道,但我個人有簽證關係,沒辦法這麼瀟灑走掉…當然 相信解決問題 還 是有除了離職以外 更好的方式吧

nh60211as08/01 11:49你們自己沒有驗收條件就算結案很奇怪餒

seanhiroshi08/01 11:52想辦法把他捧起來 之後好處理

B098869808808/01 11:55之前不是有啟智說不用code review review是要看什麼

B098869808808/01 11:55 怎麼現在那種人又消失了

airtsubasa08/01 11:56影響正常運作的bug還能算量?

大部份情況都是能運做的,但某些例外狀況 原本可能沒被測出來吧......

wuyiulin08/01 11:58直接講不聽就投到他直屬主管那邊啊,他主管不喬是他的

wuyiulin08/01 11:58事情,出問題就把他們兩個推出去扛刀。

※ 編輯: yestheway (1.145.105.198 澳大利亞), 08/01/2024 12:07:33

tsaigi08/01 12:01就不要修啊 大不了一起爛

hobnob08/01 12:05接案出身?

hobnob08/01 12:06感覺你團隊問題也蠻大的欸..

我也覺得他是做接案出身的…並不是說做接案的人不好,但真的看過太多這類接案工程師 ,那種虛應一應故事 交差了事的做事方式… 但我們是在做SaaS自己的產品,原本的開發節奏雖然不算快,但要求很嚴謹 該做的都要 做到位review才會過。但現在為了敢這個新專案,完全亂了陣腳,搞出一堆「技術債」 到頭來這些不健康的程式,也是必有人要來修….

※ 編輯: yestheway (1.145.105.198 澳大利亞), 08/01/2024 12:34:11

lazarus112108/01 12:22沒有維運過自己開發的工程師,很大機率是垃圾量產機

Tenno08108/01 12:35我好奇他寫好的時候不用先跑一下看看有無問題嗎?

ssccg08/01 12:42開規格、測案開嚴一點,然後叫他自己全測過

pilor08/01 12:51好像我們公司

sowulo08/01 13:14看起來是驗收機制有問題吧 而且主管不會覺得一個人開發

sowulo08/01 13:14要三個人維護哪裡怪怪的嗎

talkmyself08/01 13:15離職阿,那mo簡單

lilicoco52008/01 13:20出個包讓老闆痛

ma72108/01 13:38需求定義不明確吧,怎麼會有因環境程式出錯

rereterry08/01 13:43如果要你們幫忙,那功能設計初期,你們應該可以加入吧

rereterry08/01 13:43!就要求流程,把merge的權力拿走,既然老闆要你們幫忙

rereterry08/01 13:43,我是不認為你們無法要求啦!這樣也可以讓這個工程師

rereterry08/01 13:43知道你們的要求,不急著他可以一步到位的改寫成符合你

rereterry08/01 13:43們的想像,但至少你們的工作也可以隨著紀錄一起被看到

lwecloud08/01 13:47樓上把老闆想得太美好了,老闆都覺得這樣的人超讚

j11223344608/01 13:47他現在credit 太高了 出個問題就會讓你老闆好好考慮

j11223344608/01 13:47這麼問題了

lwecloud08/01 13:47要怎麼從老闆眼中的紅人拿權力

hakama9908/01 13:54qa問題吧 你們的嚴謹只有rd?

littlebroken08/01 13:58第五行怪怪的 正常QA驗出issue修就是了 為啥搞得像

wuyiulin08/01 13:58等等 corner case 沒測出來你找RD不找QA?

wuyiulin08/01 13:59還是你司 RD有加領 QA薪水?

littlebroken08/01 13:59在救火? 他的東西在進dev branch前應該就要完整了

hakama9908/01 14:01寫很糟糕被qa退太多次就要review電他啊 你們根本沒有你

hakama9908/01 14:01說的嚴謹吧

的確QA那邊也有些問題。就我所知這個案子急著要上,當初連code review, unit test 什麼都沒做,直接送QA,QA那可能連完整的文件都沒有,只測到主要的use case,但很多 corner case都沒cover到。PM拿去給sales team 作demo時開始發現一堆corner case有問 題,才陸續開了一堆bug ticket,然後我們就被拉進來擦屎了… 而PM向老闆報告時,當然也只會說「90%都在短時間內完成了」 之類漂亮的話,畢竟PM也 要懂生存,老闆聽了當然也開心…扯遠了XD

chses91037208/01 14:01快點逃,我說真的,這個是制度問題,今天解決了下次

chses91037208/01 14:01還是重蹈覆徹

hakama9908/01 14:01只有rd自我要求

littlebroken08/01 14:02啊還有你們沒peer review跟1:1? code亂寫就提出來講

sjr50008/01 14:03這種品質還沒速度的話....

starburs08/01 14:04他很爛但是很聰明,你們很強但是很笨

我們確實是一群單純的工程師,確實不太精明…所以現在才在苦惱該怎麼辦..

Lordaeron08/01 14:08產量高就CHATGPT 就好了,但測不出來就SIT 做什麼的?

stepnight08/01 14:21你們團隊有問題,他也有問題

stepnight08/01 14:21看內文,照你們團隊正常的流程

stepnight08/01 14:21三個人下去做,案子可能趕不出來?

stepnight08/01 14:21他一人做你們兩人擦屎,結果是趕得出來

stepnight08/01 14:21這就是老闆要的,老闆只在乎這個

stepnight08/01 14:21要先領悟到這點,其他的對老闆都不是問題

ilede08/01 14:49這也很神奇阿...還要別人幫忙擦屎反而做得出來?

s06yji308/01 15:00主管在幹嘛...沒救了。不離職就自己吞了。

descent08/01 15:11老闆不知道你們修 bug 的貢獻嗎?

很多沒做技術職的人,真的不會理解,去修那10%的bug,可能比從零開始做那90%,還要 耗時費力…

arcade042508/01 15:30木已成舟,不想單純繼續擦屎就兩條路 1. 把他 merge

arcade042508/01 15:30權限拔掉,每次commit 都有人 互相 review,讓這份專

arcade042508/01 15:30案回歸正常開發流程,就算有屎也會變成大家一起拉的

arcade042508/01 15:30

arcade042508/01 15:302. 離職

對…目前只能希望,如果還要他繼續開發新功能,至少先回來follow正常開發流程。若是 又要為了趕進度,重道覆轍搞得一團亂,真的大家會瘋掉…

x2016508/01 15:48先工作流程紀錄給老闆看,後面再離職

※ 編輯: yestheway (1.145.107.162 澳大利亞), 08/01/2024 15:57:03 ※ 編輯: yestheway (1.145.107.162 澳大利亞), 08/01/2024 15:58:54

NTUTM0408/01 16:03恭喜變成專業擦屎工

recorriendo08/01 16:33好爽 本篇簡直軟工如何飛黃騰達的教學文 有收藏有推

atst208/01 16:33你們去幫他處理後續的bug,付出的成本有估算過嗎? 老闆知道

tw1150908/01 16:34公司就這樣啊,我曾經支援一個專案,程式一看就是一坨會

tw1150908/01 16:34動的大便,但聽說他主管對他評價很高,後來他離職接他工

tw1150908/01 16:34作的人也離職了XD

atst208/01 16:34嗎? 有沒有加班? 如果有,加班費有拿嗎?

atst208/01 16:34個人是覺得,你看來短期內走不了,那最少該拿的拿一拿.

atst208/01 16:36其他案子如何,後續有沒有坑,不是你們要負責的。

v8686106208/01 16:38真慘

v8686106208/01 16:39快逃;)

abc092200108/01 16:45表示老闆覺得你們平常開發方式不好

fatb08/01 16:56其實他在開發過程中應該就要一直在測試了

fatb08/01 16:57其實這種工程師就走專案的 以完成為第一目標 不能說對錯

newuser08/01 17:01我亂猜 這位工程師是不是印度人 以我遇過同樣的 是印度人

pot123408/01 17:14你們原本開發時間是他的兩倍以上嗎,是的話可以叫他多花

pot123408/01 17:14一倍時間寫test,然後你們去找別的工作

ku7208/01 18:06你們沒救了 老闆都是看結果的 老闆喜歡他 你以為是不知道你

ku7208/01 18:06們幫他擦屁股? 老闆喜歡他 是因為他可以讓你們有幾倍的產出

ku7208/01 18:06

ab4daa08/01 18:43我也好想專門拉屎別人擦QQ

brunchbug08/01 19:21離職 或 讓他爆

eugene3143108/01 19:30原來是bug的產量高,建議可以改一下標題

lchcoding08/01 19:49不試一下 pair programming 嗎?

lchcoding08/01 19:49反正會吵,何不一開始就吵...

lchcoding08/01 19:49當他要拉的時候,就電到他不敢拉

codehard08/01 19:55讓他爆,公司不會一個案子就死掉,如果是也是遲早。

hsiang104908/01 20:32要上線才在測?

xephon08/01 20:42你們驗收條件,只驗Happy Path,那放他過也是正常的

xephon08/01 20:42你們就是鼓勵這樣產出的工程師

NDark08/01 20:50錢若不花在教育 就會花在監獄

NDark08/01 20:50如果不能提早測試 那就要Allocate足夠的測試期

NDark08/01 20:51預測時程通常都是不準的原本預估拿來測試的時間都不夠

NDark08/01 20:51所以有經驗的人都會叫你早點測試 這樣早點反應時程規格問題

NDark08/01 20:59因為有時候不符規格不是開發的錯是規格就開得不準確

benben99408/01 22:43環境制度問題 你們也知道很趕

benben99408/01 22:43時間產品品質三角都要兼顧

jlhc08/01 23:03通常這種 promote 也很快, 因為老闆喜歡 impacted 也大

jlhc08/01 23:03說真的擦擦屁股就能跟著一起 promote 不好嗎?

Baternest08/01 23:32幫忙的人沒有原本負責的項目嗎?多少要推脫一點 不要全

Baternest08/01 23:33力去處理別人造成的問題 (是說大概已經來不及了...)

AoShenFengYu08/02 00:21這種人我遇過兩位,一位被開除了,另一位正在 pip

viper970908/02 00:28覺得是制度的問題+1

internetms5208/02 01:50公司衡量產出的方式有問題,這要tech lead或cto層

internetms5208/02 01:51級去講,反應沒有得到改善,唯一能做的就是想辦法

internetms5208/02 01:51將驗收條件納入衡量

internetms5208/02 01:51而且驗收條件要先訂這件事pm也不一定願意,吃力不

internetms5208/02 01:51討好

internetms5208/02 01:51應該是蠻複雜的多個位置的偷懶結果

sachung2808/02 05:53說不定他有意的 他一人在忙接PM一堆要求壓榨 你們這些

sachung2808/02 05:53老人在納涼也不幫忙或講話 那時間不夠當然先求會動 快

sachung2808/02 05:53做完你們以為能拿成果想進去分功勞 踩到屎剛好而已

sachung2808/02 06:02換個角度說不定變成接到屎缺辛苦完成後受到老闆賞識

sachung2808/02 06:03 結果老人嫉妒開始搞排擠和貶低能力還把程式改得面目

sachung2808/02 06:03全非很難維護

sachung2808/02 06:14剛到新公司-努力求站穩-老人慢慢做-PM看新人完成比較

sachung2808/02 06:14快工作一直指派-老人覺得請到好人樂得繼續慢慢來-新人

sachung2808/02 06:14愈用愈賭爛開始程式亂寫-接內文後續?

abraxas08/02 07:35每次都趕,每次都這樣,還不快跑?

Csongs08/02 09:14他紀錄會很漂亮 大老闆會升他

flysonics08/02 09:30勸你快跑吧 這人搞不好之後就變你老闆了

flysonics08/02 09:31他已經紅了 你現在再跟老闆提什麼要求 在老闆眼裡都像

flysonics08/02 09:31扯他後腿

stepnight08/02 09:32看原Po的回應,只想把鍋甩在新人身上

stepnight08/02 09:32把事情塑造成老人們天神下凡拯救專案

stepnight08/02 09:32而老闆卻只看到新人好棒棒

stepnight08/02 09:32搞不好老闆也看透不說破你們在搞啥啦

flysonics08/02 09:35你現在最多就是盡量自保 抓好自己手裡的開發工作 避免

flysonics08/02 09:35被assign去幫他擦屁股 不然就是跟他打好關係 看看有沒

flysonics08/02 09:35有機會他被premote時你能分點羹

lturtsamuel08/02 09:55不就是自食惡果的故事嗎== 找個新人來幫忙 舊的人偷

lturtsamuel08/02 09:55懶一行扣都沒看過 到品管才被抓出來 說你們沒問題誰

lturtsamuel08/02 09:55會信==

lturtsamuel08/02 09:56假如產品跟你無關 為什麼叫你來救火?假如有關 為什

lturtsamuel08/02 09:56麼你之前敢放著都讓他弄?

lturtsamuel08/02 10:04新人也很難知道貴公司的產品會有什麼奇怪的 edge cas

lturtsamuel08/02 10:04e,你們訓練沒做好,品質不把關,規格和程式也不好好

lturtsamuel08/02 10:04寫把 edge case 變成必經的檢查,那還想怎樣咧

lturtsamuel08/02 10:16最後 你們公司的期望也有問題 到底想要會動就好的poc

lturtsamuel08/02 10:16還是品質好的產品?前者的話新人幹得很好 事後要修bu

lturtsamuel08/02 10:16g也是期望中的事 後者的話那為什麼不遵守開發流程?

你腦補的太多了….

※ 編輯: yestheway (1.158.19.142 澳大利亞), 08/02/2024 10:34:01

ericthree08/02 10:45能力不行啊 讓他做雜事吧

ericthree08/02 10:45如果老闆敢讓他主導那就陪他做垃圾

flash540808/02 11:07Ptt真多大開眼界的事 沒有code review 沒test 隨便me

flash540808/02 11:07rge 沒有dev infra嗎

wuwaiter08/02 11:20離職啊, 公司有這樣的人 也不會有好發展

wuwaiter08/02 11:20你的能力這麼強的話 到那邊都會發光

LiebeLion08/02 11:42真正問題是qa沒做好

LiebeLion08/02 11:42QA要負責edge case

LiebeLion08/02 11:43再怎麼趕也不可能不Q就進

s06yji308/02 11:53原po要一直解釋了,網路上超多腦補怪XD

LiebeLion08/02 12:12demo就遇到的case不能算corner吧

jeffreyup108/02 12:36我單純在想為啥沒有pr code review

MonkeyCL08/02 12:39他以後可能是你主管

GoalBased08/02 13:16你反映給主管,主管不往上反應,某種程度主管覺得他那

GoalBased08/02 13:16樣是比較符合公司要的,或者老闆認為那是公司要的

sachung2808/02 13:26講完整點別人也不用腦補

zenuo08/02 14:15推回給原開發的維護,都測出問題還要別人改?

worf08/02 16:54改得很辛苦 他怎麼可以置身事外 這不是很奇怪嗎

worf08/02 16:54你上面的人又不知道你改得很辛苦 他的BUG很多

mathrew08/02 18:00誰叫你們要那麼認真 debug,當然是挖洞給他跳啊

A009112708/02 19:00建議直接把這篇貼給主管看

SuperTaco08/02 19:13推mathrew

luke7208/02 20:09這種反應沒用,別傻了,上面一堆code review的也是幻想

luke7208/02 20:10code review是有政治權力的人review賤民的code,你有權嗎

luke7208/02 20:12這種人就是政治滿分,直接跟老闆說他是專家,你不懂

luke7208/02 20:13他的code快爆炸前就會拖你去當墊背,他置身事外

luke7208/02 20:17你能做的有兩個,一個是當練功,未來這些都是你的履歷

luke7208/02 20:19一個是提高政治敏感度,務必在他起手式就阻絕掉讓他自爆

luke7208/02 20:21當然他可能會找你主管下手,把鍋全丟你身上,這就沒辦法

joke070908/02 20:47收藏了 之後朝著方向練習

agra08/02 23:02「每個sprint都不斷接新ticket卻最後QA不過」,聽起來敏捷的

agra08/02 23:02部分就很歪啦

imhaha08/02 23:08不要放出來拜託

Ghamu08/03 01:25這好像有看過那個漫畫 這個人最後會榮譽跳巢漲薪水 後面公

Ghamu08/03 01:25司會多請更多工程師來擦屁股 順便抱怨說怎麼這些新來的工程

Ghamu08/03 01:25師比不上之前那位做超快的

Ghamu08/03 01:29說不定還會說你看不懂他程式是因為你程度不足勒 呵呵

Ghamu08/03 01:31這種事情通常是要反映給一個老闆信得過的資深工程師或是主

Ghamu08/03 01:31管 讓他們去講才有用 你們平行的工程師去跟不懂技術的老闆

Ghamu08/03 01:31講可能還會被視為眼紅要中傷人勒

prag22208/03 04:10看你有沒天天加班在擦屁股

prag22208/03 04:10能上來抱怨感覺還能接受?

crustancean08/03 10:48都Q過一堆問題了,還在QA的問題

wade5208/03 11:37跟老闆談條件,讓你做專案,換你亂寫,他來debug

eulbos08/03 13:14老闆會覺得新人很有潛力 願意栽培

henrylin808608/03 18:14你們問題在開發流程,你不即時Review誰剛來會知道

henrylin808608/03 18:14公司的coding style,除非你們有linting tool在管

henrylin808608/03 18:14控,但顯然沒有。再來是CI,你們Merge沒做test的話

henrylin808608/03 18:14,工程師們大多都是自己測爽了就上code,畢竟有太

henrylin808608/03 18:14多隱性的Bug在一開始是很難發現的,而工程師又很清

henrylin808608/03 18:14楚自己設計的程式邏輯,通常會無意識避開沒被考慮

henrylin808608/03 18:14進去的邏輯。

henrylin808608/03 18:17沒在前面時間寫unit test跟code review,那就是後

henrylin808608/03 18:17面花時間debug。老闆不是軟體出身不會懂,那你們真

henrylin808608/03 18:17的只能自求多福。

yoyoyolin08/03 21:21接案就是要快啊。接案通常沒在code review很正常,套句

yoyoyolin08/03 21:21sa 講的話,功能可以跑就好。你能做的就是跟對方溝通,

yoyoyolin08/03 21:21請對方要測過再commit。不然這真的無解

NDark08/03 22:51外包或支援就是看單子辦事 單子沒提到的當然就不算

NDark08/03 22:51這就是自研開發與外包支援的差異

NDark08/03 22:52這才是為什麼開單/開規格的人本應該把規格/測試寫清楚

NDark08/03 22:52所以才會有找外援反而更浪費人力的情況發生

NDark08/03 22:53因為原本可能對自己人可以模糊一點的需求必須改寫很清楚

NDark08/03 22:54規格裡面沒有限制效能資源的話

NDark08/03 22:54做的人浪費資源記憶體也只能默默自己事後改

rogerlarger08/04 18:27這樣產一堆垃圾也算產量高嗎