Segfaultにオブジェクトを追加するとき<vector>

Question

2次元配列を行列として使用し、boost :: ublas :: matrixを使用するという2つの異なる方法でコードを書きました。このオブジェクトを最初のケースに追加するときは機能していますが、2番目のケースではセグメンテーション・フォルトが発生します。私は第二の方法を使用したいので、なぜ誰かがsegfaultを取得していることを知っていれば、私は感謝します。

コード：

img.h

#include <Magick++.h> 
#include <string> 
#include <boost/numeric/ublas/matrix.hpp> 
#include <boost/numeric/ublas/io.hpp> 

using namespace boost::numeric::ublas; 
using namespace std; 
using namespace Magick; 

class Img 
{ 
    public: 
     Img(); 
     Img(const string path2file); 

     unsigned int width, height; 
     string filename; 
    private: 
     typedef struct pix 
     { 
      Quantum R; 
      Quantum G; 
      Quantum B; 
     } pix; 

     matrix<pix> p; 

     pix **pixels; 
    string format; 
}; 

img.cpp

Img::Img(const string path2file) 
{ 
Image file; 
unsigned int i, j; 
Color pixel; 

file.read(path2file); 

filename = path2file; 
width = file.size().width(); 
height = file.size().height(); 

// begin of first way 
pixels = (pix**)malloc(sizeof(pix*)*height); 
for(i=0 ; i<height ; ++i) 
    pixels[i] = (pix*)malloc(sizeof(pix)*width); 

for(i=0 ; i<height ; ++i) 
{ 
    for(j=0 ; j<width ; ++j) 
    { 
    pixel = file.pixelColor(j, i); 

    pixels[i][j].R = pixel.redQuantum(); 
    pixels[i][j].G = pixel.greenQuantum(); 
    pixels[i][j].B = pixel.blueQuantum(); 
    } 
} 
// end of first way 

// begin of second way 
p.resize(height, width); 
for(i=0 ; i<height ; ++i) 
{ 
    for(j=0 ; j<width ; ++j) 
    { 
    pixel = file.pixelColor(j, i); 

    p(i, j).R = pixel.redQuantum(); 
    p(i, j).G = pixel.greenQuantum(); 
    p(i, j).B = pixel.blueQuantum(); 
    } 
}*/ 
} 
// end of second way 

私は、このコードは、セグメンテーション違反の原因ではないことを確信しています。 しかし、私は、私は（最初に働いている、唯一の第二の方法のための）セグメンテーションフォールトを取得していますメインプログラムで使用する場合：

main.cppに

#include <iostream> 
#include <stdio.h> 
#include "img.h" 
#include <vector> 

using namespace std; 

int main(void) 
{ 
std::vector<Img> files; 
files.push_back(Img("files/mini.bmp")); 
return 0; 
} 

Answer 1

ロードプログラムをGDBにして作る、それがクラッシュ。 gdbコンソールbtまたはバックトレースに入力すると、すべての呼び出しのスタックフレームが取得され、segfaultの原因を確認できます。

Answer 2

valgrindの下でプログラムを実行すると、segfaultの原因と場所を見つけるのに役立ちます。

Answer 3

pixels = (pix**)malloc(sizeof(pix*)*height); 

これは奇妙な、malloc関数は、ポインタに（無効）のポインタではなく、ポインタを返しますが、それは私が推測[] []配列のために働く可能性がありそうです。再校正した後、OKと思われます。p

「Img」クラスと「file」という名前の変数を使用してインスタンス化する別のImageの両方を混乱させる可能性があります。

一部のコードも欠落しています（たとえば、 'p'は何ですか）が表示されますが、それは問題ありません。もし私があなただったら、プログラムがクラッシュした場所を狭めるためにprintf（）をここにかけることにしました。 私の最善の策は、p（i、j）の索引付けが範囲外である可能性があるということです。

幸運を祈る！

Answer 4

問題は私が書いたように定義されていました。 は次のようになります。

vector<Img*> files; 

代わりの

vector<Img> files; 

と

Img* tmp = new Img("path_to_file"); 

に

Img tmp("path_to_file"); 

からオブジェクトを初期化する変更私は一日中コーディングの非常に疲れています、だから私は嘲笑をする多くの間違い。

ありがとうございました！