Discussion :

[Flow_75]

Bonjour,

Je cherchais un moyen de verifier si le fichier etait ouvert par une application.

Je suis tombé sur ce sujet : https://www.developpez.net/forums/d2...ichier-ouvert/

Qui proposait une solution via un try/catch.

Le sujet datant d'un moment (Fin 2006), je me suis dis que j'allais vous demander si le framework avait assez évolué pour proposer une solution plus élégante.

Merci de vos lumières.
F.

[Noxen]

Contrairement aux apparences cette approche est correcte, à cause du TOCTTOU : Time Of Check To Time Of Use. C'est-à-dire que si la vérification et l'utilisation ne constituent pas une opération atomique l'état de la resource peut être invalidé avant l'utilisation et provoquer une erreur.

Code :

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

1
2
3
4
5
6
7
8
if(CanOpenFile(file))
{
  OpenAndUseFile(file);
{
else
{
 ReportError("Can not open file");
}

Concrètement avec ce code si le fichier est disponible à l'appel de CanOpenFile mais qu'il ne l'ai plus au moment de l'appel à OpenAndUseFile (fichier supprimé, ouvert par un autre processus, réseau déconnecté, radiations solaires inopinées, apocalypse zombi...) il y aura une erreur non gérée, il faudra de toute façon se protéger des erreurs inattendues. Donc autant utiliser l'approche optimiste avec gestion des erreurs :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
bool TryOpenAndUseFile(string filePath)
{
  try
  {
    OpenAndUseFile(filePath);
    return true;
  }
  catch(Exception e)
  {
    ReportError(e);
    return false;
  }
}

[Flow_75]

D'accord.
Merci de ton aide.

Cordialement.
F.