mon programme qui lit dans un fichier .ase ne se termine jamais !

mon programme qui lit dans un fichier .ase ne se termine jamais ! - C++ - Programmation

Marsh Posté le 26-01-2003 à 14:41:40    

il semblerait que mon programme fait une boucle infinie.
la variable "nb_indices[b]"qui est vers la fin de la fonction "lecture"  s'affiche mais pas les "hello"
je n'ai pas d'erreurs, le programme s'exécute mais ne sort jamais de la fonction.
pouvez vous m'aider ?
voici le code :

Code :
  1. #include <stdio.h>
  2. #include <iostream.h>
  3. #include <string.h>
  4. #include <conio.h>
  5. #include <stdlib.h>
  6. //#include <GL/glut.h>
  8. #define nb_vertices 20000
  9. #define nb_faces 20000
  11. struct donnees
  12. {
  13.  float tab_vertices [nb_vertices][3] ;
  14.  int tab_sommets [nb_faces][3] ;
  15. };
  16.     donnees tab_struct[50] ;
  17. int nb_indices[100] ;
  19. int i=0 ;
  20. int j=0 ;
  21. int k=0 ;
  22. int n=0 ;
  23. int m=0 ;
  24. int l, b ;
  25. int a=0 ;
  27.     void lecture ()
  28.     {
  30.  char tmp [100] ;
  31.  char tmp2 [100] ;
  32. char tmp3 [100] ;
  33.  int temp [nb_vertices] ;
  34.  char passe_ligne [200] ;
  37.  char nom_fichier[] = "bob.ase" ;
  40.  FILE *pointeur ;    //pointeur sur le fichier  
  42.  if ((pointeur = fopen(nom_fichier, "r" )) == NULL)
  43.           return ;
  45. do
  46.  {
  47.   fscanf(pointeur, "%s", tmp) ;
  48.  if(strcmp ("*MESH_VERTEX", tmp) == 0)
  49.  {
  50.   a++ ;
  51.  }
  52.  }
  53.  while (!feof(pointeur)) ;
  55. fseek (pointeur, 0,SEEK_SET) ;
  57.  for (b=1;b<=a;b++)
  58. {
  60.  do
  61.   {
  63.    fscanf(pointeur, "%s", tmp) ;
  64.   }
  65.   while (strcmp ("*MESH_VERTEX", tmp) != 0) ; //passe toutes les chaines en revue jusqu'a qu'a ce que tmp soit égal à chaine ("*MESH_VERTEX" )
  68.   do//rempli le tableau de structure avec des coordonnées de vertices
  69.   {
  71.    fscanf (pointeur, "%d%f%f%f%s", &(temp[0]) ,&(tab_struct[b].tab_vertices[i][0]),
  72.    &(tab_struct[b].tab_vertices[i][1]),&(tab_struct[b].tab_vertices[i][2]), tmp) ;
  74.    cout << tab_struct[b].tab_vertices[i][0]<<" "<<tab_struct[b].tab_vertices[i][1]<<" "<< tab_struct[b].tab_vertices[i][2]<<endl ;
  75.   i++ ;
  77.   }
  78.        while (strcmp (tmp, "}" )!=0) ;
  82.   do
  83.   {
  85.   fscanf(pointeur, "%s", tmp2) ;
  86.   }
  87.   while (strcmp ("A:", tmp2) != 0) ;
  91.  do //rempli le tableau de la structure avec le numero des sommets
  92.  {
  95.   fscanf (pointeur, "%d%s%d%s%d", &(tab_struct[b].tab_sommets[j][0]),tmp,
  96.     &(tab_struct[b].tab_sommets[j][1]),tmp, &(tab_struct[b].tab_sommets[j][2])) ;
  97.   //cout << tab_struct[b].tab_sommets[j][0]<<" "<<tab_struct[b].tab_sommets[j][1]<<" "<<tab_struct[b].tab_sommets[j][2]<< endl ;
  99.   fgets (passe_ligne, 200, pointeur) ; //saute une ligne dans le fichier où les données ne servent pas
  100.   fscanf (pointeur, "%s%s%s", tmp3, tmp, tmp);
  101.   j=j+1 ;
  102.   l=l+1 ;
  103.  }
  104.  while (strcmp(tmp3, "}" )!=0) ;
  106.  nb_indices[b] = l*3;
  107.  cout << nb_indices[b]<<endl ;
  108.  l=0 ;
  109.     }
  110.     cout<< "hello1"<<endl ;
  113.      fclose (pointeur) ;
  116.     }
  118.    
  120.  
  121. void main ()
  122. {
  123. lecture () ;
  124. cout<< "hello2"<<endl ;
  126. }


Message édité par airseb le 26-01-2003 à 14:42:15
Reply

Marsh Posté le 26-01-2003 à 14:41:40   

Reply

Marsh Posté le 26-01-2003 à 15:39:14    

:hello:

Reply

Marsh Posté le 26-01-2003 à 15:47:25    

:ouch: désolé, je sors tout de suite

Reply

Marsh Posté le 26-01-2003 à 15:49:43    

++Taz a écrit :

:ouch: désolé, je sors tout de suite


 
il va faire une attaque cardique le petit père taz  [:tinostar]


---------------
From now on, you will speak only when spoken to, and the first and last words out of your filthy sewers will be "Sir!"
Reply

Marsh Posté le 26-01-2003 à 15:57:59    

pourquoi fais tu une première passe pour compter les sections *MESHVERTEX ?
 
a priori ca sert à rien :heink:  
 
Et puis sinon des noms de variables explicites ca serait déjà une bonne idée pour du code lisible; par ce que a, b, i ... :o


Message édité par schnapsmann le 26-01-2003 à 15:58:24

---------------
From now on, you will speak only when spoken to, and the first and last words out of your filthy sewers will be "Sir!"
Reply

Marsh Posté le 26-01-2003 à 16:06:10    

SchnapsMann a écrit :

pourquoi fais tu une première passe pour compter les sections *MESHVERTEX ?
 
a priori ca sert à rien :heink:  
 
Et puis sinon des noms de variables explicites ca serait déjà une bonne idée pour du code lisible; par ce que a, b, i ... :o


 
je compte les *MESHVERTEX  pour savoir combien il y a d'objets
et combien de fois la boucle for doit s'éxécuter (une fois par objet)

Reply

Marsh Posté le 26-01-2003 à 16:09:43    

airseb a écrit :


 
je compte les *MESHVERTEX  pour savoir combien il y a d'objets
et combien de fois la boucle for doit s'éxécuter (une fois par objet)  


 
C'est bien ce qui me semblait, la première passe ne sert à rien. En  traitant les sections à la volée ca marcherait aussi bien puisque tu n'as pas besoin de connaitre le nombre de sections pour en traiter une.
 
Pour ce qui est de la nom terminaison de ton programme je te conseille le débugueur parce que la il doit sans doute boucler sur une erreur de parsing.


---------------
From now on, you will speak only when spoken to, and the first and last words out of your filthy sewers will be "Sir!"
Reply

Marsh Posté le 26-01-2003 à 16:12:31    

SchnapsMann a écrit :


 
C'est bien ce qui me semblait, la première passe ne sert à rien. En  traitant les sections à la volée ca marcherait aussi bien puisque tu n'as pas besoin de connaitre le nombre de sections pour en traiter une.
 
Pour ce qui est de la nom terminaison de ton programme je te conseille le débugueur parce que la il doit sans doute boucler sur une erreur de parsing.


excuse moi mais : c quoi une erreur de parsing et comment je dois utiliser le debugueur ?

Reply

Marsh Posté le 26-01-2003 à 19:16:14    

Une erreur de parsing, c'est une erreur d'interprétation d'un texte (un fichier texte, dans ton cas). Avec le débuggeur, tu pourras vérifier au fur et à mesure les résultats des "fscanf", et trouver où se situe le problème. Mais pour apprendre à l'utiliser, mieux vaux lire la doc fournie avec (et surtout, ça dépend de ton environnement de travail).

Reply

Marsh Posté le 26-01-2003 à 22:09:11    

Quelques conseils
1. indente un peux mieux...
2. évite les do... while pour parser, vaut mieux un while tout seul, le "do" implique que le code est executé au moins une fois, c'est dangereux!!
3. fscanf revoie comme valeur de retour le nombre paramètres lus, ça pourrait t'aider.
4. %d%f%f%f à éviter, ou alors écris le séparateur entre les formats, c'est plus clair.
5. un mélange de C et de C++, c'est assez casse-gueule (fscanf et cout, ça fait tache, ainsi que stdio.h et iostream.h )
6. fscanf et "%s%s%s", c'est trèèès casse geule
7. les variables globales sont à proscrire dans la majorité des cas!!!


---------------
Angel Eyes, j'en raffole tous les matins... :pt1cable:  
Reply

Marsh Posté le 26-01-2003 à 22:09:11   

Reply

Marsh Posté le 27-01-2003 à 03:14:04    

feof n'est vrai que lorsqu'une lecture a échoué, pas dès que la prochaine va échouer.
 
Je ne me suis pas attardé sur le reste...


---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone
Reply

Marsh Posté le 27-01-2003 à 09:20:13    

Angel_Eyes a écrit :

Quelques conseils
1. indente un peux mieux...
2. évite les do... while pour parser, vaut mieux un while tout seul, le "do" implique que le code est executé au moins une fois, c'est dangereux!!
3. fscanf revoie comme valeur de retour le nombre paramètres lus, ça pourrait t'aider.
4. %d%f%f%f à éviter, ou alors écris le séparateur entre les formats, c'est plus clair.
5. un mélange de C et de C++, c'est assez casse-gueule (fscanf et cout, ça fait tache, ainsi que stdio.h et iostream.h )
6. fscanf et "%s%s%s", c'est trèèès casse geule
7. les variables globales sont à proscrire dans la majorité des cas!!!


 
merci pour tes conseils  :D  
mais je voudrais savoir pourquoi : fscanf et "%s%s%s" c casse gueule ?

Reply

Marsh Posté le 27-01-2003 à 13:34:02    

un scanf (sscanf, fscanf, ...) s'arrete sur le premier caractère ne correspondant pas à la valeur recherchée.
 
par exemple %d s'arrete à une lettre, une virgule ou autre chose n'étant pas un chiffre.  
 
pour %s, il lit une chaine de caractères et s'arrete sur un caractère interprèté comme séparateurs d'une manière parfois assez tordue (espaces, fin de ligne, fin de chaîne,... pas toujours celui que l'on croit), c'est pas évident à maitriser si on commence et c'est une belle source d'erreurs.  
 
Faut être sur de la manière dont scanf va séparer ton %s%s%s.


---------------
Angel Eyes, j'en raffole tous les matins... :pt1cable:  
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed