这是一个创建于 2530 天前的主题，其中的信息可能已经有所发展或是发生改变。
之前同事在我做的功能模块中写了一个需求，(因为那时候我忙别的需求领导让同事写了)。
后来我发现，实现不是很好有些地方没考虑到，因为同事对我这块内容了解不是特别多所以采用的解决方案我认为不是最佳的。
我现在如果改的话，重写工作量对我来说不是很多，能写的更好一点吧，代码也能精简很多，但是这样的话感觉同事的代码就白写了，会让人感觉觉得不好吧。如果不改，后续我在这个模块如果有新需求，我要兼顾同事写的这块地方写考虑更多吧。
Ps：并不是说同事技术问题，因为在一个不熟悉的模块上去实现一个功能，可能不会详细去了解别人的代码，就实现来说怎么快怎么写。
后来我发现，实现不是很好有些地方没考虑到，因为同事对我这块内容了解不是特别多所以采用的解决方案我认为不是最佳的。
我现在如果改的话，重写工作量对我来说不是很多，能写的更好一点吧，代码也能精简很多，但是这样的话感觉同事的代码就白写了，会让人感觉觉得不好吧。如果不改，后续我在这个模块如果有新需求，我要兼顾同事写的这块地方写考虑更多吧。
Ps：并不是说同事技术问题，因为在一个不熟悉的模块上去实现一个功能，可能不会详细去了解别人的代码，就实现来说怎么快怎么写。
 |
1
hamasaki1999 2017-12-22 11:44:09 +08:00
跟同事沟通一下，然后重写
|
 |
2
lifesimple OP @hamasaki1999 #1 是啊 担心这样会让同事感觉不好，写的几百行代码白写了，对同事来说功能实现能用也不好说啥。
|
 |
3
daydaydayup 2017-12-22 11:48:20 +08:00
可以口头跟你同事说,看他反应,如果他认为你可以改的话就改.你只当一个意见者.
你把你的想法跟他说一次作为大家的交流! |
|
4
lifesimple OP @daydaydayup #3 恩 谢谢建议
|
 |
5
SuperMild 2017-12-22 11:55:44 +08:00
直接改就行了，改完找个机会用轻松的语气和同事说一声“上次谢谢你啦，帮我应急了，现在我才有时间回头再去弄弄，下次你要应急尽管叫我”
|
 |
6
ChenSir 2017-12-22 11:56:58 +08:00  2
不要。除非有 bug，导致功能异常。否则，再怎么烂的代码也是好代码。
|
 |
7
racecoder 2017-12-22 12:01:17 +08:00
一定要先沟通啊，每次我写的代码被人莫名其妙改了之后，我都想骂人。就算我知道我写的很烂，但在我不知道的额情况下改了代码，下次要加功能的时候发现代码不正常，我是非常生气的。
|
 |
9
swulling 2017-12-22 12:52:49 +08:00 via iPhone
代码质量最重要，该修改就修改
话说你们最需要的是 code review，你同事的代码需要你 review 才能入库 |
 |
10
110male 2017-12-22 13:07:07 +08:00
线上的最好别重写，很多问题你想不到，因可能是修改了很多版的
|
 |
11
shoaly 2017-12-22 13:40:23 +08:00
你把同事的 方法留着, 自己写一个方法叫做 version2
然后已经有的就不管了, 之后新的代码 直接调用 version2 的... |
|
12
lifesimple OP |
 |
13
nicevar 2017-12-22 15:11:29 +08:00
沟通一下，然后再看怎么改，这不是什么大事，毕竟他是在不熟悉这个模块，也不是他负责的，通情达理的人不会计较的，如果是他负责的模块，你最好别动
|
 |
14
forestyuan 2017-12-22 16:42:53 +08:00
如果以后代码由你维护，改一下无所谓，如果是他维护，最好别改
|
Select Language