定义两个具有相同参数类型的方法

Question

今天我遇到了一种情况，我必须创建一个与现有方法共享相同 name, params count and params types 的方法，如下所示：

public static Department GetDepartment(string departmentName)
{
  //LOGIC
}

public static Department GetDepartment(string employeeID)
{
  //LOGIC
}

乍一看，我只是说为什么不给它起一个不同的名字并把事情做好，但我做不到！我确实想保持我正在处理的代码的可读性，我希望它是overloaded 第一个，
所以我说为什么不添加一个假参数只是为了从编译器的角度解决这个问题。

 public static Department GetDepartment(string employeeID, object fakePassWtEver)
    {
      //LOGIC
    }

这种情况的最佳做法是什么？我看到了所有可以让我的代码运行的方法，但没有一个让我满意

Answer 1

保持可读性正是为什么你应该重命名它：

Department GetDepartmentByName(...)

Department GetDepartmentByEmployeeID(...)

现在，无论何时调用该方法，一目了然您指的是哪个方法。如果您改为重载方法，则情况非常不是。

随着时间的推移，我越来越不愿意超载 - 有 quite a few subtle issues，可读性经常下降。

Answer 2

您可以通过执行以下操作来更新您的方法签名并同时使您的代码更具可读性。

public static GetDepartmentByName( string departmentName )

public static GetDepartmentByEmployeeId( string employeeId )

就我个人而言，我觉得在代码中添加冗长有助于后来的其他人理解发生了什么。它还有助于使您的方法更容易“阅读”。

Answer 3

定义2个方法：

1. public static Department GetDepartmentByDepartmentName(string departmentName)
2. public static Department GetDepartmentByEmployeeID(string employeeID)

Answer 4

如果您可以通过检查参数以某种方式区分员工 ID 和部门名称，则另一种选择是委托给其他方法。

public static Department GetDepartment(string employeeIdOrDepartmentName) {
    if (LooksLikeEmployeeID(employeeIdOrDepartmentName))
        return GetDepartmentByEmployeeID(employeeIdOrDepartmentName);
    else
        return GetDepartmentByDepartmentName(employeeIdOrDepartmentName);
}

private static Department GetDepartmentByEmployeeID(string employeeId) {
    /* ... */
}

private static Department GetDepartmentByDepartmentName(string departmentName) {
    /* ... */
}

只有在您绝对不能添加其他方法以明确起见时，您才应该这样做 - 其他答案是 100% 正确的。

Answer 5

有点晚了，但有可能，我今天遇到了完全相同的情况（构造函数重载，因此无法更改名称）。这是我的做法，小技巧，但它让我可以将所有相关的 LINQ 谓词放在同一个地方：

public BusinessStructureFilterSpecification(int responsibilityTypeId, bool dummy1 = true) : base(x => x.ResponsibleGroups.Any(x1 => x1.ResponsibilityTypeId == responsibilityTypeId))
{
    AddIncludes();
}

public BusinessStructureFilterSpecification(int userId, string dummy2 = "") : base(x => x.ResponsibleUsers.Any(x1 => x1.UserId == userId))
{
    AddIncludes();
}

现在的诀窍是使用参数名称来调用它们，如下所示：

if (responsibiltyTypeId.HasValue && !userId.HasValue)
    spec = new BusinessStructureFilterSpecification(responsibilityTypeId: responsibiltyTypeId.Value);

if (!responsibiltyTypeId.HasValue && userId.HasValue)
    spec = new BusinessStructureFilterSpecification(userId: userId.Value);