Question

功能如下：

Set::Set(Multinumber* tempArray[], int tempSize)
{
 numElements = tempSize;
 capacity = tempSize*2;
 setArray = new Multinumber*[capacity];
 for (int i=0; i<numElements; i++)
 {
  addElement(tempArray[i]);
 }
}

变量setArray在我的标题中声明为Multinumber **

类型

每当我用它调用它时会出现段错误：

Multinumber* carr[2];
carr[0]=c4;
carr[1]=c5;
Set setb(carr,2);

c4和c5已经被声明为指向正确类型的对象的指针。

非常感谢任何帮助。

编辑：下面的代码是addElement函数（缩进道歉）

const Set Set::operator+(const Set& rhs) const
{
 Set result;
 int i=0, j=0;

 while ((i < numElements) && (j < rhs.numElements))
 {
  Multinumber* toadd=new Multinumber;
  toadd=*(setArray[i]) + *(rhs.setArray[j]);
  result.addElement(toadd);
  i++;
  j++;
 }

 while ((i < numElements))
 {
  result.addElement(setArray[i]);
  i++;
 } 


 while ((j < rhs.numElements))
 {
 result.addElement(rhs.setArray[j]);
 j++;
 }


 return result;
}

编辑：基于许多cout语句，错误似乎出现在这个函数中：

 bool Set::isFull()
 {
  return (numElements == capacity);
 }

编辑：更改了数组索引，但仍然是段错误

Answer 1

数组使用从零开始的索引，因此将carr[2]设置为两长度数组中的任何内容都是未定义的行为。你应该感激这是一个段错误。： - ）

尝试：

Multinumber* carr[2];
carr[0]=c4;
carr[1]=c5;
Set setb(carr,2);

那应该照顾段错误。

Answer 2

carr[1]=c4;
carr[2]=c5;

不应该是

carr[0]=c4;
carr[1]=c5;

一条建议：如果你在调试器中加载它，比如gdb，它就会识别出罪魁祸首，你很快就会看到你的错误。

为什么这个函数是segfaulting？

2 个答案: