如何避免重复？

Question

我正在制作一个 C# 应用程序。该应用程序有两个类和多个方法。在编写代码时，我偶然发现了一个问题。我在两个类中使用相同的两个变量（XList 和 YList）和一种方法。并且可能我需要更多带有此代码的类。所以我创造了一个重​​复问题。以下是我的代码的简单版本：

public class A {
  private testEntities db = new testEntities();
  public List<int> XList = new List<int>();
  public List<int> YList = new List<int>();

  public void GetAllInfo()
  {
    // Get the data from a database and add to a list
    XList = db.Table1.ToList();
    YList = db.Table2.ToList();
  }

  public void DoStuff()
  {
    // Do Stuff with XList and YList
  }
}

public class B {
  private testEntities db = new testEntities();
  public List<int> XList = new List<int>();
  public List<int> YList = new List<int>();

  public void GetAllInfo()
  {
    // Get the data from a database and add to a list (the same as in class A)
    XList = db.Table1.ToList();
    YList = db.Table2.ToList();
  }

  public void DoDifferentStuff()
  {
    // Do ddifferent stuff with XList and YList then in class A
  }
}

我的问题是解决这个重复问题的最佳方法是什么？

经过一些研究，我发现我可能可以通过继承或组合来解决这个问题。我还读到人们选择组合而不是继承。所以我写了以下代码来解决重复：

public class DataPreparation
{
  private testEntities db = new testEntities();
  public List<int> XList = new List<int>();
  public List<int> YList = new List<int>();

  public void GetAllInfo()
  {
    // Get the data from a database and add to a list
    XList = db.Table1.ToList();
    YList = db.Table2.ToList();
  }

  // Implement other methods
}

public class A 
{
  public void MethodName()
  { 
    DataPreparation dataPreparation = new DataPreparation();
    dataPreparation.GetAllInfo();

    UseDataX(dataPreparation.XList);
    UseDataY(dataPreparation.YList);

    // Implementation UseDataX() and UseDataY()
  }
}

public class B 
{
  public void MethodName()
  { 
    DataPreparation dataPreparation = new DataPreparation();
    dataPreparation.GetAllInfo();

    VisualizeDataX(dataPreparation.XList);
    VisualizeDataY(dataPreparation.YList);

    // Implementation VisualizeDataX() and VisualizeDataY()
  }
}

如您所见，我创建了一个类来处理从数据库中获取数据。并且A类和B类使用DataPreparation类。 但这是解决重复问题的最佳方法吗？还是应该使用继承或其他方式？

Answer 1

我认为您可能应该只使用一种称为DoStuff() 的方法，而不是一种称为DoStuff() 和另一种称为DoDifferentStuff() 的方法。

然后您可以创建一个 ABC 来实现公共代码，并在派生类中以不同的方式实现抽象 DoStuff() 方法：

public abstract class Base
{
    private testEntities db = new testEntities();

    public List<int> XList = new List<int>();
    public List<int> YList = new List<int>();

    public void GetAllInfo()
    {
        // Get the data from a database and add to a list (the same as in class A)
        XList = db.Table1.ToList();
        YList = db.Table2.ToList();
    }

    public abstract void DoStuff();
}

public class A: Base
{
    public override void DoStuff()
    {
        // Do Stuff with XList and YList
    }
}

public class B: Base
{
    public override void DoStuff()
    {
        // Do ddifferent stuff with XList and YList then in class A
    }
}

（我也认为拥有这样的公共字段是个坏主意 - 但我猜测/希望这只是示例代码，而您的真实代码没有这些......）

其他代码（创建A 或B 的代码除外）将通过Base 类类型使用该对象。

Answer 2

一个简单的选择是使用inheritance。创建一个具有共享功能的基类，A 和 B 可以继承。例如：

public abstract class Base
{
    private testEntities db = new testEntities();
    public List<int> XList = new List<int>();
    public List<int> YList = new List<int>();

    public void GetAllInfo()
    {
        // Get the data from a database and add to a list
        XList = db.Table1.ToList();
        YList = db.Table2.ToList();
    }
}

public class A : Base
{
    public void DoStuff()
    {
        // Do Stuff with XList and YList
    }
}

public class B : Base
{
    public void DoDifferentStuff()
    {
        // Do ddifferent stuff with XList and YList then in class A
    }
}