Discussion :

[zooffy]

Bonjour à tous.

Pas trés familier des WinForms (je fais habituellement du Web) je butte sur un détail dans un StreamReader.

Le code que j'ai fais marche bien mais je pense que je pourrais être plus performant.
Voici le code :

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
 
        Dim corps As String = ""
        Dim sr As StreamReader
        Dim sLine As String
        Try
            sr = New StreamReader(NomFichier)
            sLine = sr.ReadLine()
            While Not sLine Is Nothing
                corps &= sLine
                sLine = sr.ReadLine()
            End While
        Finally
            'Fermeture streamreader
            If Not IsNothing(sr) Then sr.Close()
        End Try

Si je comprends bien ce code (que j'ai pris dans la FAQ) il lit ligne aprés ligne et charge la variable corps de la même manière. Mais j'ai juste besoins qu'il prenne tout le contenu du fichier et qu'il le mette dans la variable corps.

Egalement, dans le Finally, il me dit que sr est utilisé avant qu'une valeur lui soit assigné et ça me mets un avertissement qui me gêne un peu.

Pouvez vous m'aider ?

[Pol63]

pour l'avertissement, rien de génant
en mettant Dim sr As StreamReader = Nothing il disparaitra

c'est juste un avertissement, si tu sais ce que tu fais avec y a pas de soucis

de plus If Not IsNothing(sr) teste s'il a été instancié, donc encore moins de soucis

le compilateur en plus de chercher les erreurs de codage cherche ce qu'il lui semble pouvoir provoquer des erreurs à l'execution



ce code n'est pas optimisé, il stringbuilder ammène beaucoup plus de performances qu'un string pour de la concaténation multiple

de plus si tu veux un string à partir d'un fichier, y a deja un truc qui le fait

Code :

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

1
2
3
 
dim s as string 
s = System.IO.File.ReadAllText(chemindufichier)

[zooffy]

Merci sperot51, c'est exactement ce qu'il me manquait. Un truc qui fait tout d'un coup. Du coup, perf et économie de code (en bon informaticien, je suis fainéant ! )

Sinon, l'averstissement, oui, je sais que c'est pas trés grave, mais j'aime pas quand y en a, parce que, justement, ça peut générer un problème. Du coup, j'y fais la chasse pour avoir l'esprit tranquille.

Là, j'avais un peu de mal à suivre, c'est pour ça que j'ai séché.

Merci beaucoup et bon week-end.