Discussion :

[dream_of_australia]

J'essaie de récupérer certaines informations depuis un fichier, ligne par ligne. Le traitement dépend du premier caractère de la ligne.
J'ai des lignes de la forme "lettre double double double" que je dois traiter.
J'ai essayé pas mal de choses mais je suis toujours coincée.
Je ne récupère que des double égaux à 0 (ce qui veut dire que la transformation atof ne s'est pas bien passée).

Voici mon code :

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
17
18
19
20
21
22
23
24
25
26
27
28
char *file = AG_STRING(1);
    FILE *f ;
    char line[LINE_MAX] ; char tabtmp[10] ; 
    double sommets[MAXSIZE][3] ;
    int i = 0, j = 0, k = 0, nbSommets = 0 ;
 
    if ((f = fopen(file, "r")) != NULL) {
           while (fgets(line, LINE_MAX, f) != NULL) {
                 i = 0 ;
                 if (line[i] == '#') { skipLine(f) ; }
                 else if (line[i] == 'v') {
                    i = 1 ;
                    for (k=0 ; k<3 ; k++) {
                        j = 0 ;
                        while (&line[i] != " ") { i++ ; } // on avance
                        while (&line[i] != " ") {   // on copie
                            tabtmp[j] = line[i] ; 
                             i++ ; j++ ; 
                        }
                        tabtmp[j] = '\0' ; // fin de chaine
                        sommets[nbSommets][k] = atof(tabtmp) ; // conversion
                        tabtmp[0] = '\0' ; // chaine vide
                        nbSommets++ ;
                    }
                 }
           }                                                           
          fclose(f) ;                               
    }

Je voudrais un conseil pour m'aider à avancer ou bien une toute nouvelle piste si celle que j'ai suivie n'est pas la bonne.

J'ai essayé pas mal de choses (fscanf et autres) mais je n'arrive à rien!

Merci !

[Invité]

Salut

A quoi sert la première lettre ? Et lesquelles sont valides ? Et à quoi ressemble un fichier type ?

[jeroman]

Code :

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

while (&line[i] != " ")

Ce code est très douteux et ne marchera pas. Ici, tu ne compares pas deux caractères mais tu compares deux adresses, celle où est stocké le caractère dans le tableau et celle où est stocké un espace (qui correspond en fait à une chaîne littérale, située ailleurs en mémoire). La condition sera toujours vraie (opérateur != ). Ce code doit en principe provoquer une boucle infinie.

[sambalien]

C'est quoi la variable que tu utilises comme descripteur de fichier car
j'ai vu une declaration

Code :

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

FILE *f;

et ton plus bas tu mets

Code :

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

f=fopen(file,"r");

je ne vois pas la chaine qui correspond au variable file.

[Médinoc]

@sambalien:

• FILE* est le type "fichier ouvert" du C standard. La structure pointée contient typiquement un descripteur de fichier tel qu'on le connait sous POSIX.
• f est une variable de ce type, initialisée avec le retour de fopen().
• file est une variable de type char*, initialisée avec l'expression AG_STRING(1). Aucune idée de ce qu'il y a derrière cette macro.

@dream_of_australia : Peut-on voir le code de la fonction skipline() ? Vue son interface, je pense qu'elle est bugguée.

[souviron34]

déjà, pourquoi faire simple quand on peut faire compliqué

Code :

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

1
2
3
4
            while (fgets(line, LINE_MAX, f) != NULL) {
                if ( line[0] == 'v') {
                }
            }

suffit à tout passer et ne traiter que les bonnes lignes.

Ensuite, une solution simple et portable suffit pour lire la ligne (à moins que tu aies C99, auquel cas c'est encore plus simple) :

Code :

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

1
2
3
4
5
6
7
8
float f[3] ;
char c ;
 
....
 
sscanf ( line, "%c %f %f %f", &c, &f[0], &f[1], &f[2]);
for ( i = 0 ; i < 3 ; i++)
    sommets[nbSommets][i] = f ;

Si tu bénéficies du format lf, tu peux directement lire tes sommets :

Code :

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

1
2
3
4
5
6
char c ;
 
....
 
sscanf ( line, "%c %lf %lf %lf", &c, &sommets[nbSommets][0], 
           &sommets[nbSommets][1], &sommets[nbSommets][2]);

[dream_of_australia]

Merci pour toutes ces réponses !

@souviron : j'ai lu plusieurs articles déconseillant l'utilisation du scanf, qui apparemment peut tout faire planter.

@medinoc : j'avais effectivement de mon côté viré l'appel à skipline

@jeroman : cette comparaison fonctionne, quand tu mets & devant une variable tu travailles sur sa valeur et non son adresse. (si j'ai bien tout compris !!)

Tout fonctionne, en fait j'avais un problème à l'affichage du test, car je travaille avec une bibli qui n'utilise pas le formatage standard du C (les %d, %i, %f...). J'ai un peu amélioré ma fonction mais dans le principe c'est bon.

[souviron34]

@souviron : j'ai lu plusieurs articles déconseillant l'utilisation du scanf, qui apparemment peut tout faire planter.

Eventuellement si on ne sait pas prévoir correctement la taille de ses buffers, mais c'est scanf dont tu parles, qui est pour la saisie à l'écran, et il y a des manières de faire (voir sur ce forum même).

Le sscanf est d'une utilisation tout à fait sûre...

[diogene]

...
@jeroman : cette comparaison fonctionne, quand tu mets & devant une variable tu travailles sur sa valeur et non son adresse. (si j'ai bien tout compris !!)
...

Absolument pas et jeroman a raison. Si tu mets & devant l'identificateur d'une variable, tu obtiens l'ADRESSE de la variable.
Cette comparaison ne fonctionne pas. elle devrait être :

Code :

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

1
2
                        while (line[i] != ' ') { i++ ; } // on avance
                        while (line[i] != ' ') {   // on copie

Ce code est à l'évidence illogique. Je suppose que ce code devrait être :

Code :

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

1
2
                        while (line[i] == ' ') { i++ ; } // on avance
                        while (line[i] != ' ') {   // on copie

[dream_of_australia]

Oui pardon j'ai répondu vite fait mais en fait j'avais changé ça (comme je l'ai dit j'ai un peu retouché ma fonction).

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
17
if ((f = fopen(file, "r")) != NULL) {
           while (fgets(line, LINE_MAX, f) != NULL) {
                ........
                 else if (line[0] == 'v' && line[1] == ' ') {
                    i = 1 ;
                    for (k=0 ; k<3 ; k++) {
                        while (line[i] == ' ') { i++ ; }
                        marqueur = i ;
                        while (line[i] != ' ' && line[i] != '\n') { i++ ; } 
                        sommets[nbSommets][k] = (GLfloat) atof((char const *) (substr(line, marqueur, i-1))) ;
                    }
                    nbSommets++ ;
                 }
                 ........
           }
           fclose(f) ;
           ..............

Merci à tous !

[sambalien]

bonjour a tous,
dream_of_australia: j'aimerais que tu publie ton code au complet si tu ne vois pas d'inconvénients. Pourrais-tu nous dire également où dans le code tu as buggé. Merci.

[dream_of_australia]

Les utilisateurs ont relevé mes erreurs au cours de cette discussion. Merci à tous d'ailleurs!

Quant au code complet, je pense qu'il apporterait plus de confusion qu'autre chose étant donné que j'utilise des bibliothèques et qu'il y a des appels et des variables spécifiques à ces biblis que j'utilise partout.

As-tu une question spécifique, un point sur lequel je pourrais te renseigner ?

[sambalien]

ce que je veux ce n'est pas vraiment le tester mais ce de le lire et de voir si je comprends a quoi sert ton programme, si tu trouves que ca te derange je n'y ai aucun probleme

[souviron34]

en tous cas, je ré-itère que tu réinventes la roue..

Code :

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

1
2
3
4
5
6
7
8
9
 
                    i = 1 ;
                    for (k=0 ; k<3 ; k++) {
                        while (line[i] == ' ') { i++ ; }
                        marqueur = i ;
                        while (line[i] != ' ' && line[i] != '\n') { i++ ; } 
                        sommets[nbSommets][k] = (GLfloat) atof((char const *) (substr(line, marqueur, i-1))) ;
                    }
                    nbSommets++ ;

est avantageusement remplacé par :

Code :

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

1
2
3
4
5
6
                    sscanf ( &(line[1]), "%lf %lf %lf", 
                             &(sommets[nbSommets][0]),
                             &(sommets[nbSommets][1]),
                             &(sommets[nbSommets][2]));
 
                    nbSommets++ ;

et est du C plus correct...

[dream_of_australia]

@sambalien : j'ai moi-même passé 15 jours à décortiquer les apis des biblis... Je ne vois pas trop l'intérêt pour comprendre le concept de la boucle de lecture d'un fichier de mettre 500 lignes de code... Par contre si quelque chose n'est pas clair pour toi dans la fonction exposée ici je serais ravie de te répondre.

@souviron : oui sûrement ,
1 je ne connais pas d'avance le nombre d'éléments contenus dans chaque ligne (pour un autre cas if qui n'est pas présenté ic), scanf fonctionne-t-il dans ce cas ?
2 j'ai une bibli qui n'utilise pas le formatage standard en C (les lettres %d %fl etc...) et je voudrais éviter de m'emmêler les pinceaux...
3 est-ce que scanf est plus performant en termes d'utilisation mémoire et de temps d'exécution ?

j'ai essayé de mettre ton code avec scanf dans le if où il n'y a toujours que 3 éléments par ligne, mais il ne fonctionne pas (je ne récupère que des 0...).

[souviron34]

@sambalien : j'ai moi-même passé 15 jours à décortiquer les apis des biblis... Je ne vois pas trop l'intérêt pour comprendre le concept de la boucle de lecture d'un fichier de mettre 500 lignes de code... Par contre si quelque chose n'est pas clair pour toi dans la fonction exposée ici je serais ravie de te répondre.

@souviron : oui sûrement ,
1 je ne connais pas d'avance le nombre d'éléments contenus dans chaque ligne (pour un autre cas if qui n'est pas présenté ic), scanf fonctionne-t-il dans ce cas ?
2 j'ai une bibli qui n'utilise pas le formatage standard en C (les lettres %d %fl etc...) et je voudrais éviter de m'emmêler les pinceaux...
3 est-ce que scanf est plus performant en termes d'utilisation mémoire et de temps d'exécution ?

strtok ou strstr ou strchr et sscanf sont standards....

Comme je dis, libre à toi de réinventer le fil à couper le beurre...

[dream_of_australia]

Je n'ai pas dit que scanf n'était pas standard, j'ai simplement dit que j'utilise une bibli qui n'utilise pas le formatage standard (les %d, %c etc... sont différents) et que j'ai peur de m'emmêler entre des appels avec ce formatage non standard (et j'en ai beaucoup dans mon code) et des appels avec le formatage standard au milieu.

Merci pour ton aide en tout cas