通过成员数组循环给出错误的值

Question

我设置了两个 classes，Dog 和 AnotherDog。 Dog 并不是 AnotherDog 的基础 class。

在AnotherDog中，我有一个Dog对象。在那个 Dog 对象中是一个成员数组。当 AnotherDog 对象调用其 Dog 成员，然后让成员循环遍历其成员数组时，我得到了错误的结果。

#include <iostream>

class Dog
{
private:
    int m_NumberOfBarks;
    int m_Decibels[];
public:
    Dog();
    ~Dog();

    void setBarkDecibels(int decibel1, int decibel2);
    void loopDecibels();
};

Dog::Dog() : m_NumberOfBarks(2){}
Dog::~Dog(){}

void Dog::setBarkDecibels(int decibel1, int decibel2){
    m_Decibels[0]=  decibel1;
    m_Decibels[1]=  decibel2;
}

void Dog::loopDecibels(){
    for(int i=0; i<m_NumberOfBarks; ++i){
        std::cout << i << ' ' << m_Decibels[i] << std::endl;
    }
}


class AnotherDog
{
private:
    Dog m_Dog;
public:
    AnotherDog();
    ~AnotherDog();

    Dog getDog();
};

AnotherDog::AnotherDog(){
    m_Dog.setBarkDecibels(10, 100);
}
AnotherDog::~AnotherDog(){}

Dog AnotherDog::getDog(){
    return m_Dog;
}


int main(){
    AnotherDog goodDog;
    goodDog.getDog().loopDecibels();
    return 0;
}

我希望 void Dog::loopDecibels() 打印 10 和 100，以及索引。

相反，我得到了这个：

0 0
1 4196480

我做错了什么？

如何达到我想要的结果？

Answer 1

您的程序表现出未定义的行为。

 int m_Decibels[];

声明了一个指向int的指针，并且不为指向的指针分配任何内存。指针在 class 构造函数中保持未初始化状态（因为您没有初始化它）。稍后你做

m_Decibels[0]=  decibel1;
m_Decibels[1]=  decibel2;

您正在取消引用 this 指针，这是一个禁忌。要解决此问题，您可以使用固定大小的数组：

int m_Decibels[2];

硬币的另一面是，您正在 return 从您的 getDog 中获取 Dog 的实例（按值）。当您在此特定实例上设置分贝时，它对 class 的原始 dog 成员没有影响。要解决此问题，您可能需要通过引用 return 您的对象，如下所示：

   Dog& getDog(); // and corresponding change in the definition

通过成员数组循环给出错误的值

Looping thru member array gives wrong values

c++

arrays

oop

class

member