Probleme de lecture de plusieurs fichiers

Version imprimable

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:

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

03/02/2009, 16h31
Trunks

Tu ne fais jamais de inputfile.close()? :roll:

De plus, c'est

Code:

ifstream inputfile(temp.c_str());

et non pas

Code:

ifstream inputfile(temp);
03/02/2009, 16h42
Luc Hermitte

Citation:

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.
03/02/2009, 16h49
Trunks

Citation:

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 :?
03/02/2009, 17h01
Goten

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).
03/02/2009, 17h05
Trunks

Citation:

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 ;)
03/02/2009, 17h22
chicken92000

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
03/02/2009, 17h40
Luc Hermitte

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.
04/02/2009, 09h18
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
04/02/2009, 09h27
3DArchi
Citation:

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:

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 :mrgreen: (et perso, je trouve que ça améliore la lisibilité, mais tout le monde n'est pas d'accord sur ce point).
04/02/2009, 09h28
Luc Hermitte

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

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:

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

04/02/2009, 09h35
3DArchi

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

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:

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;
               }
    }
}

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:

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;
               }
            }
    }
}

04/02/2009, 10h14
chicken92000

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
04/02/2009, 10h32
3DArchi

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 ;)