构造函数中的方法,不好？

Question

我有一个Windows窗体,我有一个类检查文本文件,以确保它具有某些方面.现在我在构造函数中有方法,它看起来有点奇怪.我应该将构造函数留空,并实现一个start()类型方法并调用它？到目前为止我的代码看起来像这样

public class Seating
{
    private int segments = 0;
    public Seating()
    {
        checkInvoice();
        getSegmentCount();          
    }
}

Answer 1

我在构造函数中有方法,看起来有点奇怪

那么,为什么还有构造函数存在呢？在构造函数中使用方法是完全合法的,甚至是可能失败的方法(从而中断对象创建).很难说这是否适合您的具体情况.

一般情况下它很好.

Answer 2

在构造函数中调用非虚方法没有错.在构造函数触发时,您的父对象已完全构造.但是,您不希望调用任何虚方法,因为它们可以被子类覆盖,子类在未完全构造时会在其中执行代码.

Answer 3

一般规则是方法非常简单,不太可能在构造函数中抛出异常.如果方法可能由于任何原因(文件访问或丢失,数据库问题,空引用......)而失败,那么它不应该在构造函数中,因为人们应该期望构造函数不会失败(特别是无参数构造函数,尽管一般指导没有指明).人们不应该期望var seating = new Seating();成为错误的来源.

您可以添加Start()/ Initialize()种类的方法作为您提到的实例方法.您还可以添加一个新的静态方法,该方法在调用您需要的两个方法并返回构造函数私有之后返回Class的新实例.您可以更进一步,在新的Factory类中使用此方法(使构造函数内部).您也可以考虑其他方法.

有一点是你可以看到在这些方法中计算或检索任何值的构造函数参数(并且没有无参数构造函数),然后构造函数将只将这些参数分配给相应的字段/属性.同一程序集或其他"服务"专用程序集中的工厂方法或服务方法可以负责调用方法,获取参数,将它们传递给构造函数,以及返回类的新实例.这是我个人的最爱.

一般来说,这类问题表明该类做得太多,您可能希望将功能拆分为其他类(现有的或新的).这就是建议最后一个解决方案的原因,因此这两个方法本身可能位于另一个"服务"程序集中,但如上所述,如果需要,还有许多其他方法可以执行此操作.

更新:

以下是Microsoft构造函数指南:
构造函数使用指南

从页面引用:

最小化构造函数中完成的工作量.构造函数不应该只捕获构造函数参数或参数.这会延迟执行进一步操作的成本,直到用户使用实例的特定功能.

更新2

上面的页面在Constructor Design的名称下移动到一个活文档(这意味着它是可更新的).

Answer 4

一般来说,构造函数应该只设置对象的状态.这可能意味着调用方法.我会说运行很长的方法并不是一个好主意.您的类的用户不希望构造函数是一种昂贵的方法.

我不是你没有提到虚方法,但你永远不应该在构造函数中调用它们.

以下是如何快速恢复的示例:

public class MyBase
{
        protected MyBase()
        {
                this.VirtualMethod();
        }

        protected virtual void VirtualMethod()
        {
                Console.WriteLine("VirtualMethod in MyBase");
        }
}

public class MyDerived : MyBase
{
        private readonly string message = "Set by initializer";

        public MyDerived(string message)
        {
                this.message = message;
        }

        protected override void VirtualMethod()
        {
                Console.WriteLine(this.message);
        }
}

现在,让我们说你在其他地方有这个代码:

MyDerived d = new MyDerived("Called from constructor");

您认为在控制台上会显示什么？如果您说"由初始化程序设置",那么您是对的.

这就是为什么:

• 所有字段初始值设定项都在构造函数中的代码之前执行.
• C#编译器在用户定义的任何内容之前添加对基础构造函数的调用.在这种情况下,它调用
  MyBase哪个调用VirtualMethod().由于d的运行时类型是MyDerived,的超驰VirtualMethod()中MyDerived被
  执行.现在,由于MyDerived构造函数的主体尚未
  执行,因此this.message具有在初始化程序中给出的值
  .
• 现在MyDerived执行的构造函数的主体.
• 稍后VirtualMethod()对此实例的
  调用现在将打印出"从构造函数调用".

Answer 5

构造函数中的虚方法调用是不行的(密封类的极小例外,这使得该方法实际上是非虚拟的).

非虚方法可以是好的,但你应该确保所有成员之前已经初始化,以及非虚方法可能不调用任何虚拟成员也是如此.