Foro de elhacker.net

al introducir nombre se me cierra el programa y no lo entiendo.

Seguramente te esté dando una violación de segmento por lo siguiente:

 
char *nombre;
char *tlf

constructor:

nombre="\0";
tlf="\0";
 

No les das tamaño.

Por favor akai dime que se supone que debo de poner.

o bien declaras nombre y tlf como

char nombre[20]
char tlf[10]

o algo así, y en el constructor no haces nada, o bien, dejas la declaración como puntero y en el constructor haces lo siguiente:

nombre=new char[20] //o tu tamaño
tlf=new char[10]//o tu tamaño

En principio, si luego vas a leer una cadena, no necesitas asignarles el \0.

Por otro lado, en C++, a menos que trabajes con la clase string, que no es lo mismo que trabajar con char[] o char* no puedes hacer que nombre="\0" o tlf="\0". Sino que has de asignar caracter a caracter o copiar con funciones. Además, la cadena vacía es "" no "\0".

ok gracias.

EDITO: Para no crear un post tonto...
Díganme si el uso que le he dado a un puntero que apunta a una clase en la función bubble está bien hecho o no se debería de hacer así por favor.

#include <iostream>
using namespace std;
class base {
    private:
        bool rellena;//0 no tiene contenido . 1 tiene contenido
        char *nombre;
        char *tlf;
    public:
        base();
        void del();
        void dnuevo();
        void editar();
        void view(int n);
        void freeram();
};
base::base(){
    rellena = false;
    nombre = new char[20];
    tlf = new char[20];
};
void base::del(){
};
void base::dnuevo(){
    cout << "\nIntroduce un nombre: " ;
    cin >> nombre;
    cout << "\nIntroduce un telefono: ";
    cin >> tlf;
    rellena=true;
};
void base::freeram(){
    delete[] nombre;
    delete[] tlf;
};
void base::editar(){
    dnuevo();
};
void base::view(int n){
    cout << "---------------------------------------------------------" << endl;
    cout << "Nombre: ";
    if(rellena==true)cout << nombre;
    cout << endl;
    cout << "Tlfn: ";
    if(rellena==true)cout << tlf ;
    cout << endl;
    cout << "---------------------------------------------------------" << endl;
    cout << "1-Nuevo\t2-Eliminar 3-Salir 4-Editar 8-Pagback 9-Pagnext\t Pag: " << n << endl;
};
void bubble(int n1,int n2,base b[]){
    base *p;
    for(int i=n1;i<n2;i++){
        p=&b[i+1];
        b[i]=*p;
    }
};
 
int main(int argc,char *argv[]){
    base b[10];
    int pag=1;
    int evento=0;
    int pagmax=0;
    while(1){
        b[pag].view(pag);
        cout << "Opcion: ";
        cin >> evento;
        if(evento==1){
            pagmax=pagmax+1;
            pag=pagmax;
            b[pag].dnuevo();
        }
        else if(evento==2){
            if(pag!=1){
                bubble(pag,pagmax,b);
                pagmax--;
            }
        }
        else if(evento==3)break;
        else if(evento==4)b[pag].editar();
        else if(evento==8 && pag>1)pag--;
        else if(evento==9 && pag < pagmax)pag++;
        evento=0;
        cout << "\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n" ;
    }
    for(int i=0;i<10;i++)b[i].freeram();
    cout << "El programa se cerrara" << endl;
    cin.get();
    return 0;
}

Saludos

Hola

Para lo que quieres hacer debes sobrecargar el operador =

base operator = (const base &b);
 

base base::operator = (const base &b)
{
    rellena = b.rellena;
    strcpy(nombre, b.nombre);
    strcpy(tlf, b.tlf);
}
 

De esta forma en la función bubble te quedaría

void bubble(int n1,int n2,base b[]){
    for(int i=n1;i<n2;i++)
        b[i]=b[i+1];
}
 

Saludos

y a que se debe que me de buen resultado?

Hola

Pues se debe a que le estás diciendo al compilador cómo asignar los objetos de esa clase. Date cuenta que cuando asignas enteros el compilador sabe qué tiene que hacer, pero cuando se encuentra con objetos de la clase "base" o de cualquier clase tienes que decírselo tú.

Saludos

Ose lo que debo de hacer es darle a cada miembro su valor uno a uno.
Esque pensaba que lo que hacia al hacerlo con la clase directa lo hacia todo.

Eso es lo primero que se piensa por la costumbre de la asignación de enteros y otros tipos simples pero con objetos "no funciona" porque no sabes qué va a hacer el compilador.

El problema de la forma en que tú lo has hecho es que no has reservado memoria para p entonces te peta, aparte de que el operador = no lo has definido.

Lee sobre la sobrecarga de operadores porque es muy útil, entre otras cosas para ordenar los elementos o compararlos con <, >, ==, != o imprimir con cout, etc

Saludos

En principio, la importancia de sobrecargar el operador de asignación o de realizar un constructor de copia en una clase viene por lo siguiente:

(Dependiendo del compilador, pero en principio)
Si no están definidos, se copia campo a campo.

Problema? Si tienes memoria dinámica, y copias campo a campo, tienes dos punteros apuntando al mismo sitio. La copia es borrada (por ejemplo, pasabas por copia en una función) y con ello se libera la memoria dinámica a la cual apuntaba. Ahora, el original tiene un campo que apunta a un espacio liberado, mala suerte, violación de segmento cuando accedas y fin del programa.

Solución? constructores de copia y sobrecarga de operadores de asignación, para en caso que tu clase trabaje con memoria dinámica, replicar dicha memoria de forma manual, y no copiando el puntero. (por ejemplo, volviendo a declarar un vector y copiando todos los elementos manualmente, en vez que el compilador simplemente te copie el puntero).