确定不要在构造函数中使用所有参数吗？ - 重新审视

Question

该类在构造函数中注入所有依赖项，但一次仅使用其中一个依赖项。这被认为是糟糕的设计吗？

public class OrderPayment
{
    ICreditCardPayment _ccPayment;
    ICashPayment _cashPayment;

    public OrderPayment(ICreditCardPayment ccPayment, ICashPayment cashPayment)
    {
        _ccPayment = ccPayment;
        _cashPayment = cashPayment;
    }

    private void PrepareOrder(Order order)
    {
        // Do stuff with the order
    }

    public PaymentResult PayByCreditCard(Order order)
    {
        PrepareOrder(order);
        return _ccPayment.Pay(order);
    }

    public PaymentResult PayByCreditCard(Order order)
    {
        PrepareOrder(order);
        return _cashPayment.Pay(order);
    }   
}

另一种方法是：

public class OrderPayment
{
    private void PrepareOrder(Order order)
    {
        // Do stuff with the order
    }

    public PaymentResult PayByCreditCard(Order order, ICreditCardPayment ccPayment)
    {
        PrepareOrder(order);
        return ccPayment.Pay(order);
    }

    public PaymentResult PayByCreditCard(Order order, ICashPayment cashPayment)
    {
        PrepareOrder(order);
        return cashPayment.Pay(order);
    }   
}

这会使函数调用稍微复杂化。即使不是每个构造函数参数都被使用，您会使用第一个看起来更干净的吗？考虑到 DI 框架必须实例化潜在的重类，即使它们可能不会全部被使用，我不确定这有多好。

那么您会使用哪一个呢？或者也许有不同的实现？

Answer 1

我会从 ICashPayment 和 ICreditCardPayment 重构或提取通用接口。您的代码示例表明您的方法都在调用 xPayment.Pay，这看起来是您的通用接口方法的良好候选者。

public interface IPayment 
{
    PaymentResult Pay(Order order);
}

您的更专业的接口可以继承并构建在它的基础上。

一般来说，我会避免让构造函数（或任何方法）接受未使用的参数，或者如果使用一个参数，则不使用另一个参数。这通常表明您没有在适当的抽象级别上操作，或者您的类/方法有太多职责。

Answer 2

您需要一个需要满足以下所有条件的操作：

• 付款方式
• 支付的金额
• 正在支付的订单
• 付款前需要根据订单和支付的金额进行哪些处理

您正在尝试定义一些操作顺序与操作中正在执行的内容无关。依赖注入也可以在方法级别使用。

你需要一个像这样的方法：

public PaymentResult Pay(Amount amount, Order order, IOrderService orderService,
    IPaymentService paymentService) {
  var updatedOrder = orderService.Process(order); // don't alter the original in
                                                  // case you need to roll back
  var result = paymentService.Pay(amount, updatedOrder);
  return result; // this result should include the updated order, so that the system
                 // can determine what to do upon successful payment
}