如何在c#中删除模棱两可的方法调用

Question

这里我有一个简单的场景，有两种方法，我从彼此那里得到一个模棱两可的调用：

这是我的代码：

public IEnumerable<JobsViewModel> GetJobsViewModels(Guid vesselId, int status, Func<JobsNoSubsYpdcResult, bool> predicate = null)
    => predicate == null
        ? Mapper.Map<IEnumerable<JobsViewModel>>(_procedureService.Tech_GetJobsNoSubsYPDC(vesselId, status))
        : Mapper.Map<IEnumerable<JobsViewModel>>(_procedureService.Tech_GetJobsNoSubsYPDC(vesselId, status).Where(predicate));

public IEnumerable<JobsViewModel> GetJobsViewModels(Guid vesselId, int status, Func<JobsViewModel, bool> predicate = null)
    => predicate == null
        ? GetJobsViewModels(vesselId, status)
        : GetJobsViewModels(vesselId, status).Where(predicate);

我不喜欢更改方法的名称，但从第二个开始我收到错误消息：

模棱两可的调用

我想像我想要做的那样从第二个调用第一个，有人知道我怎样才能在不更改方法名称的情况下做到这一点，否则我应该更改它们？

Answer 1

处理这个问题的最好方法是不要有两个模棱两可的重载，这样你就不需要记住如何在你调用它们的任何地方对它进行特殊处理。

由于您的第二个重载调用第一个，我想说最合乎逻辑的行动原因是删除该委托的默认参数值：

public IEnumerable<JobsViewModel> GetJobsViewModels(Guid vesselId, int status, Func<JobsViewModel, bool> predicate)
    => predicate == null
        ? GetJobsViewModels(vesselId, status)
        : GetJobsViewModels(vesselId, status).Where(predicate);

---

您还应该对指定委托参数调用此方法的能力进行推理，但明确指定null。如果这不是真的意味着可能无论如何（即三元表达式的第一部分不会真正使用），那么我将简化整个方法：

public IEnumerable<JobsViewModel> GetJobsViewModels(Guid vesselId, int status, Func<JobsViewModel, bool> predicate)
    => GetJobsViewModels(vesselId, status).Where(predicate);

---

现在，说了这么多，无论如何我都没有真正看到第二种方法的意义，因为在外面我可以简单地塞上.Where(...) 过滤器。由于该方法没有对这个谓词做任何聪明的事情，比如传递它并在较低级别上执行它，或者优化它获取数据的方式，所以这实际上只是语法糖。

然而，这只是我的意见，我承认我不知道这一切将如何被使用。我真正的答案是第一部分，你真的应该避免声明模棱两可的方法。

---

另请参阅此问题：Breaking change in method overload resolution in C# 6 - explanation?。

当 Roslyn 进入现场时，重载分辨率发生了变化，他们决定记录更改并使其破坏而不是重新引入旧的“错误”。

Answer 2

您需要提供可选参数并让编译器清楚其类型，即使值为null：

GetJobsViewModels(vesselId, status, (Func<JobsNoSubsYpdcResult, bool>)null)

---

编辑：在 OP 提出的特定情况下，Lasse's solution 更可取。

另一方面，此答案中显示的解决方案通常可以应用，也可以在消除另一个重载没有可选参数的重载时（例如，在调用某些 @ 时） 987654322@构造函数）。

Answer 3

这样做的唯一方法是用适当类型的值填充可选参数，以便编译器知道要选择哪个重载。例如：

public IEnumerable<JobsViewModel> GetJobsViewModels(
    Guid vesselId,
    int status,
    Func<JobsViewModel, bool> predicate = null)
{
    // We're never filtering by JobsNoSubsYpdcResult, but this
    // satisfies overload resolution
    Func<JobsNoSubsYpdcResult, bool> resultPredicate = null;
    return predicate == null
        ? GetJobsViewModels(vesselId, status, resultPredicate)
        : GetJobsViewModels(vesselId, status, resultPredicate).Where(predicate);
}

或者为了避免重复：

public IEnumerable<JobsViewModel> GetJobsViewModels(
    Guid vesselId,
    int status,
    Func<JobsViewModel, bool> predicate = null)
{
    // We're never filtering by JobsNoSubsYpdcResult, but this
    // satisfies overload resolution
    Func<JobsNoSubsYpdcResult, bool> resultPredicate = null;
    var allResults = GetJobsViewModels(vesselId, status, resultPredicate);
    return predicate == null
        ? allResults : allResults.Where(predicate);
}

或者为了避免更多重复，引入你自己的扩展方法：

public static IEnumerable<T> OptionalWhere<T>(
    this IEnumerable<T> source,
    Func<T, bool> predicate) =>
    predicate == null ? source : source.Where(predicate);

然后你可以重写这两种方法来使用它：

public IEnumerable<JobsViewModel> GetJobsViewModels(Guid vesselId, int status, Func<JobsNoSubsYpdcResult, bool> predicate = null) =>
    Mapper.Map<IEnumerable<JobsViewModel>>(_procedureService.Tech_GetJobsNoSubsYPDC(vesselId, status))
        .OptionalWhere(predicate);

public IEnumerable<JobsViewModel> GetJobsViewModels(
    Guid vesselId,
    int status,
    Func<JobsViewModel, bool> predicate = null)
{
    // We're never filtering by JobsNoSubsYpdcResult, but this
    // satisfies overload resolution
    Func<JobsNoSubsYpdcResult, bool> resultPredicate = null;
    return GetJobsViewModels(vesselId, status, resultPredicate)
        .OptionalWhere(predicate);
}

（如果您愿意将null 转换为Func<JobsNoSubsYpdcResult, bool> 或为此使用静态字段，您可以使用表达式主体方法。）

Answer 4

如果你真的想要你所说的，你需要传递正确的谓词参数，例如GetJobsViewModels(vesselId, status, new Func<JobsNoSubsYpdcResult, bool>( x => true ));