Discussion :

[Kriegor D Will]

Yo,

Je rencontre un petit problème avec une structure et un pointeur.

Voici mes structures :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
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;
};
 
struct cellule_compagnie
{
  Compagnie comp;
  struct cellule_compagnie* suivant;
};

Dans un fichiers de code, j'initialise la liste :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
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 = 'A';
  cellule->suivant = NULL;
  liste = cellule;

  return liste;
}

La ligne rouge me pose problème, voici le message du shell :

Code :

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

1
2
In function ‘initialisationCompagnie’:
gestion_liste_compagnie.c:22:21: attention : assignment makes pointer from integer without a cast [enabled by default]

J'ai essayé quelques modifs de syntaxe mais ça me fait même des erreurs après, du cou je sais pas si c'est un bug ou qu'il y a vraiment un souci.

Merci à vous et bonne soirée !

[kwariz]

Bonsoir,

le problème est exactement ce que le compilateur te dit : tu donnes une valeur entière (le code ascii du caractère 'A') à une variable qui attend un pointeur, un char* en l’occurrence.
'A' est un caractère, "A" est la chaîne composée du caractère 'A' suivi du 0 terminal. Je pense que tu voulais utiliser une chaîne plutôt qu'un caractère.

[Kriegor D Will]

Ah.

En fait les structures sont ainsi et je ne peux les modifier.
Donc si je te suis bien, le char *nom est en fait un char nom[], c'est bien ça ?
Ah ben oui ce serait stupide sinon d'avoir le nom d'une compagnie qui ne fait qu'une lettre. Autant pour moi je n'ai pas suffisamment réfléchit.

Merci de m'avoir éclaircit.

Krieg

[kwariz]

Je t'en prie, mais attention aux différences entre un char* (allocation mémoire, pointeurs) et un char [] (tableau). Réfère toi aux tuto de base pour ne pas tomber dans les pièges traditionnels

Bonne continuation.

[Kriegor D Will]

Par contre je n'arrive pas à écrire la bonne syntaxe. Du coup j'avais modifié en faisant ça :

cellule->comp.nom[5] = "AB";

Pourtant l'erreur est toujours présente.

Je pense que je ne comprend pas quelque chose d'essentiel en fait.

[kwariz]

Pour faire court, cellule->comp.nom[5] permet d'accéder aux sixième caractère de la chaine cellule->comp.nom. Tu peux par exemple faire cellule->comp.nom="blabla" ...
cf. Les chaines de caractères et tableaux et pointeurs en C.

[Kriegor D Will]

D'accord merci, en effet ma détresse me fait écrire n'importe quoi.

Merci et passes une bonne soirée !

Krieg

[ternel]

attention tout de même.
Bien que déclaré comme un char*, si tu fais ton cellule->comp.nom="blabla", tu risque d'avoir de très gros problème.

Essaie de changer pour un const char*. "blabla" est une chaine de caractères constant. (un const char *, presque mais pas tout à fait un const char[]).

Tenter d'en modifier un caractère est une erreur. (nom[1]='c' échouerait avec une segmentation fault)

[Bktero]

D'ailleurs, le compilateur devrait émettre un warning pour dire que les qualificatifs ne sont pas en accord. Par exemple avec MinGW :

Code :

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

D:\...\main.c|19|warning: assignment discards 'const' qualifier from pointer target type [enabled by default]|

[Franck.H]

L'idéal serait d'allouer un espace pour ta chaîne après tu peux faire ce que tu veux, sans oublier bien sûr de libérer la mémoire une fois que tu n'en a plus besoin.

[Kriegor D Will]

Mhh étrange lors de la compilation tout se passe bien, même avec les extension -Wall et -ansi, il n'y a ni warnings ni erreurs.
Mais je n'ai pas le droit de modifier les composants des structures dans cet exercice.

Par contre après je n'arrive pas à afficher le nom en mettant un :

Code :

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

printf("%s\n",cellule->comp.nom)

C'est bizarre on dirait que ça le saute en fait.

J'ai vraiment du mal avec les pointeur en fait là... concernant l'acronyme...

Je veux l'initialiser pour qu'il contienne la chaîne "AFR"

Du coup je saisir la ligne :

Code :

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

cellule->comp.acronyme = "AFR"

Mais ça me met l'erreur suivante :

Code :

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

erreur: incompatible types when assigning to type ‘char[3]’ from type ‘char *’

[Metalman]

"AFR" est un pointeur "char*".

acronyme est de type "char[3]", c'est à dire un tableau.

Pour réaliser ton opération, tu dois copier le contenu du pointeur dans le tableau !
strcpy fait cela !

Mais ton tableau de 3 caractère est risqué, car on n'a pas la place de mettre le '\0' final !
Il faudrait mettre un char[4] ou faire très très attention et ne travailler qu'avec les strN* (strncpy, strnlen, etc... au lieu des versions classiques strcpy, strlen...)

[Kriegor D Will]

Bizarre je croyais que char c* voulait dire la même chose que char c[].

Car si je remplace le :

Code :

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

char acronyme[3];

par :

Code :

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

char *acronyme;

ça fonctionne en initialisant avec :

Code :

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

cellule->comp.acronyme = "..."

Donc le seul moyen d'initialiser le tableau écrit en char c[] dans la structure est de se servir de strcpy ?

[Bktero]

http://gradot.wordpress.com/2012/08/...ointeurs-en-c/

Peut-être qu'un jour je reprendrais la réaction de la version longue de cet article pour Developpez...

[Kriegor D Will]

J'ai réussi à écrire dans le chaîne avec strcpy, maintenant j'aimerai pouvoir l'afficher, mais je n'y arrive pas avec un simple printf("%s\n", ...) comment pourrais-je l'afficher autrement ?

[Metalman]

Quel est le printf exact que tu entres ?

[Kriegor D Will]

Code :

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

printf("%s\n", cellule->comp.nom);

[Metalman]

On est bien d'accord que ton code ressemble à quelquechose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
  Cellule_compagnie *cellule = malloc(sizeof(*cellule));
 
  cellule->comp.nom = malloc(42 * sizeof (char));
  cellule->comp.nom = strcpy(cellule->comp.nom, "Metalman");
  // OU 
  //cellule->comp.nom = strdup("Metalman");
  strncpy(cellule->comp.acronyme, "Me");
  cellule->suivant = NULL;
 
  printf("nom : %s\nacronyme : %s\n", cellule->comp.nom, cellule->comp.acronyme);

[Kriegor D Will]

Je n'avais pas exactement ça mais en copiant ton code le printf ne s'effectue pas non plus, je ne comprends pas pourquoi. Et j'ai bien appelé ma fonction dans mon main.

[Metalman]

Je viens de voir une TERRIBLE erreur.

Code :

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

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

Est à remplacer par :

Code :

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

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