创建对象数组时出现分段错误

时间:2015-10-09 20:29:49

标签: c++ arrays pointers object

下面是我想要开始工作的代码。我想创建一个BankTeller对象数组。根据我的理解,当调用一个新的BankTeller()时,它会创建一个指向该对象的指针,并将其保存到计数器中。然后我尝试插入到psotion,但是当我使用gdb逐步执行并且程序崩溃时,会出现分段错误。

BankTeller **tellers; 
BankModel(int count){
    srand (time(NULL)); //ignore this
    *tellers = new BankTeller[count];
    for(int i=0; i <= count; i++){
        tellers[i] = new BankTeller();
    }

    return;
}

4 个答案:

答案 0 :(得分:2)

我认为你的代码比它可能需要的更复杂。我会考虑使用std容器,例如std::vector或更适合您需求的容器。一般来说,你应该避免多个间接层,除非它确实是必要的,在这种情况下它似乎不是。

了解指针

您首先声明BankTeller **tellers,这是指向BankTeller指针的指针。导致代码中的段错误的行是*tellers = new BankTeller[count];。这一行返回一个指向BankTeller个对象数组的指针,但你的双**声明说它应该是一个数组到指针BankTeller个对象。分配的值仍然被解释为一个地址(它不是)并最终尝试访问无效的内存位置,从而触发段错误。

相反它应该是*tellers = new BankTeller*[count];。注意开始括号前的*。这一行为您提供了一个指针BankTeller个对象的数组。

简单示例

为了说明,忘记BankTeller s并让我们回到基元。

#include <iostream>
using namespace std;

int main()
{
        const size_t max = 3;
        int **nums;

        cout << "Creating arrays...";
        nums = new int*[max];             // <<---- not: nums = new int[max];
        for(size_t i = 0; i < max; ++i)
                nums[i] = new int(i);
        cout << "done" << endl;

        cout << "Contents: ";
        for(size_t i = 0; i < max; ++i)
                cout << *nums[i] << ' ';  // <<---- dereferenced twice to reach actual value
        cout << endl;

        cout << "Deleting arrays...";
        for(size_t i = 0; i < max; ++i)
                delete nums[i];
        delete[] nums;
        cout << "done" << endl;
        return 0;
}

请注意,这与前面描述的情况相同。要运行它,请将代码放在名为test.cpp的文件中并使用此命令(GNU / Linux):

➜  /tmp  g++ test.cpp -o test && ./test
Creating arrays...done
Contents: 0 1 2
Deleting arrays...done
➜  /tmp

如果要在调试器中检查它,请将-ggdb添加到上面的g++命令,以确保将调试符号添加到二进制文件中。然后,您可以使用b <linenumber>(例如b 10)设置断点和p <variable_name>,(例如p numsp *nums等)来打印地址和值。

但同样,你不需要像这样使用原始指针。您可以而且应该使用标准模板库中的容器。

您的代码重构

我使用std::vector而不是双指点重写了下面的示例代码。

#include <iostream>
#include <vector>
using namespace std;

class BankTeller
{
public:
        BankTeller() {
                cout << "Created BankTeller\n";
        }
        ~BankTeller() {
                cout << "Destroyed BankTeller\n";
        }
};

class BankModel
{
public:
        BankModel(size_t count) {
                // remember to throw exception if count <= 0
                for(size_t i = 0; i < count; ++i)
                        _tellers.push_back(new BankTeller());
                cout << "Created BankModel\n";
        }

        ~BankModel() {
                // consider using iterators
                for(size_t i = 0; i < _tellers.size(); ++i) {
                        delete _tellers[i];
                        _tellers[i] = 0;
                }
                _tellers.clear();
                cout << "Destroyed BankModel\n";
        }

private:
        vector<BankTeller*> _tellers;
};

int main() {
        BankModel *model = new BankModel(5);
        delete model;
        return 0;
}

在我的系统(GNU / Linux)中构建和运行它如下所示:

➜  /tmp  g++ tellers.cpp -o tellers
➜  /tmp  ./tellers
Created BankTeller
Created BankTeller
Created BankTeller
Created BankTeller
Created BankTeller
Created BankModel
Destroyed BankTeller
Destroyed BankTeller
Destroyed BankTeller
Destroyed BankTeller
Destroyed BankTeller
Destroyed BankModel

希望这有助于您理解指针和使用STL的好处。

答案 1 :(得分:1)

您的tellers变量是指向指针的指针。所以当你这样做时

*tellers = new BankTeller[count]

您必须确保已将内存正确分配给BankTeller*

答案 2 :(得分:0)

欢迎使用C ++的动态数组!正确的写作方式是:

BankTeller* tellers = new BankTeller[count];  

您不需要指向此指针的指针。

然而,根本不要这样做!使用std::vector<BankTeller>

答案 3 :(得分:0)

试试这个 -

BankTeller **tellers;

tellers = new BankTeller* [count];

for(int i=0; i <= count; i++)
        {
            tellers[i] = new BankTeller();
        }

你可以在这里轻松看到差异。