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

[MadSix]

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

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

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

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

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

V 友们给出出主意。

[handpr]

@MadSix #10 和我现在上手的项目一样.举个例子.之前的后端让前端自己写死控制哪个按钮显示不显示

[fredweili]

强化 code review ，不允许随便 merge

[yangxiaopeipei]

code review 怎么过的

[FrankAdler]

原来代码写的烂还有这么一个高大上的名字

[whoosy]

我认为只是单纯的水平菜，身边技术稍微强点的基本上对代码都有洁癖

[RightHand]

拆分，推出去

[7h3d4wn]

把这个人要到你组里来负责后续维护
你的核心问题是以后维护麻烦，那就人跟着项目走
其他没什么更好办法了

[liyanan]

能不能反噬一下他自己？这个项目维护归属 op 看看能不能换成他，自己种的果子自己吃吧~

[MadSix]

@yangxiaopeipei 描述里说过了
这项目负责人是个万年老大哥，人不做代码 review 而且他位高权重的咱也管不着

[h272377502]

现在有 ai 了，感觉问题不是很大，让 ai 基于他的代码转化成可维护的，不断校验每步的输出。

[foxkiller]

你们不 Code Review 啊？

[7h3d4wn]

换句话说，是系统性问题，或者说，管理问题造成了你当下的 dilemma
你又改变不了整个管理风格，那你基本就只能成全这哥们儿的防御性

[yanw]

可能单纯就是菜

[MadSix]

@foxkiller
描述里说过了
我们的其他开发都是做的，但这个项目是技术老大哥“亲自”负责。人不怎么管事，代码 review 也不做。位高权重我们也管不着

[sparklee]

我所理解的 防御性编程 是提前返回, 考虑到各种异常情况提前 return, 减少 else 减少条件判断嵌套的写法

[chocolate518]

这不叫防御性编程吧，没有代码规范就是这样了 如果不想操心就是反正也不是不能跑的心态，如果觉得别扭就推动代码规范

[wanguorui123]

在包一层外壳继续拉💩💩💩

[villivateur]

禁止污染“防御性编程”词条

[mansurx]

交叉管理确实没什么办法了，但可以把情况先写在周报里，避免锅从天上来。

后面维护可以考虑引入 ai 解释/重构这段代码……专门克制这种“防御性编程”

[S9Yh4wIFsBG7jnE4]

世界是一个草台班子 这话没错的