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

[MadSix]

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

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

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

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

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

V 友们给出出主意。

[me1onsoda]

你跟你技术老大多学学吧，他怎么就 cover 住了呢，带着这么“祸害”走了这么多年？

[Jame00001]

直接跟上级表明，自己组做耗时耗力需要增加大量工时，安排那个人做成本更低。为了避免公司利益损失，请领导出面指明让他维护。

[zhao8681286]

我以为是正常逻辑写完了花了 20%的代码其他 80%的用来处理异常情况的防御性编程呢，结果不就是只有自己能看懂的屎山代码吗？

[alexsz]

@h272377502 我也觉得应该让 AI 解释代码

[MadSix]

@me1onsoda 跟他学不来，人家根基深厚位高权重。有问题向下施压或者要求公司调动资源重做项目就行了。上一个就这么黄的。人自己又不干活，不用吃屎。
我算哪根葱跟他学，学完就废了

[icyalala]

这是防御性编程的概念： https://zh.wikipedia.org/zh-cn/防御性编程
这是好事

[k9982874]

你们这流程有问题，接收前拉个验收会，叫上相关负责人和技术主管，一个模块一个模块的过，列出来哪些系统问题和需求问题需要改，说明质量不行拒绝接收。

话说“防御性编程”现在已经是指在项目里拉屎，增加维护难度了么？

[bianhui]

没有改不了的代码，只有改不了的人。在软件工程还没有普及的时候，大家的开发都是一把梭哈

[nthin0]

理解岔了，我看到的第一反应也是：防御性编程难道不是好事吗😂

[sofukwird]

对领导说这位同事的代码水平过高，自己维护不了他的代码

[uiosun]

@wusheng0 不，大佬你这是“防御性编程”，我们不考虑编程，单纯的“防御性代码”

还是有区别的，虽然扣文字很无聊，但楼主遇到的这种人，连代码都没写好，更别提编程/软件工程了……

[junkk]

防御性编程是防御式设计的一种具体体现，它是为了保证，对程序的不可预见的使用，不会造成程序功能上的损坏。

不是防御开除式编程，拉屎就拉屎，不要重新定义老的词语，什么毛病。

[hahiru]

应该是菜。
我指的是公司。
如果是国企当我没说。

[F7TsdQL45E0jmoiG]

这种代码往往能顺利的通过 sonar 的扫描

[MadSix]

@hahiru 一针见血，很到位，我的互联网嘴替

[ooee2016]

@IMelon #2 遇到过所有的方法参数定义都是 void Fun(string a1, string a2, string a3, string a4, string a5);

[rehoni]

"我现在很想跟上面反馈一下这个问题，但是我这种跨管辖范围的做法总是不太好，而且也不知道提了有没有用。难道就这么把这口屎吃下来不成。"
现状是
1. 你是第一责任人，到时候维护他没事，你背锅
2. 你已经在吃屎了
我觉得 OP 也别管那么多了，他这种行为已经深刻影响了你的利益，不想沦落到代码和你只有一个能跑的话。就低情商，就把事情办得难看，让大家都挂不住面子，当个刺猬。

[yuwangG]

卷起来

[ersic]

也许他就是这么菜呢

[cathub86]

一模一样了 2 月 20 接受了 一个屎山 要把底层的 es 换成 ck 代码没注释 问他还爱答不理
组长： 有这么难么 就换个组件 不是很简单么
我： 我要理清楚业务流程才能改 代码没注释
组长：那你问啊 人就在你旁边