C++ core dumped のエラーの改善方法

Question

1. 前提、実現させたいこと

add 命令でsizeを超えたときにsizeを二倍にした配列を確保しそこにデータをコピーする。
その後元のデータを削除しdataに新しい配列を登録するというものです

2,発生している問題

Aborted(core dumped) というエラーが出てしまう

3, 該当のソースコード

#include<iostream>
using namespace std;
class DoubleArray{
public:
int size;
double *data;
int pos;
DoubleArray(){
size = 8;
pos = 0;
data = new double[size];
}
DoubleArray(int sz){
size = sz;
pos = 0;
data = new double[sz];
}
void add(double d){
if(pos < size){
data[pos] = d;
pos++;
}else{
double *newdata;
newdata = new double[size * 2];
for(int i = 0; i < pos; i++){
newdata[i] = data[i];
}
delete[] data;
data = newdata;
data[pos] = d;
pos++;
}
}
void print(){
cout << data[0];
for(int i = 1; i < pos; i++){
cout << ", " << data[i];
}
cout << endl;
}
DoubleArray(const DoubleArray& orig){
pos = orig.pos;
size = orig.size;
data = new double[size];
for(int i = 0; i < pos; i++){
data[i] = orig.data[i];
}
}
~DoubleArray(){
delete[] data;
}
};
int main(){
DoubleArray d1;
cout << d1.size << "," << d1.pos << endl;
for(int i = 0; i < 20; i++){
d1.add(i);
}
cout << d1.size << "," << d1.pos << endl;
DoubleArray d2 = d1;
d2.add(100.0);
cout << d2.size << "," << d2.pos << endl;
d1.print();
d2.print();
}

Comments

[episteme]

これコンパイル通ります? newdataが未定義なんだけど...

Answer 1

• DoubleArray::add() で newdata が未宣言。コンパイルできないはず
• DoubleArray::add() で data に対して delete[] ではなく delete が使われている
• DoubleArray コピーコンストラクタで、this->size が未初期化で参照されている

追記:

• DoubleArray::add() でバッファが足りないときに size を更新していないため、pos が16以上になると範囲外アクセスを起こす

Comments

[episteme]

+ 要素数0のとき、print() は正しく動かない

[fere]

回答を参考に修正したのですが結局同じエラーが出てしまいます。
sizeを二倍にしたnewdataを作るところの書き方が悪いのかなとも思うのですがどう書けばいいのかよくわからないです...

[int32_t]

> 回答を参考に修正したのですが結局同じエラーが出てしまいます。

修正が間違っているかもしれないので、修正後のコードを質問文に追記してください。

今更ですけども、「コードのどこで core dump するのか」を調べるのが先決ですね。コンパイルするときに -g フラグをつけて、デバッガ上で実行してみてください。

[fere]

本文を編集しているのでそちらを見てくださると幸いです
cygwin上で画像のように打ったのですが特に何も出てこないのですが何か間違っているのでしょうか...?

[int32_t]

gdb q2_1.exe で起動して、
run で実行、停止したら bt するとよいでしょう。

残りのバグについては回答に追記します。

[fere]

解決しました。
ご丁寧にありがとうございました。