能否将不同仓储传入泛型方法以复用ID解析与名称查询逻辑？

这个需求完全可以落地，而且在当前这种重复度很高的场景下，做统一抽象的收益远高于新增的少量实现成本，完全不存在得不偿失的问题。

为什么这个场景适合抽公共方法 #

对比两段重复代码可以发现，所有流程逻辑是100%一致的，只有三个变量存在差异：

• 从表单字典取值时用的Key名（CountryId / SubjectId）
• 拿到有效ID后调用的仓储查询方法（两个仓储的方法名、返回实体类型都不一样）
• 从返回实体中取展示名称的字段（虽然两个实体都有Name字段，但不需要强行做公共约束）

这些差异完全可以通过参数传入泛型方法解决，不需要改动现有仓储的原有结构。

最灵活的实现方式（无侵入改造） #

不用强行给两个仓储加公共继承接口，直接通过委托传参的方式实现泛型方法，完全兼容现有仓储的方法签名：

/// <summary>
/// 统一处理表单ID取值、仓储查询、名称提取的公共逻辑
/// </summary>
/// <typeparam name="T">仓储返回的实体类型</typeparam>
/// <param name="formDataDictionary">提交的表单数据字典</param>
/// <param name="formKey">表单中ID字段的Key名</param>
/// <param name="getEntityByIdFunc">按ID查询实体的仓储委托</param>
/// <param name="getNameSelector">从实体中提取名称的选择器</param>
/// <returns>最终用于展示的搜索名称</returns>
private string GetSearchedEntityName<T>(
    Dictionary<string, string> formDataDictionary,
    string formKey,
    Func<int, T> getEntityByIdFunc,
    Func<T, string> getNameSelector)
{
    // 统一处理字典取值、类型转换、ID合法性校验逻辑
    if (!formDataDictionary.TryGetValue(formKey, out var idValue) 
        || !int.TryParse(idValue, out var entityId) 
        || entityId <= 0)
    {
        return string.Empty;
    }

    // 调用传入的仓储方法查询实体
    var entity = getEntityByIdFunc(entityId);
    // 提取实体名称，空值兜底返回空字符串
    return entity != null ? getNameSelector(entity) : string.Empty;
}

原来的两段重复代码，直接替换成两次方法调用即可，没有任何重复的流程代码：

// 获取Country对应的搜索名称
var searchedCountryName = GetSearchedEntityName(
    formDataDictionary,
    "CountryId",
    id => _countriesRepository.GetCountryById(id),
    entity => entity.Name);

// 获取Subject对应的搜索名称
var searchedSubjectName = GetSearchedEntityName(
    formDataDictionary,
    "SubjectId",
    id => _subjectsRepository.GetFullSubjectDetailsById(id),
    entity => entity.Name);

哪些情况会让抽象变得得不偿失 #

抽象本身没有问题，问题出在过度设计上，只要避开下面这两个坑，就不会出现复杂度高于收益的情况：

• 不要为了泛型方法强行给现有仓储改造出统一的IBaseRepository<T>接口：两个仓储的查询方法命名不同、返回的实体类型没有公共继承关系，硬改方法名、加继承关系属于无意义的侵入改动，反而会增加维护成本
• 不要强行约束所有实体必须实现IHasName接口：如果后续碰到实体的名称字段是FullName、Title这类不同命名的情况，靠接口约束根本兜不住，不如直接通过传入的名称选择器委托来的灵活

额外的优化收益 #

把这些共性逻辑抽到公共方法里后，后续调整逻辑（比如要给ID解析失败加日志、要给实体不存在的情况加告警）的时候，只需要改公共方法一处就行，不用在每个重复逻辑块里挨个修改，长期维护的成本会低很多。

内容的提问来源于stack exchange，提问作者Username_null