Discussion :

[kraoui]

Bonjour à tous et à toutes !

Je m'adresse à vous car j'ai un petit problème de saisie lors de l'utilisation de la fonction fgetsJ'ai donc besoin que l'utilisateur entre une suite de caractères pour que celle-ci soit stockée dans un tableau de type char.
J'aimerais par la suite retranscrire cette suite.

Je ne m'occupe pas de vider le buffer pour le moment car j'aimerais que ma fonction fgets fonctionne d'abord.

Voici le code de ma fonction :

Code c :

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
17
 
#include <stdlib.h>
#include <stdio.h>
#include <windows.h>
#include <string.h>
#include "fonction.h"
 
void modifNom (char hero[100],int taille) // Fonction qui permet de changer le nom d'un hero.
{
    printf("==============-------- Choix du nom de %s --------==============\n", hero);
 
    printf("\nVeuillez entrer le nouveau nom de %s ne depassant pas %d caracteres\net sans espaces\n", hero, taille); // Affiche le nom actuel du hero
    fgets(hero, sizeof hero, stdin);// lit le nouveau nom et l'enregistre
    printf("Le nouveau nom est : ");
    printf("%s \n", hero); // affiche le nouveau nom du héro
    system("PAUSE");
}

Mon problème est que quand la fonction se lance dans mon programme, elle affiche tout en bloc et s'arrête à l'instruction system("PAUSE").

Et, du coup, elle ne renvoie rien dans la variable hero et le nom de mon héros est remplacé par du vide.

Si quelqu'un remarque une erreur dans mon code ou si quelqu'un a une astuce à me donner,

Je suis toute ouïe !

[Obsidian]

Si quelqu'un remarque une erreur dans mon code ou si quelqu'un a une astuce à me donner,
Je suis toute ouïe !

Bonjour, c'est subtil mais l'erreur est dans le prototype de la fonction :

Code :

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

void modifNom (char hero[100],int taille)

Les tableaux sont passés par pointeur lorsqu'ils sont transmis en paramètre, si bien qu'un sizeof tab, pourtant tout-à-fait dans les règles par ailleurs, renverra 4 ou 8 sur un PC 32 bits ou 64 bits, au lieu des 100 attendus.

Ensuite, si « tout s'affiche en bloc » jusqu'à ton system(), c'est probablement que le buffer clavier n'était pas vide lorsque tu as appelé ta fonction. Il faut donc le nettoyer d'abord mais, de grâce, surtout pas avec fflush(stdin).

[Abacar94]

Bonjour à tous et à toutes !

Je m'adresse à vous car j'ai un petit problème de saisie lors de l'utilisation de la fonction fgetsJ'ai donc besoin que l'utilisateur entre une suite de caractères pour que celle-ci soit stockée dans un tableau de type char.
J'aimerais par la suite retranscrire cette suite.

est il obliger de use fgets si non tu peut de contenter de

Code :

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

1
2
3
 
printf("\nVeuillez entrer le nouveau nom de %s ne depassant pas %d caracteres\net sans espaces\n", hero, taille); // Affiche le nom actuel du hero
    scanf("%s",&hero);  // lit le nouveau nom et l'enregistre

[gangsoleil]

Bonjour,

L'utilisation de scanf est déconseillé pour lire des entrées non formatées, notamment pour des problèmes de taille de chaîne saisie. fgets est la bonne solution.

Je ne peux que conseiller la lecture de la page sur les entrées/sorties https://emmanuel-delahaye.developpez...ees-solides-c/

[kraoui]

Bonjour, c'est subtil mais l'erreur est dans le prototype de la fonction :

Code :

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

void modifNom (char hero[100],int taille)

Les tableaux sont passés par pointeur lorsqu'ils sont transmis en paramètre, si bien qu'un sizeof tab, pourtant tout-à-fait dans les règles par ailleurs, renverra 4 ou 8 sur un PC 32 bits ou 64 bits, au lieu des 100 attendus.

Ensuite, si « tout s'affiche en bloc » jusqu'à ton system(), c'est probablement que le buffer clavier n'était pas vide lorsque tu as appelé ta fonction. Il faut donc le nettoyer d'abord mais, de grâce, surtout pas avec fflush(stdin).

Merci Beaucoup pour ta réponse !
J'ai changer dans ma fgets le sizeof hero en une constante : 100.

J'ai aussi vider le buffer avant et après le fgets
Mon code ressemble donc à ceci et ça fonctionne parfaitement

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
 
void modifNom (char hero[100],int taille) // Fonction qui permet de changer le nom d'un hero.
{
    clean(hero);
    printf("==============-------- Choix du nom de %s --------==============\n\n", hero);
    printf("\nVeuillez entrer le nouveau nom de %s ne depassant pas %d caracteres\net sans espaces\n", hero, taille); // Affiche le nom actuel du hero
    fgets(hero, 100, stdin);// lit le nouveau nom et l'enregistre
    clean(hero);
    printf("Le nouveau nom est : ");
    printf("%s \n", hero); // affiche le nouveau nom du héro
    system("PAUSE");

Et le code de ma fonction purger contenu dans ma fonction clean :

Code c :

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
17
18
19
20
21
 
static void purger()
{
    int c = 0;
    while (c != '\n' && c != EOF)
    {
        c = getchar();
    }
}
void clean(char *chaine)
{
    char *p = strchr(chaine, '\n');
    if (p)
    {
        *p = 0;
    }
    else
    {
        purger();
    }
}

[kraoui]

est il obliger de use fgets si non tu peut de contenter de

Code :

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

1
2
3
 
printf("\nVeuillez entrer le nouveau nom de %s ne depassant pas %d caracteres\net sans espaces\n", hero, taille); // Affiche le nom actuel du hero
    scanf("%s",&hero);  // lit le nouveau nom et l'enregistre

En faite j'aurai du changé mon code avant de poster mais je voulais qu'on puisse entrer un nom avec espace .
Sinon en effet, c'est le premier code que j'avais

[kraoui]

Bonjour,

L'utilisation de scanf est déconseillé pour lire des entrées non formatées, notamment pour des problèmes de taille de chaîne saisie. fgets est la bonne solution.

Je ne peux que conseiller la lecture de la page sur les entrées/sorties https://emmanuel-delahaye.developpez...ees-solides-c/

J'y jette un oeil maintenant

[Médinoc]

Merci Beaucoup pour ta réponse !
J'ai changer dans ma fgets le sizeof hero en une constante : 100.

Pourquoi ne pas passer taille?

[kraoui]

Pourquoi ne pas passer taille?

J'aurai pu en effet :p. Je t'avoue que j'ai changé ça vite fait au boulot :p