[SOLUCIONADO] Duda programación en C

[GerardoVF78]

Buenas, estoy intentando hacer un código que pida por pantalla el nombre, la edad y el teléfono de una persona para añadirlo a una lista. El código es el siguiente:

Código c:

#include <stdio.h>
#include <stdlib.h>
#include <windows.h>
 
struct Datos{
    char *nombre;
    int edad;
    double telef;
    struct Datos *sig;
};
 
typedef struct Datos miembro;
 
miembro *CrearMiembro(miembro *Lista, double telefono, int anios, char *name){
    
    miembro *NuevoMiembro, *aux;
    NuevoMiembro = (miembro *) malloc(sizeof(miembro));
    
    if(NuevoMiembro != NULL){
        NuevoMiembro->edad = anios;
        NuevoMiembro->telef = telefono;
        NuevoMiembro->nombre = name;
        NuevoMiembro->sig = NULL;
    }
    
        if (Lista == NULL){
            Lista = NuevoMiembro;
        }
        else{
            aux = Lista;
            
            while(aux->sig != NULL){
                aux = aux->sig;
            }
            
            aux = NuevoMiembro;
        }
    
    return Lista;
}
 
void ImprimirLista(miembro *Lista){
    
    miembro *aux;
    
    aux = Lista;
    
    printf("La Lista contiene los siguientes datos: \n");
    
    while(aux!=NULL){
        
        printf("Edad: /%d Telefono: %d Nombre: %s -> \n", aux->edad, aux->telef, aux->nombre);
        aux = aux->sig;
        
    }
    
    printf("NULL");
}
 
int main(){
    
    miembro *Lista1;
    char *Nombre;
    int age;
    double teleph;
        
    Lista1->edad = 19;
    Lista1->telef = 657485748;
    Lista1->nombre = 'Alfonso';
    
    Lista1->sig = NULL;
    
    printf("Indique el nombre que quiere introducir: ");
    scanf("%s", Nombre);
    printf("\n");
    
    fflush(stdin);
    
    printf("Indique el telefono: ");
    scanf("%d", &teleph);
    printf("\n");
    
    printf("Indique la edad: ");
    scanf("%d", &age);
    
    Lista1 = CrearMiembro(Lista1, teleph, age, Nombre);
    
    ImprimirLista(Lista1);
    
    
    
    system("PAUSE");
}
 

No da ningún error de compilación, sin embargo, al ejecutarlo deja de funcianar y no sé por qué.

[eferion]

Código C:

int main()
{
    miembro *Lista1;
 
    // ...        
 
    Lista1->edad = 19;
    Lista1->telef = 657485748;
    Lista1->nombre = 'Alfonso';
}
 

Lista1 está declarada como puntero... pero no está incializada... es un puntero que apunta a una dirección de memoria aleatoria a la cual muy seguramente no tengas acceso.

Antes de escribir algo en Lista1 hay que inicializarla:

Código C:

miembro* Lista1 = (miembro*)malloc(sizeof(miembro));
 

Y, por supuesto, no has de olvidar que por cada malloc que hagas tienes que añadir un free para liberar la memoria... que estás finalizando el programa sin liberar la memoria que has reservado y eso no está bien visto en ninguna parte.

Un saludo

[GerardoVF78]

Gracias por tu comentario, ahora permite escribir los datos, sin embargo, en la propia ejecución, cuando va a mostrar los datos se "crashea" el programa, dejo el nuevo código:

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <windows.h>
 
struct Datos{
    char *nombre;
    int edad;
    double telef;
    struct Datos *sig;
};
 
typedef struct Datos miembro;
 
miembro *CrearMiembro(miembro *Lista, double telefono, int anios, char *name){
    
    miembro *NuevoMiembro, *aux;
    NuevoMiembro = (miembro *) malloc(sizeof(miembro));
    
    if(NuevoMiembro != NULL){
        NuevoMiembro->edad = anios;
        NuevoMiembro->telef = telefono;
        NuevoMiembro->nombre = name;
        NuevoMiembro->sig = NULL;
    }
    
        if (Lista == NULL){
            Lista = NuevoMiembro;
        }
        else{
            aux = Lista;
            
            while(aux->sig != NULL){
                aux = aux->sig;
            }
            
            aux = NuevoMiembro;
        }
    
    return Lista;
}
 
void ImprimirLista(miembro *Lista){
    
    miembro *aux;
    
    aux = Lista;
    
    printf("La Lista contiene los siguientes datos: \n");
    
    while(aux!=NULL){
        
        printf("Edad: /%d Telefono: %d Nombre: %s -> \n", aux->edad, aux->telef, aux->nombre);
        aux = aux->sig;
        
    }
    
    printf("NULL");
}
 
int main(){
    
    miembro* Lista1 = (miembro*)malloc(sizeof(miembro));
    
    char *Nombre;
    int age;
    double teleph;
    
    Lista1 = NULL;
    
    printf("Indique el nombre que quiere introducir: ");
    scanf("%s", &Nombre);
    printf("\n");
    
    fflush(stdin);
    
    printf("Indique el telefono: ");
    scanf("%d", &teleph);
    printf("\n");
    
    printf("Indique la edad: ");
    scanf("%d", &age);
    
    Lista1 = CrearMiembro(Lista1, teleph, age, Nombre);
    ImprimirLista(Lista1);
    
    free(Lista1);
    
    
    system("PAUSE");
}
 

[eferion]

Te estás haciendo un lío con los punteros:

Si haces:

Lista1 = malloc()

y después haces

Lista1 = CrearMiembro

¿qué sucede con la primera reserva de memoria? se pierde, dejando por el camino lo que se llaman fugas de memoria. El efecto de estas fugas de memoria es que la máquina cada vez va a tener menos memoria disponible, ya que no se libera, hasta que llega un punto en el que se le agota la memoria libre y algo acaba cascando.

Es decir, si llamas a CrearMiembro (que en tu primera versión no lo hacías hasta después de intentar acceder a los miembros de Lista1) no necesitas hacer malloc.

Además, en el main sigues teniendo punteros sin inicializar:

Código C:

char *Nombre;
 
// ...    
 
printf("Indique el nombre que quiere introducir: ");
scanf("%s", &Nombre);
printf("\n");
 

¿Dónde está la memoria reservada para almacenar la cadena de Nombre? En ningún sitio.

Puedes optar por calcular un tamaño lo suficientemente grande como para que entre cualquier nombre y evitar el uso de memoria dinámica:

Código C:

char Nombre[100];
 

Por otro lado, si analizamos la función CrearMiembro:

Código C:

miembro *CrearMiembro(miembro *Lista, double telefono, int anios, char *name)
{
    miembro *NuevoMiembro, *aux;
    NuevoMiembro = (miembro *) malloc(sizeof(miembro));
    
    if(NuevoMiembro != NULL){
        NuevoMiembro->edad = anios;
        NuevoMiembro->telef = telefono;
        NuevoMiembro->nombre = name; // <<--- 1
        NuevoMiembro->sig = NULL;
    }
    
    if (Lista == NULL){
        Lista = NuevoMiembro;
    }
    else
    {
        aux = Lista;
            
        while(aux->sig != NULL){
            aux = aux->sig;
        }
            
        aux = NuevoMiembro; // <<--- 2
    }
    
    return Lista;
}
 

1. Las cadenas de texto no se pueden copiar así... o bien declaras nombre como un array de tamaño fijo o bien haces una reserva dinámica de memoria para que entre la cadena que quieres copiar y, a continuación, tienes que usar strcmp para copiar una cadena en otra. Si haces una copia de punteros tienes que ser consciente de que la dirección de memoria apuntada es compartida y tienes que manejarla con cuidado.

2. ¿Aquí no tendría que ser aux->sig = NuevoMiembro?

Y, bueno, ya puestos, lo mejor sería que empezases a trastear con el depurador de código. Es una herramienta imprescindible en este mundillo y es mejor aprender a usarla con programas pequeños.

Un saludo