FOR-Schleifen Problem

[guinnes]

Zitat von Luckie:

Und wo ist jetzt der Fehler? Dass die Schleife rückwärts durchlaufen wird ist eine Optimierung des Compilers.

Dürfte wohl eine Zugriffsverletzung sein, oder ?
Womit willst du die 1. Zahl vergleichen ?

[delphinub23]

Nein, eine Zugriffverletzung tritt nicht auf. Das liegt sicherlich an dem Startwert 16 oder? Ich möchte die aktuelle Zahl mit der vorherigen vergleichen um auszuschließen, dass keine nochmal auftritt.

[guinnes]

Zitat von delphinub23:

Das liegt sicherlich an dem Startwert 16 oder? Ich möchte die aktuelle Zahl mit der vorherigen vergleichen um auszuschließen, dass keine nochmal auftritt.

Das ist mir schon klar. Meine Frage war: Womit vergleichst du bei I = 0 ?
Richtig, mit rZahl[-1] und das gibt es nicht
Setze ausserhalb der Schleife den Zufallswert von rZahl[0] und lass die Schleife von Low(rzahl) + 1 bis High(rzahl) laufen

edit
Dazu kann es natürlich immer noch vorkommen, dass Zahlen doppelt sind, weil du ja immer nur mit der letzten Zahl vergleichst

[delphinub23]

Ok. Das macht Sinn.

So bekomme ich generierte Zahlen:

Delphi-Quellcode:

procedure TFmMain.btnGenClick(Sender: TObject);
var
  rZahl: array[0..15] of integer;
  LastIndex, temp, i: integer;
begin
  // Generate randoms
  for i := Low(rZahl)+1 to High(rZahl) do
  begin
    temp := Random(16);
    rZahl[i] := temp;
    txtRandoms.Text := txtRandoms.Text + IntToStr(rZahl[i]) + '; ';
  end;
end;

Aber so kommen einige Zahlen mehrfach vor und das will ich unterbinden.
Gibt es noch einen besseren Ansatz, als meinen Quark?

[rollstuhlfahrer]

Nimm eine Liste (z.B. TList) und lasse diese nach dem Zufallsprinzip sortieren. Dann hast du deine Zahlen immer in unterschiedlicher Reihenfolge.

Bernhard

PS: Tipp: TList.Sort()

[guinnes]

Zitat von delphinub23:

Gibt es noch einen besseren Ansatz, als meinen Quark?

Klar. Nimm ein Array of Boolean, initialisiere es ändere die While-Schleife so, dass du die neu gezogene Zahl als Index in das Array benutzt und schau nach, ob da schon ein True drinsteht, wenn ja-> neue Zahl ziehen, wenn nein -> das Feld im Array auf true setzen.

[delphinub23]

Danke für eure Hilfe! Ich probiere es mal mit dem Boolean-Array.

[guinnes]

Das macht aber nur dann Sinn, wenn du x Zahlen aus y Möglichkeiten ziehen willst. Willst du jede Zahl einmal haben, ist der Vorschlag mit der Liste natürlich besser

[Aphton]

Ich habe einen einfacheren und leichter nachvollziehbaren Vorschlag für dich:
Zuerst - du benutzt Random(16) - dh es wird ein Zufallswert zwischen 0 und 15 zurückgeliefert. Nennen wir diese Zahl X um alles variabel zu halten. (X=16)
Übrigens muss X >= Länge von rZahl sein, damit keine Zahl doppelt vorkommt! (dh es müssen genug Zahlen vorhanden sein)

- Erstelle ein dynamisches Array der Länge X
- Fülle diese mit den Indexwerten (Also DynArr[0] := 0; DynArr[1] := 1, ..)
- Gehe alle Elemente von rZahl durch (i)
- ermittle ein Zufallsindexwert j := Random(Length(DynArr))
- führe folgende Zuweisung durch: rZahl[i] := DynArr[j]
- Lösche Element j aus DynArr

[himitsu]

Zitat von delphinub23:

Nein, eine Zugriffverletzung tritt nicht auf.

Geh mal in die Projektoptionen und aktiviere dort die Überlauf- und die Bereichsprüfung.

Und schon kommt deine Zugriffsverletzung.
Du hast nur Glück, daß an der falschen stelle, wo zu zugreifst zufällig etwas anderes liegt und es somit nicht gleich knallt.

Zitat:

Delphi-Quellcode:

except
  on E: Exception do
  begin
    // Irgendwo ein Fehler passiert? - 0-Array zurückgeben
    ZeroMemory(@Result, SizeOf(Result));
    raise E; // Exception nach oben weiter geben (benötigt??)
  end;
end;

Beim ZeroMemory, muß beachtet werden, da es hier auf dieses statische Array angepaßt ist ... also je nach Typ, muß es anders gelöst werden.

Zitat:

(benötigt??)

is besser so, auch wenn man es kürzen kann.

Delphi-Quellcode:

except
  ZeroMemory(@Result, SizeOf(Result));
  raise;
end;