下面是我想要开始工作的代码。我想创建一个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;
}
答案 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 nums
,p *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();
}
你可以在这里轻松看到差异。