面向对象·重构·设计模式

设计模式六大原则

单一职责原则
开闭原则
里氏替换原则
依赖倒置原则
接口隔离原则
迪米特法则

代码的坏味道

Long Method(过长方法)

public void reserve(Integer orderId){
    //此处省略1000行代码
    //行复杂度1000，分支复杂度100
}

方案：拆分成独立的子方法

//level 1
public boolean reserve(Integer orderId){
    ReserveContext context = fetchData(orderId);
    
    if(!precheckInventory(context)){
        insufficientInventory(context);
        return false;
    }
    
    if(reserveEntirely(context)){
        return true;
    }
    
    insufficientInventory(context);
    return false;
}

//level 2
private ReserveContext fetchData(Integer orderId){
    //此处有100行
}

//level 2
private boolean precheckInventory(ReserveContext context){
    //此处有100行
}

//level 2
private boolean insufficientInventory(ReserveContext context){
    //其他代码100行
    addInventoryDemand();
    notifyLackGoodsCancel();
}

//level 2
private boolean reserveEntirely(ReserveContext context){
    //其他代码100行
    deductInventory();
    updateOrder();
    notifyPick();
    notifyAutoDeliver();
}

//level 3
private boolean addInventoryDemand(){
    //此处有100行
}

//level 3
private boolean deductInventory(){
    //此处有100行
}

//level 3
private boolean updateOrder(){
    //此处有20行
}

//level 3
private boolean notifyPick(){
    //此处有30行
}

//level 3
private boolean notifyAutoDeliver(){
    //此处有125行
}

//level 3
private boolean notifyLackGoodsCancel(){
    //此处有50行
}

Large Class(过大类)

方案：把不同的业务拆分到独立的类中

class InventoryServiceImpl implements InventoryService {
    //level 3
    private boolean addInventoryDemand(){
        //此处有100行
    }
    
    //level 3
    private boolean deductInventory(){
        //此处有100行
    }
}

interface ReserveListener{
    void onInventoryInsufficient(ReserveContext context);
    void onReserveSuccess(ReserveContext context);
}

abstract class EmptyReserveListener implements ReserveListener {
    void onInventoryInsufficient(ReserveContext context){};
    void onReserveSuccess(ReserveContext context){};
}

class UpdateOrderListener extends EmptyReserveListener {
    void onReserveSuccess(ReserveContext context){
        //此处有20行
    }
}

class NotifyPickListener extends EmptyReserveListener {
    void onReserveSuccess(ReserveContext context){
        //此处有30行
    }
}

class NotifyAutoDeliverListener extends EmptyReserveListener {
    void onReserveSuccess(ReserveContext context){
        //此处有125行
    }
}

class NotifyLackGoodsCancelListener extends EmptyReserveListener {
    void onInventoryInsufficient(ReserveContext context){
        //此处有125行
    }
}

class CompositeReserveListener implements ReserveListener {
    private List<ReserveListener> listeners = new ArrayList<>();
    
    void onInventoryInsufficient(ReserveContext context){
        for(ReserveListener each : listeners){
            each.onInventoryInsufficient(context);
        }
    }
    void onReserveSuccess(ReserveContext context){
        for(ReserveListener each : listeners){
            each.onReserveSuccess(context);
        }
    }
}
class ReserveService {
    private InventoryService inventoryService;
    
    
    
    //level 2
    private boolean insufficientInventory(ReserveContext context){
        //其他代码100行
        addInventoryDemand();
        notifyLackGoodsCancel();
    }
    
    //level 2
    private boolean reserveEntirely(ReserveContext context){
        //其他代码100行
        deductInventory();
        updateOrder();
        notifyPick();
        notifyAutoDeliver();
    }
}

Duplicated Code(重复的代码)

方案：抽取到公共的类，模版方法

Redis redis = initRedis();
Integer randomValue = Random.randomInt();
redis.set("key", randomValue, NX,PX,1000)
try{
    //do something in lock
    //此处省略200行...
}finally{
    redis.eval("if redis.call('get', KEYS[1]) == ARGV[1] then return redis.call('del', KEYS[1]) else return 0 end", "key", randomValue)
}

abstract class RedisLockTemplate {
    void doWithLock(){
        Redis redis = initRedis();
        Integer randomValue = Random.randomInt();
        redis.set("key", randomValue, NX,PX,1000)
        try{
            doWithInLock();
        }finally{
            redis.eval("if redis.call('get', KEYS[1]) == ARGV[1] then return redis.call('del', KEYS[1]) else return 0 end", "key", randomValue)
        }
    }
    
    abstract void doWithInLock();
}

class OrderServcieImpl extends RedisLockTemplate {
    void doWIthInLock(){
        //此处省略200行
    }
}

interface Callback{
    void doWithInLock();
}

class RedisLockTemplate {
    void doWithLock(RedisCallBack callBack){
        Redis redis = initRedis();
        Integer randomValue = Random.randomInt();
        redis.set("key", randomValue, NX,PX,1000)
        try{
            callBack.doWithInLock();
        }finally{
            redis.eval("if redis.call('get', KEYS[1]) == ARGV[1] then return redis.call('del', KEYS[1]) else return 0 end", "key", randomValue)
        }
    }
    
class OrderServcieImpl {
    private RedisLockTemplate lock = new RedisLockTemplate();

    public void syncOrder(Order order){
        lock.doWithLock(() -> {
            ...
            //此处省略200行  
        });
    }
}

方案：定义接口，生命周期

interface Servlet {
    void init();
    void service(ServletRequest req, ServletResponse res)
    void destroy();
}

class container {
    void void service(ServletRequest req, ServletResponse res) {
        Servlet servlet = getServlet();
        servlet.init();
        servlet.service(req, res);
        servlet.destroy();
    }
}

Long Parameter List(过长參数列表)

//出库
void stockOut(String shipmentId, String trackingNumber, String boxCode, Boolean basketFlag, String fromFacilityId, String userName, String packager);

方案：提取参数对象，Builder模式

Class StockRequest {
    String shipmentId;
    String trackingNumber;
    String boxCode;
    Boolean basketFlag;
    String fromFacilityId;
    String userName;
    String packager;
    
    //getter
    //setter
}

void stockOut(StockRequest stockRequest);

Switch Statements(过多的分支)

if(partyId = 65588){
   ...
   ...
   ...
   ...
   ...
   //此处省略100行
} else if(partyId = 65530){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
} else if(partyId = 65540){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
} else if(partyId = 65539){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
} else if(partyId = 65542){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
}

方案：提取接口，策略模式

    interface PartyProcessor {
        void processParty()
    }
    
    class FDProcessor implements PartyProcessor {
        void processParty(){
            ...
            //此处省略100行
        }
    }
    
    class AZProcessor implements PartyProcessor {
        void processParty(){
            ...
            //此处省略100行
        }
    }
    
    Map<Integer, PartyProcessor> processorByPartyId = ...;
    
    processorByPartyId.get(partyId).processParty();

if(condition1){
   ...
   ...
   ...
   ...
   ...
   //此处省略100行
} else if(condition2){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
} else if(condition3){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
} else if(condition4){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
} else if(condition5){
    ...
    ...
    ...
    ...
    ...
    //此处省略100行
}

    interface Strategy {
        boolean applyable(condition);
        void process()
    }
    
    class Strategy1 implements Strategy {
        boolean applyable(condition){
            return condition == condition1;
        }
        
        void process(){
            ...
            //此处省略100行
        }
    }
    
    class Strategy2 implements Strategy {
        boolean applyable(condition){
            return condition == condition2;
        }
    
        void process(){
            ...
            //此处省略100行
        }
    }
    
    List<Strategy> strategies = ...;
    
    for(Strategy each : strategies){
        if(each.applyable(condition)){
            each.process();
        }
    }

Feature Envy(依恋情结)

class InventoryController {
    public void inventoryLoss(String dispatchSn, String user){
        //page controller logic 100行
        
        InventoryItem inventoryItem = getInventoryItem(dispatchSn);
        vOrderSn = generateVOrderSn();
        Product product = getProduct(dispatchSn)
        
        doInventoryLoss(inventoryItem, user, inventoryItem.getFacilityId, vOrderSn, product.getGoodName(), inventoryItem.getUnitCost)
    }
}

class InventoryService {
    void doInventoryLoss(InventoryItem inventoryItem, String user, String facilityId, String vOrderSn, String goodName, String goodPrice){
        //处理业务500行
    }
}

方案：迪米特法则，接口只传递必要参数，精简参数列表

Divergent Change (多种变化)

class OrderServiceImpl {
    Order createSaleOrder(...);
    Order createPurchaseOrder(...);
    Order createStockOrder(...);
    
    void saleOrderPay();
    void saleOrderConfirm();
    void purchaseOrderStockIn();
    void stockOrderStockIn();
}

方案：拆分成不同类，各自负责某个方向

class OrderFactory {
    Order createSaleOrder(...);
    Order createPurchaseOrder(...);
    Order createStockOrder(...);    
}

class OrderService {
    void saleOrderPay();
    void saleOrderConfirm();
    void purchaseOrderStockIn();
    void stockOrderStockIn();
}

class SaleOrderService {
    Order createSaleOrder(...);
    void saleOrderPay();
    void saleOrderConfirm();
}

class InventoryOrderService {
    Order createPurchaseOrder(...);
    Order createStockOrder(...);    
    void purchaseOrderStockIn();
    void stockOrderStockIn();
}

Shotgun Surgery(霰弹式改动)

public void businessA(){
    ...200行
    //库存-供应商退货
    ...300行
}

public void businessB(){
    ...200行
    //库存-入库
    ...300行
}

public void businessD(){
    ...200行
    //库存-出库
    ...300行
}

public void businessE(){
    ...200行
    //库存-盘亏
    ...300行
}

public void businessF(){
    ...200行
    //库存-盘盈
    ...300行
}

方案：单一职责，把一类事情交给一个类来处理


class InventoryService {
    public void suppleirReturn(){
        //库存-供应商退货
    }
    
    public void inStock(){
        //库存-入库
    }
    
    public void outStock(){
        //库存-出库
    }
    
    public void loss(){
        //库存-盘亏
    }
    
    public void regain(){
        //库存-盘盈
    }
    
}

public void businessA(){
    ...200行
    inventoryService.suppleirReturn();
    ...300行
}

public void businessB(){
    ...200行
    inventoryService.inStock();
    ...300行
}

public void businessD(){
    ...200行
    inventoryService.outStock();
    ...300行
}

public void businessE(){
    ...200行
    inventoryService.loss();
    ...300行
}

public void businessF(){
    ...200行
    inventoryService.regain();
    ...300行
}

参考书目

《java与模式》
《重构:改善既有代码的设计》
《重构与模式》

人面猴
序言：七十年代末，一起剥皮案震惊了整个滨河市，随后出现的几起案子，更是在滨河造成了极大的恐慌，老刑警刘岩，带你破解...
沈念sama阅读 159,835评论 4赞 364
死咒
序言：滨河连续发生了三起死亡事件，死亡现场离奇诡异，居然都是意外死亡，警方通过查阅死者的电脑和手机，发现死者居然都...
沈念sama阅读 67,598评论 1赞 295
救了他两次的神仙让他今天三更去死
文/潘晓璐我一进店门，熙熙楼的掌柜王于贵愁眉苦脸地迎上来，“玉大人，你说我怎么就摊上这事。” “怎么了？”我有些...
开封第一讲书人阅读 109,569评论 0赞 244
道士缉凶录：失踪的卖姜人
文/不坏的土叔我叫张陵，是天一观的道长。经常有香客问我，道长，这世上最难降的妖魔是什么？我笑而不...
开封第一讲书人阅读 44,159评论 0赞 213
港岛之恋（遗憾婚礼）
正文为了忘掉前任，我火速办了婚礼，结果婚礼上，老公的妹妹穿的比我还像新娘。我一直安慰自己，他们只是感情好，可当我...
茶点故事阅读 52,533评论 3赞 287
恶毒庶女顶嫁案：这布局不是一般人想出来的
文/花漫我一把揭开白布。她就那样静静地躺着，像睡着了一般。火红的嫁衣衬着肌肤如雪。梳的纹丝不乱的头发上，一...
开封第一讲书人阅读 40,710评论 1赞 222
城市分裂传说
那天，我揣着相机与录音，去河边找鬼。笑死，一个胖子当着我的面吹牛，可吹牛的内容都是我干的。我是一名探鬼主播，决...
沈念sama阅读 31,923评论 2赞 313
双鸳鸯连环套：你想象不到人心有多黑
文/苍兰香墨我猛地睁开眼，长吁一口气：“原来是场噩梦啊……” “哼！你这毒妇竟也来了？” 一声冷哼从身侧响起，我...
开封第一讲书人阅读 30,674评论 0赞 203
万荣杀人案实录
序言：老挝万荣一对情侣失踪，失踪者是张志新（化名）和其女友刘颖，没想到半个月后，有当地人在树林里发现了一具尸体，经...
沈念sama阅读 34,421评论 1赞 246
护林员之死
正文独居荒郊野岭守林人离奇死亡，尸身上长有42处带血的脓包…… 初始之章·张勋以下内容为张勋视角年9月15日...
茶点故事阅读 30,622评论 2赞 245
白月光启示录
正文我和宋清朗相恋三年，在试婚纱的时候发现自己被绿了。大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
茶点故事阅读 32,115评论 1赞 260
活死人
序言：一个原本活蹦乱跳的男人离奇死亡，死状恐怖，灵堂内的尸体忽然破棺而出，到底是诈尸还是另有隐情，我是刑警宁泽，带...
沈念sama阅读 28,428评论 2赞 254
日本核电站爆炸内幕
正文年R本政府宣布，位于F岛的核电站，受9级特大地震影响，放射性物质发生泄漏。R本人自食恶果不足惜，却给世界环境...
茶点故事阅读 33,114评论 3赞 238
男人毒药：我在死后第九天来索命
文/蒙蒙一、第九天我趴在偏房一处隐蔽的房顶上张望。院中可真热闹，春花似锦、人声如沸。这庄子的主人今日做“春日...
开封第一讲书人阅读 26,097评论 0赞 8
一桩弑父案，背后竟有这般阴谋
文/苍兰香墨我抬头看了看天上的太阳。三九已至，却和暖如春，着一层夹袄步出监牢的瞬间，已是汗流浃背。一阵脚步声响...
开封第一讲书人阅读 26,875评论 0赞 197
情欲美人皮
我被黑心中介骗来泰国打工，没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留，地道东北人。一个月前我还...
沈念sama阅读 35,753评论 2赞 276
代替公主和亲
正文我出身青楼，却偏偏与公主长得像，于是被迫代替她去往敌国和亲。传闻我的和亲对象是个残疾皇子，可洞房花烛夜当晚...
茶点故事阅读 35,649评论 2赞 271

面向对象·重构·设计模式

设计模式六大原则

代码的坏味道

Long Method(过长方法)

方案：拆分成独立的子方法

Large Class(过大类)

方案：把不同的业务拆分到独立的类中

Duplicated Code(重复的代码)

方案：抽取到公共的类，模版方法

方案：定义接口，生命周期

Long Parameter List(过长參数列表)

方案：提取参数对象，Builder模式

Switch Statements(过多的分支)

方案：提取接口，策略模式

Feature Envy(依恋情结)

方案：迪米特法则，接口只传递必要参数，精简参数列表

Divergent Change (多种变化)

方案：拆分成不同类，各自负责某个方向

Shotgun Surgery(霰弹式改动)

方案：单一职责， 把一类事情交给一个类来处理

参考书目

推荐阅读更多精彩内容

方案：单一职责，把一类事情交给一个类来处理