您觉得《C# 敏捷原则、模式和实践》中的数据访问部分（使用 SQL Server）怎么样？

Question

您如何看待这样的数据访问代码：

public void AddCusotmer(Cusotmer customer)
{
   //save customer into database
   ...

   // save payment type
   SavePaymentType(customer);

   //save other data
   ...
}

private void SavePaymentType(Customer customer)
{
   if(customer.PaymentType is XXXPayment)
   {
      var payment = customer.PaymentType as XXXPayment;
      //save payment to XXXPayments table in db
      ...
   }
   else if(customer.PaymentType is YYYPayment)
   {
      var payment = customer.PaymentType as XXXPayment;
      //save payment to YYYPayments table in db
      ...
   }
   ...
}

就我个人而言，我对这样的代码感觉不太好（使用“is”来检测类型来决定做什么），但是作者 Robert Martin 说没关系，因为它只是在 DAL 中，因此稍微违反 OCP 是可以接受的。

你觉得怎么样？

Answer 1

这样的代码对我来说不太好。
您可能正在执行自己的 O/RM，因此不了解所有细节。

但使用接口可能会有所帮助（在这种情况下，付款实体将被 DAL 代码污染）......闻起来不太好。

因此，类的注册可能会完成这项工作：

private void SavePaymentType(PaymentType )
{
   if (paymentType == null)
       throw new NotSupportedException("Handle nulls too");
   IClassPersister persister;
   if (!paymentType2Persister.TryGetValue(paymentType.GetType(), out persister))
      throw new ORMException(string.Format("Cannot find persister for {0}", paymentType.GetType().Name))
   persister.Save(paymentType);
}

在应用程序启动期间的某个地方，您可以注册 PaymentTypes：

paymentType2Persister.Add(typeof(XXXPayment), new XXXPaymentPersistor);
paymentType2Persister.Add(typeof(YYYPayment), new YYYPaymentPersistor);
// etc

因此，当您需要添加另一种付款类型时，您必须为其实现持久化程序并注册它。
对我来说，这看起来比原始代码好得多。

干杯。