Discussion :

[Kropernic]

Hello,

Ce matin dans les logs d'erreur, une IOException sur l'accès à un fichier (je me demande bien par quel process le fichier était déjà ouvert mais c'est un autre débat).

J'ajoute donc un peu de sécurité à mon code et je suis arrivé à ceci :

Code :

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

1
2
3
4
5
6
7
8
9
        While True
            Try
                sw = New IO.StreamWriter(storageFolder & "\" & Me.DTO.Store.Code & "\" & fileName, True)
                Exit While
            Catch ex As Exception
                Threading.Thread.Sleep(1000)
                Continue While
            End Try
        End While

Même si je pense que ça fera bien le taff, je me demande s'il n'y a pas plus propre comme manière de faire.

Laissez aller votre créativité ^^.

[worm83]

Salut,

Déjà faut penser à fermer et disposer un stream (même si le dispose fait le close dans ce cas-ci), sinon tu te retrouve avec un handler sur ton fichier qui ne sera pas libéré.

Ensuite, à quoi sert ton Thread.Sleep (1000 en plus ça fait beaucoup) ?? Ce n'est certainement pas ici que tu doit décider de combien de temps le thread doit attendre.

Cordialement.

[Kropernic]

Salut,

Déjà faut penser à fermer et disposer un stream (même si le dispose fait le close dans ce cas-ci), sinon tu te retrouve avec un handler sur ton fichier qui ne sera pas libéré.

Ensuite, à quoi sert ton Thread.Sleep (1000 en plus ça fait beaucoup) ?? Ce n'est certainement pas ici que tu doit décider de combien de temps le thread doit attendre.

Cordialement.

Le 1000, c'est un détail, je pourrais bien sûr mettre moins...

C'est juste sur la technique du while true avec le try catch dedans que je me pose des questions.

Quant au close du stream, cela se fait après que le traitement sur le fichier ait été fait. Vu que le code ne concerne que l'accès au fichier, il n'est normal que tu ne le vois pas...

[sinople]

Faudrait un peu le contexte de ton bout de code et de ton application.

Selon moi, si l'écriture dans un fichier fait suite à une action de l'utilisateur (export donnée dans un fichier par exemple). Il est peut être judicieux de mettre une boîte de dialogue Réessayer, Abandon pour contrôler ta boucle et avoir une deuxième condition de sortie. Parce que le fichier est peut être effectivement ouvert ailleurs pour une bonne raison :-)

S'il s'agit d'une écriture en tâche de fond (un fichier de log par exemple), à priori je poserai un compteur pour sortir de la boucle au bout d'un moment (à voir comment cette nouvelle sortie est gérée).