[C++] Vérifier l'existence de fichiers

Bonjour,

J’essaye de vérifier l’existence de deux fichiers envoyés en paramètres de mon programme C++ :

#include <iostream>
using std::cout;
using std::endl;

bool fileExists(const char* fileName)
{
    FILE* fp = NULL; // will not work if you do not have read permissions
    fp = fopen(fileName, "r");
    
    if (!fp)
        return false;
    
    fclose(fp);
    return true;
} 

int main(int argc, char* argv[])
{
    if (argc != 3)
    {
        cout << "Le programme a besoin de deux arguments pour démarrer :"
             << endl << "(1) Le fichier de la base de donnée, au format BLAST"
             << endl << "(2) Le fichier de la protéine d'entrée, au format FASTA."
             << endl;
        exit(1);
    }

    // On vérifie si les fichiers existent
    if (!fileExists(argv[1]) || !fileExists(argv[2]))
    {
        cout << "Merci de renseigner des fichiers valides." << endl;
        exit(1);
    }
    
    return 0;
}

Est-ce une bonne manière de procéder ?
Il existe peut-être une méthode plus simple, plus pro ?

C++17 à la rescousse !!!

https://en.cppreference.com/w/cpp/filesystem/exists

C’est un problème plus complexe qu’il n’en a l’air.

En premier lieux, je dirais que exists ne suffit pas. Car il ne vérifie ni que le chemin spécifié est un bien fichier ni que tu puisses le lire !

Je te conseil de voir la fonction std::filesystem::status plutôt.

Sinon, il y a toujours le problème de désynchronisation. Tu vérifies à un instant T que le fichier existe mais si ça se trouve 3 lignes de code plus loin, il n’existe plus.

En C, cette fonction aurait été bien utile, en C++, j’ai un peu du mal à comprendre l’utilité. On préfère gérer des execptions en C++. On tente de lire le fichier, si ça echoue on regarde pourquoi.

On ouvre le fichier tout simplement, et en général avec les flux, pas avec le RAII-less fopen.

Cette option m’intéresse, qu’entends-tu par là ?
Pourquoi déconseilles-tu fopen ?

Parce qu’en cas de retour prématuré comme une exception par exemple, l’opération duale (fclose()) ne sera jamais appelée. Certes, on peut encapsuler dans un unique_ptr avec deleter dédié, mais c’est tellement plus simple d’utiliser un *fstream…

https://alexandre-laurent.developpez.com/cpp/retour-fonctions-ou-exceptions/