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
    }
}

如何避免重复？

2 个答案: