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

[bingyiyu]

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

[auh]

@bingyiyu 技术是个什么东西？以技服人？格局小了。

[webmasterisadog]

@bingyiyu 等你到了大厂这种级别的公司，更要步步为营啊。

高技术的人的通病就是情商往往太低，太把项目当成事，以为是自己的，结果都是别人的。这其实都是私欲在作怪。

[queuey]

@360511404 一个人也不建议在 master 上写代码

[fredli]

继续杠，技术问题有把握没什么可让步的

[pinkSlime]

时光荏苒 最终我们都会变成 40 老大哥

[bingyiyu]

@pinkSlime 希望到那天我不会那么顽固，可以虚心接受问题，马上做出调整

[zliea]

master 权限全收回。
其实我一直不理解为啥全局返回统一结构不统一处理，让业务专注业务不好么？

[bingyiyu]

@ztechstack 我也不知道，刚 node 转 java 。测试给我提 bug 的时候，我才意识到 feign 内部服务异常未统一处理，再加的拦截器，自认为思路应该是没问题的，总不能每个调用的地方都插个 ifelse 判断状态码吧哈哈，太蠢了。

[zliea]

@bingyiyu feign 这个我没细测试过，之前一直用的 resttemplate，如果需要专门处理就自己实例化一个 resttemplate bean 。讲道理 feign 的 configuration 应该是能够在每个 client 上指定的，但 configuration 不能添加 @configuration，添加了肯定就是全局的。
其实我的意思就是老大哥那边的框架应该进行统一封装，如果让我来做的话，每个接口封装一下 response，我也很不爽。

[gablic]

40 了还在看代码，是个狠人建议你别惹

[hcen1997]

@bingyiyu 之前我在的某外包项目就是 http 服务组件写了很多, 每一个调用的地方插入 ifelse
要考虑到所有人的技术水平, 说不定有人就是习惯了复制 ifelse 呢?

我还是推荐在自己的代码和模块中保持自己的坚持(如果你能坚持的话)
但是和其他人一起的部分, 就要遵守惯例, 不要动

[darknoll]

在 master 上写代码就很强势，我喜欢

[stillyu]

40 岁，已经完成了原始积累，来上班就是混日子的

[a719031256]

@hcen1997

我才开始敲代码时也很鄙视 ifelse 写法，现在我很喜欢这种写法，简单暴力不会出错，有更多空闲时间摸鱼搞自己的东西，现在的心态就是工作是工作，工作一切以实用为主，剔除掉一切不稳定的因素，避免浪费自己太多时间，也避免加班，996 之类的情况

[ckdxc]

先把自己那个拦截器改回来, 然后再把他没有用包装类那个 bug 转给他

[no1xsyzy]

@auh 同样的话，我也完全可以丢给你来堵你这句话本身。

[X0ray]

@stillyu 你 40 岁就能完成原始积累？太猛了。
我 40 岁房贷还没有还一半。积蓄的话，顶上天了也就 100 w 左右。100w 能干啥是个问题。

[X0ray]

我建议楼主不要去杠，做好准备换一个公司，对大家都好。

[hodur]

decode 可以细到 client 粒度,都返回 repsonse 对象并不好.
看了一些回复,这样的老大哥不是个例

[xiaoming1992]

我们也有项目中在接口调用的时候插入 if else 判断状态码，我的解决方案是自己另封装一个 http 请求函数，只在自己负责的部分使用，尽最大可能不碰其他人的代码，不给自己找事