演算子のオーバーロード：マルチプル化行列

Question

私は、コンストラクタを使って動的にそれらを割り当てるマトリックスと、値を設定するためのistream演算子のためにいくつかのオーバーロード演算子関数を設定しようとしています。私は2つの行列（ランタイムエラー）を乗算しようとすると問題が発生しました。

ご協力いただければ幸いです。 ps：私はテンプレートに精通していません。ここで

は私の*演算子関数です：

matrixType operator*(matrixType m) 
{ 
    if(Rows==m.Cols && Cols==m.Rows) 
    { 
     matrixType m3(Rows,m.Cols); 
     for(int i=0; i<Rows;i++) 
     { 
      for(int j=0; j<Cols; j++) 
      { 
       { 
        for(int g=0; g<Cols;g++) 
         m3.matrix[i][j]+=matrix[i][g] * m.matrix[g][j]; 
       } 
      } 
     } 
     return m3; 
    } 
} 

そして、ここでは私のコピー演算子関数です：

matrixType& operator=(matrixType& m) 
{ 
    for (int i =0; i<Rows; i++) 
    { 
     for (int j=0; j<Cols; j++) 
       matrix[i][j]=m.matrix[i][j]; 
    } 

    return *this; 
} 

Answer 1

私はあなたには、いくつかのランタイムエラーを投稿する場合は、コードを実行したときに、それが良いことだと思います。とにかく、私はあなたの乗算演算子の機能に問題が見つかりました：あなたは立方メートルを返すとき、それは別のスコープで定義されている

、したがってそれは、定義されていない、どのようにかの範囲内でreturn立方メートル文を入れについて：

matrixType operator*(matrixType m) 
{ 
    if(Rows==m.Cols && Cols==m.Rows) 
    { 
     matrixType m3(Rows,m.Cols); 
     for(int i=0; i<Rows;i++) 
     { 
      for(int j=0; j<Cols; j++){ 
        {for(int g=0; g<Cols;g++) 
        m3.matrix[i][j]+=matrix[i][g] * m.matrix[g][j];} 
     } 
     return m3; 
    } 
     // Throw some error if your assertion is not satisfied, perhaps 
    } 
} 

Answer 2

お客様のjループに間違ったColsが使用されています。

for (for(int j=0; j<m.Cols; j++) 

どこiを見て、j、およびg変数が限界値は、3つのループのためのものであるかを確認する添字に使用されます：あなたはそこにm.Colsをしたいです。