Impossible d'accéder au pointeur de vidage

Question

Je crée une liste liée qui contient des noeuds avec le "dataItem" d'un pointeur vide. Le but de ceci est de permettre au nœud de contenir n'importe quel type de données. Cependant, je suis incapable d'accéder aux données du pointeur vide, même lorsque ces données sont castées au type correct.

Mon code ressemble à ceci:

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
typedef struct Student 
{ 
    char stuName[51]; 
    char stuMajor[5]; 
    double GPA; 
    char stuID[10]; 
}student; 

typedef struct Node 
{ 
    union{ 
     void *dataPtr; 
     int countr; 
    }dataItem; 
    int link; 
    struct Node* next; 
}node; 
void readData(struct Node *); 

void main(){ 
    node head; 
    node temp; 
    readData(&temp); 
    student *ptr = (student *)(temp.dataItem.dataPtr); 
    printf("%s %d", ptr->stuName, ptr->GPA);//breaks here because unable to access memory 
} 

void readData(struct Node *link) 
{ 
    link = (node *)malloc(sizeof(node)); 
    student *ptr = (student *)malloc(sizeof(struct Student)); 
    printf("enter the student name : "); 
    fflush(stdin); 
    scanf("%[^\n]", ptr->stuName); 
    printf("enter the student's major : "); 
    fflush(stdin); 
    scanf("%[^\n]", ptr->stuMajor); 
    printf("enter the student GPA : "); 
    scanf("%lf", &(ptr->GPA)); 
    printf("enter the student ID : "); 
    fflush(stdin); 
    scanf("%[^\n]", ptr->stuID); 
    link->dataItem.dataPtr = ptr; 
} 

Je sais que j'ai certainement un pointeur mal quelque part, je ne suis pas sûr comment. J'ai également le noeud dans ma fonction readData point à une nouvelle malloc de noeud parce que je veux un nouveau noeud chaque fois que readData est appelé quand j'implémente la liste liée plus loin.

Answer 1

Votre code est très cassé,

1. Vous n'avez pas inclus un fichier d'en-tête, vous avez besoin d'au moins stdlib.h pour malloc() et stdio.h pour printf() et scanf().

2. Votre définition main() est erronée, car main() doit renvoyer int.

3. Vous fflush(stdin) qui est un comportement indéfini.

4. Vous ignorez la valeur de retour de scanf().

5. Vous supposez que malloc() renvoie toujours un pointeur valide.

6. Vous avez appelé readData() qui n'est pas encore déclaré.

Mais le plus important erreur, est que vous avez passé l'adresse « node temp-readData() et vous malloc() Éd, mais vous ne retournez pas un pointeur vers elle, perdant ainsi toutes les modifications apportées à l'intérieur readData() qui sera ne fonctionne pas de toute façon car il n'est pas déclaré au moment où vous l'appelez.

je fixe votre code, parce que je sais que vous ne l'avez pas aimé ma réponse, mais vérifier les correctifs qui sont liés à la réponse, et maintenant il fonctionne comme je m'y attendais

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

typedef struct Student 
{ 
    char stuName[51]; 
    char stuMajor[5]; 
    double GPA; 
    char stuID[10]; 
} student; 

typedef struct Node 
{ 
    union{ 
     void *dataPtr; 
     int countr; 
    } dataItem; 
    int   link; 
    struct Node* next; 
} node; 

void readData(struct Node **link); 


int main() 
{ 
    node *head; 
    student *ptr; 

    readData(&head); 

    ptr = head->dataItem.dataPtr; 
    if (ptr != NULL) 
     printf("%s\t%g", ptr->stuName, ptr->GPA); 
    return 0; 
} 

void readData(struct Node **link) 
{ 
    student *ptr; 
    if (link == NULL) 
     return; 
    *link = malloc(sizeof(node)); 
    if (*link == NULL) 
     return; 
    memset(*link, 0, sizeof(node)); 

    ptr = malloc(sizeof(struct Student)); 
    if (ptr == NULL) 
     return; 
    printf("enter the student name : "); 
    if (scanf("%50[^\n]%*c", ptr->stuName) != 1) 
     ptr->stuName[0] = '\0'; 

    printf("enter the student's major : "); 
    if (scanf("%4[^\n]%*c", ptr->stuMajor) != 1) 
     ptr->stuMajor[0] = '\0'; 

    printf("enter the student GPA : "); 
    if (scanf("%lf%*c", &(ptr->GPA)) != 1) 
     ptr->GPA = 0; 

    printf("enter the student ID : "); 
    if (scanf("%9[^\n]%*c", ptr->stuID) != 1) 
     ptr->stuID[0] = 0; 

    (*link)->dataItem.dataPtr = ptr; 
} 

J'ai aussi ajouté certains correctifs de sécurité à scanf() ajoutant le modificateur de longueur pour empêcher le débordement de tampon, et également supprimer le '\n' arrière avec spécificateur "%*c", cela ne fonctionnera pas si plusieurs espaces suivent la valeur, mais vous pouvez le tester en attendant simplement en appuyant sur Entrée/Retour, si vous voulez une entrée plus sophistiquée, vous devez utiliser autre chose au lieu de scanf().

Answer 2

Ce:

void readData(struct Node *link) 
{ 
    link = (node *)malloc(sizeof(node)); 

Vous jetez la valeur du paramètre link qui a été passé, l'attribution à la place une nouvelle allocation par malloc. C'est votre problème le plus important, je pense. Une solution simple serait de supprimer la ligne effectuant le malloc.

Answer 3

Vous souhaitez transmettre un pointeur à un pointeur (ou l'adresse d'un pointeur) à un nœud de la fonction readData(). readData alloue un nouveau noeud et le remplit; ce qui se passe dans votre code est que readData() obtient un copy de l'adresse de temp, écrase la copie de cette adresse avec une nouvelle adresse, celle obtenue de malloc (qui est invisible pour le code appelant dans main()) puis remplit cet objet mallocé. Au retour de readData l'objet mallocé est inaccessible.

Qu'est-ce que vous vouliez faire est la suivante:

void main(){ 
    node head; 
    node *tempAddr; // pointer 
    readData(&tempAddr); // pass address of that pointer 

puis utilisez *tempAddr dans le principal lieu de temp. Se détendre se plaindrait que vous ne devriez pas lancer le résultat de malloc, mais je pense que c'est bien. Je ne me soucie pas non plus de la valeur de retour de main, ou de ses arguments.