可以调用构造函数初始值设定项列表中的函数吗?

问题描述 投票:0回答:3

我的直觉是不是。我现在的情况如下:

class PluginLoader
{
   public:
      Builder* const p_Builder;
      Logger* const p_Logger;

      //Others
};

PluginLoader::PluginLoader(Builder* const pBuilder)
   :p_Builder(pBuilder), p_Logger(pBuilder->GetLogger())
{
   //Stuff
}

或者我应该更改构造函数并从构造

Logger* const
的位置传递
PluginLoader
吗?

c++ constructor initialization member-initialization
3个回答
42
投票

这完全没问题,很正常。

p_Builder
在它之前被初始化。


38
投票

你拥有的很好。不过,我只是想警告你要小心不要这样做:(GMan提到过这一点,我只是想把它说清楚)

class PluginLoader
{
   public:
      Logger* const p_Logger;   // p_Logger is listed first before p_Builder
      Builder* const p_Builder;

      //Others
};

PluginLoader::PluginLoader(Builder* const pBuilder)
   :p_Builder(pBuilder),
    p_Logger(p_Builder->GetLogger())   // Though listed 2nd, it is called first.
                                       // This wouldn't be a problem if pBuilder 
                                       // was used instead of p_Builder
{
   //Stuff
}

请注意,我对您的代码进行了 2 处更改。首先,在类定义中,我在 p_Builder 之前声明了 p_Logger。其次,我使用成员p_Builder来初始化p_Logger,而不是参数。

这些更改中的任何一个都可以,但是它们一起引入了一个错误,因为 p_Logger 首先被初始化,并且您使用未初始化的 p_Builder 来初始化它。

请始终记住,成员按照它们在类定义中出现的顺序进行初始化。并且您将它们放入初始化列表中的顺序无关紧要。


1
投票

非常好的做法。

我建议这样做(但这纯粹是个人层面的):

不要在构造函数中调用函数,而是将它们分组在 init 函数中,这只是为了灵活性:如果您以后必须创建其他构造函数。

© www.soinside.com 2019 - 2024. All rights reserved.