If ... or ... then verwirrt mich

[hellow554]

zack nagel aufm kopf getroffen ^^
okay
also bewirkt das beginupdate, dass die indexe (indicies sieht genauso bescheuert aus ^^) gleich bleiben und sich nicht verschieben, was bei mir ja der fall wäre, da ich ja einen lösche...
okay okay verstanden und danke

[DeddyH]

Nein, BeginUpdate sorgt nur dafür, dass die Items während der Abarbeitung nicht neu gezeichnet werden, solange bis das entsprechende EndUpdate folgt.

[dominikkv]

Zitat von hellow554:

also bewirkt das beginupdate, dass die indexe (indicies sieht genauso bescheuert aus ^^) gleich bleiben und sich nicht verschieben

Nope

Zitat:

TStrings simply keeps track of when the list of strings is being changed. Some descendants of TStrings use this information to perform certain actions, such as telling a control to repaint, when updates are complete.

Es wird also nur die "Es-hat-sich-was-verändert"-Nachricht unterdrückt, die das Neuzeichnen anstößt.

[himitsu]

Wenn du von vorn nach hinten durchgehst und was löschst und auch beim Löschen den Index hochzählst, dann überspringst du den nachfolgenden Eintrag, da dieser ja an die Stelle des Gelöschten rutscht.

a) Schleife rückwärts
b) oder beim Löschen nicht weitergehn

[samso]

Edit: Ok wieder zu langsam.

Delphi-Quellcode:

if (combobox1.items.Strings[i] = 'YASU') or (combobox1.Items.Strings[i] = 'DT') then 
  combobox1.Items.Delete(i);

ist nicht dasselbe wie

Delphi-Quellcode:

if combobox1.items.Strings[i] = 'DT' then 
  combobox1.Items.Delete(i); 
if combobox1.Items.Strings[i] = 'YASU' then 
  combobox1.Items.Delete(i);

weil zwischen den beiden IF's der Index i nicht geändert wird. Bei der ersten Version ändert sich der Index. Wenn also 'DT' und 'YASU' in der Liste direkt hintereinander stehen, wird bei der ersten Version nur DT gelöscht, bei der zweiten Version wird erst 'DT' gelöscht dann rückt 'YASU' an die Position von 'DT' und dann wird 'YASU' gelöscht.

[rollstuhlfahrer]

Zitat von samso:

Delphi-Quellcode:

if (combobox1.items.Strings[i] = 'YASU') or (combobox1.Items.Strings[i] = 'DT') then 
  combobox1.Items.Delete(i);

ist nicht dasselbe wie

Delphi-Quellcode:

if combobox1.items.Strings[i] = 'DT' then 
  combobox1.Items.Delete(i); 
if combobox1.Items.Strings[i] = 'YASU' then 
  combobox1.Items.Delete(i);

Der Quellcode macht exakt das gleiche also ist diese Bemerkung nicht ganz richtig.

Zitat von samso:

weil zwischen den beiden IF's der Index i nicht geändert wird. Bei der ersten Version ändert sich der Index. Wenn also 'DT' und 'YASU' in der Liste direkt hintereinander stehen, wird bei der ersten Version nur DT gelöscht, bei der zweiten Version wird erst 'DT' gelöscht dann rückt 'YASU' an die Position von 'DT' und dann wird 'YASU' gelöscht.

JA.

Bernhard

[rwachtel]

Zitat von rollstuhlfahrer:

[...] Der Quellcode macht exakt das gleiche also ist diese Bemerkung nicht ganz richtig. [...]

Nein, das macht er nicht - wie Du ja auch eine Zeile später selbst bestätigst:

Zitat:

[...]

JA.

Bernhard

[DeddyH]

Mit einem else würde er dasselbe machen.

Delphi-Quellcode:

if combobox1.items.Strings[i] = 'DT' then
  combobox1.Items.Delete(i)
else if combobox1.Items.Strings[i] = 'YASU' then
  combobox1.Items.Delete(i);

[Sir Rufo]

1. Dafür nimmt man kein REPEAT...UNTIL sondern ein WHILE...DO

Delphi-Quellcode:

i := 0;
while i < ComboBox1.Items.Count do
  begin
  end;

, denn so gibt es keine Zugriffsfehler, wenn keine Elemente in der Liste stehen.
2. Was passiert bei Delete?
ComboBox1.Items

Code:

(0)Hallo
(1)YASU
(2)DT
(3)du da

wenn wir jetzt zu dieser Zeile ( i ist 1) kommen
ComboBox1.Items.Delete( i ); dann sieht ComboBox1.Items so aus

Code:

(0)Hallo
(1)DT
(2)du da

Wenn jetzt im nächsten Schritt i um 1 erhöht wird ( i ist dann ja 2 ), dann wird die (jetzige) Zeile 1 nicht mehr überprüft!
Also sollte man nach einem Delete den Index nicht höchzählen, weil einem dann eine Zeile immer entwischt