[讨论] 产量高品质差的工程师

楼主: 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