Discussion :

[chicken92000]

Bonjour,

Je veux intégrer une partie de mon code dans une fonction.

J'ai d'abord essayé de mettre mon code directement dans le main et tout se passe bien.

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
#include <vector>
#include <stdio.h>
#include <windows.h>
#include <iostream>
 
using namespace std;
 
int main(void)
{
    vector<char*> directory_name;
    WIN32_FIND_DATA File;
    HANDLE hSearch;
    int i;
    i=0;
 
    hSearch = FindFirstFile("*.*", &File);
    if (hSearch != INVALID_HANDLE_VALUE)
    {
        do { if (File.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY){            
             //printf("%s\n", File.cFileName);
             directory_name.push_back(File.cFileName);
             printf("%s\n",directory_name[i]);}
        } while (FindNextFile(hSearch, &File));
 
        FindClose(hSearch);
    }
    return 0;
}

Mais maintenant quand j'essaye de mettre dans une fonction spécifique, je n'arrive même pas à lancer la fonction.

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 <vector>
#include <stdio.h>
#include <windows.h>
#include <iostream>
 
using namespace std;
 
void vector_directory_name (void)
{
vector<char*> directory_name;
WIN32_FIND_DATA File;
HANDLE hSearch;
int i;
i=0;
 
    hSearch = FindFirstFile("*.*", &File);
    if (hSearch != INVALID_HANDLE_VALUE)
    {
        do { if (File.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY){            
             directory_name.push_back(File.cFileName);
             printf("%s\n",directory_name[i]);}
        } while (FindNextFile(hSearch, &File));
 
        FindClose(hSearch);
    }
}     
 
int main(void)
{ 
    vector_directory_name();
    return 0;
}

Je suis désolé mais je débute en c++ et j'apprécierai votre aide.

Merci

David

[white_tentacle]

Il n'y a rien qui me choque à première vue.

À part une chose, je ne suis pas sûr que tu aies le droit de stocker le résultat de File.cFileName --> utilise plutôt un vector<string> pour stocker les chaînes de caractères.

Qu'entends-tu exactement par "je n'arrive même pas à lancer la fonction" ? Ça ne compile pas ? Ça ne s'exécute pas ?

[chicken92000]

Désole de ne pas être très précis.

Si ça compile bien, l'exe semble semble fonctionné.

Mais le printf("%s\n",directory_name[i]) ne s'affiche pas dans ma fenêtre.

ce qui me fait dire que peut être je n'arrive même pas dans ma fonction.

David

[white_tentacle]

Ce n'est pas possible que tu ne rentres pas dans ta fonction .

Tu peux d'ailleurs le vérifier en mettant un printf("toto"); au début de ta fonction (note, en c++, on préfère cout << "toto"; en ayant inclus l'en-tête <iostream> )

Ce qui est probable, c'est que tu ne rentres pas dans ta boucle car FindFirstFile ne trouve rien. En effet, File n'est pas initialisé, donc tu dois tomber sur le cas INVALID_HANDLE_VALUE.

Tu es sous Visual, il y a un mode pas-à-pas que peux lancer (avec F9 il me semble) qui devrait bien t'aider.

[chicken92000]

Maintenant cela semble fonctionner. Je ne comprends pas trop.

Par rapport vector<string>. J'ai essayé mais lors de la compilation il me retourne une erreur 31 C:\Dev File\test_repertoire.cpp [Warning] cannot pass objects of non-POD type `struct std::basic_string<char, std::char_traits<char>, std::allocator<char> >' through `...'; call will abort at runtime

[Noxen]

std::string a une méthode char* c_str() qui renvoie un pointeur sur une chaîne de caracètre C-style. Donc tu peux stocker des chaînes dans un vector<string>, et utiliser tes fonctions demande un char* avec c_str().

Code :

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

1
2
3
 
vector<string> directories;
MaFonctionQuiPrendUnCharEtoile(directories[3].c_str());

De même, std::string a un constructeur qui prend un char*, donc tu peux stocker des chaînes C-style dans un vector<string>, il y aura un appel implicite au constructeur de std::string.

Code :

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

1
2
3
 
vector<string> directories;
directories.push_back(File.cFileName()); // Du moins si l'objet ne peut pas renvoyer une std::string.

[Médinoc]

Attention: c_str() retourne un char const *.

[chicken92000]

Encore merci pour votre aide.

Par rapport à l'affichage vaut il mieux utiliser cout ou printf?

David

[Médinoc]

En C++, je dis cout sans hésiter.

[chicken92000]

Maintenant j'essaye de renvoyer le vector que j'ai crée dans ma fonction vector_directory_name au niveau de mon main.
Mais je ne vois pas quel types mettre au niveau du prototype de ma fonction pour le retour.

David

[chicken92000]

Merci encore, j'ai trouvé ma réponse.

David