Discussion :

[aminebakr]

Bonjour,

Ce que j'ai actuellement: une fonction qui cherche dans un répertoire des fichiers et me renvoie si le fichier que je cherche se trouve bien dans le répertoire.

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
       DIR *dir;
struct dirent *ent;
if ((dir = opendir ("C:\\Users\\to122333\\Desktop\\tableau\\")) != NULL) {
  /* print all the files and directories within directory */
  while ((ent = readdir (dir)) != NULL) {
 
 
string nomFichier = ent->d_name;
        if(!(nomFichier == ".")&& !(nomFichier == "..")){
            if(CheckSubstring(nomFichier, tableauRechercher ) ){              // tableauRechercher c'est un string (un caractère) que je recherche dans nomFichier
                cout << "tableau  trouvee " << nomFichier<< endl;
                break;
            }
 
 
        }
 
  }
  closedir (dir);
} else {
  /* could not open directory */
  perror ("");
 
}

Ce que je veux faire : Au lieu d'écrire cette fonction à chaque fois dans le main ( je l'utilise 4 fois) j'aimerais juste faire appelle à elle + quand le fichier est trouvé dans le répertoire, l'ouvrir.

Je vous remercie d'avance pour votre aide.

[ternel]

bah, fait une fonction et appelle-la. C'est pas ca?

[aminebakr]

Si c'est ca, mais ca ne veut pas ou plutôt j y arrive pas ( créer la fonction dir) :/ :'(

[ternel]

Qu'entends-tu par "ca veut pas"?
Ca ne compile pas?
Montre nous le code de ton essai (au moins la fonction et celle l'appelant), et la version avant la fonction.

[aminebakr]

C'est juste l'écriture de la fonction hors main qui marche pas. Le reste est bon.

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
 
DIR* opendir(const char* chemin,string nomTableau)
{
    DIR *dir
    struct dirent *ent;
if ((dir = opendir ("C:\\Users\\to122333\\Desktop\\tableau\\")) != NULL) {
  /* print all the files and directories within directory */
  while ((ent = readdir (dir)) != NULL) {
 
 
string nomFichier = ent->d_name;
        if(!(nomFichier == ".")&& !(nomFichier == "..")){
            if(CheckSubstring(nomFichier, nomTableau ) ){
                cout << nomFichier<< endl;
                break;
            }
 
 
        }
 
  }
  closedir (dir);
} else {
  /* could not open directory */
  perror ("");
 
}
}

[Bousk]

- si tu espères retourner quelque chose, faut commencer par avoir une directive return
- retourner un DIR* implique que l'appelant devra faire le closedir lui-même, pas top
> tu peux utiliser std::unique_ptr en surchargeant le second template qui est la fonction de libération afin de fournir un objet qui s'auto-détruirait correctement à la fin de sa portée
- tu fais déjà un closedir dans ta fonction, donc tu retournerais quoi ?
- dans un tel cas, mettre en place une classe de manipulation de dossier serait préférable imo
- std::filesystem arrive en C++17 !

C'est quoi le but de ta fonction ? Juste ouvrir un dossier ? Trouver un fichier à l'intérieur ? Ouvrir le fichier trouvé ? ... ?

[ternel]

J'irai chercher boost::file_system, en attendant le C++17.

Visiblement, tu trouves que c'est bon.
Un bon compilateur (correctement utilisé) trouverait qu'on sort d'une fonction non void sans return.

Active les warnings de compilation, et corrige-les.