Discussion :

[LaseLiep]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

#include "tp3.h"

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

[Skyounet]

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

[Strab]

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 ?

[LaseLiep]

Oupsss, toutes mes excuses ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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);
}

[LaseLiep]

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 :

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

[Strab]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char nom[MAX_STR];

Mieux vaut déclarer

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Emmanuel Delahaye]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
   if (nouveau != NULL)
   {

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

	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...

[LaseLiep]

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?

[Emmanuel Delahaye]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char nom[MAX_STR];

Mieux vaut déclarer

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char *nom;

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

[Emmanuel Delahaye]

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.

: 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.

[LaseLiep]

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

[Emmanuel Delahaye]

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.

[LaseLiep]

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 !!!

[Strab]

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

[Emmanuel Delahaye]

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

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

[LaseLiep]

Je vous remercie.