[Pointeur structure] char *nom

Version imprimable

Voir 40 message(s) de cette discussion en une page

Heu non en supprimant le pointeur ça me fait une dizaine d'erreurs. J'ai modifié un peu mon main et maintenant ça affiche correctement.

Voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Cellule_compagnie *initialisationCompagnie()
{
  Cellule_compagnie *cellule = malloc(sizeof(*cellule));
  Liste_compagnie liste = malloc(sizeof(*liste));
 
  if(liste == NULL || cellule == NULL)
    exit(EXIT_FAILURE);
 
  cellule->comp.nom = malloc(42 * sizeof (char));
  cellule->comp.nom = strcpy(cellule->comp.nom, "Metalman");
  strncpy(cellule->comp.acronyme, "ATA", 3);
  cellule->suivant = NULL;
  liste = cellule;
 
  printf("%s\n",cellule->comp.nom);
 
  return liste;
}

Et voici mon appelle dans le main :

Code:

Liste_compagnie *liste_compagnie = initialisationCompagnie();

Cela fonctionne bien mais j'ai plusieurs warnings dont je dois à tout prix me débarrasser :

Code:

1
2
3
4
main.c: In function ‘main’:
main.c:19:3: attention : implicit declaration of function ‘initialisationCompagnie’ [-Wimplicit-function-declaration]
main.c:19:38: attention : initialization makes pointer from integer without a cast [enabled by default]
main.c:19:20: attention : unused variable ‘liste_compagnie’ [-Wunused-variable]

18/05/2013, 23h46
Metalman
Nononon...
Tu DOIS mettre :

Code:

Cellule_compagnie *cellule = malloc(sizeof(cellule));

S'il y a + d'erreurs, c'est parce qu'il y en a VRAIMENT + !
Je veux bien les voir...

EDIT2 : Il a bien raison, car c'est un sizeof(*cellule) donc une variable, et non un type comme je le pensais !

1) Implicit declaration : tu as mis la déclaration de ta fonction APRES l'avoir appelée... il faut mettre la déclaration AVANT, ou inclure sa signature dans le .h

2) Le problème de "without a cast", il me faut la ligne exacte qui fait cela...
En gros tu as dû inverser des pointeurs et des références...
La fonction demande un char*, tu as mis un char ou un char** par exemple.

3) C'est une variable que tu n'utilises jamais... donc retire la des déclarations !

Bref, cacher des warnings en gardant un problème de malloc ça ne réglera AUCUN problème...

EDIT : Il y a une phrase assez célèbre et lancée sous forme de blague : "Ca compile donc ça marche".
Cette phrase est évidemment fausse avec le C (et sur la très grande majorité des langages de programmation), fais attention à ne pas tomber dedans !
Pour être sûr que ton programme fonctionne "techniquement" plutôt bien il faut au minimum faire cela :
- Compiler avec l'ensemble des flags, c'est-à-dire avec : -W -Wall -Werror -ansi -pedantic si tu codes en C89, ou avec -Wextra -Wall -Werror -std=c99 -pedantic
- Exécuter avec l'outil Valgrind sur Linux (ou Dr.Memory sur Windows/MinGW) et les options : valgrind --show-reachable=yes --leak-check=full ./ton_programme (ou à peu près drmemory -show_reachable -leak_check ton_programme dans un prompt MS-DOS)
Si ton programme passe ces 2 points, mais qu'il n'agit pas comme tu le souhaites, alors les défauts proviennent quasiment à coup sûr de ta logique de développement

D'accord, je sais que dès qu'il y a des warnings il faut modifier pour que tout reste propre et fonctionne correctement. Pour ça que j'affiche ici ce que me renvoi le shell.

Pour la compilation j'utilisais uniquement gcc -Wall -ansi pour faire mon code, c'est tout ce que je connaissais.

Bon je vais tout récapituler avec les structures et le code nécessaire, sachant que je n'ai pas le droit de modifier les structures.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 typedef struct compagnie Compagnie; typedef struct cellule_compagnie Cellule_compagnie; typedef Cellule_compagnie *Liste_compagnie; struct compagnie { char *nom; char acronyme[3]; Liste_avion avion_compagnie; }; /* Liste des compagnies */ struct cellule_compagnie { Compagnie comp; struct cellule_compagnie* suivant; };
Voici mon appel de la fonction pour initialiser la compagnie de base dans le main :

Code:

Cellule_compagnie *liste_compagnie = initialisationCompagnie();

Je viens de me rendre compte pour l'histoire du prototypes dans le .h que je l'avais inclus uniquement pour le fichier gestion_liste.c au lieu du main.c vu que c'est lui qui l'appelle. Merci.

En ce qui concerne la variable inutilisé, c'est normal car je suis en période de test pour l'instant, voir si ce que j'ai déjà écrit fonctionne avec les autre fichiers de mon programme.
Le problème du cast à disparu avec l'inclusion du .h dans le main, le seul warning restant est la variable inutilisée, je te remercie.

J'ai une question par rapport à ce que tu as écrit concernant l'affichage de ma chaîne de caractère après strcpy.

Code:

cellule->comp.nom = malloc(42 * sizeof (char));

J'aimerai savoir à quoi correspond cette ligne ? Que fait la machine quand on écrit ceci ?

Merci pour tout en tout cas ! :ccool:

19/05/2013, 11h44
diogene

@Metalman
L'appel typique à malloc pour allouer N élements de type T est T * p = malloc(N*sizeof *p);.
On doit avoir :

Code:

Cellule_compagnie *cellule = malloc(sizeof *cellule);
19/05/2013, 21h26
Metalman

En effet, je suis trop habitué à sizeof(un_type) plutôt qu'une variable...
Du coup le *cellule je le prenais pour un pointeur et non pour le type pointé...
Merci de douter de mes posts du coup ! (et merci à Diogène pour me reprendre du coup)

Donc ta ligne (à toi Kriegor) est tout à fait valable en effet, et je me suis lamentablement laissé tromper par mes habitudes.
Donc encore une leçon à retenir (surtout pour moi) : ne surtout pas rester dans ses habitudes, et toujours remettre en cause ses idées ! :P

Sinon, ça :

Code:

cellule->comp.nom = malloc(42 * sizeof (char));

Ca va allouer un espace mémoire pour 42 caractères (donc 41 + le '\0' final) pour le champ "nom" de la structure "comp".
Evidemment : cela fonctionne sur des char*, mais pas sur des char[xx] (avec 'xx' un entier) qui sont déjà alloués !
Tu pourras donc y accéder plus tard, depuis : cellule->comp.nom[0]
jusqu'à : cellule->comp.nom[41] (qui DEVRAIT contenir un '\0' dans la majorité des cas usuels... sauf si tu conserves quelque part sa taille et que tu utilises les strN*)
23/05/2013, 15h13
Kriegor D Will

Merci pour ton explication et pour les réponses de tout le monde.
J'ai une nouvelle question.

Imaginons que l'on créé une variable de type entier et que l'on demande à l'utilisateur de saisir un entier à l'aide d'un scanf :

Code:

scanf("%d",&entier);

Ce qu'il se passe lorsque l'utilisateur entre un caractère, et que notre programme fonctionne dans une boucle, ça fait un gros bazar.

Comment faire pour faire en sorte que l'utilisateur ne puisse entrer que des caractères numérique ? Je sais faire une boucle qui redemande à l'utilisateur de rentrer à nouveau un nombre si celui qu'il a rentré est négatif ou trop grand mais comment faire pour les caractères ?

Merci ! :mrgreen:
23/05/2013, 16h34
ternel

c'est là tout le problème de scanf.

scanf lit une entrée formatée. Or, la saisie utilisateur n'est pas formatée, sauf si c'est un pipe, tel que "cat fichier | programme".

la solution est présent en plusieurs endroit sur le forum et dans la faq, cherche "saisie sécurisée".

Je ne suis plus capable de t'expliquer la bonne manière de coder ca.

Voir 40 message(s) de cette discussion en une page