liste Linken dans les problèmes Visual Studio 2010

Question

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

typedef struct node { 
    int value; 
    struct node *next; 
}LLIST; 



LLIST *list_add(LLIST **p, int i) 
{ 
    if (p == NULL) 
     return NULL; 
    LLIST *first = malloc(sizeof(LLIST)); 
    if (first == NULL) 
     return NULL; 
    first->value = *first; 
    *p = first; 
    first->value = i; 
} 


int main (int argc, char** argv) { 
    int i=0; 

LLIST *first = NULL; 
list_add(&first, 0); 


    return (EXIT_SUCCESS); 
} 

me génère des erreurs comme

IntelliSense: une valeur de type "void *" ne peuvent pas être utilisés pour initialiser une entité de type "LLIST *"

à la ligne malloc dans list_add pouvez-vous m'aider ??? Quand je tape le code, aucune erreur n'apparaît. IntelliSense m'aide à construire le code du nœud ... mais quand vous compilez cela, cela ... pouvez-vous m'aider à résoudre le problème?

Answer 1

Vous devez jeter le résultat de malloc:

LLIST *first = (LLIST*)malloc(sizeof(LLIST)); 

modifier:

Vous devez déclarer "d'abord" avant toute autre chose:

LLIST *list_add(LLIST **p, int i) 
{ 
    LLIST* first; 
    if (p == NULL) 
     return NULL; 
    first = malloc(sizeof(LLIST)); 
    if (first == NULL) 
     return NULL; 
    // first->value = *first; 
    *p = first; 
    first->value = i; 
    return first; 
} 

Answer 2

compilez-vous à ce que C code? En C++, il n'y a pas de conversion d'un void * vers d'autres types de pointeurs. Vérifiez que votre fichier a une extension .c.

En outre, vous avez une erreur ici:

first->value = *first; 

On peut imaginer, que ce soit vous ou le compilateur sont getting confus par cela.

Answer 3

Bien sûr, cela provoquera une erreur. Comme Neil a répondu à C++ il n'y a pas de conversion d'un void * à d'autres types de pointeurs.

Si vous voulez le faire en C++ utiliser les éléments suivants:

LLIST *list_add(LLIST **p, int i) 
{ 
    if (p == NULL) 
     return NULL; 
    LLIST *first = (LLIST*)malloc(sizeof(LLIST)); 
    if (first == NULL) 
     return NULL; 
    //first->value = *first; 
    *p = first; 
    first->value = i; 
} 

Et s'il vous plaît reconsidérer l'affectation

first->value = *first; 

dans votre code