在C ++中初始化对象数组时出错

时间:2015-06-03 07:21:14

标签: c++ arrays initialization

当我运行试图初始化一个对象数组的代码,并且一个代码具有无效值时,它似乎没有调用构造函数,这将设置一个正确的默认值。下面的代码产生输出:

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,这也不是默认值。

2 个答案:

答案 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