程式設計師過關斬將--論商品促銷程式碼的優雅性

架構師修行之路發表於2019-06-03

程式設計師過關斬將--論商品促銷程式碼的優雅性

背景介紹

據我所知,幾乎所有的網際網路公司都帶有和電商有關的專案,而且在大多數公司裡面還是舉足輕重的重頭戲,比如京東,淘寶。既然有電商專案,必然會涉及到商品,一旦有商品就會有各種促銷活動,比如 滿100減20,三八婦女節9折等等類似活動。作為一個coder怎麼才能在實現產品狗的需求下,最小改動程式碼,最優雅的實現呢。今天菜菜不才,就D妹子的問題獻醜一番。以下以.netCore c#程式碼為例,其他語言類似。

D妹子版本

首先D妹子有一個商品的物件,商品裡有一個價格的屬性,價格的單位是分

 class Product
    {
        //其他屬性省略
        public int Price { get; set; }
    }
複製程式碼

下面有一個滿100減20的活動,在結算價格的時候程式碼是這樣的

 public int GetPrice()
        {           
            Product p = new Product();
            int ret = p.Price;
            if (p.Price >= 100*100)
            {
                ret = ret - 20 * 100;
            }
            return ret;
        }
複製程式碼

有問題嗎?按照需求來說沒有問題,而且計算的結果也正確。但是從程式藝術來說,其實很醜陋。現在又有一個全場9折的活動,恰巧有一個商品參與了以上兩個活動,而且還可以疊加使用(假設活動參與的順序是先折扣後滿減)。這時候D妹子的程式碼就變成了這樣

 public int GetPrice()
        {
            Product p = new Product();
            //9折活動
            int ret = p.Price * 90 / 100;
            //滿減活動
            if (ret >= 100 * 100)
            {
                ret = ret - 20 * 100;
            }
            return ret;
        }
複製程式碼

假如現在又來一個類似活動,那這塊程式碼還需要修改,嚴重違反了開放關閉原則,而且頻繁修改已經上線的程式碼,bug的機率會大大增高。這也是D妹子領導罵她並且讓她codereview的原因。

優化版本

那具體要怎麼優化呢?修改程式碼之前,我還是想提醒一下,有幾個要點需要注意一點:

  1. 商品菜菜認為有一個共同的基類比較好,這樣就有了一個所有商品的控制點,為以後統一新增屬性留一個入口。好比一個閘道器係統,為什麼會誕生閘道器這個元件呢,因為有了它我們能方便的統一新增認證,授權,統計等一些列行為。
  2. 任何促銷的活動最好有一個基類,作用類似商品基類。
  3. 對於商品而言,任何促銷活動是商品的行為變化點,影響到的是最終的商品價格,所以獲取價格這個行為要做特殊的處理。
  4. 不同種類的促銷活動應該能自行擴充套件,不會影響別的型別促銷活動。
  5. 不同種類的促銷活動能疊加使用(其實這裡涉及到每個活動計算的標準是商品原價還是促銷之後價格的問題)。

基於以上幾點,首先把商品的物件做一下抽象

 //商品抽象基類
    abstract class BaseProduct
    {
        //商品價格,單位:分
        public int Price { get; set; }
        //獲取商品價格抽象方法
        public abstract int GetPrice();

    }
    //抽象商品(比如話費商品),繼承商品基類
    class VirtualProduct : BaseProduct
    {
        public override int GetPrice()
        {
            return this.Price;
        }
    }
複製程式碼

接下來活動的基類也需要抽象出來

//各種活動的抽象基類,繼承要包裝的型別基類
    abstract class BaseActivity : BaseProduct
    {

    }
複製程式碼

有的同學會問,這裡為什麼要繼承商品的基類呢?主要是為了活動的基類能巢狀使用,這樣我就可以實現多個活動同時使用,如果不明白沒關係,帶著這個問題接著往下看

實現一個打折的活動

//打折活動基類,支援多個商品同時結算
    class DiscountActivity : BaseActivity
    {
        BaseProduct product = null;

        public DiscountActivity(int discount, BaseProduct _product)
        {
            Discount = discount;
            product = _product;
        }

        //折扣,比如 90折 即為90
        public int Discount { get; set; }
        //獲取折扣之後的價格
        public override int GetPrice()
        {
            return product.GetPrice() * Discount / 100;
        }
    }
複製程式碼

實現一個滿減的活動,而且支援自定義滿減條件

  class ReductionActivity : BaseActivity
    {
        BaseProduct product = null;
        //滿減的對應表
        Dictionary<int, int> reductMap = null;

        public ReductionActivity(Dictionary<int, int> _redutMap, BaseProduct _product)
        {
            reductMap = _redutMap;
            product = _product;
        }
        //獲取折扣之後的價格
        public override int GetPrice()
        {
            var productAmount = product.GetPrice();
            //根據商品的總價獲取到要減的價格
            var reductValue = reductMap.OrderByDescending(s => s.Key).FirstOrDefault(s => productAmount >= s.Key).Value;
            return productAmount - reductValue;
        }
    }
複製程式碼

現在我們來給商品做個促銷活動吧

            VirtualProduct p = new VirtualProduct() {  Price=1000};           
            //打折活動
            DiscountActivity da = new DiscountActivity(90, p);
            var retPrice= da.GetPrice();
            Console.WriteLine($"打折後的價格{retPrice}");
            //還能疊加參加滿減活動
            Dictionary<int, int> m = new Dictionary<int, int>() ;
            m.Add(200, 5); //滿200減5
            m.Add(300, 10); 
            m.Add(500, 20);
            m.Add(1000, 50);
            //這裡活動能疊加使用了
            ReductionActivity ra = new ReductionActivity(m, da);
            retPrice = ra.GetPrice();
            Console.WriteLine($"打折滿減後的價格{retPrice}");

            ReductionActivity ra2 = new ReductionActivity(m, ra);
            retPrice = ra2.GetPrice();
            Console.WriteLine($"再打折後的價格{retPrice}");
複製程式碼

輸出結果:

打折後的價格900
打折滿減後的價格880
再打折後的價格860
複製程式碼

現在我們終於能優雅一點的同時進行商品的滿減和打折活動了

進化到多個商品同時促銷

以上程式碼已經可以比較優雅的能進行單品的促銷活動了,但是現實往往很骨感,真實的電商場景中多以多個商品結算為主,那用同樣的思路怎麼實現呢?

  1. 由於這次需要實現的是多商品促銷結算,所以需要一個自定義的商品列表來作為要進行結算的物件。此物件行為級別上與單品類似,有一個需求變化點的抽象:獲取價格
//商品列表的基類,用於活動結算使用
    class ActivityListProduct : List<BaseProduct>
    {
        //商品列表活動結算的方法,基類必須重寫
        public virtual int GetPrice()
        {
            int ret = 0;
            base.ForEach(s =>
            {
                ret += s.GetPrice();
            });
            return ret;
        }
    }
複製程式碼
  1. 把多商品促銷活動的基類抽象出來,供不同的促銷活動繼承使用,這裡需要繼承ActivityListProduct,為什麼呢?和單品的類似,為了多個子類能夠巢狀呼叫
//商品列表 活動的基類,繼承自商品列表基類
    internal abstract class BaseActivityList : ActivityListProduct
    {

    }
複製程式碼
  1. 建立一個打折和滿減活動
//打折活動基類,支援多個商品同時結算
    class DiscountActivityList : BaseActivityList
    {
        ActivityListProduct product = null;
        public DiscountActivityList(int discount, ActivityListProduct _product)
        {
            Discount = discount;
            product = _product;
        }
        //折扣,比如 90折 即為90
        public int Discount { get; set; }
        public override int GetPrice()
        {
            var productPrice = product.GetPrice();
            return productPrice * Discount / 100;
        }
    }
    //滿減的活動
    class ReductionActivityList : BaseActivityList
    {
        ActivityListProduct product = null;
        //滿減的對應表
        Dictionary<int, int> reductMap = null;

        public ReductionActivityList(Dictionary<int, int> _redutMap, ActivityListProduct _product)
        {
            reductMap = _redutMap;
            product = _product;
        }
        //獲取折扣之後的價格
        public override int GetPrice()
        {
            var productAmount = product.GetPrice();
            //根據商品的總價獲取到要減的價格
            var reductValue = reductMap.OrderByDescending(s => s.Key).FirstOrDefault(s => productAmount >= s.Key).Value;
            return productAmount - reductValue;
        }
    }
複製程式碼

先來一波多商品促銷活動

VirtualProduct p = new VirtualProduct() { Price = 1000 };
            VirtualProduct p2 = new VirtualProduct() { Price = 1000 };
            ActivityListProduct lst = new ActivityListProduct();
            lst.Add(p);
            lst.Add(p2);
            DiscountActivityList dalist = new DiscountActivityList(80, lst);
            Console.WriteLine($"打折後的價格{dalist.GetPrice()}");
            DiscountActivityList dalist2 = new DiscountActivityList(90, dalist);
            Console.WriteLine($"打折後的價格{dalist2.GetPrice()}");

            DiscountActivityList dalist3 = new DiscountActivityList(90, dalist2);
            Console.WriteLine($"打折後的價格{dalist3.GetPrice()}");

            //還能疊加參加滿減活動
            Dictionary<int, int> m = new Dictionary<int, int>();
            m.Add(200, 5); //滿200減5
            m.Add(300, 10);
            m.Add(500, 20);
            m.Add(1000, 50);

            ReductionActivityList ral = new ReductionActivityList(m, dalist3);
            Console.WriteLine($"再滿減打折後的價格{ral.GetPrice()}");
複製程式碼

結算結果:

打折後的價格1600
打折後的價格1440
打折後的價格1296
再滿減打折後的價格1246
複製程式碼

現在基本上可以讓D妹子不被捱罵了

神化版本見留言區

知道D妹子為什麼取名D妹子嗎?


新增關注,檢視更精美版本,收穫更多精彩

image

相關文章