projet: trajets d'avions

**BnY** · 08/04/2006, 15h56

re salut a tous... je continue mes questions sur le meme topic pour pas en creer plein...^^

je souhaiterai juste avoir une petite précision sur la fonction fgets... j'ai lu les tutoriels etc.. mais je ne suis pas sûr d'un point:

cette fonction lit un fichier ligne par ligne... mais est-ce qu'elle lit une seule ligne et a nous de la faire fonctionner en boucle ou est-ce qu'elle lit ligne par ligne mais toute seule ?? je penche pour la premiere solution mais bon...

merci a vous de m'éclairer

**BnY** · 08/04/2006, 16h01

bon désolé en fait je crois que je viens de trouver la réponse tout seul

en fait elle lit une ligne jusqu'au retour a la ligne ...c 'est ca ? et donc pour lire toutes les lignes il faut la mettre dans une boucle ?!

**Emmanuel Delahaye** · 08/04/2006, 16h19

Envoyé par BnY

fgets()... cette fonction lit un fichier ligne par ligne...

Non. Elle lit une ligne dans un fichier texte. Pour lire le fichier, il faut boucler, bien sûr...

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

**BnY** · 08/04/2006, 16h33

oki merci c'était bien ce qu'il me semblait comme indiqué plus haut... merci bien !!

**BnY** · 08/04/2006, 17h31

bon je voudrais vérifier que je ne fais pas d'erreurs...

voici une partie de 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
 
      FILE* flot;
      flot=fopen("Nomfich.txt", "r");
 
      for ( j=0; j<=nbrlignes; j++)
         {
             fgets(chaine, strlen(chaine), flot);
             i=4;
             while(chaine[i]!='\t')
                 {
                     NomAero[i]=chaine[i];
                     i++;
                  }
         }

est-ce que la boucle for est adequate et bien utilisée pour lire toutes les lignes avec fgets dans le code ci-dessus ??
et sinon d'apres vous ce code ferait quoi par la suite ? ( c'est juste pour vérifier que ce que j'écris est bon ... )

merci ^^

**Emmanuel Delahaye** · 08/04/2006, 17h50

Envoyé par BnY

bon je voudrais vérifier que je ne fais pas d'erreurs...

fopen() peut échouer...

Code :

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

1
2
3
4
5
 
   flot=fopen("Nomfich.txt", "r");
   if (flot != NULL)
   {
      ...

for() ne va pas (le nombre de lignes n'est pas connu). La lecture peut donc echouer. C'est la fonction de lecture qui esr testée.
Code : Sélectionner tout - Visualiser dans une fenêtre à part
```
1
2
3
 
    while (fgets(..., flot) != NULL)
    {
```

Ce qui interesse fgets(), c'est la taille du tableau :

Code :

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

1
2
3
 
    while (fgets(chaine, sizeof chaine, flot) != NULL)
    {

L'analyse de la ligne, dépend de la définition des données...

**BnY** · 08/04/2006, 17h59

oki c'est ... parfait ^^

de plus, je viens de voir le lien que tu m'as donné avec tes notes et avec tout ca je pense pouvoir me sortir de pas mal de situations avec les fichiers

merci encore

je reviens en cas de nouveau problème ^^

**BnY** · 08/04/2006, 18h13

ah dsl juste une tite précision ...

dans le code:

Code :

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

1
2
while (fgets(chaine, sizeof chaine, flot) != NULL) 
    {

est ce que la fonction fgets est appelée et lit donc chaque ligne au debut de chaque répétition du while ou est- ce que c'est juste un appel pour vérifier sa valeur et alors il faut refaire appel à la fonction dans le while ...?? :

Code :

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

1
2
3
4
5
6
while (fgets(chaine, sizeof chaine, flot) != NULL) 
    { 
       fgets(chaine, sizeof chaine, flot);
       i=4;
       etc...
    }

**Emmanuel Delahaye** · 08/04/2006, 18h24

Envoyé par BnY

Code :

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

1
2
while (fgets(chaine, sizeof chaine, flot) != NULL) 
    {

est ce que la fonction fgets est appelée et lit donc chaque ligne au debut de chaque répétition du while.

Faut-il vraiment qu'on réponde aux questions basiques ? Ouvre ton livre de C...

**BnY** · 08/04/2006, 18h40

euh et si j'ai pas de livre de C et que je ne l'ai pas vu marqué sur les tutoriels, je l'invente ???

:
d'autant plus que c'est plus court d'ecrire "reponse1" ou "reponse2"en repondant a ma question que de reposer une question comme reponse...

d'apres moi il faut refaire appel a la fonction, c'est bien ca ??

:

si je n'abuse pas trop de ton temps si précieux, qu'il t'empeche de repondre a des questions basiques, pourrais tu essayer d'etre simpa 30sec et de repondre juste par oui ou par non à cette question ...

ce n'est pourtant pas dur d'etre gentil, et on en meurt pas encore...
ou dois-je m'attendre à encore un flot d'aigreur dû à on ce sait quoi...

je te remercie quand meme d'avance de m'accorder 30 sec de plus...
en espérant te voir un jour de bonne humeur avec moi !!

**Emmanuel Delahaye** · 08/04/2006, 19h51

Envoyé par BnY

euh et si j'ai pas de livre de C

Il y a des réferences sur mon site, y compris un livre en ligne gratuit.

et que je ne l'ai pas vu marqué sur les tutoriels, je l'invente ???

Pour apprendre la sémantique de while(), ça devrait aller quand même...

d'autant plus que c'est plus court d'ecrire "reponse1" ou "reponse2"en repondant a ma question que de reposer une question comme reponse...

Le problème n'est pas là. C'est toi qui est là pour apprendre à reflechir, et moi, je ne suis pas là pour répondre à tes QCM...

d'apres moi il faut refaire appel a la fonction, c'est bien ca ??

:

Essaye, ça va pas de bouffer le cul...

si je n'abuse pas trop de ton temps si précieux, qu'il t'empeche de repondre a des questions basiques, pourrais tu essayer d'etre simpa 30sec et de repondre juste par oui ou par non à cette question ...

ce n'est pourtant pas dur d'etre gentil, et on en meurt pas encore...
ou dois-je m'attendre à encore un flot d'aigreur dû à on ce sait quoi...

Apprends aussi à trouver les réponses par toi même...

je te remercie quand meme d'avance de m'accorder 30 sec de plus...
en espérant te voir un jour de bonne humeur avec moi !!

Je t'ai passé des référence sur des articles de mon site qui montrent des exmples d'utilisation de fgets(). Ca ne te suffit pas ? Fait des essais, tu verras bien. Prend toi un peu en main. Il n'y aura pas toujours un couillon pour te la tenir quand tu va pisser... C'est pas possible d'être aussi peu entreprenant. C'est le résultat de 30 ans d'étatisme ?

**BnY** · 08/04/2006, 20h07

bon j'en connais un qui va etre content ^^ lol : tu me reprochais de ne pas mettre assez de code, voila du code lol ...

bon que je recapitule... j'ai un fichier qui se nomme FicCarAer.txt qui se présente sous cette forme:

Code :

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

1
2
3
4
5
6
Id Aer   Nom Aer         Fus Horaire  ...
====================================
1           Orly         GMT+0  ...
2           Roissy       GMT+0  ...
16          Le Caire     GMT+2  ...
...

les colonnes de champs sont séparées par deux tabulations et les lignes ne comportent que des caractères (pas de chiffres)

voila une partie de mon code qui me sert à demander a l'utilisateur un aeroport de départ, ouvrir ce fichier, trouver l'aeroport en question, et en extraire les données correspondant (pour l'instant on se contente de l' Id)... :

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
#include <stdio.h>
#include <string.h>
 
void main ()
{
   char Ad[25],Adtemp[25], chaine[200], chtemp[200] ;
   int i, IdAd ;
 
   printf("Entrez l' aeroport de départ\n");
   scanf("%s", &Ad);
   FILE*flot=fopen("FicCarAer.txt", "r"); 
   if(flot!=NULL)        
     { 
        while(fgets(chaine, sizeof chaine, flot)!=NULL); 
          {
             fgets(chaine, sizeof chaine, flot); 
             i=4; 
             while(chaine[i]!='\t') 
                 { 
                    Adtemp[i]=chaine[i]; 
                     i++; 
                  } 
             if (strcmp(Adtemp,Ad)
                {
                   i=0;
                   while(chaine[i]!='\t')
                       {
                          chtemp[i]=chaine[i];
                          i++;
                       }
                   IdAd=atoi(chtemp); 
                }
          }
          if (feof(flot))
             {
                puts ("EOF");
             }
 
         if (ferror(flot))
             {
                perror (FicCarAer);
             }
 
   fclose(flot) ;
     }
 
      else
         {
            printf("erreur d'ouverture de fichier\n");
         }
......... etc .....
}

1/ est ce que déjà avec ce code j'obtiens bien ce que je vous dis au début selon vous ??
2/ pour "sauter" les caractères de l'Id afin d'avoir seulement le nom de l'aéroport je mets :

Code :

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

i=4;

est-ce que cela paraît bon?? pasqu'en fait je me dis qu'il y a deux caractères dans l'Id : soit '1' et '\s' soit '1' et '6' par exemple pour les cas Num Aer = 1 ( orly ) et Num Aer = 16 ( le caire ), les 2 caracteres étant suivis des 2 tabulations on arrive à " i=4; "...
-> si quelqu'un a une meilleure idée... ?! et si quelqu'un a une idée dans le cas où on aurait pas deux caractères dans les cas : Num Aer = 1, 2, 3... mais donc qu'un seul suivi de deux tabulations...?!

je pense par exemple à quelquechose du genre :

Code :

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

1
2
3
4
if (chaine[1]='\t')
  i=3;
  else 
     i=4;

à insérer.. mais bon suis pas trop trop sûr....

3/ enfin si quelqu'un avait quelques explications sur:

if (ferror(flot))
{
perror (FicCarAer);
}

genre est-ce qu'il n'ya pas d'erreurs, qu'est-ce que la fonction perror(nomfich) fait exactement ??

voila et n'hésitez pas à me dire toutes les erreurs que le code contient !!^^

merci par avance

**BnY** · 08/04/2006, 20h24

Essaye, ça va pas de bouffer le cul...

si j'avais un compilateur j'essaierai, et quand j'irais a mon bahut lundi je pourrais essayer... pour l'instant je n'en ai pas donc j'essaie de poser la question sur un forum... qui ne tente rien n'a rien
et je sais je peux télécharger un compilateur, mais je n'ai pas non plus les fichiers en question donc j'aurais du mal a me servir d'un programme manipulant des fichiers avec des fichiers fictifs... nan ?!

et moi, je ne suis pas là pour répondre à tes QCM...

je ne t'ai jamais forcé à répondre à mes questions, d'autres tels que skyrunner me repondaient, et avec bien plus de gentillesse... donc ne te sens pas forcé de répondre à mes "QCM" comme tu dis... et la question n'est pas là non plus, je te fais savoir que soit tu reponds, soit tu ne le fais pas, je ne force personne,.. mais par contre evite de répondre à côté avec une remarque du genre:

Faut-il vraiment qu'on réponde aux questions basiques ? Ouvre ton livre de C...

puisqu'elle ne sert à rien de constructif...

Apprends aussi à trouver les réponses par toi même...

je fais avec les moyens du bord... quand je cherche une reponse, je commence par reflechir au problème avec mes connaissances de base qui ne sont pas enormes en C... apres je cherche sur des tutoriaux en ligne et si je ne trouve pas je demande sur des forums... je vois pas où est le problème dans ma démarche... et comme je tai dit je n'ai pas de compilateur pour le moment donc je ne peux pas essayer mes essais...

quoiqu'il en soit, je t'invite à répondre à mes questions ou à ne pas y répondre selon ton choix... mais désormais à éviter les remarques qui n'ont rien à voir... car je n'en vois toujours pas l'intérêt sur un forum d'entraide... voila merci quand meme de ton aide

**Emmanuel Delahaye** · 08/04/2006, 20h31

Envoyé par BnY

si j'avais un compilateur j'essaierai,

http://www.codeblocks.org/

**BnY** · 08/04/2006, 20h38

oki merci mais comme je t'ai dit les fichiers sont inexistants donc le programme ne va jamais les trouver donc il ne va pas marcher...

bon d'accord ca me servira au moins pour quelques trucs comme la sémantique de while() comme tu disais ...

bon et sinon qu'est ce que tu penses du code plus haut a priori ca va ?? tu vois qqs erreurs ?? enfin si tu avais le temps de répondre aux trois questions que je pose à la fin ce serait super ...

merci d'avance !!

**Kr00pS** · 08/04/2006, 22h56

Je ne pense pas être la meilleur personne pour parler de ça.

Mais bon, j'ai quelques remarques :

1. Remplace scanf par fgets.

2. Aère ton code car c'est assez difficile à lire.

3. Inclus stdlib.h car la fonction int atoi(const char* string); s'y trouve.

4. Passe une ligne à la fin de ton code pour éviter le warning : no newline at end of file.

5. Avec scanf, quand tu passes l'adresse d'une chaîne c'est sans le & vu que l'on peut considerer cela comme un pointeur (je me trompe ?).

6. Dans la ligne 23 de ton code, tu as oublié de fermer la parenthèse.

7. Toujours dans cette même ligne, tu dois complèter strcmp par == 0.

8. Dernièrement j'ai découvert que c'était mieux de mettre int main(void).

9. Tu as mis un point virgule après ta boucle while.

10. Je pense qu'une boucle do while; serait plus appropriée.

Et c'est tout ce que je remarque (je suis assez fatigué)

Kr00pS

**BnY** · 08/04/2006, 23h31

oki merci c'est simpa

j'ai corrigé pas mal d'erreurs et suivi tes conseils

merci encore et bonne soirée

**BnY** · 09/04/2006, 03h45

bon a nouveau j'ai une question ...

voila j'ai un fichier qui se présente comme ceci:

Code :

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

1
2
3
4
5
type avion    categorie         construsteur    distfranch    vitcroisi
=======================================================================
A320          Gros porteur      Airbus          2592          350
Falcon2000    Affaires          Dassault        3100          430
...

je rappelle que le fichier est composé exclusivement de caractères et que les colonnes sont séparées par deux tabulations

je demande a l'utilisateur d'indiquer l'avion qu'il veut, je le compare aux noms d'avions de mon fichier en le lisant ligne par ligne, dés que j'ai la correspondance je mets en mémoire toutes les caractéristiques qui en découlent...

ca peut donner quelquechose du genre: ( j'ai sauté les étapes de déclaration et certaines variables peuvent porter a confusion, n'hésitez pas à me poser des questions

)

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
 
printf("Entrez le nom de l'avion désiré\n");
scanf("%s", avion);
 
FILE*pfcav=fopen("FicCarAv.txt", "r"); 
if(pfcav!=NULL)        
{ 
   while(fgets(chaine, sizeof chaine, pfcav)!=NULL)
   {
      fgets(chaine, sizeof chaine, pfcav); 
      i=0;
      while(chaine[i]!='\t') 
      { 
         aviontemp[i]=chaine[i]; 
         i++; 
      } 
 
      if (strcmp(aviontemp,avion)==0)
         /* je compare le nom donné par l'utilisateur et celui de la ligne du fichier*/
      {
         a=sizeof avion;
         i= a+3;
         /*pour sauter les 2 tab*/
 
         while(chaine[i]!='\t')
         {
            categorie[i]=chaine[i];
            i++;
         }
         /*mise en mémoire de la catégorie*/
 
         i=i+3;
         /* re saut des 2 tab*/
 
         while(chaine[i]!='\t')
         {
            constructeur[i]=chaine[i];
            i++;
         }
 
         i=i+3;
         while(chaine[i]!='\t')
         {
            distf[i]=chaine[i];
            i++;
         }
 
         DistF=atoi(distf);
         /* conversion en entier pour la distance c'est mieux*/
 
         i=i+3;
         while(chaine[i]!='\t')
         {
            vitc[i]=chaine[i];
            i++;
         }
 
         VitC=atoi(vitc);
      }
   }
}

... mais je trouve que la répétition des while est lourde. donc je me suis dit que je pouvais alléger le tout en créant une fonction que j'appelerai a chaque fois et qui prendrait comme argument la caractéristique en question, comme la catégorie de l'avion, et l'entier "i" pour savoir ou commencer la lecture de la chaine qui comprte la ligne du fichier... ce qui donnerait un truc du genre:

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
 
printf("Entrez le nom de l'avion désiré\n");
scanf("%s", avion);
 
FILE*pfcav=fopen("FicCarAv.txt", "r"); 
if(pfcav!=NULL)        
{ 
   while(fgets(chaine, sizeof chaine, pfcav)!=NULL)
   {
      fgets(chaine, sizeof chaine, pfcav); 
      i=0;
 
      fonction(aviontemp,i);
      if(strcmp(aviontemp,avion)==0)
      {
         a=sizeof avion
         i=a+3;
         fonction(categorie,i);
         b=sizeof categorie;
         i=i+b+3;
         fonction(constructeur,i);
 
         ....
 
      }
   }    
}

avec la fonction qui serait :

Code :

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

1
2
3
4
5
6
7
8
9
10
void fonction ( char* arg1, int arg2)
{
   j=arg2;
 
   while(chaine[j]!='\t')
   {
      arg1[j]=chaine[j];
      j++;
   }
}

bon pour tous les courageux qui restent voila à présent mes petites questions... ^^

déjà est-ce que dans l'ensemble le remplacement par une fonction comme je l'ai fait est bon, j'veux dire n'ai-je rien oublié et est-ce que la fonction ferait bien le boulot attendu: remplir la chaine "catégorie" par exemple à l'aide d'une partie de la chaine issue de la lecture de la ligne du fichier??? et est ce que la fonction vaut vraiment le coup dans mon cas ou risque-t-elle de poser des problèmes??

juste une vérification: dans le cas sans fonction à part, je ne marque que :

i=i+3;

en effet je considère que mon i a gardé sa valeur changée par le while après celui-ci,

pour le cas avec la fonction je considère qu'il ne change pas avec l'éxécution de la fonction et donc je suis obligé de lui incrémenter la taille de la chaine d'avant... vous me suivez ???

et la question est : est ce que j'ai bon dans mon raisonnement ?? je sais que dans les tutoriels il y a des passages sur les variables globales et locales, et j'ai deja regardé donc ca ne sert a rien de m'envoyer un lien, je ne suis pas sûr de mon coup même en lisant les cours donc si quelqu'un a l'habitude, ce serait simpa de me confirmer ceci

merci !

bon et sinon je suis toute ouïe à vos remarques et commentaires, si vous voyez des erreurs.... !!

merci à tous les courageux qui m'ont lu jusqu'au bout

**granquet** · 09/04/2006, 04h07

déjà est-ce que dans l'ensemble le remplacement par une fonction comme je l'ai fait est bon

dans l'ensemble, c'est une bonne pratique qui ameliore la lisibilité du code. dans le cas precis ...
ouvre un bouquin de C.

en effet je considère que ...

si tu compte considerer longtemps, il serait temps d'ecrire la norme BnY et de developper ton compilateur.
ouvre un bouquin de C.

bon et sinon je suis toute ouïe à vos remarques et commentaires

voui voui, ouvre un bouquin de C ET d'algorythmie

si vous voyez des erreurs.... !!

pleins ... j'ai un conseil ...
ouvre un bouquin de C.

je ne comprend pas bien ... tu dis etre eleve ... tu n'as jamais eu de cours d'algo?

Emmanuel Delahaye est quelqu'un de competent et de bon conseils; le C c'est difficile. alors t attaquer sans rien comprendre (tu n'y comprend visiblement rien: je ne te confierais pas une seule ligne de code dans un de mes projets: desolé) a un langage difficile sur un projet consequent ... le s utilisateurs du forum ne sont pas la pour ramasser les miettes. si tu n'as pas eu de cours, ouvre un bon bouquin (c'est faisable ...).

enfin, je m'arrete la, ce que j'ai dis, les autres ont essayé de te le faire comprendre; sans succés apparement.

**BnY** · 09/04/2006, 05h13

non je n'ai jamais eu de cours d'algo je suis élève dans une école d'ingénieurs en méca et l'info c'est pas notre fort... j'ai pas le temps d'acheter un bouquin de C, on a des polys de cours et ils comportent la base. C'est tout.

on a un projet d'info a faire, et je pose mes questions sur un forum comme c'est le but d'un forum... je suis d'accord avec toi: je n'y comprends rien, mais si le but d'un forum est de laisser les personnes qui passent leur vie derriere un ecran à critiquer les débutants et bien tant mieux pour vous !! éclatez vous bien !!

je fais de mon mieux pour apprendre à coder avec mon ordi et internet, et j'attends un peu d'aide de la part de gens expérimentés, au moins pour m'aiguiller et me dire ou sont mes erreurs, mais visiblement très peu de ces personnes n'ont leur place dans un forum, puisqu'il s'agit de fournir des réponses, et non des critiques. la critique est bonne quand elle fait évoluer la question...

dsl mais mes questions ne te sont pas destinées... je ne cherche pas un pretentieux experimenté mais bien quelqu'un de juste assez simpa pour accorder un peu de son temps aux autres... en remontant plus haut tu verras que ce genre de personnes existe, et il est agréable d'avoir a faire à eux...

je n'ai peut etre pas eu de cours d'algo mais peut etre que tu manques de cours de social... maintenant tu fais ce que tu veux de ta vie, mais comme je l'ai deja precisé, je ne force personne à m'aider, mais si tu ne souhaites pas m'aider essaie de mieux occuper ton temps que critiquer les autres et répéter 4 fois la meme chose...

merci de ton intervention : elle fait bien avancer les choses !!

et pour info je n'atends pas que tu me confies une ligne d'un de tes projets... j'essaie deja de faire le mien vois tu...

projet: trajets d'avions

C

Vue hybride

Discussions similaires

Partager

Partager