Hilfe bei AccessViolation

[Dejan Vu]

Auch nett:

Delphi-Quellcode:

// Loakle IP auslesen (!)
  GetLocalIPs(Memo1.Lines);

  // Falls keine IP vorhanden, dann N.N. eintragen, ...
  if not GetLocalIPs(Memo1.Lines) then
    StatusBar1.Panels[3].Text := 'IPv4-Adresse: N. N.'
  else
    ...

1. 'Loakle' vs. 'Lokale'
2. Kommentar zu 'GetLocalIPs' = 'Lokale IP auslesen'...
3. Du ließt die ja 2x aus...

Delphi-Quellcode:

var
  hasLocalIP : Boolean;
...

  hasLocalIP := GetLocalIPs(Memo1.Lines);

  // Falls keine IP vorhanden, dann N.N. eintragen, ...
  if hasLocalIP then
    StatusBar1.Panels[3].Text := 'IPv4-Adresse: ' + Memo1.Lines.Text;
  else
    StatusBar1.Panels[3].Text := 'IPv4-Adresse: N. N.'
    ...

Und beherzige den Tipp von alda, der dir empfiehlt, deine ziemlich lange Methode in einzelne kleine Methoden zu unterteilen, sodaß jede Methode genau eine Sache macht (Grid füllen, IP-Adresse ermitteln etc.). Jedem dieser Methoden verpasst Du einen aussagekräftigen Namen und schon sparst Du dir die Kommentare (wie alda das gesagt hat).

[Sir Rufo]

Am Besten finde ich eigentlich das hier

Delphi-Quellcode:

finally
  ini.free; // Objekt freigeben, sonst EInvalidPointer except.
end;

Man beachte, das ist keine lokale Variable und in der Variablen steht immer noch die Referenz auf eine zerstörte Instanz.

Grundsätzlich ist der gesamte Code sehr grauslich und kann gut als Beispiel für "So nicht machen!" genommen werden.

[himitsu]

Dass dieses "reg" global ist, war mir schon aufgefallen, aber mir war so, als sei "ini" lokal.
Man könnte sich ja nun nochmal fragen, wo reg und ini definiert sind. (z.B. als private in der Form oder ganz global)

Aber da sieht man es ... es gibt mal ein "wichtigen" Kommentar, aber den übersahen Viele gleich.
Also wenn, dann FreeAndNil, damit das mit dem "ungültigen" Zeiger (siehe Sir Rufo) zu beheben,
aber der Kommentar ist wiedersprüchlich, denn gerade durch diesen Code kann es zu solchen Fehlern kommen. Er behebt sie jedenfalls nicht.


Beim "kopieren nach temp"-CopyFile kann man froh sein, daß hier mal keine Fehlerbehandlung vorgenommen wäre, denn sonst würde es hier auch nochmal knallen und die ursprüngliche Fehlermeldung vom Rewrite/Write vernichten, wenn es dort nicht gut ablief.

[Sir Rufo]

Das reg wird aber dort nicht wieder freigegeben und zähle ich jetzt mal zum regulären Initialisierungsteil der Form-Instanz (muss man erst mal so annehmen).

Das mit ini ist aber böse, weil danach nicht mehr überprüft werden kann, ob die Instanz noch lebt oder nicht.

[himitsu]

Sagen wir es einfach mal so: Die Instanz wird (scheinbar) nur lokal benutzt, also gehört die Variable auch lokal.

[Dejan Vu]

Zitat von Sir Rufo:

Am Besten finde ich eigentlich das hier...Man beachte, das ist keine lokale Variable und in der Variablen steht immer noch die Referenz auf eine zerstörte Instanz.

Komm schon. No risk, No fun.