Question

我会将代码最小化到您需要查看的内容。

我有3个班级：Customer，Courier和Order。

class Customer extends AbstractRegisteredUser implements CustomerInterface {
}

class Courier extends AbstractRegisteredUser implements CourierInterface {
}

class Order extends AbstractEntity implements OrderInterface {

    private $customer;
    private $courier;

    public function isUserAssociated(RegisteredUserInterface $user) {

        switch( $user->GetEntityType() ) {
        case 'Customer':
            return $this->isCustomerAssociated($user);
        case 'Courier':
            return $this->isCourierAssociated($user);
        }

        return false;
    }

    private function isCustomerAssociated(CustomerInterface $customer) {
        return ( $this->customer->getId() === $customer->getId() );
    }

    private function isCourierAssociated(CourierInterface $courier) {
        return ( $this->courier->getId() === $courier->getId() );
    }
}

正如你所看到我在那里有一个switch语句我不想这样做，所以我想出这样做：

class Customer extends AbstractRegisteredUser implements CustomerInterface {
    public function isAssociatedWithOrder(OrderInterface $order) {
         return ( $this->getId() === $order->getCustomerId() );
    }
}

class Courier extends AbstractRegisteredUser implements CourierInterface {
    public function isAssociatedWithOrder(OrderInterface $order) {
         return ( $this->getId() === $order->getCourierId() );
    }
}

我现在可以从isUserAssociated类和那个丑陋的switch语句中删除isCustomerAssociated，isCourierAssociated和Order方法。

现在，当我想检查客户是否与给定订单相关时，我

// $user could be a customer or courier object.
if( !$user->isAssociatedWithOrder($order) ) {
}

而不是

if( !$order->isUserAssociated($customer) ) {
}

这是一个解决方案，需要更少的代码，更少的方法，更容易在眼睛，但这样做是否正确？ Customer和Courier类不应该了解Order吗？这会被视为对不应该承担这一责任的班级负责吗？

任何帮助都会非常感谢。

Answer 1

我认为您的解决方案是有效的，当更多用户类型加入时会发生什么？它使用不断增加的switch语句和方法使您的订单类陷入困境。

它还增加了关注点的分离，因为与订单关联的用户是用户关注的问题，而不是订单。

将方法放在不同的类中以删除switch（）语句

1 个答案: