Git操作不规范，战友提刀来相见。

年终奖都没了，还要扣我绩效，门都没有，哈哈。

这波骚Git操作我也是第一次用，担心闪了腰，所以不仅做了备份，也做了笔记，分享给大家。

文末留言抽奖，聊聊你的年终奖。

问题描述

小A和我在同时开发一个功能模块，他在优化之前的代码逻辑，我在开发新功能。

小A在我之前把代码提交到了测试分支，我想提交我的新功能代码到测试分支时发现巨多冲突，脑袋瞬间就炸了，Boom一声惊雷响啊。

PS：因为小A的需求不急，但是改动巨大；我的需求很急，马上要提测，否则就延期扣绩效了，说真的，我着急了，哈哈哈。

分析一下

1. 首先解决冲突浪费时间，我的新功能代码每次提测到测试分支都需要解决冲突。

2. 我在测试分支解决冲突，只能按照小A优化后的代码逻辑的去解决，和我自己的分支逻辑并不一致。

3. 交付给测试同学的测试分支代码，和我自己分支的代码不一致，这种测试是没有意义的。

反思出问题的原因

1. 工厂模式使用的不合理

2. 任务分配的不合理

代码层面

TIPS:以下代码示例语言为Go

因为是工厂设计模式，我负责的实现类A和他的实现类B虽然没有直接关系。但是因为他修改了工厂类中的方法定义。

比如之前工厂类中的接口是这么定义的

package factorytype xxx interface {   GetXxxx(ctx context.Context, req aaa.aa) (res bbb.bb, err error)  }

但是小A优化(修改)了工厂类中的接口定义：

package factorytype xxx interface {   GetXxxx(ctx context.Context, req ccc.cc) (res ddd.dd, err error)  }

这样就导致了一个问题：

我想合并我的代码到测试分支也必须将我的实现类像小A一样，修改传参类型和返回类型。

但是我们都在不同的分支上开发，我是没有他定义的类型ccc.cc,ddd.dd的。

我又不能直接把他定义的ccc.cc,ddd.dd要过来，在我自己的分支上开发，一是因为需求不一致，小A的上线周期会比我长；二是这种操作本身就不规范。

解决问题

1.代码层面：

我们想到的方案是合理使用interface

工厂类中方法的入参和出参设置为interface{}类型

package factorytype xxx interface {   GetXxxx(ctx context.Context, req interface{}) (res interface{}, err error)  }

这样就比较容易进行扩展了。

2.Git层面：

方法1的入参和出参设置为interface{}类型的方案，并没有从根本上解决我们的问题。

原因是这样的：

小A的需求是整体优化工厂类和各个实现类的入参、出参，优化内部逻辑，抽取方法。

小A的迭代优化修改变动很大，导致和我实现的新需求有比较大的冲突。

但是他的Git分支又在我之前提交到了测试环境，导致我无法正常提交我的代码。

如果我要提交就要解决各种冲突，解决冲突就要按照小A的优化逻辑去改，提测分支和我自己分支的不一致，难顶啊。

考虑到小A的修改暂时不需要提测，上线周期也比较长。

最终方案：

最终的解决方案是这样的：

1. 从远程的测试分支拉取了一个备份分支，删除小A提交的远程测试分支
2. 把我本地需要测试的分支提交到测试分支，交付测试（因为我的需求很急，而小A的需求并不急）

相关命令

这波骚操作我也是第一次用，担心闪了腰，所以不仅做了备份，也做了笔记，分享给大家：

Git 重命名远程分支

先重命名本地分支

git branch -m 旧分支名称  新分支名称

删除远程分支

git push --delete origin 旧分支名称

上传新修改名称的本地分支

git push origin 新分支名称

修改后的本地分支关联远程分支

git branch --set-upstream-to origin/新分支名称

推荐阅读

【Git必知必会】多人协同开发，紧急修复线上bug的操作指南。

总结

开发起来一时爽，维护起来火葬场。

Git操作不规范，战友提刀来相见！

呼应一下开篇，这是临时解决办法的一个经验分享。肯定还有最优解，但是对我来说不重要，再不使用骚操作就该扣绩效了。

年终奖都没了，还要扣我绩效，门都没有，哈哈。

一起进步 抱团取暖

就业环境不好，就业压力大，除了提高自己更应该抱团取暖，互相帮助：

点这里—>加入就业互助交流群👏👏👏

支持一波

最后，如果你觉得这期内容不错的话，一定要三连支持一波。

下方有我的公众号卡片，欢迎扫码关注，领取免费学习资料。

来源地址：https://blog.csdn.net/w425772719/article/details/128625156