作者w0005151 (蓝厅)
看板Soft_Job
标题Re: [讨论] 怎麽跟自以为是的同事相处
时间Wed Oct 26 11:50:40 2022
提供一点不一样的看法
※ 引述《leo5916267 (封膜猎人)》之铭言：
: 也许在软体也蛮容易遇到类似个性的同事
: 我们是新创公司，我进去前已就有一个前端工程师，他从0建构了整个产品A
代表他能力不算差
而且产品A有很多他习惯的practice
: 我是产品B的前端，刚好我们产品线不急，被拉去支援他们 改版
: 但在合作上就觉得跟他相处很不舒服
: 可能是把我当竞争对手吧？
: 喜欢用高姿态/批判的方式codereview,
有些人讲话是比较机歪
但review comment只要不是「这边写得好烂」这种垃圾话
应该都还是有点价值
然後个人认知中的code review
最主要的目的，是要让你的code能符合既有的style
细到变数命名，粗到目录结构、pattern的运用
有没有重造轮子，还是有既有的utility function可以用但你不知道？
很多工程师（包含小弟我XD
都会有自己的写法最屌的错觉
对於没看过或不熟悉的做法总是看不顺眼
code review就是在确保
一个repo在多人维护下也能有一定程度的一致性
而不是每次都要在review过程中讨论出一个很猛的写法或架构
: 而我对他提出写法的意见，才提开头一句
: 就霹雳啪拉回了十句，顺带挑我程式毛病，我觉得更像是用公事来打压别人
想像一下，你从0开始写出了整个专案
一个新来的工程师，不了解既有的convention，上来就想照自己的方式写
你会不会觉得奇怪？
: 就讲不得，而整个团队都对他很头痛，但又要依赖他做事情，很多文件需求都没写清楚，很多事情都绑在他身上，而且专案架构维护性蛮差的，我看了整整一个月才懂他的思路，大概就是小孩子拿AK的感觉
脾气不好，但又依赖他做事，就更代表他确实能力不错
文件需求不清楚是小公司满常见的，而且需求应该是PM负责吧怎麽会是工程师来写需求？
: 我们做事不得不都要照的他的方式做事，但他又很自我中心，跟他配合心力大概4成是处理情绪问题4成才是程式问题
由他建立起的专案，按照他的方法做事，应该没什麽问题
因为你也是短期支援而已，要是被你一通乱改改到他看不懂
後面要收拾的也不是你
: 我网路上找过类似的关键字
: 攻击性强的同事
: 自以为是的同事
: 他的性格满符合上面相关搜寻找到的描述
: 不知道各位前辈是怎麽应对的
: 我现在是当练EQ，大概还要半年改版完忍忍
这段都是个人心情的部分，就不多做评论
: 程式部分就消极应对，我有好的想法就跟别人讨论，在他的专案只用他写过的方式做
有好的想法，也要看专案性质跟时程
不一定你在A专案的好做法，到B专案也适用
尤其前端又是生态系很乱的一个环境
光React管理CSS的做法，就有三种以上知名的solution
每过一两年都会看到「原本那套过时了，现在应该要用这个啦」的说法
如果你想套新的作法，当然就是要跟身为原作者的他讨论
很多情况都是一段你看起来很粪的code
但前人已经想过几种可能的优化方法，只是受限於某些条件无法这样做
或是大家判断这边再改动的可能性很低，乾脆就先放着能动就好
因为没有实例，所以我也无法给你太具体的建议
不过如果想做refactor的话，最好还是要有条理的整理出
旧做法哪里不好，新做法哪里好
是改善了效能、改善可读性、改善扩充性，还是什麽呢？
如果你真的对自己的做法很有信心
也可以直接做个prototype後PR就丢出来
把重要的人全部拉进去当reviewer
要是多数人认同你的做法，只有他在PR里面乱喷
就算最後没办法merge，那也是帮助你在公司建立credit
他则是在败自己的人品
那也许有一天专案出事，主管就会想到要找你去救
到时候自然是你想怎麽写就怎麽写
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 125.228.191.217 (台湾)
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1666756242.A.FE3.html
1F：→ DrTech: code review不仅是一致性的问题而已，还有程式码品质，执 10/26 12:25
2F：→ DrTech: 行效率。 10/26 12:26
要优先度来说小弟还是认为一致性是最优先的
如果有一定规模的专案，一般的feature照原本的practice写
效能跟可读性应该都不会差到哪
3F：推 gooseduck: 推这篇 10/26 12:32
4F：→ WaterLengend: 中肯推 10/26 12:46
5F：→ leolarrel: code review 的好处很多,我个人另加一项:"事前预防总比 10/26 13:03
6F：→ leolarrel: 事後补救来的好",赶上了上线时间又如何,bug在客户面前 10/26 13:04
7F：→ leolarrel: 爆了,客户一样翻脸拍桌 10/26 13:04
8F：推 Josesan: 中肯，推 10/26 13:09
※ 编辑: w0005151 (125.228.191.217 台湾), 10/26/2022 13:44:46
9F：推 NDark: bug在客户面前爆 是没QA 跟code review关系? 10/26 14:35
10F：推 s78513221: 客户容忍度高可以让客户QA 10/26 15:00
11F：推 Belieeve: 推～前端的话一致性很重要，因为可以不一致的可能性太多 10/26 16:19
12F：→ Belieeve: 了 10/26 16:19
13F：推 happylei: 推 10/26 17:24
14F：推 kurtsgm: 中肯XD 10/26 17:27
15F：推 mathrew: 推这篇，工程师最常犯的一件事情就是，只顾做自己的事 10/26 17:28
16F：→ mathrew: 有时候眼界没放这麽高，事情不是你想的那麽简单 10/26 17:28
17F：推 smalldra: 这篇正解 10/26 18:06
18F：→ longlongint: 客户爆掉的时候已经交接给学弟的学弟了 10/26 18:21
19F：推 Lhmstu: 确实，一致性超重要 10/26 18:31
20F：推 justaID: 觉得蛮中肯 10/26 18:48
21F：推 IamTD: 同意这篇 10/26 19:42
22F：推 molopo: 推 10/26 20:33
23F：推 Roderickey: 推 10/26 20:45
24F：推 jasonwung: 推 10/26 21:00
25F：推 leveger0903: 我蛮认同这篇 10/26 23:38
26F：→ leveger0903: 以前我也是看不惯我家RD头的老旧写法 後来进来的一 10/26 23:43
27F：→ leveger0903: 些同事差不多心态 开发用当代最炫炮的php写法 也没 10/26 23:43
28F：→ leveger0903: 做code review 10/26 23:43
29F：→ leveger0903: 这些人後来拍拍屁股离职 留下一堆不太好维护的程式 10/26 23:46
30F：→ leveger0903: 码 就由待在这的同事维护 我也有接到这种的 出事的 10/26 23:46
31F：→ leveger0903: 时候感到头痛 10/26 23:46
32F：推 vi000246: 有些人重构或是design pattern玩过头了 都会写出一些很 10/27 01:39
33F：→ vi000246: 难懂的code 用IDE追半天还看不懂在写啥 10/27 01:39
34F：推 KanzakiHAria: 笑死XDDDDDDDDD 10/27 05:17
35F：推 james732: 推 10/27 08:30
36F：推 baobomb: 中肯 code review cmts就算是回一大堆 但如果都是有价值 10/27 12:43
37F：→ baobomb: 的cmt 那也没什麽问题 10/27 12:43
38F：→ baobomb: 能够愿意理性讨论的都还是好同事 最白烂的是被留了cmt直 10/27 12:44
39F：→ baobomb: 接不甩你按resolved 的 10/27 12:44
40F：推 stillcolor: 有一说一，工程师也要学沟通 10/27 13:02
41F：→ Ekmund: 我的实际经验是...想太美了 就算对方败人品大家都知道 只 10/27 14:39
42F：→ Ekmund: 要架构是他做的 团队有时程压力就不可能对他的权限/职务 10/27 14:39
43F：→ Ekmund: 做任何异动 就算某个part是你做起来 你处理比较准比较快 10/27 14:39
44F：→ Ekmund: 会嘴你的还是会嘴 最终管理层也只是认知到“这两个人以後 10/27 14:39
45F：→ Ekmund: 尽量别排一起” 接着两边摸头 回到原案依然你做你的他嘴 10/27 14:39
46F：→ Ekmund: 他的 什麽都不会变 10/27 14:39
47F：→ Ekmund: 唯一解法 该翻脸翻脸 就事论事公开地吵 吵到大家受不了 10/27 14:42
48F：→ Ekmund: 但先决条件是你不怕离开团队也不怕当黑脸 10/27 14:42
49F：→ viper9709: 推这篇 10/27 18:05
50F：推 Nitricacid: 推 10/27 19:55
51F：推 gundamdx: 推，想要大量重构，也要有出问题站出来负责的心态 10/28 06:45
52F：推 Csongs: 中肯 10/28 13:21
53F：推 fly19920820: 推这篇 10/29 16:42
54F：推 ph90119: 推推 10/31 23:09
55F：推 overhead: 推。当菜鸟时不明白，後来才明白这番道理 11/10 01:51