Discussion :

[arnofly]

Bonjour,

Je ne suis pas encore alaise avec StreamReader & StreamWriter et je viens de m'apercevoir qu'il faut tjrs les fermer apres chaque utilisation. J'ai une petite question toute bête concernant "Dispose".


Admettons la procédure suivante :

Code :

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

1
2
3
4
5
6
7
Dim SR As System.IO.StreamReader
 
SR = "x"
SR.Close
 
SR = "y"
SR.Close

Est-ce utile de mettre un "SR.Dispose" après chaque utilisation ds la procédure (2 ds cet exmple) ou bien 1 seul suffit en fin de procédure ? Peut-être aussi qu'un seul "SR.Dispose" suffirait, placé à la fermeture d'une Form par exemple ?

Merci de m'éclairer sur ce petit détail qui me turlupine

[Faladin]

Assurément, lorsque tu dispose de ton stream (writer ou reader) tu ne pourras pas l'utiliser ensuite, car tu laisses tomber les ressources utiliser pour cet objet. Donc faut pas le faire 2 fois.

À savoir s'il faut le fermer après utilisation ou le fermer sur la fermeture de formulaire, je peux pas te dire ce qui est mieux par manque d'expérience, mais je serais tenté de le faire après la fin de l'utilisation, simplement pour ne pas l'oublier dans mon code.

Mais bon, quelqu'un peut sûrement nous apporté plus de précision...

[bizet]

Salut

Tu dois faire un close une fois le streamReader utilisé, sinon tu ne pourras pas lire un autre flux.

Le Dispose dit au systeme que tu n'utiliseras plus cette variable, qu'il peut donc allouer l'espace mémoire à autre chose.

Le Dispose se fait donc une seule fois et a la fin.

[harz62]

Admettons la procédure suivante :
Est-ce utile de mettre un "SR.Dispose" après chaque utilisation ds la procédure (2 ds cet exmple) ou bien 1 seul suffit en fin de procédure ? Peut-être aussi qu'un seul "SR.Dispose" suffirait, placé à la fermeture d'une Form par exemple ?

Il n'y a qu'un seul flux de fichier qui peut avoir des droits sur un fichier (droit en écriture). Les classes héritant de la classe Abstraite font appel à l'API de Windows, Windows va lui même se charger de verrouiller le fichier. La méthode Dispose sert à libérer les droits d'un fichier à d'autres programmes ainsi que les ressources natives utilisées par la classe.

Le second appel de la méthode Dispose doit normalement déclencher une exception ObjectDisposedException. Pour pouvoir réutiliser cet objet il faut l'instancier à nouveau.

[arnofly]

merci les gars !

C'est nettement plus claire maintenant.

Si j'ai bien compris, meme si j'utilise le fichier en lecture, je dois dabord faire un ".Close" avant de passer en écriture. Que la variable du StreamReader s'appelle "toto" et celle du StreamWriter "titi" on s'en fou, faut fermer d'adord ! C'est bien ça ?

[Faladin]

oui a cause de la gestion de windows comme Harz a dit

[arnofly]

OK, et bien merci à tous.
Du coup j'ai fais un peu de nettoyage ds mon code

Bonne journée

[Pol63]

et sur certaines classes, dispose appelle close
(voir avec reflector)

[harz62]

C'est pas plutôt l'inverse ?

[Pol63]

non, ca serait pas drôle

quand tu disposes un datareader, t'en as plus besoin donc tu peux le closer
par contre quand tu le close, t'en as ptete encore besoin, donc faut pas le disposer

[StormimOn]

Au contraire, en général lorsqu'il y a une méthode Close et une méthode Dispose, la méthode Close appelle le Dispose et non l'inverse.

Pour preuve, le code de la méthode Close des classes Stream, StreamWriter, StreamReader et d'autres sûrement (merci reflector ^^)

Code :

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

1
2
3
4
5
public override void Close()
{
    this.Dispose(true);
    GC.SuppressFinalize(this);
}

Évidemment, suivant les classes la méthode Close n'appelle pas forcément le Dispose, cela dépend de l'implémentation

[Pol63]

je me fiais au datareader que j'avais dejà regardé à une époque et qui lui fait l'inverse (vive l'homogénéité du code )

Code :

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

1
2
3
4
5
Protected Overridable Sub Dispose(ByVal disposing As Boolean)
    If disposing Then
        Me.Close
    End If
End Sub

mais dans un sens fermer un fichier, c'est autre chose ...

[Faladin]

En résumé, pour un débutant (comme moi!), vaut mieux ne pas se fier à Microsoft et appeller les deux méthodes pour ne pas se tromper!

[Pol63]

non car sinon tu disposes 2x le stream

le mieux est d'utiliser reflector, quand on veut savoir ce que ca fait exactement, au moins on peut etre sur

ou sinon msdn, au début on a l'impression que c'est étrange et incomplet (un peu différent de vb6)
mais une fois qu'on a compris comment msdn s'utilise on s'apercoit que tout est bien documenté, et d'ailleurs sur l'aide de close du stream c'est surement écrit que ca dispose ...

[Faladin]

Ouais j'avais compris, ce que je voulais dire c'est de pas appeller la méthode Close en se disant que probablement ça dispose ou vice-versa, mieux vaut clairement appeller ceux qu'on veut quand nécessaire et pas se fier à ce que l'on croit...

Mais bon je m'égare, il est tard, je rentre...

[Sehnsucht]

Ou alors pour éviter d'avoir à se poser trop de question, se servir de "Using",
Exemple tiré du Snippet "Lire un fichier délimité": (il porte sur un TextFieldParser mais c'est pareil pour un Stream)

Code :

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

1
2
3
4
5
6
7
8
9
 
Dim champs As String()
Using parseur As New TextFieldParser("C:\Test.txt")
    parseur.SetDelimiters(",")
    While Not parseur.EndOfData
        champs = parseur.ReadFields()
        ' Ajouter le code pour utiliser les données de la variable champs
    End While
End Using

Comme on peut le voir il n'y a ni appel à Close ni à Dispose mais c'est ce que fait le End Using, il appelle implicitement Dispose (qui lui appelle Close)