値を代入しようとするとC++コードが壊れます

Question

私はC++を初めて使用しており、ダブルポインタを使用して多次元配列のコードを記述しようとしています。これは私のコードです：

クラス宣言：

class magicMat{ 

    private: 
     int** ptrnum; 

    public: 
     void init(int); 
     void Display(int); 
     void set(int); 
     void message(); 
     void errorhandling(int);  
}; 

関数定義：

void magicMat::init(int input) 
{  
    ptrnum=new int*[input]; 

    for (int row=0;row<input;row++) 
     ptrnum[row]=new int[input]; 

    for(int x=0;x<input;x++) 
    { 
     for (int y=0;y<input;y++) 
     { 
      *(ptrnum[x]+y)=0; 
     } 
    } 
} 

void magicMat::set(int input) 
{ 
    int row=1,col=input/2,otherdiag=0; 

    for(int value=1;value<=input*input;value++) 
    { 
     if (*(ptrnum[row]+col)>0) 
     { 
      row=row+2; 
      if(row>input) 
       row=row-input; 

      col--; 
      if(col<1) 
       col=input; 
     } 
     *(ptrnum[row]+col)+=value; 
     *(ptrnum[0]+col)+=value; 
     *(ptrnum[row]+0)+=value; 

     if (row==col) 
      *(ptrnum[0]+0)+=value;   

     if (row+col==input+1) 
      otherdiag+=value;     
/*                  */ 
/*  Determine where new row and col are        */ 
/*                  */ 
     row--; 
     if (row < 1)      /* If row exceeds side then */ 
      row = input;     /* goto other side.   */ 
     col++; 
     if (col > input)     /* If col exceeds side then */ 
      col = 1; 
    }  
} 

主な機能：

int main() 
{ 
    int num; 
    magicMat newMat; 
    newMat.message(); 
    while(1) 
    { 
     cin>>num; 
     if (cin.good()) 
     { 
      newMat.errorhandling(num); 
     } 
     else if (!isdigit(num)) 
     { 
      cout<<"Please enter only digits"<<endl; 
     }  
     newMat.init(num); 
     newMat.set(num); 
     newMat.Display(num); 
    } 
    cout<<"\nBye bye!\n"<<endl; 
    return 0; 
} 

それはinit機能で動作しますが、setで関数私はそれが最初に破る値をチェックしようとするsetデータ関数内のifステートメント。

Answer 1

あなたはあなたの配列の範囲外に足を踏み入れています。ただ、一例として、私はあなたのコードを実行し、set機能の終了時にこれらの行を見て、5からinputの値を設定し、私の最初の数字として5を入力した場合：の終わりに

row--; 
if (row < 1) 
    row = input; 

最初にset関数のループを通過すると、rowは1に等しくなります。これらの3行が実行され、結果としてrowは5に等しく設定されます。 setであなたの次のループの開始時に

、あなたがこれを行う：

if (*(ptrnum[row]+col)>0) 

問題はptrnumが有効なインデックスが0-4からある意味5×5の配列、ということです。しかし、前回のループの最後では、rowを5と設定したため、範囲外のインデックス番号がptrnumとなり、プログラムがクラッシュします。

デバッガでコードをステップ実行するか、実行方法がわからない場合は、少なくともcoutの文をset関数に入れて、変数が設定されていることを確認できるようにしてくださいあなたが考えるべきものに設定するべきです。

幸運を祈る！