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.