这是一个创建于 1620 天前的主题，其中的信息可能已经有所发展或是发生改变。
领导安排一项任务，熟悉另外同事写的功能，接手这块工作，以后加需求，出现问题都由我来处理。
然后因为这块有性能问题，便说要优化下，也私信跟我说这个。后来发现几处便私信给领导，领导看见后便把聊天记录发到小群里。
所以，这个事情我也不知道该怎么办，万一改代码没有经过他们，不大好；
其实比较反感修改别人的代码，或者一般会私下跟对方反馈哪里有问题，然后问对方要不要我改下，因为有的对方也可能忘记。
有的时候，默默做这些事情，没有提交记录，好像以为没什么事情，其实发现问题和给出合理的解决方案占用一定的时间，最后结果在他们眼里可能很简单。
然后因为这块有性能问题，便说要优化下，也私信跟我说这个。后来发现几处便私信给领导，领导看见后便把聊天记录发到小群里。
所以，这个事情我也不知道该怎么办，万一改代码没有经过他们，不大好；
其实比较反感修改别人的代码，或者一般会私下跟对方反馈哪里有问题，然后问对方要不要我改下，因为有的对方也可能忘记。
有的时候，默默做这些事情，没有提交记录，好像以为没什么事情，其实发现问题和给出合理的解决方案占用一定的时间，最后结果在他们眼里可能很简单。
 |
1
plainboiledwater 2020-06-08 16:46:47 +08:00
感觉简单问题被复杂化了，既然你要接手，要改需求，优化也是很自然的事情，肯定也要一起做了。
那看到代码有性能问题，敢改就直接改；不敢改就去和同事核对一下，为什么要没看到一处就发私信给领导呢？难道是为了邀功吗？ 如果真要邀功，整个优化做完以后出一份优化报告才是最好的吧。 |
 |
2
sunny2580839896 2020-06-08 17:01:54 +08:00
星号没碰到你这样的同事
|
 |
3
liyang5945 2020-06-08 17:08:12 +08:00
第二段第二句的主语是谁?我有点懵逼
|
 |
4
swulling 2020-06-08 17:09:02 +08:00
你们公司没有 Code Review 么，你改，原来的作者 review
代码又不是个人财产，是公司财产，谁都能改 |
 |
5
l00t 2020-06-08 17:24:32 +08:00
领导都明确了让你接手这个工作，加需求改 bug 都由你来处理，那么这些代码都是移交给你了，你才是责任人，要改动当然不再需要通过他们了啊。
你要搞清楚，这已经是你的代码。哪怕不是你写的，现在也已经是你的代码了。你有不清楚的地方可以问原作者，但是你让别人改等于是让别人来干你的活。 |
 |
6
yizhimamong OP @l00t 但还是要经过原开发同事，可能领导不放心，毕竟是比较复杂的模块。
|
|
7
yizhimamong OP @plainboiledwater 这个没有邀功的意思，我不是很喜欢做这种事情。因为接手，然后领导说私信跟我说让我优化的，我发现问题后就跟他说了，也是第一次直接跟领导说，之前都是直接反馈给原同事，然后咨询下由他还是我本人改。
|
|
8
yizhimamong OP @swulling 可能有些同事不喜欢别人改他的代码，之前会上有说过这种事情。
|
 |
9
22too 2020-06-08 17:38:43 +08:00
@yizhimamong 必须经过啊。至少别人看一下啊。万一业务理解有出入，你改错了。这个锅谁背。
|
|
10
yizhimamong OP @swulling 没有代码评审，之前都是我主动发现其他同事的代码问题然后给原作者反馈的，所以这也占了一部分时间，并没有写到日报里去，有点吃力不讨好的事情
|
 |
11
kevinzhwl 2020-06-08 20:32:50 +08:00 via iPhone
@yizhimamong 这和喜欢不喜欢有半毛钱关系。代码是公司的，公司指定你负责，你就要动。如果有 review，那是另外一个工作。
简单事情搞这么复杂 |
 |
12
ljpCN 2020-06-08 22:30:05 +08:00 via Android
你们公司管理有点不是很开诚布公呀。这种纯技术工作，就应该所有相关的人一起商讨确认的。
|
 |
13
janus77 2020-06-08 22:34:49 +08:00 via iPhone
你再看看你第一段最后一句话：以后出问题找你。
这你不改？我要是这同事我求着你改，改了以后问题你就不用找我去了。 |
 |
14
catch 2020-06-09 08:24:51 +08:00 via iPhone
默默做？
|
Select Language