chaine de caracteres et structure

Bonjour,
j'ai créer une structure et je n'arrive pas à créer une fonction qui me remplisse le champ nom. Merci pour votre aide.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

struct stjoueur { char nom [25]; char couleur; }; typedef struct stjoueur JoueurA;   void SaisieNomCouleur (struct stjoueur * JoueurA) { char nomA [25] ;   printf ("\n Entrer le nom du premier joueur : "); gets (nomA); *JoueurA->nom = nomA; printf ("\n%s", nomA);

---

Bonjour,

il faudrait que tu verifie la portée des varialbes.
Dans ta methode SaisirNomCouleur
Tu crée un tableau de caracteres dans lequel tu stockes ta chaine.
Ensuiste tu colles l adresse de ce tableau (espace mémoire en fait) dans ta structure.
Au moment de terminer la fonction (avant de rendre la main a la fonction appelante), on libere la memoire des variables locales (notement ton tableau de char), donc ta structure regarde maintenant a une adresse qui d'etre liberé...

Perdu !

Par contre regarde du coté des fonction strcpy, voir mieux strncpy voir si besoin se fait sentir memcpy.

Voila voila, a toi maintenant

P.S. : met un "\n" a la fin de ton printf : printf( "\n%s\n", taStruct->nom );

Code:

---

*JoueurA->nom = nomA;

---

2 erreurs sur cette ligne:
- JoueurA est un pointeur sur struct stjoueur. Pour accéder à un champ d'une structure dont on a l'adresse, l'opérateur -> suffit :

Code:

---

JoueurA->nom = ...

---

- On ne peut pas copier un tableau dans un autre en faisant tableau1 = tableau2. D'ailleurs, on ne peut pas écrire tableau1 = ....
Pour copier un tableau nomA dans un tableau JoueurA->nom, utiliser, si ces tableaux sont des chaînes de caractères standard, la fonction strcpy

Recommandation : Ne pas utiliser gets mais fgets avec stdin comme paramètre fichier

merci,
je vais essayer vos recomandations.

Si j'utilise strcpy, mon argument de fonction n'est pas obligatoirement un pointeur sur structure?

T a deux solutions,
soit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

struct stjoueur { char nom [25]; char couleur; }; typedef struct stjoueur JoueurA;   void SaisieNomCouleur (struct stjoueur * JoueurA) { char nomA [25] ;   printf ("\n Entrer le nom du premier joueur : "); gets (nomA); strcpy(JoueurA->nom,nomA); printf ("\n%s", JoueurA);

---

Ou plus simplement :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

struct stjoueur { char nom [25]; char couleur; }; typedef struct stjoueur JoueurA;   void SaisieNomCouleur (struct stjoueur * JoueurA) {     printf ("\n Entrer le nom du premier joueur : "); gets (JoueurA->nom); printf ("\n%s", JoueurA->nom);

---

Citation:

---

Envoyé par olivier1212

merci,
je vais essayer vos recomandations.

Si j'utilise strcpy, mon argument de fonction n'est pas obligatoirement un pointeur sur structure?

---

Re,

si tu te pose encore ce genre de question c'est que tu n'est pas encore Opé sur le passage de parametres en C, et la y a pas de solutions miracles :
- lectures de cours
- compréhension
- tests personnels

Sans ca ce sera dur de passer l étape superieure

Merci pour tes encouragements. J'aurai surement d'autres questions qui viendront sur la sécurisation des entrées clavier.
Mais comme le dit le proverbe "c'est en forgeant qu'on devient forgeron".8O

En attendant j'ai pondu ça :

Code:

---

1 2 3 4 5 6 7

void SaisieNom (struct stjoueur * JoueurX) { char nomX [25] ; printf ("\n Entrer le nom du premier joueur : "); fgets (nomX, 25, stdin); strncpy (JoueurX->nom, nomX, 25); JoueurX->nom[24] = '\0';

---

Est-ce qu'il faut mettre le \0 directement après le dernier caractère tapé ou comme je l'ai fait dans la dernière case?

Citation:

---

Envoyé par olivier1212

Est-ce qu'il faut mettre le \0 directement après le dernier caractère tapé ou comme je l'ai fait dans la dernière case?

---

c'est la fonction fgets qui s en charge, donc ta chaine de caractere contient deja \0,
apart ca en mettant \0 dans nom[24] , c'est indiquer que le nom comporte exactement 24 caracteres, ce qui n est pas le cas, "james" \0 doit se trouver dans nom[5] pas dans nom[24];
fgets fait tout le travail pour toi

Quand je declare une chaine : char nom [25], le dernier élement se trouve en nom[24].
Dans tous les cas merci pour les renseignements.

Salut,

Fallait peut être pas mettre le :
typedef struct stjoueur JoueurA;
puisque tu a réécrit " struct stjoueur " :

Code:

---

void SaisieNomCouleur (struct stjoueur * JoueurA)

---

En gros le JoueurA représente lui aussi le type de ta struture " struct stjoueur " moi j'aurais écrit sa par exemple :

Code:

---

void SaisieNomCouleur (JoueurA * joueurX_ou_autre)

---

j'espère ne pas avoir dit de bêtises .....

ok ça marche merci