改同事代码

[yizhimamong]

领导安排一项任务，熟悉另外同事写的功能，接手这块工作，以后加需求，出现问题都由我来处理。

然后因为这块有性能问题，便说要优化下，也私信跟我说这个。后来发现几处便私信给领导，领导看见后便把聊天记录发到小群里。

所以，这个事情我也不知道该怎么办，万一改代码没有经过他们，不大好；

其实比较反感修改别人的代码，或者一般会私下跟对方反馈哪里有问题，然后问对方要不要我改下，因为有的对方也可能忘记。

有的时候，默默做这些事情，没有提交记录，好像以为没什么事情，其实发现问题和给出合理的解决方案占用一定的时间，最后结果在他们眼里可能很简单。

[plainboiledwater]

感觉简单问题被复杂化了，既然你要接手，要改需求，优化也是很自然的事情，肯定也要一起做了。
那看到代码有性能问题，敢改就直接改；不敢改就去和同事核对一下，为什么要没看到一处就发私信给领导呢？难道是为了邀功吗？

如果真要邀功，整个优化做完以后出一份优化报告才是最好的吧。

[sunny2580839896]

星号没碰到你这样的同事

[liyang5945]

第二段第二句的主语是谁?我有点懵逼

[swulling]

你们公司没有 Code Review 么，你改，原来的作者 review

代码又不是个人财产，是公司财产，谁都能改

[l00t]

领导都明确了让你接手这个工作，加需求改 bug 都由你来处理，那么这些代码都是移交给你了，你才是责任人，要改动当然不再需要通过他们了啊。

你要搞清楚，这已经是你的代码。哪怕不是你写的，现在也已经是你的代码了。你有不清楚的地方可以问原作者，但是你让别人改等于是让别人来干你的活。

[yizhimamong]

@l00t 但还是要经过原开发同事，可能领导不放心，毕竟是比较复杂的模块。

[yizhimamong]

@plainboiledwater 这个没有邀功的意思，我不是很喜欢做这种事情。因为接手，然后领导说私信跟我说让我优化的，我发现问题后就跟他说了，也是第一次直接跟领导说，之前都是直接反馈给原同事，然后咨询下由他还是我本人改。

[yizhimamong]

@swulling 可能有些同事不喜欢别人改他的代码，之前会上有说过这种事情。

[22too]

@yizhimamong 必须经过啊。至少别人看一下啊。万一业务理解有出入，你改错了。这个锅谁背。

[yizhimamong]

@swulling 没有代码评审，之前都是我主动发现其他同事的代码问题然后给原作者反馈的，所以这也占了一部分时间，并没有写到日报里去，有点吃力不讨好的事情

[kevinzhwl]

@yizhimamong 这和喜欢不喜欢有半毛钱关系。代码是公司的，公司指定你负责，你就要动。如果有 review，那是另外一个工作。
简单事情搞这么复杂

[ljpCN]

你们公司管理有点不是很开诚布公呀。这种纯技术工作，就应该所有相关的人一起商讨确认的。

[janus77]

你再看看你第一段最后一句话：以后出问题找你。
这你不改？我要是这同事我求着你改，改了以后问题你就不用找我去了。

[catch]

默默做？