C# Forearch meldet Fehler bei mehr als 1 Durchlauf

[Murphy9904]

Hi,
ich will Die Objekte aus meiner List Box in ein String schreiben,
meiner meinung nach müsste das so funktionieren:

MyListBoxItem[] items = new MyListBoxItem[ListBoxRight.Items.Count];
                    ListBoxRight.SelectedItems.CopyTo(items, 0);
                        foreach (var item in items)
                        {
                            String += item.Name + " / " + item.Alias; //<-- Fehler: Object reference not set to an instance of an object.
                        }

tus aber nich, was mach ich falsch?

Danke schonmal
LG Murphy

 

[Apllepie]

Hallo Murphy,

ich kann da jetzt so auf Anhieb keinen Fehler sehen, aber warum machst du es nicht direkt so:

String s = String.Empty;
foreach(Item i in ListBoxRight.SelectedItems)
  s += string.format("{0}/{1}", i.Name, i.Alias);

 

[toeffi]

@ Murphy. Wenn du eine Frage hast und das Programm einen Fehler wirft, dann bitte ich dich das du die Fehlermeldung mitpostest, schließlich können wir nich ahnen was da schief geht.

Weiterhin möchte ich euch beide bitten, Strings nicht mit dem += operator zu verbinden.
Hier die Erklärung:Warum sollte man den StringBuilder bei komplexen String-Operationen vorziehen?

 

[crushin]

String += item.Name

da ist der fehler, du musst eine Intanz vom Typ string anlegen und diese dann meinetwegen bla nennen;

also

string bla = string.Empty;

und dann inder foreach:

bla += item.Name

//achja stringbuilder ist tatsächlich zu empfehlen, da er für jede string operation ein neues objekt anlegt und diese ggf. nicht sauber aufgeräumt werden..

 

[Murphy9904]

Ja das hatt ich ja gemacht...(habs nur vergessen mitzuposten)

den Code vom Freeman4gu blick ich iwie nich ganz..

 

[haze4real]

dein fehler ist simple du initialisierst ein array mit der größe der Items in der Liste, aber "kopierst" nur die items der SelectedItems enumeration... daher bekommst du eine NullReferenceException sobald die Anzahl der Items größer ist als die der SelectedItems

 

[SteveP]

Hi,

was gibt es da nicht zu verstehen?
Er kopiert die SelectedItems nicht nochmal in eine temporäre Liste, sondern greift innerhalb der foreach-Schleife direkt auf alle selektierten items zu!
Am Anfang wird ein Leerstring initialisiert, und dann wird diesem immer wieder ein String dran gehangen mit der String.Format() Methode...


Grüße,
Stephan

 

[Murphy9904]

ok..
Also ich hab den Fehler..
ich lese die "Items" ja aus ner list box aus und er schaut sich die Gesamtanzahl an und bearbeitet aber nur die Selektierten (eig will ich ja alle)

sorry leichtsinnsfehler..

 

[Apllepie]

huch, stimmt das += ist natürlich auch dämlich ^^
würds dann wohl eher so machen

s = string.format("{0}{1}/{2}", s, i.Name, i.Alias);

hmm, obwohl das natürlich trotzdem jedes mal ein neues Objekt anlegen würde ^^
also doch besser den Stringbuilder

 

[ljnk]

Stringbuilder macht bei der Konkatenation von 3 Strings keinen Sinn.

 

[toeffi]

Wie kommst du auf 3?

 

[roker002]

String.Concat macht doch Sinn... es sind die ganzen 300 Millisekunden schneller als die "+" Operator. Naja ich habe den vergleich auf eine gesamte Methode angewandt und nicht auf einen Befehl. Ich muss noch anmerken, ich habe Dualcore mit 3 GHz Pro Core. ein langsamerer PC würde dafür nicht mehr 300 ms brauchen, sondern mehr.