Schleife schleift nicht wie erwartet (aber jetzt schon)

[TBx]

Halo,

nimm mal aus Zeile 27 diesen Code ganz raus:

Delphi-Quellcode:

listview1.ItemIndex:=listview1.ItemIndex+1;

Stattdessen fügst Du zwischen Zeile 5 und 6 folgendes ein:

Delphi-Quellcode:

listview1.ItemIndex:=i;

Das Inkrementieren des Itemindexes ist föllig überflüssig, da Du zu jeder Zeit den gewünschten ItemIndex in i stehen hast.

Der Fehler tritt auf, weil beim letzten Schleifendurchlauf der Index auf einen Wert erhöhtz werden würde, der nicht gültig ist.

Hope it helps

onlinekater

PS: Trotz roten Kastens, da die Lösung von Gollum den Index an der falschen Stelle setzt.

[Muetze1]

Habe ich schon in Beitrag #2 geschrieben und

Zitat:

Zitat von toyoman

Die Änderung macht allerdings keinen Unterschied

[TBx]

@Muetze1: Ich denke mal, die Anmerkung, dass die Änderung keinen Unterschied bringe bezog sich nicht auf Deinen Beitrag, sondern auf den ersten von Gollum :-)

[toyoman]

Zitat:

Zitat von Hawkeye219

Hallo,

Gegen das Speicherleck solltest du ebenfalls etwas unternehmen. Die Stringliste wird nicht freigegeben, falls der Anwender die Suchschleife vorzeitig beendet. Bei einen Einsatz von TRY..FINALLY könntest du auch den Cursor an einer Stelle restaurieren.

Gruß Hawkeye

ich hab doch liste.free drin ? nicht gesehen? das sollte doch reichen oder?

[toyoman]

Zitat:

Zitat von onlinekater

@Muetze1: Ich denke mal, die Anmerkung, dass die Änderung keinen Unterschied bringe bezog sich nicht auf Deinen Beitrag, sondern auf den ersten von Gollum :-)

stimmt :)

[inherited]

Zitat:

Zitat von toyoman

ich hab doch liste.free drin ? nicht gesehen? das sollte doch reichen oder?

Nein, leider nicht. Wenn es zB zu einem Fehler kommt wird die ganze prozedur abgebrochen, er kommt also garnicht zu deinem .free und gibt sie nicht frei. Dafür gibt es den try..finally-Block. Der bewirkt, dass wenn etwas in dem try-Teil schief geht, bevor er abbricht auf jeden Fall noch der finally-Block ausgeführt wird. Wenn alles glatt läuft sowieso.
Damit kannst du sicherstellen, dass die liste auf jeden Fall freigegeben wird.

[toyoman]

ok das ist ein argument! danke

Zitat:

Zitat von inherited

Zitat:

Zitat von toyoman

ich hab doch liste.free drin ? nicht gesehen? das sollte doch reichen oder?

Dafür gibt es den try..finally-Block. Der bewirkt, dass wenn etwas in dem try-Teil schief geht, bevor er abbricht auf jeden Fall noch der finally-Block ausgeführt wird. Wenn alles glatt läuft sowieso.
Damit kannst du sicherstellen, dass die liste auf jeden Fall freigegeben wird.

[maerc]

Zitat:

Zitat von toyoman

Zitat:

Zitat von Muetze1

Schlaufe? Nicht eher eine Schleife? :gruebel:

Wird durch " btnoeffnen.Click; " vllt. ein Eintrag in der ListView oder der Liste entfernt?

von mir aus schleife.. ;)

dann änder doch mal den threadtitel, damit man weiß um was es geht ;) :wiejetzt: