同事防御性编程影响项目怎么办

2024-04-17 09:51:13 +08:00
 MadSix

背景如下: 我们有多个组,我带组 1. 同事 A 不在我的组,属于组 2. 现在同事 A 正在开发一个新项目,他是目前的主力,我安排人配合他一起完成,但目前开发内容上交集很少,拆的比较开。

问题如下: 1.目前已经明确,这个项目开发完之后,全权移交给我组负责后续开发维护,我是第一责任人。 2.最近组里兄弟跟我反馈,这哥们的代码,太过防御性了点,代码几乎是不可维护的。干什么都是梭哈,除了他自己没人看得懂写的是什么。我稍微看了一眼,想在他的代码上加功能我看了都满头大汗,无从下手。 3.这个系统其实有个前身,后来他介入之后,也是这套打法。到后来成功的没办法继续维护了,只能推倒重做一套,继续放任,就是历史重演。

我很想提这个问题,但他又不归我管。他防御性编程其实我不是很在意,别人自有维护饭碗的方式,但是影响到我后续开发了,我就很难受。

现在最困难的是: 这个项目并不是由组 2 的组长带的,而是一个技术老大亲自带的,所以找组 2 组长沟通没用。 至于这个技术老大。emm 说实话不太管事,只管下任务,也不关心代码质量和项目细节,所以跟他反馈等于没说。平时开发告诉他做完了就行,代码 review 他都不做,要不然也不能让这哥们毫无心里负担得这么干。奈何技术老大位高权重资历深厚,咱们也没什么办法。

我现在很想跟上面反馈一下这个问题,但是我这种跨管辖范围的做法总是不太好,而且也不知道提了有没有用。难道就这么把这口屎吃下来不成。

V 友们给出出主意。

21786 次点击
所在节点    职场话题
123 条回复
handpr
2024-04-17 10:15:25 +08:00
@MadSix #10 和我现在上手的项目一样.举个例子.之前的后端让前端自己写死控制哪个按钮显示不显示
fredweili
2024-04-17 10:15:33 +08:00
强化 code review ,不允许随便 merge
yangxiaopeipei
2024-04-17 10:15:41 +08:00
code review 怎么过的
FrankAdler
2024-04-17 10:16:26 +08:00
原来代码写的烂还有这么一个高大上的名字
whoosy
2024-04-17 10:19:34 +08:00
我认为只是单纯的水平菜,身边技术稍微强点的基本上对代码都有洁癖
RightHand
2024-04-17 10:19:46 +08:00
拆分,推出去
7h3d4wn
2024-04-17 10:21:02 +08:00
把这个人要到你组里来负责后续维护
你的核心问题是以后维护麻烦,那就人跟着项目走
其他没什么更好办法了
liyanan
2024-04-17 10:21:18 +08:00
能不能反噬一下他自己?这个项目维护归属 op 看看能不能换成他,自己种的果子自己吃吧~
MadSix
2024-04-17 10:21:23 +08:00
@yangxiaopeipei 描述里说过了
这项目负责人是个万年老大哥,人不做代码 review 而且他位高权重的咱也管不着
h272377502
2024-04-17 10:21:29 +08:00
现在有 ai 了,感觉问题不是很大,让 ai 基于他的代码转化成可维护的,不断校验每步的输出。
foxkiller
2024-04-17 10:23:10 +08:00
你们不 Code Review 啊?
7h3d4wn
2024-04-17 10:23:21 +08:00
换句话说,是系统性问题,或者说,管理问题造成了你当下的 dilemma
你又改变不了整个管理风格,那你基本就只能成全这哥们儿的防御性
yanw
2024-04-17 10:23:57 +08:00
可能单纯就是菜
MadSix
2024-04-17 10:24:56 +08:00
@foxkiller
描述里说过了
我们的其他开发都是做的,但这个项目是技术老大哥“亲自”负责。人不怎么管事,代码 review 也不做。位高权重我们也管不着
sparklee
2024-04-17 10:25:01 +08:00
我所理解的 防御性编程 是提前返回, 考虑到各种异常情况提前 return, 减少 else 减少条件判断嵌套的写法
chocolate518
2024-04-17 10:25:37 +08:00
这不叫防御性编程吧,没有代码规范就是这样了 如果不想操心就是反正也不是不能跑的心态,如果觉得别扭就推动代码规范
wanguorui123
2024-04-17 10:27:06 +08:00
在包一层外壳继续拉💩💩💩
villivateur
2024-04-17 10:28:04 +08:00
禁止污染“防御性编程”词条
mansurx
2024-04-17 10:28:37 +08:00
交叉管理确实没什么办法了,但可以把情况先写在周报里,避免锅从天上来。

后面维护可以考虑引入 ai 解释/重构这段代码……专门克制这种“防御性编程”
S9Yh4wIFsBG7jnE4
2024-04-17 10:29:42 +08:00
世界是一个草台班子 这话没错的

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://ex.noerr.eu.org/t/1033145

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX