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

Segmentation fault when creating an array of objects

本文关键字：分段 错误 数组 创建对象      更新时间：2023-10-16

下面是我试图开始工作的代码。我想创建一个BankTeller对象数组。据我所知，当调用新的BankTeller()时，它会创建一个指向该对象的指针，我将该指针保存到teller中。然后我尝试插入到那个位置，但是当我使用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;
}

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

<标题>理解指针h1> 开始声明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 nums, p *nums等)来打印地址和值。

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

你的代码重构

我已经重写了下面的示例代码，使用std::vector代替双指针。

#include <iostream>
#include <vector>
using namespace std;
class BankTeller
{
public:
        BankTeller() {
                cout << "Created BankTellern";
        }
        ~BankTeller() {
                cout << "Destroyed BankTellern";
        }
};
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 BankModeln";
        }
        ~BankModel() {
                // consider using iterators
                for(size_t i = 0; i < _tellers.size(); ++i) {
                        delete _tellers[i];
                        _tellers[i] = 0;
                }
                _tellers.clear();
                cout << "Destroyed BankModeln";
        }
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的好处。

您的tellers变量是指向指针的指针。所以当你输入

*tellers = new BankTeller[count]

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

欢迎来到c++的动态数组!正确的写法是:

BankTeller* tellers = new BankTeller[count];

不需要指针指向指针。

但是，不要这样做!使用std::vector<BankTeller>

试试这个- 

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

相关文章：