Problema con la STL vector..

[gorystyle]

Hola tengo este código:

Código C++:

#ifndef PERSONA_H
#define PERSONA_H
 
 
class Persona
{
    public:
        Persona();
        virtual ~Persona();
        void setEdad(int);
        void setNombre(char*);
        int getEdad();
        char* getNombre();
    protected:
    private:
        int edad;
        char *nombre;
};
 
#endif // PERSONA_H
 

Código C++:

#include <iostream>
#include <cstring>
#include "persona.h"
 
Persona::Persona()
{
    //ctor
}
 
Persona::~Persona()
{
    //dtor
}
 
void Persona::setEdad(int e)
{
    edad = e;
}
 
void Persona::setNombre(char *nom)
{
    nombre = new char[strlen(nom)];
    strcpy(nombre,nom);
}
 
int Persona::getEdad()
{
    return edad;
}
 
char* Persona::getNombre()
{
    return nombre;
}
 

aqui la clase main

Código C++:

#include <iostream>
#include <vector>
#include "persona.h"
 
using namespace std;
 
int main()
{
    vector <Persona*> personas;
    char r;
    do
    {
        char *n;
        int e;
        Persona *p;
        p = new Persona;
 
        cout << "\nNombre: ";
        cin >> n;
        cout << "\nEdad: ";
        cin >> e;
        p->setEdad(e);
        p->setNombre(n);
        personas.push_back(p);
        cout << "\nContinuar? : ";
        cin >> r;
    }while(r=='s');
 
    cout << "\n\nMostrando.." << endl;
 
    for(int i=0; i<personas.size(); i++)
    {
        cout << "\nNombre: " << personas[i]->getNombre();
    }
 
    return 0;
}
 

bien el programa debe preguntar nombre y edad y crear una persona y almacenarlo en el vector personas hasta que el usuario diga que no quiere continuar.. y luego mostrar los nombres.. el programa lo hace muestra los nombres pero termina creo q abruptamente pq me sale un dialogo de error en windows que dice cerrar programa, buscar solucion en linea, depurar el programa.. pq será?? creo que es pq no libero la memoria que utilizo de ser esto cierto como lo hago?? gracias!

[alexg88]

Tienes que liberar la memoria de cada puntero que creas con new.

Tendrás que recorrer el vector y utilizar delete para destruir la memoria que habías reservado anteriormente con new.

Saludos.

[gorystyle]

Iniciado por alexg88

Tienes que liberar la memoria de cada puntero que creas con new.

Tendrás que recorrer el vector y utilizar delete para destruir la memoria que habías reservado anteriormente con new.

Saludos.

gracias por responder.. pero como haria eso? podrias decirme? no tengo nada de experiencia con liberar memoria jeje

[alexg88]

Pues es muy sencillo, cada vez que usas new estás reservando memoria para un dato y cuando quieras liberar esa memoria sólo tienes que usar delete.

En tu caso, los punteros que utilizas los guardas en el vector, por lo tanto, no tienes más que recorrer el vector e ir utilizando delete por cada uno de ellos.

Ejemplo:

Código C++:

for(int i=0; i<personas.size(); i++)
    {
       delete personas[i];
    }
 

Saludos.

[gorystyle]

Iniciado por alexg88

Pues es muy sencillo, cada vez que usas new estás reservando memoria para un dato y cuando quieras liberar esa memoria sólo tienes que usar delete.

En tu caso, los punteros que utilizas los guardas en el vector, por lo tanto, no tienes más que recorrer el vector e ir utilizando delete por cada uno de ellos.

Ejemplo:

Código C++:

for(int i=0; i<personas.size(); i++)
    {
       delete personas[i];
    }
 

Saludos.

hola gracias por responder... he colocado lo que me has dicho pero ahora es peor sale el error al ingresar el primer nombre, antes salia al final cuando mostraba los datos jeje

[sam90]

Para empezar hay un error.
Haces:
char * n;
cin >> n;

Pero nunca reservaste memoria para n. Yo declararia a n con memoria estatica.
No se si eso solucione tus problemas. Pero el error que tienes no se debe a liberar memoria. Esta puede molestar cuando utilizas demasiadas cosas. dos o tres llamados a new no haran colapsar tu programa.

Saludos

[gorystyle]

SOLUCIONADO...!

aqui el código completo:

Código C++:

#include <iostream>
#include <vector>
#include "persona.h"
 
using namespace std;
 
int main()
{
    vector <Persona*> personas;
    char r;
    do
    {
        char n[40];
        int e;
        Persona *p;
        p = new Persona;
 
        cout << "\nNombre: ";
        cin.getline(n,40);
        cout << "\nEdad: ";
        cin >> e;
        p->setEdad(e);
        p->setNombre(n);
        personas.push_back(p);
        cout << "\nContinuar? : ";
        cin >> r;
    }while(r=='s');
 
    cout << "\n\nMostrando.." << endl;
 
    for(int i=0; i<personas.size(); i++)
    {
        cout << "\nNombre: " << personas[i]->getNombre();
    }
 
    for(int i=0; i<personas.size(); i++)
    {
       delete personas[i];
    }
    return 0;
}
 

Código C++:

#include <iostream>
#include <cstring>
#include "persona.h"
 
Persona::Persona()
{
    //ctor
}
 
Persona::~Persona()
{
    //dtor
}
 
void Persona::setEdad(int e)
{
    edad = e;
}
 
void Persona::setNombre(char *nom)
{
    nombre = new char[strlen(nom)];
    strcpy(nombre,nom);
}
 
int Persona::getEdad()
{
    return edad;
}
 
char* Persona::getNombre()
{
    return nombre;
}
 

Código C++:

#ifndef PERSONA_H
#define PERSONA_H
 
 
class Persona
{
    public:
        Persona();
        virtual ~Persona();
        void setEdad(int);
        void setNombre(char*);
        int getEdad();
        char* getNombre();
    protected:
    private:
        int edad;
        char *nombre;
};
 
#endif // PERSONA_H