作者Lordaeron (Terry)
看板Soft_Job
标题Re: [讨论] 产量高品质差的工程师
时间Thu Aug 1 20:39:32 2024
※ 引述《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 版。
--
open source projects:
https://github.com/terrylao/
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 220.141.69.61 (台湾)
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1722515974.A.C71.html
※ 编辑: Lordaeron (220.141.69.61 台湾), 08/01/2024 20:43:37
※ 编辑: Lordaeron (220.141.69.61 台湾), 08/01/2024 21:37:16
※ 编辑: Lordaeron (220.141.69.61 台湾), 08/01/2024 21:52:43
1F：推 yestheway: 你说这些每个人都知道…这麽说好了 就像是交通规则，大 08/01 22:01
2F：→ yestheway: 家也都知道，但当你老板对你快马加鞭，要你闯红灯、超 08/01 22:01
3F：→ yestheway: 速，要你十分钟到达目的地时，你还停下礼让行人吗… 08/01 22:01
4F：→ Lordaeron: 这不就好了，神人啊...我只知越急的写得越烂 08/01 22:53
5F：→ Lordaeron: 要是搞到天天加班就是专产垃圾了，但不包括神人。 08/01 22:53
6F：→ Lordaeron: 忘了，闯红灯害人害己。就算是开救护车也不一定安全。 08/01 23:40
7F：推 yestheway: 这些google就找得到的教科书理论和工成流程都很好，但 08/02 09:09
8F：→ yestheway: 除非你今天是老板，否则很难有公司让你完全照着最新 08/02 09:09
9F：→ yestheway: 最完美的流程走。你可能工作经验不是很多，看得还比 08/02 09:10
10F：→ yestheway: 较单纯，多工作几年，你就会理解现实和理论往往很难并 08/02 09:10
11F：→ yestheway: 行… 08/02 09:10
12F：推 stepnight: 不是阿，回文你都能理解现实与理论差距 08/02 09:22
13F：→ stepnight: 那你怎麽不能理解新人为了时程只能交烂Code 08/02 09:22
14F：→ stepnight: 的这件事？你也很奇怪，不怎麽抱怨老板 08/02 09:22
15F：→ stepnight: 反而花篇幅抱怨赶出结果的新人 08/02 09:22
16F：→ stepnight: 你还不如多把篇幅着墨在阿新人就是故意摆烂 08/02 09:22
17F：→ stepnight: 从新人变老人换个屁股脑子也跟着换了是吗？ 08/02 09:22
18F：嘘 B0988698088: 这只就没料废文ID你们跟他认真啥0.0.. 08/02 10:03
19F：推 yestheway: 我们并没有人去赶这个新人，甚至怕他开发时间不够，才 08/02 10:21
20F：→ yestheway: 省去了这些unit test, integration test, code review 08/02 10:21
21F：→ yestheway: 等等工作。光是不用写测试，已经多出很多时间了！ 08/02 10:21
22F：→ yestheway: 大家虽然都有自己工作要忙，但能够帮的都做到位，有应 08/02 10:21
23F：→ yestheway: 必答，有技术问题也是一步步带。就算时间再短，你是不 08/02 10:21
24F：→ yestheway: 是在敷衍应付，还是有用过心，尤其是很多结案的，真的 08/02 10:21
25F：→ yestheway: 不要以为没人看得出来欸...... 08/02 10:21
26F：→ Lordaeron: 有料的废文王B0988698088，先回我上一篇吧。 08/02 10:58
27F：→ Lordaeron: 原作先生，你就针对我的问题回一篇即可。 08/02 10:59
28F：→ Lordaeron: unit test 做不做根本没人在意，SIT 做就可以反应问题 08/02 11:00
29F：→ Lordaeron: 而原作先生，你不用质疑我的工作经验。这你挑不来的。 08/02 11:01
30F：→ wuwaiter: 没有赶新人 然後通篇在干樵新人写的不合老人的习惯 08/02 11:24
31F：→ stepnight: 你们没在赶新人，所以这专案是没时程压力？ 08/02 11:38
32F：→ stepnight: 照原文看新人是来「帮忙」专案，怎麽帮到 08/02 11:38
33F：→ stepnight: 被老板、主管，委以重任V2功能给他做？ 08/02 11:38
34F：→ stepnight: 既然是「帮忙」你们怎麽搞到主导者看起来变新人 08/02 11:38
35F：推 yestheway: 公司原本人手就吃紧了，就我所知，老板承诺了客户开发 08/02 12:23
36F：→ yestheway: 这个专案，在这情况下，只好再徵个工程师来开发，而这 08/02 12:23
37F：→ yestheway: 名新人也有过多年专案开发经验，就让他负责接下开发了 08/02 12:23
38F：→ yestheway: 。之後就发生了我前面文章的故事… 08/02 12:23
39F：→ yestheway: 我认为你可能想的太可怕了，事实上西方公司的团队文化 08/02 12:23
40F：→ yestheway: ，没有这麽显着的「老人」 「新人」上下关系或什麽压榨 08/02 12:23
41F：→ yestheway: 新人，我也在台湾工作过多年，我懂你在说什麽，但真的 08/02 12:23
42F：→ yestheway: 不是那样…大家都是平起平坐的同事关系。我们一起吃饭 08/02 12:23
43F：→ yestheway: 时，也常关心他工作和生活状况等，大家下班也是5点准 08/02 12:24
44F：→ yestheway: 时走，不是你想的那个样子…至於他的工作成果，就如前 08/02 12:24
45F：→ yestheway: 文所说，又快又惊人XD… 08/02 12:24
46F：推 sachung28: 除非你有办法证明他的程式品质差到不如重写 且同样时 08/02 12:42
47F：→ sachung28: 间内产出成果不如旧有开发模式 不然都是多讲的 08/02 12:42
48F：→ sachung28: 关系不差又平等 为啥不直接找他谈 而是这边diss品质差 08/02 13:07
49F：→ Lordaeron: 就不能针对我的质疑回应就是了？我没要你证明品质哦。 08/02 14:03
50F：推 codehard: 承诺也是可以跳票的，多的是在客人前面乱开支票的老板 08/02 21:36
51F：→ codehard: ，反正那个案子不delay，先签下来才有之後的事。 08/02 21:36
52F：→ Lordaeron: 看你够不够力，不然被罚到倒赔是可能的。 08/02 22:07
54F：→ CoNsTaR: 这篇原 Po 就 csfgsj 分身啊，大家跟他认真啥 08/03 00:17
56F：→ CoNsTaR: 笑死 08/03 00:26
57F：推 Ghamu: 这就跟大家都在赶时间谁在看红绿灯一样吧 理论上看到红灯要 08/03 01:40
58F：→ Ghamu: 停下来 但我在赶时间啊 所以我闯红灯错了吗？ 当然没错啊 08/03 01:40
59F：→ Ghamu: 你看拓海过弯不踩刹车用水沟盖跑法不就得第一了？ 所以我过 08/03 01:40
60F：→ Ghamu: 弯不踩刹车有问题吗？ 都是以成败论英雄啦 时间在赶实在管 08/03 01:40
61F：→ Ghamu: 你什麽什麽红绿灯啦 08/03 01:40
62F：→ Lordaeron: @CoNsTaR 你脑补的，可以继续. 08/03 08:50
63F：→ Lordaeron: 然後一群C++ GEEKS 在哪抱团。 08/03 08:51
64F：→ Lordaeron: 然後"对於同事的coding style感到很感冒" 在哪抱着暖. 08/03 09:02
65F：→ Lordaeron: C++真的没问题，真的是用的人的问题，但自己人就没问题 08/03 09:03