Autor Tema: Error en contructor de copia (Leído 3801 veces)

Checho360 · « **en:** Lunes 28 de Marzo de 2011, 21:39 »

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);
             ~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 elementosn";
                   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 vector1n";
    cin>>n;
    matrix A(n);
    A.scan_matrix();
    cout<<"Introduzca el orden del vector2n";
    cin>>m;
    matrix B(m);
    B.scan_matrix();
    
    matrix D = A + B;
    
    D.print_matrix ();
    
    system("pause");return 1;
}
 
 

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!!!

Checho360

Bueno me había confundido de código

m0skit0

Fíjate que el constructor de copia no recibe matrix A, sino matrix& A.

Checho360

Cita de: "m0skit0"

Fíjate que el constructor de copia no recibe matrix A, sino matrix& A.

Pero si no le pongo el & tampoco compila :S

m0skit0

Cita de: "Checho360"

Pero si no le pongo el & tampoco compila :S

Yo no he dicho que no le pongas el & querido amigo. Y si no dices qué errores te da, pues difícilmente podremos ayudarte.

Checho360

Aqui estan los errores:

Uploaded with ImageShack.us

Epa

Buenas.

El error no esta en el contructor copia (que de hecho ni lo usas) sino en el operador +

Este deberia ser:

matrix& matrix::operator+ (const matrix& A){

y la instancia de clase C

matrix* C= new matrix(A.size);

Ya que sino estaria devolviendo un objeto local a ese metodo.

Para usar el contructor copia tendiras que hacer

matrix C(A +

;

en lugar de

matrix C = A + B;

Saludos

Checho360

Hola Epa. Tienes razón en ese sitio ni siquiera utilizo el constructor copia, pero de todas formas, cuando quito las lineas 14 a 21 (declaración de constructor copia e implementación) el programa funciona perfectamente; por qué es eso??

Un saludo!!

Epa

La verdad me mataste jaja

No se tanto de C++ como de C.

Per curisamente si cambias el contructor copia por

matrix::matrix(const matrix& A)

Compila correctamente.

Quizas alguien que sepa mas de obejtos pueda dar una explicacion de esto

Saludos

Checho360

OK jajaja a ver entonces si alguien dice; de momento cambio lo que tu dices de const en el copia jaja

Un saludo y muchas gracias por contestar

Checho360

Bueno pues tienes razón Epa; modificando eso para ponerle un const compila y funciona perfectamente:

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 (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 elementosn";
                   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 vector1n";
    cin>>n;
    matrix A(n);
    A.scan_matrix();
    cout<<"Introduzca el orden del vector2n";
    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;
}
 
 
 

A ver si alguien puede decir por que pasa esto :S

Un saludo!!

m0skit0

El error que te da es porque no tienes constructor por defecto. Es decir, declaras

Código: C++

matrix D
 

pero esto no lo puedes hacer porque tu constructor pide un argumento.

Saludos.

Checho360

Cita de: "m0skit0"

El error que te da es porque no tienes constructor por defecto. Es decir, declaras

Código: C++
matrix D

pero esto no lo puedes hacer porque tu constructor pide un argumento.

Saludos.

Si pero de todas formas si fuera por eso no debería de funcionar al poner esto ? :

Código: C++

    matrix D(n);
    D = A + B;
 

Es que así tampoco funciona

m0skit0

Es que el constructor copia es matrix::matrix(const matrix& A) y no matrix::matrix(matrix& A) (léete esto de principio a fin).

Checho360

Cita de: "m0skit0"

Es que el constructor copia es matrix::matrix(const matrix& A) y no matrix::matrix(matrix& A) (léete esto de principio a fin).

Bueno pues así es entonces; esa es la forma de crear el constructor copia y lo tenía mal declarado...

Un saludo y muchas gracias a todos!

SoloCodigo

Autor Tema: Error en contructor de copia (Leído 3801 veces)

Checho360

Error en contructor de copia

Checho360

Re: Error en contructor de copia

m0skit0

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia

m0skit0

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia

Epa

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia

Epa

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia

m0skit0

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia

m0skit0

Re: Error en contructor de copia

Checho360

Re: Error en contructor de copia