Valeur de retour de vecteur problématique - pas vraiment mise à jour?

Question

Je vais avoir ce problème bizarre: quand mon programme atteint cette méthode:

//Returns the transpose matrix of this one 
RegMatrix RegMatrix::transpose() const{ 
    RegMatrix result(numCol,numRow); 
    int i,j; 
    for(i=0;i<numRow;++i) 
     for(j=0;j<numCol;++j){ 
      result._matrix[j][i] = _matrix[i][j]; 
     } 

     return result; 
} 

il bloque tout à coup ...

Quand je l'ai couru avec mon débogueur VS, il semblait que tout va bien, la nouvelle matrice a été rempli avec les valeurs pertinentes, jusqu'à la ligne return result; qui, pour une raison mystérieuse a retourné un vecteur de matrice vide.

Où est-ce que je vais mal ??

---

Voici mon implémentation pour le constructeur de copie:

//CCtor of RegMatrix      
RegMatrix::RegMatrix(const RegMatrix &other): numRow(other.getRow()), numCol(other.getCol()){ 

    //Create 
    _matrix = vector<vector<MyDouble> >(other.getRow()); 
    int i,j; 
    for(i=0; i < numRow; i++) 
     _matrix[i] = vector<MyDouble>(other.getCol()); 

    //Copy Matrix 
    for(i=0;i<numRow; ++i){ 
     for(j=0;j<numCol; ++j){ 
      _matrix[i][j] = other._matrix[i][j]; 
     } 
    } 
} 

---

Mon opérateur d'affectation mise en œuvre:

//RegMatrix = RegMatrix 
RegMatrix& RegMatrix::operator=(const RegMatrix rhs){ 
    assert(numRow == rhs.getRow() && numCol == rhs.getCol()); 
    if(*this != rhs){ 
     int i,j; 
     for(i=0;i<numRow;++i) 
      for(j=0;j<numCol;++j){ 
       _matrix[i][j] = rhs._matrix[i][j]; 
      } 
    } 

    return *this; 
} 

Answer 1

En supposant que MyDouble a un constructeur de copie correcte, vous devriez être en mesure de réduire votre constructeur de copie à ceci:

RegMatrix::RegMatrix(const RegMatrix &other):numRow(other.getRow()), numCol(other.getCol()), 
    _matrix(other._matrix) 
{ } 

Voyez ce qui vous fait.

Éditer: Votre opérateur d'affectation peut poser un problème si les colonnes et les lignes ne sont pas égales. Vous lancez une affirmation dans ce cas, donc le programme va avorter. Est-ce que c'est ce que tu veux? Ne préféreriez-vous pas que l'affectation modifie les colonnes et les lignes pour qu'elles correspondent aux nouvelles valeurs? Si oui, vous pouvez simplement faire ceci:

RegMatrix & RegMatrix::operator=(const RegMatrix & rhs) { 
    if(this == &rhs) 
     return *this; 
    numRow = rhs.getRow(); 
    numCol = rhs.getCol(); 
    _matrix = rhs._matrix; 
    return *this; 
} 

Answer 2

Vous retournez la matrice par valeur. Un constructeur de copie s'implique. Comment votre constructeur de copie est-il défini?