TIPS:以下代碼示例語(yǔ)言為Go
問(wèn)題描述
小A和我并行開(kāi)發(fā),他在優(yōu)化之前的代碼邏輯,我在開(kāi)發(fā)新功能。
小A在我之前把代碼提交到了測(cè)試分支,我想提交我的新功能代碼到測(cè)試分支時(shí)發(fā)現(xiàn)巨多沖突。
首先解決沖突浪費(fèi)時(shí)間,我的新功能代碼每次提測(cè)都需要解決沖突。
再者我再測(cè)試分支解決沖突,只能按照小A優(yōu)化后的代碼邏輯的去解決,和我自己的分支邏輯并不一致。
交付給測(cè)試同學(xué)測(cè)的代碼,和我自己分支的代碼不一致,這種測(cè)試是沒(méi)有意義的。
反思出問(wèn)題的原因
代碼層面
因?yàn)槭枪S設(shè)計(jì)模式,我負(fù)責(zé)的實(shí)現(xiàn)類A和他的實(shí)現(xiàn)類B雖然沒(méi)有直接關(guān)系。但是因?yàn)樗薷牧斯S類中的方法定義。
比如之前工廠類中的接口是這么定義的
package factorytype xxx interface { GetXxxx(ctx context.Context, req aaa.aa) (res bbb.bb, err error) }復(fù)制代碼
但是小A修改了工廠類中的接口定義:
package factorytype xxx interface { GetXxxx(ctx context.Context, req ccc.cc) (res ddd.dd, err error) }復(fù)制代碼
這樣就導(dǎo)致了一個(gè)問(wèn)題:
我想合并我的代碼到測(cè)試分支也必須將我的實(shí)現(xiàn)類A修改傳參類型和返回類型。
但是我們都在不同的分支上開(kāi)發(fā),我是沒(méi)有他定義的類型ccc.cc,ddd.dd的。
我又不能直接把他定義的ccc.cc,ddd.dd要過(guò)來(lái),在我自己的分支上開(kāi)發(fā),一是因?yàn)樾枨蟛灰恢拢的上線周期會(huì)比我長(zhǎng),二是這種操作本身就不規(guī)范。
解決問(wèn)題
從代碼設(shè)計(jì)上優(yōu)化:
我們想到的方案是合理使用interface
把工廠類中要實(shí)現(xiàn)的接口方法的入?yún)⒑统鰠⒃O(shè)置為interface{}類型
package factorytype xxx interface { GetXxxx(ctx context.Context, req interface{}) (res interface{}, err error) }復(fù)制代碼
這樣就比較容易進(jìn)行擴(kuò)展了。
從git操作上優(yōu)化:
但是入?yún)⒑统鰠⒃O(shè)置為interface{}類型的辦法并沒(méi)有從根本上解決我們的問(wèn)題。
原因是這樣的:
小A的需求是整體優(yōu)化工廠類和各個(gè)實(shí)現(xiàn)類的入?yún)?、出參,?yōu)化內(nèi)部邏輯,抽取方法。
小A的修改導(dǎo)致和我的實(shí)現(xiàn)邏輯有比較大的沖突。
但是他的git提交又在我之前提交到了測(cè)試環(huán)境,導(dǎo)致我無(wú)法提交我的代碼,如果要提交就要解決各種沖突。解決沖突就要按照小A的優(yōu)化邏輯去改,給到測(cè)試同學(xué)測(cè)的有和我自己分支的不一致。
難頂啊。
考慮到小A的修改暫時(shí)不需要提測(cè),上線周期也比較長(zhǎng)。
最終的辦法是這樣的:
從遠(yuǎn)程的測(cè)試分支拉取了一個(gè)備份分支
刪除遠(yuǎn)程的測(cè)試分支
把我本地需要測(cè)試的分支提交到測(cè)試分支,交付測(cè)試。
git 重命名遠(yuǎn)程分支
1.先重命名本地分支
git branch -m 舊分支名稱 新分支名稱復(fù)制代碼
2.刪除遠(yuǎn)程分支
git push –delete origin 舊分支名稱復(fù)制代碼
3.上傳新修改名稱的本地分支
git push origin 新分支名稱復(fù)制代碼
4.修改后的本地分支關(guān)聯(lián)遠(yuǎn)程分支
git branch –set-upstream-to origin/新分支名稱復(fù)制代碼
總結(jié)
開(kāi)發(fā)起來(lái)一時(shí)爽,維護(hù)起來(lái)火葬場(chǎng)。
操作不規(guī)范,親人兩行淚。
作者:王中陽(yáng)Go鏈接:https://juejin.cn/post/7104258964732575775