[DEBUTANT] Problème de lecture avec un fscanf

Version imprimable

Bonjour tout le monde !

Voilà je viens expooser ici un problème qui commence à me griller quelques neurones :D !!

Je possède un fichier contenant 7 nombres séparés par un espace chacun ;
Exemple : 0 368 183 34 60 0 0

Je sougaite lire et "interpréter" ces nombres. Pour ça, j'ai recours à un fscanf ... or si j'enregistre après dans un fichier les variables changées par le fscanf, je m'aperçois en fait que lros de la lecture, seul 1 nombre sur 2 est pris en compte, les autres restant à 0.

Exemple (complémentaire d'au-dessus) : 0 0 183 0 60 0 0

Voici mon code complet :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
 
//------------------------------------------------------------------------------
//---- Chargement de la map ----------------------------------------------------
//------------------------------------------------------------------------------
void affichageMap(SDL_Surface *ecran, SDL_Surface *background, FILE* map, SDL_Surface *spriteChoisi[])
{
//---- Déclaration des variables -----------------------------------------------
SDL_Rect position, partieRestante;
 
int choixSprite = 0;
//------------------------------------------------------------------------------
 
 
//---- Affichage du Background -------------------------------------------------
SDL_BlitSurface(background, NULL, ecran, NULL);
//------------------------------------------------------------------------------
 
//---- On lit le contenu du fichier & on affiche les sprites -------------------
 
if(map != NULL)
{
    while(!feof(map))
    {
    //---- On lit le fichier sprite, abscisse , ordonnée--------------------
    fscanf(map,"%ld %ld %ld %ld %ld %ld %ld\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
    //----------------------------------------------------------------------
 
    //---- On affiche le sprite correspondant ------------------------------
    SDL_BlitSurface(spriteChoisi[choixSprite], &partieRestante, ecran, &position);
    //----------------------------------------------------------------------
    }
    SDL_Flip(ecran);
 
}
//------------------------------------------------------------------------------
 
//---- Fermeture du fichier ----------------------------------------------------
fclose(map);
//------------------------------------------------------------------------------
 
}

Voilà je pense que vous avez compris mon problème ....

Dans l'attente d'une réponse,

Bonne continuation !

27/05/2006, 10h34
Trap D

Tu peux déjà tester le retour de fscanf pour connaître le nombre de conversions réussies.
De toutes façon, comme d'hab, fscanf est une fonction difficile à utiliser, fais une saisie de la ligne avec fgets et travaille avec strtol qui permet de savoir où s'arrêtent les conversions.

Salut,

Tu demandes au fscanf de convertir des long (%ld) alors que ta première variable, du moins, est un int. Un %d devrait suffire.
Sinon, mêmes remarques que Trap D.

[EDIT]
avec ce code, ça fonctionne très bien

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
int main(int argc, char *argv[])
{
    int ret = 0 ;
    char ligne1[] = "0 368 183 34 60 0 0\n",
         ligne2[] = "0 0 183 0 60 0 0\n" ;
 
    int choixSprite ;
    SDL_Rect position, partieRestante ;
 
    ret = sscanf(ligne1,"%ld %ld %ld %ld %ld %ld %ld\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
    printf("\n%d champs convertis\n", ret) ;
    printf("  choixSprite = %d\n", choixSprite) ;
    printf("  position.x = %d\n", position.x) ;
    printf("  position.y = %d\n", position.y) ;
    printf("  partieRestante.w = %d\n", partieRestante.w) ;
    printf("  partieRestante.h = %d\n", partieRestante.h) ;
    printf("  partieRestante.x = %d\n", partieRestante.x) ;
    printf("  partieRestante.y = %d\n", partieRestante.y) ;
 
    ret = sscanf(ligne2,"%ld %ld %ld %ld %ld %ld %ld\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
    printf("\n%d champs convertis\n", ret) ;
    printf("  choixSprite = %d\n", choixSprite) ;
    printf("  position.x = %d\n", position.x) ;
    printf("  position.y = %d\n", position.y) ;
    printf("  partieRestante.w = %d\n", partieRestante.w) ;
    printf("  partieRestante.h = %d\n", partieRestante.h) ;
    printf("  partieRestante.x = %d\n", partieRestante.x) ;
    printf("  partieRestante.y = %d\n", partieRestante.y) ;
 
  return 0 ;
}

Nota : J'ai utiliser sscanf, car je n'ai pas la structure de ton fichier.
N'aurais-tu pas un problème dans l'écriture des variables dans ton fichier ?

27/05/2006, 13h13
Pingouin

Merci de vos réponses.

Ce qui m'étonne, c'est que pour l'enregistrement du fichier, j'ai fait exactement la même chose, mais avec un fprintf, et ça fonctionne très bien !

Elijha > Le problème de la solution que tu me proposes, c'est que tu lis une chaîne et nonp pas un fichier. Or je dois lire ces fichiers, je ne peux pas tout stocker en mémoire ... je vais me pencher quand même dans cette direction pour voir un peu si je peux faire qqch
27/05/2006, 13h21
Trap D

ben tu fais un fgets suivi d'un sscanf...

Grr je ne vois vraiment le problème ... voilà mon code maintenant avec un fgets suivi d'un sscanf ... toujours le même problème !!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
//------------------------------------------------------------------------------
//---- Chargement de la map ----------------------------------------------------
//------------------------------------------------------------------------------
void affichageMap(SDL_Surface *ecran, SDL_Surface *background, FILE* map, SDL_Surface *spriteChoisi[])
{
//---- Déclaration des variables -----------------------------------------------
SDL_Rect position, partieRestante;
 
int choixSprite = 0;
char lecture[30] = {0};
//------------------------------------------------------------------------------
 
 
//---- Affichage du Background -------------------------------------------------
SDL_BlitSurface(background, NULL, ecran, NULL);
//------------------------------------------------------------------------------
 
//---- On lit le contenu du fichier & on affiche les sprites -------------------
 
if(map != NULL)
{
    while(!feof(map))
    {
    //---- On lit le fichier sprite, abscisse , ordonnée------------------------
    fgets(lecture, sizeof(lecture), map);
    //--------------------------------------------------------------------------
 
    //---- Conversion des données recueillies en fgets -------------------------
    sscanf(lecture, "%d %d %d %d %d %d %d\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
    //--------------------------------------------------------------------------
 
    //---- On affiche le sprite correspondant ----------------------------------
    SDL_BlitSurface(spriteChoisi[choixSprite], &partieRestante, ecran, &position);
    //--------------------------------------------------------------------------
    }
    SDL_Flip(ecran);
 
}
//------------------------------------------------------------------------------
 
//---- Fermeture du fichier ----------------------------------------------------
fclose(map);
//------------------------------------------------------------------------------
 
}

Cette fois, si après je fais un fprintf pour voir ce qui a été converti .... toujours le même problème, 1 valeur sur 2. Mais si je fais un fprintf de la chaïne %s lecture, ça m'affiche bien ce qu'il est censé y avoir !

Merci de votre aide !

27/05/2006, 14h07
AjJi

On peut voir comment tu ouvres ton fichier ? :)
(juste au cas où)
fscanf retourne le nombre d'elements lus, tu devrais déjà tester..
le \n dans le fscanf ne sert à rien aussi.
27/05/2006, 15h34
Pingouin

J'ouvre le fichier avec un simple fopen en r ...

Ajout de quelques tests de retour de fonctions.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
// Chargement de la map 
void affichageMap(SDL_Surface *ecran, SDL_Surface *background, FILE* map, SDL_Surface *spriteChoisi[])
{
// Déclaration des variables 
SDL_Rect position, partieRestante;
 
int choixSprite = 0;
char lecture[30] = {0};
 
// Affichage du Background 
SDL_BlitSurface(background, NULL, ecran, NULL);
 
// On lit le contenu du fichier & on affiche les sprites
 
if(map != NULL)
{
    while(fgets(lecture, sizeof(lecture), map))
    {
 
        // Conversion des données recueillies en fgets 
        if(sscanf(lecture, "%d %d %d %d %d %d %d", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y) != 7) {
            printf("Erreur, format incorrect !\n");
            return;
        }
 
        // On affiche le sprite correspondant 
        SDL_BlitSurface(spriteChoisi[choixSprite], &partieRestante, ecran, &position);
    }
    SDL_Flip(ecran);
 
}
 
if(feof(map)
    fclose(map);
else {
    printf("Erreur lecture fichier !\n");
    fclose(map);
    return;
}
 
}

Les commentaires c'est bien mais les grandes lignes de commentaires - juste pour separer c'est vraiment chiant.

27/05/2006, 16h38
Pingouin

Skyrunner > merci je vais regarder ça dans un moment.

PS : pour les commentaires je m'y retrouve mieux avec ma méthode, mais ça chacun fait comme il veut !
27/05/2006, 16h57
Trap D

Tu es sûr que 30 octets c'est assez large ? 7 nombres plus un espaces ça risque de dépasser.
27/05/2006, 18h51
Pingouin

Skyrunner > J'ai faitl e test que tu m'as suggéré, et il ne renvoie rien, pas de message, comme si tout c'était bien passé. C'est quoi encore cette bizarrerie ?

Trap-D > Non 25 ou 30 suffisent, puisque si j'affiche le contenu de lecture (en char) dans un fichier, j'ai toute la ligne ...
27/05/2006, 18h53
Skyounet

Citation:

Envoyé par Pingouin

Skyrunner > J'ai faitl e test que tu m'as suggéré, et il ne renvoie rien, pas de message, comme si tout c'était bien passé. C'est quoi encore cette bizarrerie ?

Trap-D > Non 25 ou 30 suffisent, puisque si j'affiche le contenu de lecture (en char) dans un fichier, j'ai toute la ligne ...

Apres le fgets essaye d'afficher la ligne sur la sortie standard et apres le sscanf affiche toutes les variables aussi.
27/05/2006, 19h34
Pingouin

Skyrunner > Voilà ce que ça donne ...

Si juste après le fgets j'affiche lecture %s (grâce à un fprintf), j'obitens bien la suite de chiffres attendue.

Si j'affiche les variables après sscanf, voilà ce que j'obtiens :

choixSprite est correct ;
position.x est à 0 ;
position.y est correct ;
partieRestante.w est à 0 ;
partieRestante.h est correct ;
partieRestante.x est à 0 ;
partieRestante.y est correct ;

j'ai donc bien une valeur sur deux prise en compte, et le reste est quand même modifié (j'avais initialisé toutes les variables à 23) mais toujours en 0.
28/05/2006, 12h53
Elijha

Salut,

Pourrais-tu nous donner le format de ton fichier de sprites ?

J'ai essayé avec un fichier au format texte contenant plusieurs lignes de 7 valeurs, et cela fonctionne bien avec un fscanf(...).

Citation:

87 43 56 41 84 74 69
99 96 80 68 12 29 15
61 30 78 57 90 38 36

Voilà l'enregistrement :
Code:

1 2 3 4 5 6 void enregistrementSprite(FILE* fichier, int choixSprite, SDL_Rect position, SDL_Rect partieRestante) { fprintf(fichier, "%d %d %d %d %d %d %d\n", choixSprite, position.x, position.y, partieRestante.w, partieRestante.h, partieRestante.x, partieRestante.y); }
et voilà la structure du fichier :

Citation:

0 141 211 60 60 0 0
0 201 211 60 60 0 0
0 261 211 60 60 0 0
0 321 211 60 60 0 0
0 381 211 20 60 0 0
0 149 401 60 60 0 0
0 149 461 60 60 0 0
0 149 521 60 39 0 0

Ca c'est le fichier enregistré, et les 0 présents sont des 0 volontaires. Si je changeais un paramètre, je n'aurais pas 0. L'enregistrement fonctionne donc bien.

Essaye avec un petit programme à côté pour déterminer ton problème. Je vois ce seul moyen.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
#include <stdio.h>
#include <stdlib.h>
 
typedef struct{
  signed int x, y;
  unsigned int w, h;
} SDL_Rect;
 
int main(int argc, char *argv[])
{
  FILE *sprite ;
 
  int ret = 0 ;
  int choixSprite ;
  SDL_Rect position, partieRestante ;
 
  sprite = fopen("sprite.txt", "r+") ;
  if(sprite==NULL)
    {
      printf("Erreur d'ouverture du fichier\n") ;
      exit(0) ;
    }
 
  while(!feof(sprite))
    {
      ret = fscanf(sprite,"%d %d %d %d %d %d %d\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
      printf("%d champs convertis\n", ret) ;
      printf("  choixSprite = %d\n", choixSprite) ;
      printf("  position.x = %d\n", position.x) ;
      printf("  position.y = %d\n", position.y) ;
      printf("  partieRestante.w = %d\n", partieRestante.w) ;
      printf("  partieRestante.h = %d\n", partieRestante.h) ;
      printf("  partieRestante.x = %d\n", partieRestante.x) ;
      printf("  partieRestante.y = %d\n", partieRestante.y) ;
      printf("\n") ;
    }
 
  fclose(sprite) ;
 
  return 0 ;
}

Ce petit programme fonctionne correctement (chez moi ;)). Essaye de le compiler et de l'executer, peut-être que cela t'aidera à trouver le problème.
Nota : Le fichier sprite.txt correspond au format que tu as posté.

28/05/2006, 15h20
Pingouin
Elijha > Pourquoi tu redéfinis la structure SDL_Rect, au lieu d'includer directment la SDL ?

A ce propos, je suis allé voir dans les fichiers SDL, et DSL_Rect est défini comme suit : (moi ça ne me dit rien mais peut-être vous) ...
Code:

1 2 3 4 5 typedef struct SDL_Rect { Sint16 x, y; Uint16 w, h; } SDL_Rect;
J'essaie ça tout de suite
28/05/2006, 15h23
Elijha

Citation:

Envoyé par Pingouin

Elijha > Pourquoi tu redéfinis la structure SDL_Rect, au lieu d'includer directment la SDL ?

J'essaie ça tout de suite

Heu.... j'ai eu la flemme de la télécharger ;)
28/05/2006, 15h27
Pingouin
Le problème vient peut-être des variables de la structure SDL_Rect ..... ça n'a pas l'air d'être des int très communs (je ne connais pas du tout ça) :
Code:

1 2 3 4 5 6 typedef struct SDL_Rect { Sint16 x, y; Uint16 w, h; } SDL_Rect;
28/05/2006, 15h40
Elijha
Citation:
Envoyé par Pingouin

Le problème vient peut-être des variables de la structure SDL_Rect ..... ça n'a pas l'air d'être des int très communs (je ne connais pas du tout ça) :

Code:

1 2 3 4 5 6 typedef struct SDL_Rect { Sint16 x, y; Uint16 w, h; } SDL_Rect;
Je pense pas. Après qql recherche dans les headers de SDL, tu trouves les déclarations (SDL_stdinc.h) suivantes
Code:

1 2 typedef int16_t Sint16; typedef uint16_t Uint16;
et ensuite les déclarations (SDL_config_win32.h) suivantes
Code:

1 2 typedef signed short int16_t; typedef unsigned short uint16_t;
Mais on sait jamais....

[EDIT]
Bingo, tu as raison ! Avec signed short et unsigned short, une valeur sur 2 est convertie.
28/05/2006, 15h51
Pingouin

Lol, là c'est vraiment un coup de chance .... mais what is the solution ? ^^ Ne pas utiliser fscanf ni sscanf ?

Après qql brèves recherches sur fscanf, le format de lecture approprié serait "%hd" et non "%d" seulement.
Code:

1 2 3 4 ret = fscanf(sprite,"%d %hd %hd %hd %hd %hd %hd\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
Je pense (mais il reste à confirmer) que cela vient de la taille des variables. Un unsigned int = 4 octets, et un unsigned short = 2.

28/05/2006, 15h56
Elijha

Pourquoi %hd

Citation:

Envoyé par man scanf

h Indique que la conversion sera de type dioux ou n et que le pointeur suivant est un pointeur sur un short int (plutôt que sur un int).

Voila, j'espère que cela fonctionne maintenant :)
28/05/2006, 15h58
Pingouin

Youhou tu as raison !!! Ca fonctionne bien avec un %hd.

Je ne sais pas comment te remercier :oops: :D :) ... vivent les membres de ce forum !

J'aimerais juste savoir si tu as certains sites / bases de données en marque page où tu recherches toutes ces infos, ou si .... google est ton ami ^^ :lol: ?

Merci encore !
28/05/2006, 16h02
Elijha

Tu as encore trouvé ::google:
http://www.linux-france.org/article/...3/scanf-3.html

bye... (n'oublis pas le p'tit bouton :resolu:)

Citation:
Envoyé par Elijha
Après qql brèves recherches sur fscanf, le format de lecture approprié serait "%hd" et non "%d" seulement.
Code:

1 2 3 4 ret = fscanf(sprite,"%d %hd %hd %hd %hd %hd %hd\n", &choixSprite, &position.x, &position.y, &partieRestante.w, &partieRestante.h, &partieRestante.x, &partieRestante.y);
Je pense (mais il reste à confirmer) que cela vient de la taille des variables. Un unsigned int = 4 octets, et un unsigned short = 2.
Je re commande pas du tout cette technique qui n'est pas portable, mais plutôt :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 int choixSprite; int position_x; int position_y; int partieRestante_w; int partieRestante_h; int partieRestante_x; int partieRestante_y; ret = fscanf(int, "%d %d %d %d %d %d %d\n" , &choixSprite , &position_x , &position_y , &partieRestante_w , &partieRestante_h , &partieRestante_x , &partieRestante_y); if (ret == 7) { position.x = position_x; position.y = position_y; partieRestante.w = partieRestante_w; partieRestante.h = partieRestante_h; partieRestante.x = partieRestante_x; partieRestante.y = partieRestante_y; }
Le principe étant de rester indépendant des types dont on a pas la maitrise.

Le choix de int couvre de façon portable la plage - 32767 à 32767. Si cétait insuffisant, passer en unsigned int ("%u") ou long ("%ld") ou unsigned long ("%lu").