处理IMyInterface实例时：方法内过滤与外部过滤的设计抉择

在我的应用程序中，需处理一组IMyInterface实例。其中部分实例额外实现了IAnotherInterface，注意IAnotherInterface并非继承自IMyInterface。遵循单一职责原则，我单独实现了一个类，通过Process方法处理IMyInterface实例。目前面临两种设计选择：

1. 方法签名为Process(IEnumerable<IMyInterface> items)，在方法内部过滤出实现IAnotherInterface的实例；
2. 方法签名为Process(IEnumerable<IAnotherInterface> items)，由调用方在方法外部完成过滤。

两种代码方案如下：

// 方案1:
List<IMyInterface> items = GetItems(); // 此处省略相关代码
foreach(var item in items)
{
    // 此处省略前置处理逻辑

    // 调用Process(IEnumerable<IMyInterface> items)
    myProcessor.Process(items);                
            
    // 此处省略后置处理逻辑
}

// 方案2:
List<IMyInterface> items = GetItems(); // 此处省略相关代码
foreach (var item in items)
{
    // 此处省略前置处理逻辑

    // 调用Process(IEnumerable<IAnotherInterface> items)
    // -> 需先执行过滤
    var filteredItems = filterForIAnotherInterface(items);
    myProcessor.Process(filteredItems);

    // 此处省略后置处理逻辑
}

我想了解两种方案的选择依据：方案1对调用方更友好，但Process方法需承担过滤职责，可能违背单一职责；方案2虽符合职责分离，但会降低处理流程的可读性。

1. 单一职责原则的落地细节 #

• 方案2严格遵循单一职责：Process方法只负责处理IAnotherInterface实例，过滤逻辑由调用方或专门的过滤组件承担，每个模块的职责边界清晰。如果后续过滤规则（比如新增其他需要过滤的接口）或处理逻辑变更，只需修改对应模块，不会相互影响。
• 方案1的Process方法同时承担“类型过滤”和“业务处理”两个职责，未来任何一方规则变化都要改动该方法，违反开闭原则的风险更高，也会增加代码维护的复杂度。

2. 调用场景的通用性 #

• 如果绝大多数调用场景，都是从IMyInterface集合中过滤出IAnotherInterface实例再处理，方案1的“一站式”调用能减少重复代码，提升开发效率。但如果存在直接处理纯IAnotherInterface集合的场景，方案1会强制引入不必要的过滤步骤，造成性能浪费。
• 方案2灵活性更高：调用方可以根据场景选择不同的过滤逻辑（甚至复用已有过滤组件），也能直接传入已有的IAnotherInterface集合，无需额外过滤操作。

3. 代码可读性与维护成本 #

• 方案1的优势是调用代码简洁，调用方无需关注过滤细节，但代价是Process方法内部逻辑复杂度提升——其他开发者阅读代码时，必须先理清隐藏的过滤逻辑，才能明白实际处理的对象范围，增加了理解成本。
• 方案2的调用代码虽然多了一行过滤，但明确展示了“过滤→处理”的完整流程，逻辑透明度更高。如果把过滤逻辑封装成通用扩展方法（比如items.OfType<IAnotherInterface>()），还能进一步简化调用代码，兼顾可读性和简洁性。

4. 可测试性 #

• 方案2的测试更简单：测试Process方法时，直接传入IAnotherInterface实例即可，无需构造混合类型的集合；测试过滤逻辑时，单独验证规则是否正确，两者互不干扰，便于定位问题。
• 方案1的测试需要构造包含多种类型的集合，同时验证过滤和处理逻辑，测试用例复杂度更高，难以单独隔离某一部分逻辑的问题。

折中方案建议 #

如果想兼顾调用友好性和职责分离，可以采用重载方法的方式：

• 保留方案2的Process(IEnumerable<IAnotherInterface> items)作为核心处理方法，保证职责单一；
• 在处理器类中新增重载方法Process(IEnumerable<IMyInterface> items)，内部仅负责过滤并转发到核心方法。

示例代码：

public class MyProcessor
{
    // 核心处理方法，仅负责业务逻辑
    public void Process(IEnumerable<IAnotherInterface> items)
    {
        // 业务处理逻辑
    }

    // 便捷重载，仅负责过滤与转发
    public void Process(IEnumerable<IMyInterface> items)
    {
        var filteredItems = items.OfType<IAnotherInterface>();
        Process(filteredItems);
    }
}

这种方式既给了调用方选择余地：需要便捷调用时用重载方法，需要灵活控制过滤逻辑时用核心方法；同时核心处理逻辑的职责依然清晰，符合单一职责原则。

内容的提问来源于stack exchange，提问作者me.at.coding