没有太多泛型参数的 C# 代码重用

Question

我有一个名为CommercialRiskProcessor 的课程。它只有 221 行长，包括 cmets 和using。它只有 3 个注入参数，它只做一件事 - 评估一些风险。其实它的接口只有一个方法：

public interface ICommercialRiskProcessor
{
    Task<CommercialQuoteWithStatus> ApplyRules(CommercialRiskProcessorRequestInfo info, CommercialQuoteContext qtx, EFCommercialQuote quote);
}

所以，无论如何，它都不是“神”类。开头如下：

public class CommercialRiskProcessor : ServiceBase, ICommercialRiskProcessor
{
    public const string UiHardDeclineMessage = WorkflowFailure.UiDefaultWorkflowFailureMessage;
    protected ICommercialRiskRuleCollectionFactory RiskRuleCollectionFactory { get; }
    protected ICommercialRiskRuleFactory RiskRuleFactory { get; }
    protected ICommercialOverrideService OverrideService { get; }

    public CommercialRiskProcessor(
        ICommercialRiskRuleCollectionFactory riskRuleCollectionFactory,
        ICommercialRiskRuleFactory riskRuleFactory,
        ICommercialOverrideService overrideService)
    {
        RiskRuleCollectionFactory = riskRuleCollectionFactory;
        RiskRuleFactory = riskRuleFactory;
        OverrideService = overrideService;
    }

    public async Task<CommercialQuoteWithStatus> ApplyRules(CommercialRiskProcessorRequestInfo info, CommercialQuoteContext qtx, EFCommercialQuote quote)
    {
        var ruleCollection = await RiskRuleCollectionFactory.TryResolveService(info.RuleCollectionName, qtx, quote);

        if (ruleCollection == null)
        {
            return new CommercialWorkflowFailure(UiNotSupportedStateMessage);
        }

        var results = await ApplyRules(ruleCollection, qtx, quote);
        qtx.RuleEvaluationResults.AddRange(results);
        return await ProcessResults(info, results, qtx, quote);
    }

还有一些私有方法可以做一些额外的处理。这是无关紧要的。问题是我需要另一个风险处理器，称之为HomeownerRiskProcessor。如果我复制/粘贴整个文件，然后将“Commercial”重命名为“Homeowner”，那么新类将开箱即用，因为所有支持类和属性在这两组类中的调用方式相同。然而，房主和商业类别非常不同，因此所有底层对象都不同。

所以，复制/粘贴/单个替换似乎不是一种好的编码风格，不是吗？然而，利用代码重用的唯一可能方法似乎是引入泛型。这就是该死的地方：风险处理器类中使用的每个方法/属性都必须是generic-ized，同时引入一些通用的通用接口。随后，使整个类成为泛型（包括私有成员）将需要超过 10 个具有多个泛型约束的泛型参数！这显然超出了可读性，可能会比复制/粘贴/修改更糟糕。

我想知道是否有其他解决方案可以实现代码重用但没有太多的泛型参数。

如果我开始应用泛型，这就是它的样子：

    public interface IQuoteContext<TRuleEvaluationResult>
    {
        List<TRuleEvaluationResult> RuleEvaluationResults { get; }
    }

    public interface IRiskProcessorRequestInfo<TRuleCollectionName>
    {
        TRuleCollectionName RuleCollectionName { get; }
    }

    public interface IRiskProcessor<TQuoteContext, TQuote, TRequest, TQuoteWithStatus>
    {
        Task<TQuoteWithStatus> ApplyRules(TRequest info, TQuoteContext qtx, TQuote quote);
    }

    public interface IRiskRuleCollectionFactory<TQuoteContext, TQuote, TRuleCollectionName, TRuleCollection>
    {
        Task<TRuleCollection> TryResolveService(TRuleCollectionName collectionName, TQuoteContext qtx, TQuote quote);
    }

    public abstract class RiskProcessor<TQuoteContext, TQuote, TQuoteWithStatus, TRequest, TRiskRuleCollectionFactory, TRiskRuleFactory, TOverrideService, TRuleCollectionName, TRuleCollection, TRuleEvaluationResult>
        : ServiceBase, IRiskProcessor<TQuoteContext, TQuote, TRequest, TQuoteWithStatus>
        where TRequest : IRiskProcessorRequestInfo<TRuleCollectionName>
        where TRiskRuleCollectionFactory : IRiskRuleCollectionFactory<TQuoteContext, TQuote, TRuleCollectionName, TRuleCollection>
        where TQuoteContext : IQuoteContext<TRuleEvaluationResult>
    {
        public const string UiHardDeclineMessage = WorkflowFailure.UiDefaultWorkflowFailureMessage;
        protected TRiskRuleCollectionFactory RiskRuleCollectionFactory { get; }
        protected TRiskRuleFactory RiskRuleFactory { get; }
        protected TOverrideService OverrideService { get; }

        public RiskProcessor(
            TRiskRuleCollectionFactory riskRuleCollectionFactory,
            TRiskRuleFactory riskRuleFactory,
            TOverrideService overrideService)
        {
            RiskRuleCollectionFactory = riskRuleCollectionFactory;
            RiskRuleFactory = riskRuleFactory;
            OverrideService = overrideService;
        }

        protected abstract TQuoteWithStatus ToNullRuleCollectionFailure(TRequest info, TQuoteContext qtx, TQuote quote);

        public async Task<TQuoteWithStatus> ApplyRules(TRequest info, TQuoteContext qtx, TQuote quote)
        {
            var ruleCollection = await RiskRuleCollectionFactory.TryResolveService(info.RuleCollectionName, qtx, quote);

            if (ruleCollection == null)
            {
                return ToNullRuleCollectionFailure(info, qtx, quote);
            }

            var results = await ApplyRules(ruleCollection, qtx, quote);
            qtx.RuleEvaluationResults.AddRange(results);
            return await ProcessResults(info, results, qtx, quote);
        }

这只会使第一种方法ApplyRules 看起来不错。它下面的其他（私有）方法使用已声明类的更多方法/属性，这反过来会引入更多通用参数和更多约束。

Answer 1

如果我复制/粘贴整个文件，然后将“商业”重命名为 “房主”，那么新类将开箱即用，因为所有 支持类和属性在这些中被相同地调用 两组班。然而，房主和商业阶层是 非常不同，因此所有底层对象都不同。

听起来即使这些类非常不同，您也可以从中提取相同的接口。您只需将接口限制为仅风险处理所需的方法。然后风险处理器可以对实现这些接口的任何类进行操作。