当我运行试图初始化一个对象数组的代码,并且一个代码具有无效值时,它似乎没有调用构造函数,这将设置一个正确的默认值。下面的代码产生输出:
1
2
1528112104
玩具代码:
#include <iostream>
using namespace std;
class BeeBoop
{
public:
static const int MIN_X = 1;
static const int MAX_X = 2;
BeeBoop(int x);
int getX() { return x; }
bool setX(int x);
private:
int x;
};
int main()
{
BeeBoop boops[] =
{
BeeBoop(1),
BeeBoop(2),
BeeBoop(3)
};
for (int i = 0; i < 3; i++)
cout << boops[i].getX() << endl;
}
BeeBoop::BeeBoop (int x)
{
if(!setX(x))
x = MIN_X;
}
bool BeeBoop::setX(int x)
{
if (x < MIN_X || x > MAX_X)
return false;
this->x = x;
return true;
}
为什么不调用构造函数并将其设置为BeeBoop(3)的默认值?
如果我将初始化列表的顺序切换为
,甚至更奇怪...
BeeBoop boops[] =
{
BeeBoop(1),
BeeBoop(3),
BeeBoop(2)
)
...
输出变为:
1
0
2
因此它初始化为0,这也不是默认值。
答案 0 :(得分:5)
您使用名称x
作为函数参数和成员变量(可能不是一个好主意!)。因此,您需要更改:
BeeBoop::BeeBoop (int x)
{
if(!setX(x))
x = MIN_X;
}
为:
BeeBoop::BeeBoop (int x)
{
if(!setX(x))
this->x = MIN_X;
}
否则你只是修改参数x
而不是设置成员变量。 (或者,您可以使用参数和成员变量的唯一名称来避免这种歧义。)
请注意,如果您编译时启用了适当的警告(-Wshadow
),编译器就能指出您的错误:
main.cpp: In constructor 'BeeBoop::BeeBoop(int)':
main.cpp:30:24: warning: declaration of 'x' shadows a member of 'BeeBoop' [-Wshadow]
BeeBoop::BeeBoop (int x)
^
main.cpp:14:13: note: shadowed declaration is here
int x;
^
main.cpp: In member function 'bool BeeBoop::setX(int)':
main.cpp:36:25: warning: declaration of 'x' shadows a member of 'BeeBoop' [-Wshadow]
bool BeeBoop::setX(int x)
^
main.cpp:14:13: note: shadowed declaration is here
int x;
答案 1 :(得分:0)
类成员函数和函数参数的局部变量是局部变量隐藏类的数据成员。
例如,在名为x
的构造函数参数中隐藏数据成员x
BeeBoop::BeeBoop (int x)
{
if(!setX(x))
x = MIN_X;
}
因此在声明中
x = MIN_X;
在赋值的左侧,使用参数x
。
你应该使用
BeeBoop::x = MIN_X;
或
this->x = MIN_X;
存在相同问题的所有功能都必须以相同的方式更新。
考虑到最好使用qualofoer getX
声明函数const
int getX() const { return x; }
由于类的构造函数没有说明符explicit
,因此它是转换构造函数。这意味着你可以简单地写一下
BeeBoop boops[] = { 1, 2, 3 };
函数main可以通过以下方式简化:)
int main()
{
for ( BeeBoop b : { 1, 2, 3 } )
{
std::cout << b.getX() << std::endl;
}
}
程序输出
1
2
1