将方法绑定到实现类

Question

这是否会产生任何代码异味或违反 SOLID 原则？

public string Summarize()
{
IList<IDisplayable> displayableItems = getAllDisplayableItems();
StringBuilder summary = new StringBuilder();

foreach(IDisplayable item in displayableItems)
{
    if(item is Human)
        summary.Append("The person is " + item.GetInfo());

    else if(item is Animal) 
        summary.Append("The animal is " + item.GetInfo());

    else if(item is Building) 
        summary.Append("The building is " + item.GetInfo());

    else if(item is Machine) 
        summary.Append("The machine is " + item.GetInfo());
}

return summary.ToString();
}

如您所见，我的 Summarize() 与 Human、Animal 等实现类绑定。

此代码是否违反 LSP？ （还有其他 SOLID 原则吗？）

Answer 1

我闻到了一点东西......

如果您的类都实现了 IDisplayable，那么它们应该各自实现自己的逻辑来显示自己。这样你的循环就会变成更干净的东西：

public interface IDisplayable
{
    void Display();
    string GetInfo();
}

public class Human : IDisplayable
{
    public void Display() { return String.Format("The person is {0}", 
        GetInfo());

    // Rest of Implementation
}

public class Animal : IDisplayable
{
    public void Display() { return String.Format("The animal is {0}", 
        GetInfo());

    // Rest of Implementation
}

public class Building : IDisplayable
{
    public void Display() { return String.Format("The building is {0}", 
        GetInfo());

    // Rest of Implementation
}

public class Machine : IDisplayable
{
    public void Display() { return String.Format("The machine is {0}", 
        GetInfo());

    // Rest of Implementation
}

然后您可以将循环更改为更简洁的内容（并允许类实现自己的显示逻辑）：

foreach(IDisplayable item in displayableItems)
    summary.Append(item.Display());

Answer 2

似乎 IDisplayable 应该有一个显示名称的方法，因此您可以将该方法简化为类似

summary.Append("The " + item.displayName() + " is " + item.getInfo());

Answer 3

是的。

为什么不让每个类都实现一个来自IDisplayable 的方法来显示它们的类型：

interface IDisplayable
{
    void GetInfo();
    public string Info;
}
class Human : IDisplayable
{
   public string Info
   { 
    get 
    { 
        return "";//your info here
    }
    set;
   }

   public void GetInfo()
   {
       Console.WriteLine("The person is " + Info)
   }
}

然后只需按如下方式调用您的方法：

foreach(IDisplayable item in displayableItems)
{
    Console.WriteLine(item.GetInfo());
}

Answer 4

鉴于 OP 对 this answer 的评论，我认为最好的方法是创建一个自定义容器类来替换具有 containsHumans() 和 containsAnimals() 等方法的 IList<IDisplayable> displayableItems，这样您就可以封装 icky non - 多态代码在一处，并保持Summarize() 函数中的逻辑干净。

class MyCollection : List<IDisplayable>
{
    public bool containsHumans()
    {
        foreach (IDisplayable item in this)
        {
            if (item is Human)
                return true;
        }

        return false;
    }

    // likewise for containsAnimals(), etc
}

public string Summarize()
{
    MyCollection displayableItems = getAllDisplayableItems();
    StringBuilder summary = new StringBuilder();

    if (displayableItems.containsHumans() && !displayableItems.containsAnimals())
    {
        // do human-only logic here
    }
    else if (!displayableItems.containsHumans() && displayableItems.containsAnimals())
    {
        // do animal-only logic here
    }
    else
    {
        // do logic for both here
    }

    return summary.ToString();
}

当然，我的例子过于简单和做作。例如，作为Summarize() if/else 语句中逻辑的一部分，或者可能围绕整个块，您需要遍历displayableItems 集合。此外，如果您在 MyCollection 中覆盖 Add() 和 Remove() 并让它们检查对象的类型并设置一个标志，您可能会获得更好的性能，因此您的 containsHumans() 函数（和其他函数）可以简单地返回标志的状态，并且不必在每次调用时都迭代集合。

Answer 5

怎么样：

    summary.Append("The " + item.getType() + " is " + item.GetInfo()); 

Answer 6

至少它违反了 LSP 和开闭原则。

解决方案是给IDisplayable 接口添加一个Description 属性，这样summary 就可以调用

summary.Append(string.Format("The {0} is {1}", item.Description, item.GetInfo()));

这也可以通过反射来解决，因为您只获得了类的名称。

更好的解决方案是从GetInfo() 方法返回类似IDisplayableInfo 的内容。这将是一个有助于保护 OCP 的扩展点。

Answer 7

如果您无法修改 IDisplayable 或类实现并且您使用的是 .NET 3.5 或更高版本，则可以使用扩展方法。但这并没有那么好