添加参数，或创建新方法？

Question

假设我有一个长期建立的存储库，如下所示：

interface IDonutRepository
{
    public IEnumerable<Donut> GetDonuts();
}

它已经存在很长时间了，并且 GetDonuts 方法按照它的说明执行操作。然后有一天，我需要添加一个新屏幕来显示数据库中的所有甜甜圈，结果发现该方法有一个隐藏功能 - 它过滤掉所有 stale = true 的甜甜圈。但在我的新屏幕上，我想显示所有这些，甚至是陈旧的！这里最好的方法是什么？

假设这个方法在各处使用，并且默认行为需要保持不变，那么最好添加一个名为 GetAllDonuts 的新方法，该方法不执行过滤，或者我应该只是这样做将 onlyFresh 参数添加到 GetDonuts 方法中？

我猜这只是判断，但我想知道是否有更多明智的答案？

Answer 1

我将重载该方法，创建一个采用 showStale 参数的新重载，然后修改旧方法以使用新重载，为参数值传递 false 。

界面如下所示：

interface IDonutRepository
{
    public IEnumerable<Donut> GetDonuts();
    public IEnumerable<Donut> GetDonuts(bool showStale);
}

或者，如果您使用 .NET 4.0，则可以使用可选参数：

interface IDonutRepository
{
    public IEnumerable<Donut> GetDonuts(bool showStale = false);
}

Answer 2

为什么不使用可选参数？这样您就不会破坏现有代码：

interface IDonutRepository
{
    public IEnumerable<Donut> GetDonuts(bool onlyFresh);
}

实现：

public IEnumerable<Donut> GetDonuts(bool onlyFresh = false)
{
    if (onlyFresh)
        // do stuff
    else
        // do other stuff
}

Answer 3

在某种程度上，这确实取决于个人喜好...

如果您有能力更改 API，我会（个人）以一种明显不返回所有 Donut 的方式重命名当前方法 实例。我的期望是存储库的 GetDonuts 方法将获取所有甜甜圈。这可以通过参数或不同的名称来完成，具体由您自行决定。

话虽这么说，如果保持兼容性至关重要，那么采用额外参数的方法重载可能是前进的最佳选择。 （这很大程度上取决于这个 API 的使用者和地点......）

Answer 4

根据情况，人们可能会考虑引入一种用于访问甜甜圈的属性。

interface IDonutRepository
{
    IEnumerable<Donut> Donuts { get; }
    .. or ..
    IQueryable<Donut> Donuts { get; }
}

如果您使用像 Entity Framework 或 NHibernate 这样精通 Linq 的 ORM，则实现此接口相当容易。

旧的 GetDonuts 方法可以重命名为 GetFreshDonuts()，或者您可以将对它的调用重构为以下形式：

repository.Donuts.Where(x => !x.Stale)

Answer 5

软件的增长趋势之一
设计将界面与
执行。原理是关于
将模块分为公共模块和
私人部位，以便您可以更改
私处不协调
与其他模块。然而，有
进一步的区别——
公共和发布的接口。这
区别很重要，因为它
影响你的工作方式
界面。

http://www.martinfowler.com/ieeeSoftware/published.pdf