Hallo zusammen,
ich frage mich gerade, ob Lazarus an der Stelle einen Bug hat, oder warum meldet sich der Compiler mit der Warnung "Note: Local variable MyRecord is assigned but never used"?

zusammenfalten · markieren

Delphi-Quellcode:

procedure TMyClass.DeleteRecord(index: integer);
var MyRecord: TMyRecord;
      i: integer;
begin
    if index<0 then Exit;
    for i:= index to high(MyArray) do
    begin
        MyArray[i]:= MyArray[i+1];
        MyRecord:= MyArray[i];
    end;
end;

Der Code gehört zu einer Methode zum Löschen eines Records aus einem Array und funktioniert auch problemlos.

zusammenfalten · markieren

Delphi-Quellcode:

procedure TMyClass.DeleteRecord(index: integer);
var
  //MyRecord: TMyRecord;
  i: integer;
begin
    if index<0 then Exit;
    for i:= index to high(MyArray) do
    begin
        MyArray[i]:= MyArray[i+1];
        //MyRecord:= MyArray[i];
    end;
end;

der obige Code würde auch funktionieren - oder?

Grüße
Klaus

Du weist der Variablen zwar Werte zu, aber diese Werte werden niemals benutzt.
Also ist die Zuweisung überflüssig und der Compiler erkennt das eben.

Stimmt, ich hatte den Code von einer anderen Stelle in meinem Programm übernommen, da war die Zuweisung noch nötig gewesen. Hab sie jetzt rausgenommen.
Ich habe die Fehlermeldung wahrscheinlich mit "Variable xy is not used" verwechselt, die kommt dann, wenn man nur die Deklaration hinschreibt.

nebenbei bemerkt, auch wenns offtopic ist, aber es fällt mir grad auf:

Wenn Index = high(MyArray) ist, dann machs bei
MyArray[i]:= MyArray[i+1]; .
knirsch, da i+1 ausserhalb dem Array ist

Zitat von TForm1:

Ich habe die Fehlermeldung wahrscheinlich mit "Variable xy is not used" verwechselt, die kommt dann, wenn man nur die Deklaration hinschreibt.

Es ist keine Fehlermeldung, sondern nur eine Note, nichtmal eine Warnung. Fehlermeldungen führen dazu, dass am Ende die Kompilierung abbricht, wenn du nur Warnungen, Notes und Hints hast, passiert das nicht.

Gruß,
Sven

Zitat von Helmi:

Wenn Index = high(MyArray) ist, dann machs bei
MyArray[i]:= MyArray[i+1]; .
knirsch, da i+1 ausserhalb dem Array ist

Das habe ich mir auch gedacht, als ich das Schrieb, aber komischerweise funktioniert das trotzdem, ich weiß nicht warum.

Zitat von JamesTKirk:

Es ist keine Fehlermeldung, sondern nur eine Note, nichtmal eine Warnung. Fehlermeldungen führen dazu, dass am Ende die Kompilierung abbricht, wenn du nur Warnungen, Notes und Hints hast, passiert das nicht.

Das war mir schon klar, ich habe mich vielleicht nicht passend ausgedrückt.

Zitat von TForm1:

Zitat von Helmi:

Wenn Index = high(MyArray) ist, dann machs bei
MyArray[i]:= MyArray[i+1]; .
knirsch, da i+1 ausserhalb dem Array ist

Das habe ich mir auch gedacht, als ich das Schrieb, aber komischerweise funktioniert das trotzdem, ich weiß nicht warum.

Es muss nicht immer knallen, aber zumindest sollte in MyArray[High(MyArray)] nach dem Aufruf Müll stehen. Es empfiehlt sich (fast) immer beim Entwickeln die Bereichsprüfung einzuschalten. Dann werden an solchen Stellen auch Exceptions ausgelöst, so dass man nachher nicht böse Überraschungen hat.