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