Discussion :

[hjvI85]

Bonjour,

En mode Debug la ligne suivante

Code :

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

 std::ifstream FileID (FFileName, std::ios::in);

renvoie pour la variable FileID

Code :

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

<incomplete type>

.

Sachant que

Code :

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

 const char * FFileName;

auriez-vous une idée de l'origine de cette erreur (un défaut à la déclaration ?) ?

Merci d'avance.

[foetus]

C'est ton include qui n'est pas bon ? C'est #include <fstream>.

et au passage c'est plus std::ifstream::in que std::ios::in, voire l'omettre puisque que c'est la valeur par défaut d'après la documentation : "in is always set for ifstream objects (even if explicitly not set in argument mode)." (<- lien cplusplus.com en anglais)

[hjvI85]

Merci pour cette réponse rapide !
ll y a bien le #include <fstream> et j'ai toujours le message d'erreur malgré le std::ifstream::in (et même sans rien).

[koala01]

Salut, et bienvenue sur le forum.

A la base, déjà, pourquoi as tu utilisé un const char * pour représenter le nom du fichier Est-ce que tu le récupère d'une bibliothèque C quelconque

Si tu le récupère directement auprès de l'utilisateur ou que tu le définis "en dur" dans ton code, tu devrais utiliser la classe prévue pour représenter des chaines de caractères, à savoir std::string (qui nécessite l'inclusion du fichier d'en-tête <string>).

Enfin, mais ce n'est qu'un détail, tu devrais veiller à choisir correctement le nom de tes variable, histoire qu'ils représentent clairement l'usage auquel tu les destines.

Si FFileName est un nom "relativement correct" (car, pourquoi avoir deux F au début du nom un seul n'aurait-il pas suffit ) le nom FileID semble plus tôt indiquer que l'on a affaire à l'identifiant (ID) d'un fichier (File), ce qui ne correspond pas vraiment à l'usage que tu en feras .

Pourquoi ne pas l'appeler File (fichier) tout simplement, voire ... Stream (flux) étant donné que c'est ce que tu obtiens

[Bousk]

Pourrait-on avoir plus de code (si ça fait pas 300 lignes) et l'erreur complète copiée telle que le compilateur/linker/IDE l'indique, et non ce que tu penses être faux et le problème ?

[hjvI85]

Bonjour et merci pour ces réponses !

Je tente de vous introduire plus clairement le problème.
Il s'agit bien d'un code initialement en C que je souhaite retranscrire en C++. J'utilise en effet #include <string.h>L'objectif est de lire le fichier "file.txt". Le main est le suivant:

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
#include"iofile.h"
#include <iostream>
using namespace std;
int main()
{TInputFile  InputFile("file.txt",1,1);
    InputFile.open();
    TOutputFile OutputFile("iofdata.out");
    OutputFile.open();

    cout<<InputFile.firstline()<<endl;
    cout<<InputFile.firstcolum()<<endl;
    //-------------------------------------------------
    TSequence RecordSeq(InputFile.nFields(), TSequence::column);
    cout << "nombre de lingne  : " << InputFile.nLines () << endl << endl;
    cout << "nombre de colonne : " << InputFile.nColums () << endl << endl;
    cout << "nombre de fields : " << InputFile.nFields () << endl << endl;}

A l'ouverture de ce fichier, le constructeur est appelé par le programme:

.cpp

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
#include<fstream>
#include<iostream>
#include<string.h>
#include"array.h"
#include"iofile.h"


TInputFile::TInputFile (const char * AFileName, unsigned int nTline, unsigned int nTcol): Tiofile(AFileName)
   {
      config(Tiofile::onrcds);
      nTleLn=nTline;
      nTleCo=nTcol;
      if (Dimension ()){
         FRecord = new TSequence (nFlds, TSequence::column);
         FField  = new TSequence (nRcds, TSequence::line);
         FRcds = new Array<TSequence*> (nRecords());
         if (GetRecords (*FRcds)==NULL){
           std::cerr <<"GetRecords (); File Error : " << GetFileName() << std::endl;
           FRcds = NULL;
           exit(1);
         };
         FFlds = new Array<TSequence*> (nFields());
         if (GetFields (*FFlds)==NULL){
           std::cerr <<"GetRecords (); File Error : " << GetFileName() << std::endl;
           FFlds = NULL;
           exit(1);
         };
      }
     FFileID = new std::ifstream (FFileName);

   }

Au niveau du second "if" du constructeur, la fonction "GetRecords" est appelée:

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
 Array <TSequence*> * TInputFile::GetRecords (Array <TSequence*> &ARcds){
   TSequence TempSeq (FRecord->nElements, TSequence::line);
   std::ifstream * OldFileID = FFileID;
   unsigned int nbRcds = nRcds;
   std::ifstream FileID(FFileName, std::ifstream::in);
   FFileID = &FileID;
   if (!(ARcds.size() <= nRcds)) nbRcds = nRcds;
   else nbRcds = ARcds.size();

   if (FFileID->fail()){
	std::cerr <<"Erreur de manipulation du fichier" << FFileName << std::endl << std::endl;
	return NULL;
	exit(1);
   }
    else{
	InitFilePtr(nTleLn-1, nTleCo-1);
	for (int i= 0; i<nbRcds ; i++){
	   NextRecord(TempSeq);
	   ARcds[i] = TempSeq.clone();
	}
    }
   FFileID = OldFileID;
   FileID.close();
   return &ARcds;
 }

Le programme tourne jusqu'à la fin mais la console affiche "Erreur de manipulation du fichier".
En se plaçant en mode Debug, je remarque que la variable FileID est constamment déclarée en <incomplete type>.

[Bousk]

Si le programme compile alors la syntaxe est correcte.
Que l'IDE affiche des coquilles sur du texte n'est pas un problème et loin d'être rare.
Après ton code en soit est... au mieux médiocre.
FFileID est alloué, puis écrasé. Et c'est juste en regardant en diagonale 3 lignes.