组里 40 老大哥有错不改还注释他人代码怎么解

[bingyiyu]

spring cloud 项目，用的 feign，框架是老大哥搭的。前阵子测试给我提 bug 说异常了代码还继续执行。 我就查了一下加了个全局的 decode，但是要求返回值都是统一的自定义 Respose 对象，拦截器里拿这个对象去反序列化。我们的代码绝大部分是标准返回自定义 Respose 对象的，但是老大哥偷懒直接返回了 VO 未拿 Respose 对象包装导致反序列化失败。 我给提了我去全局扫一遍帮他改，支支吾吾说要报告领导（产品还没上线而且又不牵扯业务逻辑），然后加了个 ifelse 把我的逻辑绕过去了(还没绕对，我 debug 一看还是有问题)。 然后就进入了扯皮阶段。好说歹说领导给同意我的做法了。我就把她的 ifelse 去掉了。提交至 master 。 然后第二天骚气的来了，前端给我提 bug 。 我一看好家伙他直接把我的拦截器给注释掉了，导致 feign 异常未捕获继续执行了

[falcon05]

修改回来然后，git commit -m 再注释掉我跟你没完!

[Aliennnnnn]

提刀

[timle1029]

你们的 code review 呢？为什么他能自说自话把你的代码去掉

[bingyiyu]

@timle1029 他的账号是有 master 分支权限的，说起来有点搞笑，他一直在 master 上写代码。我给提了也不听，虽然现在项目还没上线问题不大

[bingyiyu]

@timle1029 华生你发现了盲点

[HarryHook]

master 上写代码也太骚了

[360511404]

@HarryHook
我只在 master 上写代码,因为只有我一个人...我就是 master 哈哈哈

[harde]

能出现这个问题，要么你的公司很小，要么就没有技术主管，我跟你一样一路走来，建议你要么顶掉老顽固，要么换个公司，长期以往，既对你成长不利，对心情也不好～

[bingyiyu]

@harde 膝盖中了一箭，基本被老哥说中了

[NjcyNzMzNDQ3]

查了一下就加全局的 decode 啊。。好像还是你改了拦截器，对象咋还要反序列化下才能用呢。。

首先个人担心你还不了解业务就改全局不合适，其次你让老大哥改这个全局返回需求它不香吗？

最后要是能只改自己负责的功能代码就最好了。

[MaxTan]

"她"是 40 岁的老大哥？

[uselessVisitor]

内部流转的就不要反序列化了吧

[a719031256]

最好去找人家确认一下再改，大家协商着弄比较稳妥点

40 岁的老开发，还不至于做这种蠢事

[auh]

我建议你尊重别人的一切行为。包括他的私欲。更正一个人很难。想思考一下能不能兼容一下他

[bingyiyu]

@a719031256 不，他就是懒

[bingyiyu]

@auh 以技服人，有问题我这个人一定会指出。

[bingyiyu]

@NjcyNzMzNDQ3 哎我咋管不住我这手呢，老大哥搭了框架但是一些基建啊规范啊还是不是很到位。我看到了就忍不住改

[del1214]

看透人生的表现

[webmasterisadog]

@bingyiyu #17 兄弟，你这想法很危险，也不成熟。容易碰钉子。
建议你 听听 #8 的意见。

等到你由你全权负责一个项目的时候，你再这样做。
如果是刚出来没多长时间，建议在自己能控制的领域，好好表现。

[chairuosen]

谁改出的问题继续由谁改，把 BUG 转他