Virtual C++ 6.0

Bonjour, je suis actuellement en BTS IG et j'ai un petit souci dans mon code qui doit charger un tableau à partir d'un fichier. Je le link.

Ma déclaration

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#include <iostream.h> // J'inclue #include "stdafx.h" #include "Procedurale_gestion_voiture_VC.h" #include "Procedurale_gestion_voiture_VCDlg.h" #include <fstream.h>   typedef struct // Structure Svoiture. { int Numero_voiture; char Marques_voiture[10]; char Modeles_voiture[10]; int Types_voiture; // 0 pour citadine et 1 pour sportive. int Louer_voiture; // 0 pour libre et 1 pour louer. int Prix_voiture; }SVoiture; SVoiture Voiture; SVoiture TVoiture[500];

---

J'ai penser à appeler ma procédure dans le CDialog::OnInitDialog();

Code:

---

1 2 3 4 5 6 7 8 9 10

BOOL CProcedurale_gestion_voiture_VCDlg::OnInitDialog() { CDialog::OnInitDialog(); SetIcon(m_hIcon, TRUE); // Set big icon SetIcon(m_hIcon, FALSE); // Set small icon   CProcedurale_gestion_voiture_VCDlg::Cprocedure_charger_tableau_voiture(); return TRUE;

---

Et voici la procédure qui vérifie la présence du fichier et qui charge le tableau si celui ci est présent :

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

void CProcedurale_gestion_voiture_VCDlg::Cprocedure_charger_tableau_voiture() // Charge le fichier Fvoiture {       ifstream FVoiture ("FVoiture.txt"); if (! FVoiture) { //l'ouverture a échoué MessageBox("Le programme va maintenant s'arreter.", "Fermeture de session...", MB_OK); // REPERE 1 } else { int I = 1; FVoiture >> Voiture.Numero_voiture >> Voiture.Modeles_voiture >> Voiture.Types_voiture >> Voiture.Louer_voiture >> Voiture.Prix_voiture; // REPERE 2 while (FVoiture.eof()) { TVoiture[I].Numero_voiture = Voiture.Numero_voiture; strcpy(TVoiture[I].Marques_voiture , Voiture.Marques_voiture); // REPERE 3 strcpy(TVoiture[I].Modeles_voiture , Voiture.Modeles_voiture); TVoiture[I].Types_voiture = Voiture.Types_voiture; TVoiture[I].Louer_voiture = Voiture.Louer_voiture; TVoiture[I].Prix_voiture = Voiture.Prix_voiture;   FVoiture >> Voiture.Numero_voiture >> Voiture.Modeles_voiture >> Voiture.Types_voiture >> Voiture.Louer_voiture >> Voiture.Prix_voiture; I = I + 1; } FVoiture.close(); REPERE 4 } }

---

J'ai donc deux problèmes. :calim2:

J'ai ajouter des reperes qui m'ont permis de comprendre se que faisait mon programmes. Je les ai mis sous forme de commentaire pour pas ajouter trop d'information.

Lorsque que je n'ai pas de fichier à l'emplacement requis, le programme me crée le fichier (Alors qu'il est en Ouvrir.Lecture) atteint les REPERES 2 et 4 sans passer par la boucle Alors qu'il devrait m'afficher l'erreur et le REPERE 1.

Lorsque je laisse le fichier dans le repertoire source.
Il plante sur ma boucle While sans fin.

Je n'arrive pas a comprendre mon/mes erreurs. Si quelqu'un peut m'aider ? Merci.

Je laisse aussi le fichier FVoiture si ca peut être utile.

Code:

---

1 2 3

1 peugeot hx 0 1 2000 2 peugeot hx 1 0 3000

---

Je n'ai trop cherché la logique dans votre programme.
Bon nombre de choses me dérangent.

1 -

Citation:

---

Code:

---

1 2	#include <iostream.h> // J'inclue #include "stdafx.h"

---

---

Toujours mettre #include "stdafx.h" en premier, toujours, et mettre les includes des en-tête qui ne change jamais dans le fichier "stdafx.h".
Cela accélère énormément la compilation et vous évitera bien des problèmes qui vous sembleront absconds.

2 -

Citation:

---

Code:

---

1 2 3 4 5 6 7 8 9

typedef struct // Structure Svoiture. { int Numero_voiture; char Marques_voiture[10]; char Modeles_voiture[10]; int Types_voiture; // 0 pour citadine et 1 pour sportive. int Louer_voiture; // 0 pour libre et 1 pour louer. int Prix_voiture; }SVoiture;

---

---

Ca, c'est du C pas beau, utilisez les structs C++

Code:

---

1 2 3 4 5 6 7 8 9

struct Svoiture. { int Numero_voiture; char Marques_voiture[10]; char Modeles_voiture[10]; int Types_voiture; // 0 pour citadine et 1 pour sportive. int Louer_voiture; // 0 pour libre et 1 pour louer. int Prix_voiture; };

---

3 -

Citation:

---

Code:

---

SVoiture TVoiture[500];

---

---

Beurk, le grand tableau globale, pas dynamique pour un sous.
Utilisez par exemple sun std::vector qui vous permettra d'avoir une nombre arbitraire d'élément.
Je ferais de ce vecteur (std::vector) un membre de la classe CProcedurale_gestion_voiture_VCDlg.

4 -

Citation:

---

Code:

---

1 2 3 4 5

BOOL CProcedurale_gestion_voiture_VCDlg::OnInitDialog() { ... CProcedurale_gestion_voiture_VCDlg::Cprocedure_charger_tableau_voiture(); ...

---

---

Quand on n'est dans une méthode d'une classe, il ne faut pas utiliser le nom de la classe comme sélecteur de namespace.
Voici un code bien plus concis, et plus correct en cas de fonction virtuelle.

Code:

---

1 2 3 4 5

BOOL CProcedurale_gestion_voiture_VCDlg::OnInitDialog() { ... Cprocedure_charger_tableau_voiture(); ...

---

5 -
Je ne suis pas un adepte des "ifstream". Je préfère les API non portable de Win32.:aie:
Je pense que ce n'est pas votre programme qui créé ce fichier.
C'est peut-être le processus de compilation ou de lancement du débuggeur.
Pour maîtriser finement la non création des fichiers, j'utiliserais la fonction "fopen" à la place de "ifstream".

6 -

Citation:

---

Code:

---

int I = 1;

---

---

Réservez les majuscules aux constantes et aux MACROS.

Code:

---

int i = 1;

---

7 -

Citation:

---

Code:

---

I = I + 1;

---

---

Code:

---

i++;

---

8 -
Le meilleur pour la fin

Citation:

---

Code:

---

while (FVoiture.eof())

---

---

Tant que vous êtes en erreur on boucle.:aie:
Google est notre ami à tous :
http://www.daniweb.com/forums/post155265-18.html

Merci beaucoup. Bonne continuation.