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

楼主: Lordaeron (Terry)   2024-08-01 20:39:32
※ 引述《yestheway (LKK)》之铭言:
: 大家有没有遇过这样工程师…
: 我们公司最近在开发新的专案,找了一位新来的工程师帮忙一起做。这个人Coding速度真
: 的很快,交给他的功能很快就能做出来。每个sprint下来,他也一直不停的接新ticket和
: 开发新东西。
: 最近这个新专案终于要上线了,结果QA却测出了一大堆bug!!由于数量真的太多了,但
: 又为了承诺客户如期上线,所以只好把我和其他2个工程师也叫来,一起昴下去帮忙解bug
: …
我也很好奇,怎么你们不一开始就做呢?
: 结果不去看还好,一下去看他里面的code,真的是非常可怕…又臭又长像流水帐一样,结
: 构也是乱七八糟,很多逻辑明显没有想过或设计过硬干去写出来,没有任何弹性和维护性
: ,大家花了非常多时间再改他的程式,真的改的非常辛苦...
这种code chatgpt 是可以代劳的,大概也就是哪样的光景。
为何你们不join?
: (对…我们为了赶这个专案,完全skip code review、skip unit tests 等等。二来 这
: 新专案相对独立,不影响现有系统。所以他commit 什么 就merge什么,闹得今天这下场
: 。我们的例子,正好回应前几篇某些人质疑为何要code review......)
: 最后产品虽然如期上线,但这下好了,老板和PM现在超喜欢这个工程师,后面很多v2 要
: 衍生的新功能,都要叫这位工程师来主导开发…
: 我们几个帮忙“收烂摊子”的人,听到真的有种不好的预感…一来害怕又有更多有问题的
: 程式被他写出来,后面又要花更多时间来修改;二来有种功劳你在接,烂摊子我们在收的
: 感觉…
: 我们原本找主管说这些问题,但目前公司大老板想正积极开发这项产品,他们只希望快点
: 见到结果,似乎也不太在乎原有的开发流程了,只想先快点把东西生出来,给客户demo…
: 各位如果面对这种情况,和这样的工程师该怎么办?公司想快速看到成品,找了一个产出
: 快的人,虽然短期快速看得到成果,但却后患无穷…
这种故事就真的很有趣。
但这位神人在做时,你们在做什么?
为何已经赶成这样了,他好不容易写好,哪你们改他的同时有CODE REVIEW 吗?
有: 谁REVIEW? PM? 老板? 神人? 还是互看?
这不就很神? 有空改写有空测,还有空
REVIEW,还可以用更短的时间完成且没BUG,这绝对是台湾之光。
没: 整篇是想表示你们很神? 因为他写到到快DEAD LINE 了,结果你们可以在这个
更短的时间,将他的重写完,还不用review。神囉.....
还真的是鬼月到讲鬼故事。
至于code review 囉....你是知道怎么做?
IEEE 1028-2008 lists the following review types:[6]
Management reviews
Technical reviews
Inspections
Walk-throughs
Audits
还是你只是 Software peer review?
正式同行评审的程序会定义参与者特定的角色,
进入评审及离开评审的品质准则,在同行评审程序中要确认的软件度量。
在检查过程中,会有以下的角色。
作者:建立待检查工作文件的人。
主持人:领导检查流程的人,主持人规划检查流程,并且进行协调。
朗读者:朗读整份文件的人,一次读出一部份,其他的检查者会指出有缺陷之处。
记录:在检查过程中记录大家找到缺陷的人。
检查者:检查工作文件中是否有缺陷的人。
检查流程中的各阶段包括有:计划、简介会议、准备、检查会议、修正及追踪。
以上中文来自WIKI,和英文WIKI 一致。
工程,还是以结果论英雄。
偏偏由一票没programming 背景的人,发明了一票"方法",让哪些
傻傻的programmer 去跟,还有人将他们当神拜。
不管agile, code review, 等的源头,都是没/没什么专案实绩的人发明的。
真的除了人月神话。这本书还有20th review 版。
作者: yestheway (LKK)   2024-08-01 22:01:00
你说这些每个人都知道…这么说好了 就像是交通规则,大家也都知道,但当你老板对你快马加鞭,要你闯红灯、超速,要你十分钟到达目的地时,你还停下礼让行人吗…
楼主: Lordaeron (Terry)   2024-08-01 22:53:00
这不就好了,神人啊...我只知越急的写得越烂要是搞到天天加班就是专产垃圾了,但不包括神人。忘了,闯红灯害人害己。就算是开救护车也不一定安全。
作者: yestheway (LKK)   2024-08-02 09:09:00
这些google就找得到的教科书理论和工成流程都很好,但除非你今天是老板,否则很难有公司让你完全照着最新最完美的流程走。你可能工作经验不是很多,看得还比较单纯,多工作几年,你就会理解现实和理论往往很难并行…
作者: stepnight (桃卡武康)   2024-08-02 09:22:00
不是阿,回文你都能理解现实与理论差距那你怎么不能理解新人为了时程只能交烂Code的这件事?你也很奇怪,不怎么抱怨老板反而花篇幅抱怨赶出结果的新人你还不如多把篇幅着墨在阿新人就是故意摆烂从新人变老人换个屁股脑子也跟着换了是吗?
作者: B0988698088 (废文少女小円♥)   2024-08-02 10:03:00
这只就没料废文ID你们跟他认真啥0.0..
作者: yestheway (LKK)   2024-08-02 10:21:00
我们并没有人去赶这个新人,甚至怕他开发时间不够,才省去了这些unit test, integration test, code review等等工作。光是不用写测试,已经多出很多时间了!大家虽然都有自己工作要忙,但能够帮的都做到位,有应必答,有技术问题也是一步步带。就算时间再短,你是不是在敷衍应付,还是有用过心,尤其是很多结案的,真的不要以为没人看得出来欸......
楼主: Lordaeron (Terry)   2024-08-02 10:58:00
有料的废文王B0988698088,先回我上一篇吧。原作先生,你就针对我的问题回一篇即可。unit test 做不做根本没人在意,SIT 做就可以反应问题而原作先生,你不用质疑我的工作经验。这你挑不来的。
作者: wuwaiter (挂~~有事打我手机)   2024-08-02 11:24:00
没有赶新人 然后通篇在干樵新人写的不合老人的习惯
作者: stepnight (桃卡武康)   2024-08-02 11:38:00
你们没在赶新人,所以这专案是没时程压力?照原文看新人是来“帮忙”专案,怎么帮到被老板、主管,委以重任V2功能给他做?既然是“帮忙”你们怎么搞到主导者看起来变新人
作者: yestheway (LKK)   2024-08-02 12:23:00
公司原本人手就吃紧了,就我所知,老板承诺了客户开发这个专案,在这情况下,只好再征个工程师来开发,而这名新人也有过多年专案开发经验,就让他负责接下开发了。之后就发生了我前面文章的故事…我认为你可能想的太可怕了,事实上西方公司的团队文化,没有这么显著的“老人” “新人”上下关系或什么压榨新人,我也在台湾工作过多年,我懂你在说什么,但真的不是那样…大家都是平起平坐的同事关系。我们一起吃饭时,也常关心他工作和生活状况等,大家下班也是5点准时走,不是你想的那个样子…至于他的工作成果,就如前文所说,又快又惊人XD…
作者: sachung28 (00)   2024-08-02 12:42:00
除非你有办法证明他的程式品质差到不如重写 且同样时间内产出成果不如旧有开发模式 不然都是多讲的
楼主: Lordaeron (Terry)   2024-08-02 14:03:00
就不能针对我的质疑回应就是了?我没要你证明品质哦。
作者: codehard   2024-08-02 21:36:00
承诺也是可以跳票的,多的是在客人前面乱开支票的老板,反正那个案子不delay,先签下来才有之后的事。
楼主: Lordaeron (Terry)   2024-08-02 22:07:00
看你够不够力,不然被罚到倒赔是可能的。
作者: CoNsTaR ((const *))   2024-08-03 00:16:00
#1PnMqPe6 (C_and_CPP)这篇原 Po 就 csfgsj 分身啊,大家跟他认真啥#1PoeaOee (C_and_CPP)笑死
作者: Ghamu (猫丸)   2024-08-03 01:40:00
这就跟大家都在赶时间谁在看红绿灯一样吧 理论上看到红灯要停下来 但我在赶时间啊 所以我闯红灯错了吗? 当然没错啊你看拓海过弯不踩刹车用水沟盖跑法不就得第一了? 所以我过弯不踩刹车有问题吗? 都是以成败论英雄啦 时间在赶实在管你什么什么红绿灯啦
楼主: Lordaeron (Terry)   2024-08-03 08:50:00
@CoNsTaR 你脑补的,可以继续.然后一群C++ GEEKS 在哪抱团。然后"对于同事的coding style感到很感冒" 在哪抱着暖.C++真的没问题,真的是用的人的问题,但自己人就没问题

Links booklink

Contact Us: admin [ a t ] ucptt.com