Error en contructor de copia

[Checho360]

Bueno, tengo el siguiente codigo:

Código C++:

#include <iostream>
using namespace std;
#define MAX_SIZE 10
 
class matrix {
      
      public:
             
      int size; double s[MAX_SIZE];
      
      matrix (int n){
             
             if (n > MAX_SIZE) {
                   cerr<<"error, demasiado grande"; return;
                   }
             size = n;
             }
      matrix operator = (matrix A);
      
      ~matrix () {}
      
      matrix scan_matrix ();
      
      matrix suma_matrix (matrix A); 
      
      };
      
      matrix matrix :: scan_matrix (){
             for(int i=0;i<size;i++){
                     cout<<"Introduzca el valor "<<i<<" de la matriz\n";cin>>s[i];}
                     }
      matrix matrix :: suma_matrix(matrix A){
             if(size!=A.size){cerr<<"Las matrices no pueden sumarse";}
             matrix C(A.size);
             for (int i=0;i<size;i++){
                 C.s[i]=A.s[i]+s[i];
                 cout<<A.s[i]<<"+"<<s[i]<<"="<<C.s[i]<<"\n";
                 }return C;}
                 
      matrix matrix :: operator = (matrix A){
             matrix temp (A.size);
             for (int i=0;i<A.size;i++){
                 temp.s[i]=A.s[i];
                 }
                 return temp;
                 }
      
      int main (int n, int m){
          cout<<"Introduzca el orden del vector 1\n";cin>>n;
          matrix A(n);
          A.scan_matrix();
          cout<<"Introduzca el orden del vector 2\n";cin>>m;
          matrix B(m);
          B.scan_matrix();
          matrix C=A.suma_matrix(B);
          for(int i=0;i<n;i++){cout<<C.s[i]<<" ";}
          system ("pause");
}
 

Sé que el error está en alguna parte del constructor de copia por que el programa compila y funciona si le quito esa redefinición, sin embargo no encuentro el error. A ver si alguien me puede decir donde está el error y ya de paso por qué hay que tener escrito el operador '&' en la declaración.

Un saludo y muchas gracias de antemano!!!

[Heimish2000]

El error puede estar en la declaración de temp, ya que deberías crear el objeto con un new, para lo que tienes que definir el constructor de copia (ahora, aunque digas que lo tienes, lo que tienes es el operador de asignacion)

matrix temp = new matrix(A.size);

Por otra parte, no deberías declarar los atributos como públicos (eso no dará errores de compilación ni ejecución, pero no está bien hacerlo)

[Checho360]

Iniciado por Heimish2000

El error puede estar en la declaración de temp, ya que deberías crear el objeto con un new, para lo que tienes que definir el constructor de copia (ahora, aunque digas que lo tienes, lo que tienes es el operador de asignacion)

matrix temp = new matrix(A.size);

Por otra parte, no deberías declarar los atributos como públicos (eso no dará errores de compilación ni ejecución, pero no está bien hacerlo)

Ostia tienes toda la razón; no es que no tenga definido el constructor copia es que me había confundido de código jajajja

Aqui esta el codigo de verdad:

Código C++:

#include <iostream>
using namespace std;
#define MAX_SIZE 10
 
class matrix{
      public:
             int size; double s[MAX_SIZE];
             matrix (int n);
             ~matrix() {}
             int test_matrix (int a);
             matrix scan_matrix (void);
             matrix operator + (matrix A);
             void print_matrix (void);
             matrix (matrix& A);//<---------------Aqui esta la declaracion
};
 
matrix :: matrix (matrix& A){ //<-------------Aqui esta la implementación
       size = A.size;
       for (int i=0;i<A.size;i++) s[i]=A.s[i];
       }
 
matrix :: matrix (int n){
       size = test_matrix(n);
       }
 
int matrix :: test_matrix (int a){
    if(a>MAX_SIZE){
                   cerr<<"Demasiados elementos\n";
                   a=MAX_SIZE;
                   }
    return a;
}
 
matrix matrix :: scan_matrix (void){
       for(int i=0;i<size;i++){
       cout<<"Ingresa el coef "<<i<<"\n";cin>>s[i];
       }
}
 
matrix matrix :: operator + (matrix A){
     if(A.size!=size){
                      cerr<<"Los vectores han de ser del mismo orden";
                      }
     matrix C (A.size);
     for(int i=0;i<A.size;i++){
            C.s[i]=s[i]+A.s[i];}
     return C;
     }
 
void matrix :: print_matrix (void){
     for(int i= 0;i<size;i++){
             cout<<"Suma no "<<i<<" = "<<s[i]<<"\n";
             }
     }
 
int main(){
    int n;int m;
    cout<<"Introduzca el orden del vector1\n";
    cin>>n;
    matrix A(n);
    A.scan_matrix();
    cout<<"Introduzca el orden del vector2\n";
    cin>>m;
    matrix B(m);
    B.scan_matrix();
   
    matrix D = A + B;
   
    D.print_matrix ();
   
    system("pause");return 1;
}
 

Un saludo!!!

[Heimish2000]

¿Que error te da? ¿Has probado a poner me.size y me.s[i]? (Creo que es me, ahora programo en Java y en Java es this, pero en C creo recordar que es me)

Sigo diciendo que los atributos los deberías poner en la parte privada de la clase, pero ese es otro tema.

[Checho360]

Aqui estan los errores:

http://img6.imageshack.us/i/sinttulom.gif/

[Heimish2000]

Te lo pone bien claro...

Tu, en tu constructor por copia, estás pasando la referencia a un objeto de tipo matrix (por eso el &), pero en esa linea no estás pasando la referencia, estás pasando el objeto. Lo puedes ver porque si te fijas, el operador + devuelve un matrix, no un matrix& y por eso te da error.

Soluciones: Que acepte un objeto y no su referencia o pasarle al constructor una referencia y no un objeto.

[Checho360]

Iniciado por Heimish2000

Te lo pone bien claro...

Tu, en tu constructor por copia, estás pasando la referencia a un objeto de tipo matrix (por eso el &), pero en esa linea no estás pasando la referencia, estás pasando el objeto. Lo puedes ver porque si te fijas, el operador + devuelve un matrix, no un matrix& y por eso te da error.

Soluciones: Que acepte un objeto y no su referencia o pasarle al constructor una referencia y no un objeto.

Lo consegui añadiendo un const al constructor copia y funciona:

Código C++:

#include <iostream>
using namespace std;
#define MAX_SIZE 10
 
class matrix{
      public:
             int size; double s[MAX_SIZE];
             matrix (int n);
             ~matrix() {}
             int test_matrix (int a);
             matrix scan_matrix (void);
             matrix operator + (const matrix A);
             void print_matrix (void);
             matrix (const matrix& A);//<---------------Aqui esta la declaracion
};
 
matrix::matrix(const matrix& A){ //<-------------Aqui esta la implementación
       size = A.size;
       for (int i=0;i<A.size;i++) s[i]=A.s[i];
       }
 
matrix :: matrix (int n){
       size = test_matrix(n);
       }
 
int matrix :: test_matrix (int a){
    if(a>MAX_SIZE){
                   cerr<<"Demasiados elementos\n";
                   a=MAX_SIZE;
                   }
    return a;
}
 
matrix matrix :: scan_matrix (void){
       for(int i=0;i<size;i++){
       cout<<"Ingresa el coef "<<i<<"\n";cin>>s[i];
       }
}
 
matrix matrix :: operator + (const matrix A){
     if(A.size!=size){
                      cerr<<"Los vectores han de ser del mismo orden";
                      }
     matrix C (A.size);
     for(int i=0;i<A.size;i++){
            C.s[i]=s[i]+A.s[i];}
     return C;
     }
 
void matrix :: print_matrix (void){
     for(int i= 0;i<size;i++){
             cout<<"Suma no "<<i<<" = "<<s[i]<<"\n";
             }
     }
 
int main(){
    int n;int m;
    cout<<"Introduzca el orden del vector1\n";
    cin>>n;
    matrix A(n);
    A.scan_matrix();
    cout<<"Introduzca el orden del vector2\n";
    cin>>m;
    matrix B(m);
    B.scan_matrix();
   
    matrix D = A + B;
    matrix E(A+B);
   
    D.print_matrix ();
    E.print_matrix ();
   
    system("pause");return 1;
}
 

Me podrias escribir el codigo con tus soluciones??

Un saludo!!

[Aemilius]

¿Por qué hay que tener escrito el operador '&' en la declaración.?

Para evitar un ciclo infinito. Si el constructor copia recibe el parámetro por valor, ¿adivinas a que función se va a llamar para COPIAR el valor de la variable al parámetro? ¡Al constructor copia!

¿y el const?

Si no aumentas const el constructor copia estará restringido a funcionar sólo si recibe un objeto no constante, es decir que puede ser modificado internamente. Como ya te habrás dado cuenta tal limitación implica que no puedas escribir "matrix D = A + B;" o "matrix E(A+B);" por que internamente el compilador necesita que tu constructor copia pueda recibir como parámetro objetos constantes.

[Checho360]

Muchas gracias Aemilius!!