0

我正在使用工厂 ( BillFactory) 来确定Bill要实例化哪个。每个Bills (AClassBill等BClassBill) 都可以判断提供的数据是否是范围。在因子中，我遍历每个账单并检查当前数据是否在当前账单范围内。如果它在范围内，我实例化该类的一个对象并返回它。

这是代码。

interface IBillable {
    function calculate_bill();
}

class Bill implements IBillable {

    protected $from;
    protected $to;
    protected $rate;
    protected $unit_amount;

    public function get_from() {
        return $this->from;
    }

    public function get_to() {
        return $this->to;
    }

    public function get_rate() {
        return $this->rate;
    }

    public function in_range($unit_amount) {
        $_from = $this->get_from();
        $_to = $this->get_to();
        return ($_from <= $unit_amount && $_to >= $unit_amount);
    }

    public function calculate_bill() {
        return ($this->get_rate() * $this->unit_amount);
    }

}

class AClassBill extends Bill {

    protected $from = 0;
    protected $to = 100;
    protected $rate = 3.05;

    public function __construct($amount) {
        $this->unit_amount = $amount;
    }

}

class BClassBill extends Bill {

    protected $from = 101;
    protected $to = 400;
    protected $rate = 4.29;

    public function __construct($amount) {
        $this->unit_amount = $amount;
    }

    public function calculate_bill() {
        if ($this->unit_amount >= 301) {
            return $this->get_rate() * $this->unit_amount;
        } else {
            $bill1 = $this->get_rate() * ($this->unit_amount - $this->get_from() + 1);
            $bill2 = new AClassBill($this->get_from() - 1);
            return $bill1 + $bill2->calculate_bill();
        }
    }

}

class CClassBill extends Bill {

    protected $from = 401;
    protected $to = -1; // not used
    protected $rate = 7.89;

    public function __construct($amount) {
        $this->unit_amount = $amount;
    }

    public function in_range($unit_amount) {
        $_from = $this->get_from();
        return ($_from <= $unit_amount);
    }

}

class BillFactory {

    private static $s1 = null;
    private static $s2 = null;
    private static $s3 = null;

    public static function instance($units) {
        if (is_null(self::$s1))
            self::$s1 = new AClassBill(0);
        if (is_null(self::$s2))
            self::$s2 = new BClassBill(0);
        if (is_null(self::$s3))
            self::$s3 = new CClassBill(0);

        if (self::$s1->in_range($units)) {
            $b = new AClassBill($units);
        } else if (self::$s2->in_range($units)) {
            $b = new BClassBill($units);
        } else if (self::$s3->in_range($units)) {
            $b = new CClassBill($units);
        } else {
            $b = false;
        }
        return $b;
    }

}

for ($i = 1; $i <= 500; $i++) {
    $b = BillFactory::instance($i);
    if ($b !== false) {
        printf("%10s\t%04d\t%04.2f\t%04d\t%06.2f\n", get_class($b), $i, $b->get_rate(), $i, $b->calculate_bill());
    }
}

问题出在工厂类 ( BillFactory) 中。您会注意到我创建了 3 种Bills 类型（单例模式）的 3 个虚拟实例。这就是我解决问题的方法。我的问题是我应该使用这个虚拟实例吗？他们是否违反了任何原则？

in_range另一种方法是通过将方法转换为静态方法来做到这一点。然后我不必创建实例。此外，由于此in_range方法是特定于类的属性（意味着它不会在不同的实例上改变），它应该是有意义的。在这种情况下，我必须在in_range方法中对这些值进行硬编码，或者将所有from,to和range属性设为静态。

我应该使用什么方法？你知道更好的吗？

4

1 回答 1

0

尝试重新审视设计。看起来应该只有 1 个 Bill 类（并创建 3 个不同的实例而不是 3 个不同的类），并且可能将 calculate_bill 委托给一个策略（策略模式）。

于 2012-06-29T07:07:27.620 回答