dotNETportal.cz

Dobrý den, v mém programu načítám jména z textového souboru do ListBoxu. V textovém souboru mám jedno jméno na jednom řádku.

Ale jména se do ListBoxu nenačtou, a já nevím proč.

 Dim rd As New IO.StreamReader("seznamJmen.txt")
        Dim line As String
        Do
            line = rd.ReadLine()
            If line IsNot Nothing Then
                lstSeznamJmen.Items.Add(line)
            End If
        Loop While line IsNot Nothing
        rd.Close()

Můžete mi prosím poradit, co je špatně, a jak kód upravit aby fungoval?

Děkuji za odpověď.

A zkoušel jste si tam dát breakpoint a odkrokovat to?

Panebože co je to za příšernost?! Máte alespoň přibližnou představu, co ten kód dělá? Napíšu vám korektní řešení, ale jen proto, aby jste se z něho mohl poučit.

Using reader As New StreamReader("seznamJmen.txt")
  Dim line = String.Empty
  lstSeznamJmen.BeginUpdate()
  While line IsNot Nothing
    line = reader.ReadLine()
    If line IsNot Nothing Then lstSeznamJmen.Items.Add(line)
  End While
  lstSeznamJmen.EndUpdate()
End Using

Vůbec netušíte, že při inicializaci proměnné line je její hodnota nastavena na Nothing, tudíž cyklus nemůže proběhnout ani jednou.

Nevhodné řešení je to vaše právě z důvodu, že se vše načítá do paměti. Kratší zápis <> lepší řešení. A pokud se to někdo jednou naučí špatně, potom to bude používat i na nevhodných místech (velké soubory).

Nikde netvrdím, že kratší zápis je lepší řešení. Proto jsem ale ukázal obě řešení, aby každý viděl, že jedno je co do kódu delší, ale nenačítá celý soubor do paměti.

Druhé řešení je v tomto případě naprosto v pořádku - na malém souboru to rozdíl ve spotřebě paměti nepoznáte (GC to hned sebere) a velké soubory nemá smysl cpát do ListBoxu.

Děkuju Vám za odpovědi. Řešení pomohlo, a snad se z toho něco naučím.