// 500 元订单 var order500 = function (orderType, pay, stock) { if (orderType === 1 && pay === true) { console.log('500 元定金预购, 得到 100 优惠券'); } else { order200(orderType, pay, stock); // 将请求传递给 200 元订单 } }; // 200 元订单 var order200 = function (orderType, pay, stock) { if (orderType === 2 && pay === true) { console.log('200 元定金预购, 得到 50 优惠券'); } else { orderNormal(orderType, pay, stock); // 将请求传递给普通订单 } }; } // 普通购买订单 var orderNormal = function (orderType, pay, stock) { if (stock > 0) { console.log('普通购买, 无优惠券'); } else { console.log('手机库存不足'); } }; // 测试结果: order500(1, true, 500); // 输出:500 元定金预购, 得到 100 优惠券 order500(1, false, 500); // 输出:普通购买, 无优惠券 order500(2, true, 500); // 输出:200 元定金预购, 得到 500 优惠券 order500(3, false, 500); // 输出:普通购买, 无优惠券 order500(3, false, 0);// 输出:手机库存不足
可以看到,执行结果和之前的 order
函数完全一样,但是代码的结构已经清晰很多,我们把一个大函数拆分了3个小函数,去掉了许多嵌套的条件分支语句。
但是,还有不足之处,可以看到请求在链条中的顺序非常僵硬,传递请求的代码被耦合在了业务函数中。这依然是违反开放-封闭原则的,如果有一天要增加300元预定或者去掉200元预定,以为着必须修改这些业务函数内部。就像一根环环相扣打了死结的链条,如果要增加、拆除或者移动一个节点,就必须得先砸烂这根链条。