Tema: Ayuda C++ Overload operator.

Ya se que estaba mal




Amigos,

Me he topado con un problema que no logro entender, espero me puedan ayudar.

Estoy realizando un API para el manejo de matrices, y tengo el problema de Sobrecargar el operador de asignación '='

La clase se llama 'Matriz' y el operador esta declarado de la siguiente manera:

Código:

Matrix operator = ( Matrix rightM );

Esta declaración funciona cuando asigno explícitamente.

Código:

Matrix obj1, obj2;

obj1.llenarMatriz();

obj2 = obj1;

El código anterior funciona. Pero cuando tengo una función que regresa un objeto Matriz, he intento asignarla a un objeto, no funciona, es decir la matriz se llena de ceros en este caso, el objeto es destruido antes de llegar a la funcion operator =.

Código:

const Matrix invertir (  );

...
...
...


obj2 = obj1.invertir();

Ahora lo voy a detallar un poco más.

La clase matriz utiliza el template <vector> para crear un arreglo bi-dimensional.

Código:

vector < vector< float > > _matrix;

cuando llamo al operador de asignación, esta función debe redimensionar la matriz (filas, columnas) para que se ajuste a los nuevos datos que debe contener.

A continuación una versión muy resumida de la clase matriz.

Código:

class Matrix{
public:


/// Crea una nueva Matrix de dimensiones [ Row x Cols ]
Matrix( int Rows, int Cols )
{
    _matrix.resize(_nRows);
    for( int i = 0; i < _nRows; i++ )
        _matrix[i].resize(_nCols);
}

Código:

Matrix operator = ( Matrix rightM ){

     _nRows = rightM._nRows;
     _nCols = rightM._nCols;

     _matrix.resize( rightM._nRows );

     for( int i = 0; i < rightM._nRows; i++ )
         _matrix [i].resize(rightM._nCols);


     for( int i = 0; i < _nRows; i++ )
         for( int j = 0; j < _nCols; j++ )
             _matrix[i][j] = rightM._matrix[i][j];

      return *this;
 }

static Matrix DotProduct( Matrix &leftM, Matrix &rightM )
{
   if( leftM.getCols() == rightM.getRows() )
    {
        Matrix resultM ( leftM.getRows(), rightM.getCols() );


        float partial = 0;


        for( int i = 1; i <= resultM.getRows(); i++ )
        {
            for( int j = 1; j <= resultM.getCols(); j++ )
            {
                for( int m = 1; m <= rightM.getRows(); m++ )
              
                    partial += leftM.getNumber( i, m ) * rightM.getNumber( m, j );
                    
                resultM.add( i, j, partial );
            }
            partial = 0;
        }

        return resultM;
}

private:


    /** 2-D Vector Array*/
    vector < vector<float> > _matrix;


    /** Number of Rows in Matrix*/
    int _nRows;


    /**Number of Columns in Matrix*/
    int _nCols;


};

¿Alguna de idea de que puede estar mal? O de como se sobrecarga el operador de manera correcta? Porque de acuerdo a las pruebas que he realizado el problema esta en la función sobrecargada del operador. He intentado hacer lo mismo con clases más sencillas y funcionan bien siempre y cuando no sobrecargue el operador, pero cuando lo sobrecargo, no admite objetos que retornen de Funciones. Es decir una función que retorne un objeto de la misma clase no queda asignado. Es como si no 'viviera' lo suficiente para llegar a la función '='.

buenas:

he estado repasando un poco lo de la sobrecarga, y en un par de ejemplos que he visto, siempre ponen en la definición de la sobrecarga, el tipo de objeto devuelto ACABADO con &.

Si fuera esto, tendrias que poner en tu sobrecarga:

Matrix& operator = (Matrix rightM){}

Esto tendria sentido, porque asi devuelves en el return con un *.

Te pongo el enlace de donde he estado viendo el tema: Programación en C++/Sobrecarga de Operadores - Wikilibros

Opino lo mismo que jakala, es más incluso te recomendaría que le pases la matriz por referencia en el parámetro 'rightM', ya que al pasar el parámetro tal cual estas haciendo otra copia.

En definitiva, lo ideal seria hacer que la declaración quede de la siguiente forma :

Código:

Matrix &operator = (const Matrix &rightM);

Y esto requerirá que tengas un constructor copia habilitado con la siguiente declaración :

Código:

Matrix(const Matrix &Copy);

Saludos

al fin encontre el BUG!!!

tienen razón con lo de la referencia, es para ahorrar memoria y evitar una llamada al constructor. Pero como no me salia y no sabia porque empeze a jugar con la referencia '&' a ver si a la fuerza se arreglaba jeje...


Tuve que leerme algunos libros y nada al final encontre mi respuesta aqui, pero que bien que logre resolver este problema....

"Existen tres casos generales en los cuales se llama al constructor de copia, en lugar del operador de asiganación:

1. Cuando al instanciar un objeto e iniliazarlo con valores de otro objeto:
2. Cuando se pasa el objeto por valor.
3. Cuando el objeto es retornado de una función por valor."

Ese era mi problema. Debido a que uso el Template ( vector < type > ) cuando llamo al Constructor de copia mi objeto matriz sólo quedaba definido su tamaño, el constructor no copiaba lo que contenía la matriz.

Esto es lo que tenia para el constructor de copia:

/// Defines the same parameters as 'base' Matrix
Matrix::Matrix(const Matrix& base)
{

_nCols = base._nCols;
_nRows = base._nRows;
_matrix.resize(_nRows);
for( int i = 0; i < _nRows; i++ )

_matrix[i].resize(_nCols);

}

Luego lo le agregue que ademas copie los elementos de la matriz.

/// Defines the same parameters as 'base' Matrix
Matrix::Matrix(const Matrix& base)
{

_nCols = base._nCols;
_nRows = base._nRows;
_matrix.resize(_nRows);
for( int i = 0; i < _nRows; i++ )

_matrix[i].resize(_nCols);

for( int i = 0; i < _nRows; i++ )

for( int j = 0; j < _nCols; j++ )

_matrix[i][j] = base._matrix[i][j];

}

Y eso fue todo, problema solucionado.... ahora si podre dedicarme a escribir la clase para las operaciones de la matriz!