Probleme de lecture de plusieurs fichiers

**chicken92000** · 03/02/2009, 16h07

Bonjour,

J'ai besoin d'une aide par rapport à une fonction que je suis en train de créer.

Je voudrais créer une fonction qui me permettent de lire plusieurs fichiers, en sachant que les fichiers ne sont dans les mêmes directories mais ont le même nom.
Les paramètre de la fonction sont:
file_name -> nom des fichiers qui doivent être lu
current_path -> partie commune du chemin pour les fichiers
subfolders -> Partie variable du chemin pour les fichiers
Ces 3 paramètres me permettent de contruire le chemin pour l'ouverture des différents fichiers.

Mon problème, lors de la compilation de mon code (voir ci-dessous) j'ai une erreur.
62 C:\Projects\Dev File\test_repertoire.cpp no matching function for call to `std::basic_ifstream<char, std::char_traits<char> >::basic_ifstream(std::string&)'

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
vector <string> function_path_read_file (string file_name, string current_path, const vector<string> &subfolders)
{
string input_file;
string temp;
 
    for (int i =0 ; i<subfolders.size();i++)
        //Construction de mon chemin pour ouverture du fichier qui doit être lu
        temp=current_path+'/'+subfolders[i].c_str()+'/'+file_name;    
        // le constructeur de ifstream permet d'ouvrir un fichier en lecture    
        ifstream inputfile(temp);
            if ( inputfile ) // ce test échoue si le fichier n'est pas ouvert
            {
               string ligne; // variable contenant chaque ligne lue
 
               // cette boucle s'arrête dès qu'une erreur de lecture survient
               while (getline( inputfile, ligne ) )
               {
               // afficher la ligne à l'écran
               cout << ligne << endl;
               }
    }
}

Merci par avance.

David

**Trunks** · 03/02/2009, 16h31

Tu ne fais jamais de inputfile.close()?

De plus, c'est

Code :

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

ifstream inputfile(temp.c_str());

et non pas

Code :

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

ifstream inputfile(temp);

**Luc Hermitte** · 03/02/2009, 16h42

Envoyé par Trunks

Tu ne fais jamais de inputfile.close()?

C'est inutile car implicitement exécuté à la sortie de la portée où est défini le flux fichier.

**Trunks** · 03/02/2009, 16h49

Envoyé par Luc Hermitte

C'est inutile car implicitement exécuté à la sortie de la portée où est défini le flux fichier.

Ah?! Tu viens de m'apprendre un truc là !

C'est pas mentionné dans la référence

**Goten** · 03/02/2009, 17h01

C'est pas la référence... La classe ifstream comme toute les class de la STL implémente le RAII, donc pas besoin d'appeler la méthode close(), sauf dans cas particuliers (on en a parlé dans un topic récemment).

**Trunks** · 03/02/2009, 17h05

Envoyé par Goten

La classe ifstream comme toute les class de la STL implémente le RAII, donc pas besoin d'appeler la méthode close(), sauf dans cas particuliers (on en a parlé dans un topic récemment).

Merci, je ne le savais pas ! Comme quoi, on en apprend tous les jours

**chicken92000** · 03/02/2009, 17h22

Merci beaucoup mieux.

Mais j'ai encore un problème au niveau de ma boucle for. Car la boucle ne me retourne que la dernière valeur de mon vector (le vecteur contient 4 lignes).

David

**Luc Hermitte** · 03/02/2009, 17h40

Ni ce site, ni cppreference ne sont des sites de référence sur le C++ -- ne vous laissez pas abuser.

dinkumware et roguewave sont en revanche d'excellentes références.

Ceci dit, sur ce point, j'ai comme l'impression qu'il faut lire entre les lignes dans la norme. En gros, il est suggéré que c'est comme si les basic_*fstream disposaient d'un membre de type basic_filebuf. Et donc là, nous sommes sensés en déduire que la destruction du flux fichier entraine la destruction de son membre, qui elle entraine sa fermeture. Même si dans les faits l'implémentation que l'on utilise ne dispose pas d'un tel membre.

**chicken92000** · 04/02/2009, 09h18

Bonjour,

Personne n'a une idée pour mon problème de boucle for.

Car j'ai un peu l'impression de tourner en rond, et je ne comprend pas pourquoi ma boucle ne me retourne uniquement la derniere valeur de mon vector.

Merci

David

**3DArchi** · 04/02/2009, 09h27

Envoyé par chicken92000

Bonjour,

Personne n'a une idée pour mon problème de boucle for.

Car j'ai un peu l'impression de tourner en rond, et je ne comprend pas pourquoi ma boucle ne me retourne uniquement la derniere valeur de mon vector.

Merci

David

Tu parles de ces lignes ?

Code :

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

1
2
3
 for (int i =0 ; i<subfolders.size();i++)
        //Construction de mon chemin pour ouverture du fichier qui doit être lu
        temp=current_path+'/'+subfolders[i].c_str()+'/'+file_name;

A priori, il n'y a pas de raison.
Es-tu sûr que ton vecteur contient bien plusieurs élément (fait un dump sur subfolders.size())?

P.S. : pourquoi ne pas utiliser des itérateurs pour parcourir ton tableau, c'est fait pour ça.
P.P.S. : et pourquoi ne pas utiliser un for_each. C'est aussi fait pour ça

(et perso, je trouve que ça améliore la lisibilité, mais tout le monde n'est pas d'accord sur ce point).

**Luc Hermitte** · 04/02/2009, 09h28

Affiche au fur et à mesure les noms contenus dans ton vecteur.

**chicken92000** · 04/02/2009, 09h33

Merci pour ta réponse. Et ou c'est bien cette partie.

J'affiche mon size juste avant de rentrer dans ma boucle for. Et il me retourne bien 4. Mais dans ma boucle for il ne m'affiche uniquement un ligne lors du cout de temp.

Code :

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

1
2
3
4
5
    cout << subfolders.size();
    for (int i =0 ; i<subfolders.size();i++)
        //Construction de mon chemin pour ouverture du fichier qui doit être lu    
        temp=current_path+'\\'+subfolders[i].c_str()+'\\'+"information"+'\\'+file_name;
        cout <<temp<< endl;

David

**3DArchi** · 04/02/2009, 09h35

J'aurais tendance à penser que ton problème est au remplissage du vecteur. Es-tu sûr d'y mettre 4 valeurs distinctes ?

**chicken92000** · 04/02/2009, 09h56

Bon j 'ai mis un peu plus de trace (voir ci-dessous) pour avoir plus d'information.

La console affiche donc:
function_read_write_file
4
.
..
test1
Test2
C:\Projects\Swift\Dev File\Test2\Information\events.csv
Les lignes du fichier qui a été ouvert

et je n'ai rien d'afficher pour le test1

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
void function_read_write_file (string file_name, string current_path, const vector<string> &subfolders)
{
string input_file;
string temp;
int nbr_file;
int nbr_line;
 
    cout << "function_read_write_file"<< endl;
    cout << subfolders.size()<< endl;
    for (int i =0 ; i<subfolders.size();i++)
        cout << subfolders[i].c_str()<< endl;
 
    for (int i =0 ; i<subfolders.size();i++)
        //Construction de mon chemin pour ouverture du fichier qui doit être lu    
        temp=current_path+'\\'+subfolders[i].c_str()+'\\'+"information"+'\\'+file_name;
        cout <<temp<< endl;    
        // le constructeur de ifstream permet d'ouvrir un fichier en lecture    
        ifstream inputfile(temp.c_str());
            if ( inputfile ) // ce test échoue si le fichier n'est pas ouvert
            {
               string ligne; // variable contenant chaque ligne lue
 
               // cette boucle s'arrête dès qu'une erreur de lecture survient
               while (getline( inputfile, ligne ) )
               {
               // afficher la ligne à l'écran
               cout << ligne << endl;
               }
    }
}

**3DArchi** · 04/02/2009, 10h03

Au début j'ai pensé qu'il s'agissait d'une erreur de copier/coller de ton code. Mais là, j'ai un doute : n'aurais-tu pas oublié les accolades après le for?

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
void function_read_write_file (string file_name, string current_path, const vector<string> &subfolders)
{
string input_file;
string temp;
int nbr_file;
int nbr_line;
   
    cout << "function_read_write_file"<< endl;
    cout << subfolders.size()<< endl;
    for (int i =0 ; i<subfolders.size();i++)
        cout << subfolders[i].c_str()<< endl;
    
    for (int i =0 ; i<subfolders.size();i++)
    {
        //Construction de mon chemin pour ouverture du fichier qui doit être lu    
        temp=current_path+'\\'+subfolders[i].c_str()+'\\'+"information"+'\\'+file_name;
        cout <<temp<< endl;    
        // le constructeur de ifstream permet d'ouvrir un fichier en lecture    
        ifstream inputfile(temp.c_str());
            if ( inputfile ) // ce test échoue si le fichier n'est pas ouvert
            {
               string ligne; // variable contenant chaque ligne lue
 
               // cette boucle s'arrête dès qu'une erreur de lecture survient
               while (getline( inputfile, ligne ) )
               {
               // afficher la ligne à l'écran
               cout << ligne << endl;
               }
            }
    }
}

**chicken92000** · 04/02/2009, 10h14

Bien vu l'aveugle.

Désole de t'avoir déranger pour un problème aussi bête.

Encore merci pour ta précieuse aide

David

**3DArchi** · 04/02/2009, 10h32

Une règle de codage qui peut paraître lourde mais qui évite ce genre de bug : TOUJOURS mettre des accolades (for, while, if, else ...) même s'il y a une seule ligne de code. Ça ne coûte que la première fois que tu tape ton code

Probleme de lecture de plusieurs fichiers

C++

Discussions similaires

Partager

Partager