Problema con sobrecarga operadores +=

[fingeroso]

Buenas a todos, soy nuevo y tengo un problema con un programa que no consigo completar.
Se trata de un agenda de contactos, donde debo guardar , mostrar y buscar. Todo eso practicamente está hecho pero a la hora de guardar un nuevo contacto debo hacerlo de una forma especifica para cumplir con los objetivos de la práctica.

El problema es que tengo que añadir un contacto nuevo haciendo miagenda+=micontacto en la linea 248, como veis tengo que emplear la sobrecarga de operadores y lo máximo que he conseguido es haciendo miagenda++. No consigo aclararme muy bien con este tema de sobrecarga de operadores y atributos dinamicos. A ver si podéis echarme una mano. Gracias

Código C++:

#include <iostream>
#include <conio.h>
#include <stdlib.h>
#include <fstream>
 
using namespace std;
 
 
class fecha{
 
private:
    int dd,mm,aa;
    bool valido;
public:
 
        // solicitamos al usuario los datos de fecha
    friend istream& operator >> (istream &medio, fecha &f2){
        do{
            cout << "Introduce el dia: "; medio >> f2.dd; medio.ignore();
            cout << "Introduce el mes: "; medio >> f2.mm; medio.ignore();
            cout << "Introduce el anyo: "; medio >> f2.aa; medio.ignore();
            f2.valido=f2.comprobar();
        }while (f2.valido==false);
        return(medio);
    }
 
    //mostramos al usuario los datos de fecha
    friend ostream& operator << (ostream &medio, fecha f2){
        medio << f2.dd << "/" << f2.mm << "/" << f2.aa << endl;
        return(medio);
    }
 
 
    //comprobamos si la fecha es correcta
    bool comprobar (){
        valido=true;
        if (dd<1 || dd>31) valido=false;
        if (mm<1 || mm>12) valido=false;
        if (aa<1950 || aa>2011) valido=false;
 
        return (valido);
    }
 
};
 
class direccion{
 
private:
    string calle, localidad;
    int numero;
public:
 
    //solicitamos al usuario los datos de direccion
    friend istream& operator >> (istream &medio, direccion &d2){
        cout << "Introduce la calle: ";getline (medio,d2.calle);
        cout << "Introuce el numero: "; medio >> d2.numero; medio.ignore();
        cout << "Introduce la localidad: "; getline (medio,d2.localidad);
        return(medio);
    }
 
        //mostramos al usuario los datos de direccion
    friend ostream& operator << (ostream &medio, direccion d2){
        cout << "Calle: "; medio << d2.calle << endl;
        cout << "Numero: "; medio << d2.numero << endl;
        cout << "Localidad: "; medio << d2.localidad << endl;
        return(medio);
    }
 
};
 
class contacto{
    friend class agenda;
    friend class menu;
private:
    fecha f1;
    direccion d1;
    string nombre, apellidos;
 
public:
 
    //solicitamos los datos del contacto
    friend istream& operator >> (istream &medio, contacto &c1){
        system("cls");
        cout <<"Introduce el nombre: "; getline (medio,c1.nombre);
        cout << "Introduce los apellidos: "; getline (medio,c1.apellidos);
        medio >> c1.f1;
        medio >> c1.d1;
        return(medio);
    }
 
    //mostramos los datos del contacto
    friend ostream& operator << (ostream &medio, contacto c1){
        cout << "Nombre: "; medio << c1.nombre<< endl;
        cout << "Apellidos: "; medio << c1.apellidos<< endl;
        medio << c1.f1;
        medio << c1.d1;
        cout << endl;
 
    return(medio);
 }
};
 
class agenda{
private:
    string buscarnombre;
    contacto *lista;
    bool centinela;
    int n;
public:
 
    //constructor de agenda
    agenda (){
        getch();
        system ("cls");
        cout << "constructor agenda" << endl;
        getch();
        system ("cls");
        if (n<0){
        n=0;
        lista = NULL;
    }
        centinela=true;
        return;
    }
 
    //destructor de agenda
    ~agenda (){
        getch();
        system ("cls");
        cout << "destructor de agenda" << endl;
        getch();
        system ("cls");
        if (lista!=NULL)
        delete []lista;
 
        return;
    }
 
    //constructor de copia de agenda
    agenda (const agenda &copialista){
        getch();
        system ("cls");
        cout << "constructor de copia" << endl;
        getch();
        system ("cls");
        n=copialista.n;
        lista= new contacto[n];
        for (int i=0; i<n; i++){
            lista[i]=copialista.lista[i];
        }
        return;
    }
 
 
 
    //añadir un contacto a la agenda  void operator+= (agenda micontacto1)
    void operator++(int){
        // creamos un objeto vector auxiliar de la clase contacto y le asignamos un tamaño mayor que el actual
        // de la lista de contactos
 
        contacto *aux=new contacto [n+1];
 
        //copiamos el contenido de la lista de contactos en el vector auxiliar
        for (int i=0;i<n;i++){
            aux[i]=lista[i];
        }
        // borramos el vector de la lista de contactos
        delete []lista;
        //igualamos ambos vectores
        lista=aux;
        // metemos el nuevo contacto en eñ vector de la lista de contactos
        cout << "operador de carga" << endl;
        getch();
 
        cin >> lista[n];
        //incrementamos el numero de la lista de contactos para su proxima ejecucion
        n++;
 
        return;
    }
 
 
 
    //mostrar todos los contactos de la agenda
    friend ostream& operator << (ostream &medio, agenda ag1){
 
        if (ag1.n==0){
            cout << "La lista de contactos esta vacia" << endl;
        }else{
 
        for (int i=0;i<ag1.n;i++)cout << ag1.lista[i];
        }
        cout << endl << "Presiona una tecla para volver al menu";
        getch();
        return(medio);
    }
 
 
    // buscar un contacto en la agenda
    void buscar (void){
        system ("cls");
        if (lista!=NULL){
            cout << "Introduce el nombre del contacto: ";
            getline (cin,buscarnombre);
            for (int i=0;i<n;i++){
                if (lista[i].nombre==buscarnombre){
                    cout << endl;
                    cout << lista[i];
                    centinela=false;
                }
            }
            if (centinela==true){
                cout << endl << "No existe ningun contacto con ese nombre";
            }
        }
        cout << endl << "Presiona una tecla para volver al menu";
        getch();
        return;
    }
};
 
class menu{
 
private:
    agenda miagenda;
    int eleccion;
public:
    int solicitar (void){
        do{
            system("cls");
            cout << "1- Nuevo contacto" << endl;
            cout << "2- Listar contactos" << endl;
            cout << "3- Buscar contacto" << endl;
            cout << "4- Salir" << endl << endl;
            cout << "Introduce tu eleccion: ";
            cin >> eleccion; cin.ignore();
            system ("cls");
        }while (eleccion<1 || eleccion>4);
        return (eleccion);
    }
 
    void seleccion (void){
 
        do{
            eleccion=solicitar();
            switch(eleccion){
            case 1:
                miagenda++;
                break;
            case 2:
                cout << miagenda;
                break;
            case 3:
                miagenda.buscar();
                break;
            }
        }while (eleccion!=4);
    }
};
 
int main (void){
    menu m1;
    m1.seleccion();
    getch();
    return(0);
}
 

[Gott]

no vi en ninguna parte del programa que hagas la sobrecarga del operador ++. Si pudiste hacer eso no creo que te sea dificil hacer la sobrecarga del operador +=.

[fingeroso]

Iniciado por Gott

no vi en ninguna parte del programa que hagas la sobrecarga del operador ++. Si pudiste hacer eso no creo que te sea dificil hacer la sobrecarga del operador +=.

Hice una cutre sobrecarga del operador ++ pero no consigo hacer la +=. No se si necesito crear operador suma y asignacion para poder hacer e +=. Ayuda por favor.

[SDEK]

¿miagenda += miagenda; ?? ¿ Eso quieres hacer?

[fingeroso]

Iniciado por SDEK

¿miagenda += miagenda; ?? ¿ Eso quieres hacer?

nop, quiero hacer miagenda+=micontacto;

Algo he conseguido pero ahora no me guarda en la lista de contactos, empiezo a volverme loco.
Por cierto voy a editar el highlight de arriba para dejarlo igual que este pero no me deja.

Aclaro, la instruccion miagenda+=micontacto esta en la linea 275. Los operadores para la sobrecargar empiezan en la linea 156. Se que tiene algo que ver con el bloque de instrucciones que contienen pero en serio... mi cabeza no da para más. Como veis lo he seguido intentando por mi mismo y algo he conseguido. Si pudierais ayudarme os lo agradeceria muchisimo. No pido que lo completeis (que no estaria mal jeje) pero por favor necesito consejo y ayuda.

Código C++:

#include <iostream>
#include <conio.h>
#include <stdlib.h>
#include <fstream>
 
using namespace std;
 
 
class fecha{
    friend class agenda;
private:
    int dd,mm,aa;
    bool valido;
public:
 
    // solicitamos al usuario los datos de fecha
    friend istream& operator >> (istream &medio, fecha &f2){
        do{
            cout << "Introduce el dia: "; medio >> f2.dd; medio.ignore();
            cout << "Introduce el mes: "; medio >> f2.mm; medio.ignore();
            cout << "Introduce el anyo: "; medio >> f2.aa; medio.ignore();
            f2.valido=f2.comprobar();
        }while (f2.valido==false);
        return(medio);
    }
 
    //mostramos al usuario los datos de fecha
    friend ostream& operator << (ostream &medio, fecha f2){
        medio << f2.dd << "/" << f2.mm << "/" << f2.aa << endl;
        return(medio);
    }
 
 
    //comprobamos si la fecha es correcta
    bool comprobar (){
        valido=true;
        if (dd<1 || dd>31) valido=false;
        if (mm<1 || mm>12) valido=false;
        if (aa<1950 || aa>2011) valido=false;
 
        return (valido);
    }
 
};
 
class direccion{
    friend class agenda;
private:
    string calle, localidad;
    int numero;
public:
 
    //solicitamos al usuario los datos de direccion
    friend istream& operator >> (istream &medio, direccion &d2){
        cout << "Introduce la calle: ";getline (medio,d2.calle);
        cout << "Introuce el numero: "; medio >> d2.numero; medio.ignore();
        cout << "Introduce la localidad: "; getline (medio,d2.localidad);
        return(medio);
    }
 
    //mostramos al usuario los datos de direccion
    friend ostream& operator << (ostream &medio, direccion d2){
        cout << "Calle: "; medio << d2.calle << endl;
        cout << "Numero: "; medio << d2.numero << endl;
        cout << "Localidad: "; medio << d2.localidad << endl;
        return(medio);
    }
 
};
 
class contacto{
    friend class agenda;
private:
    fecha f1;
    direccion d1;
    string nombre, apellidos;
 
public:
 
    //solicitamos los datos del contacto
    friend istream& operator >> (istream &medio, contacto &c1){
        system("cls");
        cout <<"Introduce el nombre: "; getline (medio,c1.nombre);
        cout << "Introduce los apellidos: "; getline (medio,c1.apellidos);
        medio >> c1.f1;
        medio >> c1.d1;
        return(medio);
    }
 
    //mostramos los datos del contacto
    friend ostream& operator << (ostream &medio, contacto c1){
        cout << "Nombre: "; medio << c1.nombre<< endl;
        cout << "Apellidos: "; medio << c1.apellidos<< endl;
        medio << c1.f1;
        medio << c1.d1;
        cout << endl;
 
        return(medio);
    }
};
 
class agenda{
private:
    string buscarnombre;
 
    contacto *lista;
    bool centinela;
    int n;
public:
 
    //constructor de agenda
    agenda (){
        getch();
        system ("cls");
        cout << "constructor agenda" << endl;
        getch();
        system ("cls");
        n=0;
        lista = NULL;
        centinela=true;
        return;
    }
    agenda (contacto micontacto){
        lista = new contacto[n+1];
        lista[n]=micontacto;
        return;
    }
 
    //destructor de agenda
    ~agenda (){
        getch();
        system ("cls");
        cout << "destructor de agenda" << endl;
        getch();
        system ("cls");
        if (lista!=NULL)
            delete []lista;
        return;
    }
 
    //constructor de copia de agenda
    agenda (const agenda &copialista){
        getch();
        system ("cls");
        cout << "constructor de copia" << endl;
        getch();
        system ("cls");
        n=copialista.n;
        lista= new contacto[n+1];
        for (int i=0; i<n; i++){
            lista[i]=copialista.lista[i];
        }
        return;
    }  
 
    agenda operator+(agenda ag1){
        agenda resul;
        resul.lista= new contacto [n+1];
        for (int i=0;i<n;i++){
            resul.lista[i]=ag1.lista[i];
        }
        return (resul);
    }
    agenda operator+=(contacto micontacto){
        agenda resul,ag1(*this),ag2(micontacto);
        resul=ag1+ag2;
        return(resul);
    }
 
    agenda operator=(agenda ag1){
        ag1.n=n;
        for (int i=0;i<ag1.n;i++){
            lista[i]=ag1.lista[i];
            n++;
        }
        return (*this);
    }
 
 
 
    //mostrar todos los contactos de la agenda
    friend ostream& operator << (ostream &medio, agenda ag1){
 
        if (ag1.n==0){
            cout << "La lista de contactos esta vacia" << endl;
        }else{
 
            for (int i=0;i<ag1.n;i++)cout << ag1.lista[i];
        }
        cout << endl << "Presiona una tecla para volver al menu";
        getch();
        return(medio);
    }
    void ordenar (void){
        string compara1, compara2,comparanom1, comparanom2;
 
        contacto aux;
        for (int i=0;i<n-1;i++){
 
            for (int j=i+1;j<n;j++){
                compara1=lista[i].apellidos;
                compara2=lista[j].apellidos;
 
                if (compara1>compara2){
                    aux=lista[i];
                    lista[i]=lista[j];
                    lista[j]=aux;
                }
                if (compara1==compara2){
                    comparanom1=lista[i].nombre;
                    comparanom2=lista[j].nombre;
                    if (comparanom1>comparanom2){
                        aux=lista[i];
                        lista[i]=lista[j];
                        lista[j]=aux;
                    }
                }
            }
        }
        return;
    }
 
    // buscar un contacto en la agenda
    void buscar (void){
        system ("cls");
        if (lista!=NULL){
            cout << "Introduce el nombre del contacto: ";
            getline (cin,buscarnombre);
            for (int i=0;i<n;i++){
                if (lista[i].nombre==buscarnombre){
                    cout << endl;
                    cout << lista[i];
                    centinela=false;
                }
            }
            if (centinela==true){
                cout << endl << "No existe ningun contacto con ese nombre";
            }
        }
        cout << endl << "Presiona una tecla para volver al menu";
        getch();
        return;
    }
};
 
class menu{
 
private:
    agenda miagenda;
    contacto micontacto;
    int eleccion;
public:
    int solicitar (void){
        do{
            system("cls");
            cout << "1- Nuevo contacto" << endl;
            cout << "2- Listar contactos" << endl;
            cout << "3- Buscar contacto" << endl;
            cout << "4- Salir" << endl << endl;
            cout << "Introduce tu eleccion: ";
            cin >> eleccion; cin.ignore();
            system ("cls");
        }while (eleccion<1 || eleccion>4);
        return (eleccion);
    }
 
    void seleccion (void){
 
        do{
            eleccion=solicitar();
            miagenda.ordenar();
            switch(eleccion){
            case 1:
                cin >> micontacto;
                miagenda+=micontacto;
                break;
            case 2:
                cout << miagenda;
                break;
            case 3:
                miagenda.buscar();
                break;
            }
        }while (eleccion!=4);
    }
};
 
int main (void){
    menu m1;
    m1.seleccion();
    getch();
    return(0);
}
 

[SDEK]

Aquí viene claramente como sobrecargar el operador de asignación en C++:

http://es.wikibooks.org/wiki/Programaci%C3%B3n_en_C%2B%2B/Sobrecarga_de_Operadores

Saludos,

[fingeroso]

Ya lo habia visto este fin de semana pasado, pero tanto como claramente.... jajajaja. Mañana seguiré intentandolo. La cosa esta bastante cerca de funcionar pero sigo sin verlo claro con la mierda de atributos dinamicos. Por lo menos ahora consigo compilarlo sin errores. Es decir que la sobrecarga de operadores funciona, el problema viene con el bloque de instrucciones de cada sobrecarga de operador.

Nuevamente, gracias por las respuestas.

[SDEK]

Me comprometo a revisar tu código mañana para apoyarte y explicarte. Por ahora no puedo, (trabajo) pero en el transcurso de mañana lo haré .... saludos!

[fingeroso]

Iniciado por SDEK

Me comprometo a revisar tu código mañana para apoyarte y explicarte. Por ahora no puedo, (trabajo) pero en el transcurso de mañana lo haré .... saludos!

Mil millones de gracias por mostrar interes en ayudarme. De cualquier forma, parece que ya he conseguido entender más o menos esto de la sobrecarga de operadores. . Ahora solo me queda que me cargue los datos desde un *.txt , guardarlos me los guarda pero cargarlos me trae de cabeza también.

Esperare al viernes, que tengo que presentar la práctica para colgar el codigo (ahora sí 100% funcional), el motivo no es otro que ningun compañero de clase lo copie y se lleve el los mismos meritos que yo.

De manera que INFORMO QUE YA LO HE SOLUCIONADO, gracias a todos y EL VIERNES LO COLGARE TERMINADO

[SDEK]

Que bueno, esperaremos a ver como lo solucionaste.

Saludos,

[fingeroso]

Iniciado por SDEK

Que bueno, esperaremos a ver como lo solucionaste.

Saludos,

pues, la solucion era usar solamente el operador autosuma += pasandole como parametro micontacto y no devolver nada en el return.

Me he pasado el fin de semana comiendome la cabeza para resultar ser algo mucho mas sencillo.

Ahora solo me queda cargar desde el constructor todas las variables que guardo una vez se ejecuta mi destructor.

Repito, el código lo cuelgo el viernes a la noche o sábado por la mañana. Lo colgaría antes pero no quiero arriesgarme a que algun compañero busque en google miagenda+=micontacto esperando tener suerte y se encuentre con todo el trabajo ya hecho.

[dragg04]

a ver si esto es lo que querias hacer

Código C++:

#include "stdafx.h"
#include <vector>
#include <string>
#include <iostream>
 
using namespace std; 
 
class CContacto
{
    string _nombre;
 
public:
    void Nombre(const string& nombre)
    {
        _nombre= nombre;
    }
 
    const string &Nombre()
    {
        return _nombre;
    }
 
};
 
class CAgenda
{
    vector<CContacto> _contactos;
 
public:
 
    CAgenda &operator +=(const CContacto& contactos )
    {
        _contactos.push_back(contactos);
        return *this;
    }
 
    CContacto& operator[](int pos)
    {
        return _contactos[pos];
    }
};
 
int _tmain(int argc, _TCHAR* argv[])
{
 
    CContacto contacto;
    CAgenda agenda;
    contacto.Nombre("Mi nombre");
 
    agenda+=contacto;
 
    cout<<"Nombre del contacto"<< agenda[0].Nombre();
 
    return 0;
}
 

[fingeroso]

Iniciado por dragg04

a ver si esto es lo que querias hacer

Código C++:

#include "stdafx.h"
#include <vector>
#include <string>
#include <iostream>
 
using namespace std; 
 
class CContacto
{
    string _nombre;
 
public:
    void Nombre(const string& nombre)
    {
        _nombre= nombre;
    }
 
    const string &Nombre()
    {
        return _nombre;
    }
 
};
 
class CAgenda
{
    vector<CContacto> _contactos;
 
public:
 
    CAgenda &operator +=(const CContacto& contactos )
    {
        _contactos.push_back(contactos);
        return *this;
    }
 
    CContacto& operator[](int pos)
    {
        return _contactos[pos];
    }
};
 
int _tmain(int argc, _TCHAR* argv[])
{
 
    CContacto contacto;
    CAgenda agenda;
    contacto.Nombre("Mi nombre");
 
    agenda+=contacto;
 
    cout<<"Nombre del contacto"<< agenda[0].Nombre();
 
    return 0;
}
 

Hay muchas cosas que todavia no entiendo. Pero ya lo he conseguido solucionar. Ahora tengo otro problema y es a la hora de cargar. De cualquier modo, gracias por tu respuesta.

Código C++:

void cargar (void){
        ifstream fichero ("datos.txt");
        if(fichero.good()==true){
            while(fichero.eof()==false){
                fichero >> n;
                fichero.ignore();
                for (int i=0; i<n;i++){
                    fichero >> lista[i].nombre;
                    fichero >> lista[i].apellidos;
                    fichero >> lista[i].f1.dd;
                    fichero >> lista[i].f1.mm;
                    fichero >> lista[i].f1.aa;
                    fichero >> lista[i].d1.calle;
                    fichero >> lista[i].d1.numero;
                    fichero >> lista[i].d1.localidad;
                }
            }
        }
        fichero.close();
        return;
    }
 

Como veis tengo que cargar ciertas variables que son de tipo string sin embargo cuando lo cargo de esta forma me da fallos.

[dragg04]

podrias mostrar como tienes estructurado tu archivo datos.txt, en segunda lista tiene el espacio sufiente para almacenar todos los registos?

[fingeroso]

Código C++:

void guardar (void){
        ofstream fichero ("datos.txt");
        fichero << n << endl;
        for (int i=0;i<n;i++){
            fichero << lista[i].nombre << endl;
            fichero << lista[i].apellidos << endl;
            fichero << lista[i].f1.dd << endl;
            fichero << lista[i].f1.mm << endl;
            fichero << lista[i].f1.aa << endl;
            fichero << lista[i].d1.calle << endl;
            fichero << lista[i].d1.numero << endl;
            fichero << lista[i].d1.localidad << endl;
        }
        fichero.close();
        return;
    }
 

Así es como guardo la lista de contactos en el txt.

[fingeroso]

jajajajaja que imbecil que soy pero un gran imbecil. Ahora ya esta todo solucionado y el programa cumple su cometido al 100%. Asi que nada, gracias a todos pero al final me las he apañado para conseguirlo. Mañana cuelgo el codigo.

[fingeroso]

vale, el problema al guardar los datos en el txt era tan simple como que ejecutaba un salto de linea al agregar el ultimo contacto. Este salto de linea era necesario en todos los contactos excepto en el ultimo. LA solucion era crear un condicional para que ejecute el salto de linea siempre que no sea el ultimo contacto. Despues todo carga y guarda correctamente. La practica completada al 100%.

El codigo he decidido no colgarlo porque parece ser que algun compañero ha usado mi trabajo para hacer el suyo. En cualquier caso, estoy guardando todos los trabajos y cuando termine el curso los subire a esta web para compartirlos con vosotros.