Discussion :

[olivier1212]

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 :

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

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

[BainE]

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

[diogene]

Code :

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

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

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

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

[olivier1212]

merci,
je vais essayer vos recomandations.

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

[siegfried64]

T a deux solutions,
soit :

Code :

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

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 :

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

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

[BainE]

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

[olivier1212]

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

En attendant j'ai pondu ça :

Code :

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

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?

[siegfried64]

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

[olivier1212]

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

[orfix]

Salut,

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

Code :

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

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 :

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

void SaisieNomCouleur (JoueurA * joueurX_ou_autre)

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

[olivier1212]

ok ça marche merci