[débutant] Erreur à la compilation - listes chaînées

Bonjour,

Je me remets doucement au C et aux listes chaînées, et j'ai qqs petits soucis. En effet, j'ai défini, dans un fichier .h, la structure des éléments de ma liste chaînée :

Code:

---

1 2 3 4 5

struct element { char nom[MAX_STR]; int profession; struct element* suivant; } *Element;

---

ainsi que tous les prototypes des fonctions que je vais utiliser dans mon programme.
Ensuite, dans un fichier .c je mets le code des fonctions ... et c'est quand je compile que j'ai des erreurs du style :

Citation:

---

46 D:\Mes Documents\home\tp3.c request for member `nom' in something not a structure or union

---

Je n'ai pourtant po oublier d'inclure dans mon .c le .h en y mettant :

Code:

---

#include "tp3.h"

---

Si qqn aurait une solution, qu'il me fasse signe !!!

Et si tu nous montrez la ligne 46 de tp3.c et meme la fonction qui contient cette ligne par la meme occasion.

Il eût été plus facile de t'aider avec la ligne 46...

Ne serait-ce pas un problème d'utilisation du point au lieu d'une flèche, ou vice-versa ?

Oupsss, toutes mes excuses ...

Code:

---

1 2 3 4 5 6 7 8

struct element* NouveauNoeud(char* nouvNom, int nouvProfession){ struct element* nouveau; nouveau = (struct element*) malloc(sizeof(struct element)); nouveau.nom = nouvNom; // la fameuse ligne 46 !!! nouveau.profession = nouvProfession; nouveau.suivant = NIL; return(nouveau); }

---

Starb : en effet, si je change les points en flêches, cela me fait moins d'erreurs mais j'ai une erreur qui persiste à la ligne 46 :

Citation:

---

46 D:\Mes Documents\home\tp3.c incompatible types in assignment

---

Il faut savoir que ce que tu fais à la ligne 46 te fait perdre la mémoire allouée par

Code:

---

1 2	char nom[MAX_STR];

---

Mieux vaut déclarer

Code:

---

1 2	char *nom;

---

De plus, avec la ligne 46, tu peux par exemple allouer 10 caractères à nouvNom alors que MAX_STR = 16. Comme l'utilisateur de la structure va se base sur le .h, il pensera qu'il y a toujours 16 éléments dans nom, ce qui n'est pas le cas si tu suis mon exemple. Je pense que l'erreur est une protection du compilateur pour empêcher ce genre d'erreurs. Tu ne l'auras plus en déclarant nom comme un pointeur.

Citation:

---

Envoyé par LaseLiep

Code:

---

1 2 3 4 5 6 7 8

struct element* NouveauNoeud(char* nouvNom, int nouvProfession){ struct element* nouveau; nouveau = (struct element*) malloc(sizeof(struct element)); nouveau.nom = nouvNom; // la fameuse ligne 46 !!! nouveau.profession = nouvProfession; nouveau.suivant = NIL; return(nouveau); }

---

---

C'est pas la peine de compliquer inutilement le code...

Code:

---

struct element* nouveau = malloc(sizeof*nouveau);

---

Par contre malloc() peut échouer et il manque donc un test...

Code:

---

1 2	if (nouveau != NULL) {

---

ensuite, c'est pas '.' puisque nouveau est un pointeur, mais '->' :

Code:

---

nouveau->nom = nouvNom; // la fameuse ligne 46 !!!

---

mais évidemment, ça ne fonctionne pas, car 'nouveau->nom' est un tableau de char. Utilise strcpy() si tu es sûr de la taille. Sinon strncat() ... sans te planter...

En effet, en déclarant en char *nom; je n'ai plus l'erreur, mais je ne comprends po trop : si je déclare ma variable nom comme tel, je déclare un pointeur sur un tableau de caractères, mais il faut bien que je dise combien de caractères se trouvent dans ce tableau, non? C'est pour cela que je dois mettre char nom[16]; par exemple, non?

Citation:

---

Envoyé par Strab

Il faut savoir que ce que tu fais à la ligne 46 te fait perdre la mémoire allouée par

Code:

---

1 2	char nom[MAX_STR];

---

Mieux vaut déclarer

Code:

---

1 2	char *nom;

---

---

Pas forcément. On peut avoir besoin d'une copie si la source est statique et donc unique...

Citation:

---

Envoyé par LaseLiep

En effet, en déclarant en char *nom; je n'ai plus l'erreur, mais je ne comprends po trop

---

Ben oui. Le but n'est pas, en soi, de ne plus avoir d'erreur, mais d'écrire du code correct. Je rappelle qu'un pointeur est une variable qui contient une adresse. On peut récupérer l'adresse d'une chaine de caractères si sa durée de vie est garantie, sinon, il peut y avoir des problèmes si
- l'adresse est celle d'une variable locale qui a disparu après l'appel d'une fonction
- l'adresse d'une variable statique unique d'une fonction, qui est ecrasée à chaque appel de cette fonction.

Citation:

---

: si je déclare ma variable nom comme tel, je déclare un pointeur sur un tableau de caractères, mais il faut bien que je dise combien de caractères se trouvent dans ce tableau, non? C'est pour cela que je dois mettre char nom[16]; par exemple, non?

---

Pour les raisons évoquées ci-dessus, la copie peut être indispensable. Mais pas forcément dans un tableau de char. Ca peut être aussi une copie 'dynamique' dont on récupère l'adresse dans un pointeur. (avec strdup(), par exemple, qui n'est pas standard C, mais POSIX, donc très portable.)

Evidemment, comme avec toute allocation dynamique, il faut ensuite libérer le bloc avec free() quand on en a plus besoin.

Quelle est la différence entre le point et la flêche : element.nom et element->nom ???

Citation:

---

Envoyé par LaseLiep

Quelle est la différence entre le point et la flêche : element.nom et element->nom ???

---

Qu'en pense ton livre de C ? On ne répond pas aux questions aussi basiques.

Citation:

---

Envoyé par Emmanuel Delahaye

Qu'en pense ton livre de C ? On ne répond pas aux questions aussi basiques.

---

Rahhh, j'ai po de livre(s) de C, mais j'ai des profs de C qui ne m'ont jamais parlé de la différence, mais je vais me documenter avec mon ami google.
Merci pour votre aide !!!

Le club met à ta disposition bon nombre de cours de C :
http://c.developpez.com

Citation:

---

Envoyé par LaseLiep

Rahhh, j'ai po de livre(s) de C

---

http://www-clips.imag.fr/commun/bernard.cassagne/Introduction_ANSI_C.html

Je vous remercie.