ソースコード修正時の競合と統合の解決方法

その他テクニック
2021.08.03

複数の案件が同時に動いており、１つのソースコードを複数の案件で修正する場合、ソースコード修正の競合（コンフリクト）の統合（マージ、差分取り込み）が必要になります。
統合には、全体のロジックの整合性や案件のリリース時期の違いを考慮する必要があり、機械的に判断できない場合もあるので、最終的には人の手で判断し統合する必要があります。

例えば、商品の販売価格を計算する以下のソースコードがあるとします。
（今回はjavaのソースコードであると仮定します）

・修正前ソース

public class SalesPriceCalc {

    // DBサクセサのオブジェクト
    private DBAccessor dBAccessor;
    
    // 消費税
    private final TAX_RATE = 1.05;
    
    public SalePriceCalc() {
        dBAccessor = new DBAccessor();
    }

    public int execute(int commodityId) {

        int salePrice;
        
        // 商品の値段の取得
        salesPrice = dBAccessor.getCommodityPrice(commodityId);
        
        // 最終的な販売価格を返す
        return (int) Math.floor(salePrice * TAX_RATE);
    }
}

このソースコードについて、下記３案件で修正するものとします。
案件１．消費税増税対応（３ヶ月後リリース）
案件２．店舗毎セール対応（３ヶ月後リリース）
案件３．タイムセール対応（６ヶ月後リリース）

３案件の修正後ソースはそれぞれ以下の通りとします。

・案件１

public class SalesPriceCalc {

    // DBサクセサのオブジェクト
    private DBAccessor dBAccessor;
    
    // 消費税
    private final TAX_RATE = 1.08; // 修正ポイント１
    
    public SalePriceCalc() {
        dBAccessor = new DBAccessor();
    }

    public int execute(int commodityId) {

        int salePrice;
        
        // 商品の値段の取得
        salesPrice = dBAccessor.getCommodityPrice(commodityId);
        
        // 最終的な販売価格を返す
        return (int) Math.floor(salePrice * TAX_RATE);
    }
}

・案件２

public class SalesPriceCalc {

    // DBサクセサのオブジェクト
    private DBAccessor dBAccessor;
    
    // 店舗毎割引率取得オブジェクト // 修正ポイント２－１
    private ShopPriceRateCalc shopPriceRateCalc;
    
    // 消費税
    private final double TAX_RATE = 1.05;
    
    public SalePriceCalc() {
        dBAccessor = new DBAccessor();
        shopPriceRateCalc = new ShopPriceRateCalc(); // 修正ポイント２－２
    }
    
    public SalePriceCalc(int shopId) { // 修正ポイント２－３
        dBAccessor = new DBAccessor();
        shopPriceRateCalc = new ShopPriceRateCalc(shopId);
    }

    public int execute(int commodityId) {

        int salePrice;
        double shopPriceRate;
        
        // 商品の値段の取得
        salesPrice = dBAccessor.getCommodityPrice(commodityId);
        
        // 店舗毎割引率の取得 // 修正ポイント２－４
        shopPriceRate = shopPriceRateCalc.getShopPriceRate(commodityId);
        
        // 最終的な販売価格を返す // 修正ポイント２－５
        return (int) Math.floor(salePrice * TAX_RATE * shopPriceRate);
    }
}

・案件３

import java.util.Date;

public class SalesPriceCalc {

    // DBサクセサのオブジェクト
    private DBAccessor dBAccessor;
    
    // タイムセール割引率取得オブジェクト // 修正ポイント３－１
    private TimeSaleRateCalc timeSaleRateCalc;
    
    // 消費税
    private final TAX_RATE = 1.05;
    
    public SalePriceCalc() {
        dBAccessor = new DBAccessor();
        timeSaleRateCalc = new TimeSaleRateCalc(); // 修正ポイント３－２
    }

    public int execute(int commodityId) {

        int salePrice;
        double timeSaleRate;
        
        // 商品の値段の取得
        salesPrice = dBAccessor.getCommodityPrice(commodityId);
        
        // タイムセール割引率の取得 // 修正ポイント３－３
        Date systemDateTime = new Date();
        timeSaleRate
            = timeSaleRateCalc.getTimeSaleRate(commodityId, systemDateTime);
        
        // 最終的な販売価格を返す // 修正ポイント３－４
        return (int) Math.floor(salePrice * TAX_RATE * timeSaleRate);
    }
}

リリースすることを考えると、以下の２段階に分けてソースコードの修正を統合する必要があります。

・３ヶ月後を想定

　案件１と案件２を統合

・６ヶ月後を想定

　３ヶ月後のソースと案件３を統合


案件１と案件２の統合は容易です。
以下のように機械的に統合することができます。

・３ヶ月後のソース

public class SalesPriceCalc {

    // DBサクセサのオブジェクト
    private DBAccessor dBAccessor;
    
    // 店舗毎割引率取得オブジェクト // 修正ポイント２－１
    private ShopPriceRateCalc shopPriceRateCalc;
    
    // 消費税
    private final double TAX_RATE = 1.08; // 修正ポイント１
    
    public SalePriceCalc() {
        dBAccessor = new DBAccessor();
        shopPriceRateCalc = new ShopPriceRateCalc(); // 修正ポイント２－２
    }
    
    public SalePriceCalc(int shopId) { // 修正ポイント２－３
        dBAccessor = new DBAccessor();
        shopPriceRateCalc = new ShopPriceRateCalc(shopId);
    }

    public int execute(int commodityId) {

        int salePrice;
        double shopPriceRate;
        
        // 商品の値段の取得
        salesPrice = dBAccessor.getCommodityPrice(commodityId);
        
        // 店舗毎割引率の取得 // 修正ポイント２－４
        shopPriceRate = shopPriceRateCalc.getShopPriceRate(commodityId);
        
        // 最終的な販売価格を返す // 修正ポイント２－５
        return (int) Math.floor(salePrice * TAX_RATE * shopPriceRate);
    }
}

対して、３ヶ月後のソースと案件３を統合は機械的に行うことができません。
修正ポイント２－５と修正ポイント３－４は同じ個所を修正しているので、両方の修正が反映されるようにソースを書き変えなければなりません。
そもそも、修正ポイント２－３でコンストラクタが追加になっているため、新たに追加したコンストラクタに対しても案件３の内容を反映しなければなりません。
よって、６ヶ月後のソースは以下のようになります。

・６ヶ月後のソース

public class SalesPriceCalc {

    // DBサクセサのオブジェクト
    private DBAccessor dBAccessor;
    
    // 店舗毎割引率取得オブジェクト // 修正ポイント２－１
    private ShopPriceRateCalc shopPriceRateCalc;
    
    // タイムセール割引率取得オブジェクト // 修正ポイント３－１
    private TimeSaleRateCalc timeSaleRateCalc;
    
    // 消費税
    private final double TAX_RATE = 1.08; // 修正ポイント１
    
    public SalePriceCalc() {
        dBAccessor = new DBAccessor();
        shopPriceRateCalc = new ShopPriceRateCalc(); // 修正ポイント２－２
        timeSaleRateCalc = new TimeSaleRateCalc(); // 修正ポイント３－２
    }
    
    public SalePriceCalc(int shopId) { // 修正ポイント２－３＋案件３の統合
        dBAccessor = new DBAccessor();
        shopPriceRateCalc = new ShopPriceRateCalc(shopId);
        timeSaleRateCalc = new TimeSaleRateCalc();
    }

    public int execute(int commodityId) {

        int salePrice;
        double shopPriceRate;
        
        // 商品の値段の取得
        salesPrice = dBAccessor.getCommodityPrice(commodityId);
        
        // 店舗毎割引率の取得 // 修正ポイント２－４
        shopPriceRate = shopPriceRateCalc.getShopPriceRate(commodityId);
        
        // タイムセール割引率の取得 // 修正ポイント３－３
        Date systemDateTime = new Date();
        timeSaleRate
            = timeSaleRateCalc.getTimeSaleRate(commodityId, systemDateTime);

        // 最終的な販売価格を返す // 修正ポイント２－５＋修正ポイント３－４
        return (int) Math.floor
            (salePrice * TAX_RATE * shopPriceRate * timeSaleRate);
    }
}

なお、バージョン管理システムのGitのブランチやマージの機能を使えば、これらの統合の作業が楽になります。
機械的に統合できる箇所は自動で統合できますし、機械的に統合できないブランチについても以下のような方法を用いて比較的容易に統合することができます。
（具体的な操作方法は省略します。操作方法はWebで調べられるので、必要に応じて調べると良いでしょう。）

  • Gitが出力した差分を見てエディタで編集
  • 競合の原因となったコミットを取り消しコミットし直し
  • マスター管理しているブランチを元に新たにブランチを作り直し、作り直したブランチに対して同じ修正を行う

いかがでしたでしょうか。

「ソースコード修正の競合と統合」と言うと、Gitのテクニックが求められるように思われるかもしれません。
しかし、Gitの操作が多少不得手でも、何をしなければならないのかが理解できていれば、多少手間取ったり非効率になったりすることはあっても、最終的には問題無く修正作業を行うことができます。
まずは、何をしなければならないのかを理解することに努めましょう。
（それを理解する上で、今回の記事が参考になれば幸いです）

コメント

タイトルとURLをコピーしました