在子方法中显式使用基本方法而不使用关键字 new

Question

由于类的多态性属性，下面的示例将打印AB 两次，这是预期的。

就我而言，我真的希望它先打印A，然后再打印AB。

我决定将B 中的Get() 方法从overrides 更改为new。 这解决了我的问题，但他们告诉我有不好的做法，所以我正在寻找替代方案......

想到的一件事是instantiate a new A in B.Do()，我认为这也是不好的做法......

//ORIGINAL
class Program
{
    static void Main(string[] args)
    {
        var b = new B();
        b.Do();
    }
}

public class A
{
    public virtual void Do()
    {
        var get = Get();
        Console.WriteLine(get);
    }

    public virtual string Get()
    {
        return "A";
    }
}

public class B : A
{
    public override void Do()
    {
        base.Do();

        var get = Get();
        Console.WriteLine(get);
    }

    public override string Get()
    {
        return base.Get() + "B";
    }
}

//UPDATED, USING NEW
class Program
{
    static void Main(string[] args)
    {
        var b = new B();
        b.Do();

        Console.ReadLine();
    }
}

public class A
{
    public virtual void Do()
    {
        var get = Get();
        Console.WriteLine(get);
    }

    public virtual string Get()
    {
        return "A";
    }
}

public class B : A
{
    public override void Do()
    {
        base.Do();

        var get = Get();
        Console.WriteLine(get);
    }

    public new string Get()
    {
        return base.Get() + "B";
    }
}

Answer 1

除了在B.Do 中调用Do 的基本版本，您可以更改实现如下：

public class B : A
{
  public override void Do()
  {
    // Call the base version of Get explicitly
    var getBase = base.Get();
    Console.WriteLine(getBase);
    // Call the current implementation of Get
    var get = Get();
    Console.WriteLine(get);
  }

  public override string Get()
  {
    return "B";
  }
}

这将在技术上解决您的问题，但从 OOP 的角度来看，这并不是一个真正干净的解决方案。我建议考虑一下您是否需要能够独立覆盖Get。也许更改方法的签名以便Get 始终返回应打印的字符串列表也是一个很好的解决方案（我已将Get 重命名为GetLines 以反映方法的更改目的）：

public class A
{
    public virtual void Do()
    {
        var lines = GetLines();
        foreach(var line in lines)
            Console.WriteLine(line);
    }

    public virtual IEnumerable<string> GetLines()
    {
        return new string[] { "A" };
    }
}

public class B : A
{

    public override IEnumerable<string> GetLines()
    {
        var lst = new List<string>(base.GetLines());
        lst.Add("B");
        return lst;
    }
}

Answer 2

以下是更正后的代码：

class Program
{
    static void Main(string[] args)
    {
        var b = new B();
        b.Do();
    }
}

public class A
{
    public virtual void Do()
    {
        Console.WriteLine(Get());
    }

    public virtual string Get()
    {
        return "A";
    }
}

public class B : A
{
    public override void Do()
    {
        Console.WriteLine(base.Get());
        base.Do();
    }

    public override string Get()
    {
        return base.Get() + "B";
    }
}