Problem mit Klassen

[Cöster]

Hallöle!

Ich hab mir vorgenommen, mein Programm aus mehreren Klassen bestehen zu lassen. Bisher gibt es nur 2 Klassen: Unit1 (die Hauptunit) und Bild.

Da ich zum ersten Mal mit mehreren Klassen arbeite, gibt es auch schon das erste Problem.
Hier erstmal der (noch kurze) Quelltext der Unit Bild:

Delphi-Quellcode:

unit Bild;

interface

uses
  Graphics, Dialogs;

type
  TBild = class
  private
    FCurrentFile: String;
  public
    procedure Oeffnen;
  end;

implementation

uses Unit1;

procedure TBild.Oeffnen;
begin
  If Form1.OpenPictureDialog1.Execute then
  begin
    self.FCurrentFile := Form1.OpenPictureDialog1.FileName;
    Form1.ImMain.Picture.LoadFromFile(Form1.OpenPictureDialog1.FileName);
    Form1.ImMain.Picture.Bitmap.PixelFormat := pf24Bit;
    Form1.ImMain.Visible := True;
  end;
end;

Die Klasse dient also bisher nur dazu, ein Picture zu öffnen und sich seinen Dateinamen zu merken, damit man beim Speichern nicht immer auf "Speichern unter" klicken muss.
Beim Öffnen eines Bitmaps tritt nun eine Zugriffsverletzung auf. Wenn ich die Zeile self.FCurrentFile := Form1.OpenPictureDialog1.FileName; weglasse, tritt der Fehler nicht auf.

Kann mir jemand helfen? Falls es eine bessere Möglichkeit gibt, die Dateinamen zu speichern, wäre ich natürlich auch daran interessiert.

Danke!

[mkinzler]

Ist Form1 zu diesem Zeitpunkt schon instantiiert?

BTW. ich würde das ganze mal überabeiten, so ist es nämlich nicht so toll!

[pszopp]

Hallo Zusammen,

Zitat von mkinzler:

Ist Form1 zu diesem Zeitpunkt schon instantiiert?

Ja, denn

Zitat von Cöster:

Wenn ich die Zeile
self.FCurrentFile := Form1.OpenPictureDialog1.FileName; weglasse, tritt der Fehler nicht auf.

wenn die entsprechende Zeile auskommentiert wird, würde der gleiche Fehler in der nächsten Zeile auftreten.
Daher muss Form1 instanziiert sein, da der Fehler ja nicht auftritt

Ich habe eher den Eindruck, das beim Zugriff auf Self ein Fehler entsteht.
Self scheint nil zu sein.

Gruß,
pszopp

[GuenterS]

Hast Du denn eine Instanz von TBild erzeugt?

[3_of_8]

Wenn self=nil wäre, könnte er nicht mal die Methode aufrufen, da der Methodenzeiger aus einem Pointer auf den Code und einem Pointer auf die Klasseninstanz besteht. Das heißt, wenn die Klasse nicht instantiiert wäre, würde er bevor er in die Methode springt, übelst mit ner AV abschmieren.

[Cöster]

Zitat von GuenterS:

Hast Du denn eine Instanz von TBild erzeugt?

Ich Idiot
Danke, das war's.

Zitat von mkinzler:

BTW. ich würde das ganze mal überabeiten, so ist es nämlich nicht so toll!

Danke für den Hinweis. Was schlägst du mir denn vor, wie ich das überarbeiten sollte?

[Hawkeye219]

Zitat von 3_of_8:

Wenn self=nil wäre, könnte er nicht mal die Methode aufrufen, da der Methodenzeiger aus einem Pointer auf den Code und einem Pointer auf die Klasseninstanz besteht. Das heißt, wenn die Klasse nicht instantiiert wäre, würde er bevor er in die Methode springt, übelst mit ner AV abschmieren.

Diese Aussage stimmt nur für virtuelle Methoden, nicht aber für statische wie die Methode Oeffnen im Beispiel von Cöster. Selbst wenn die Objektreferenz den Wert NIL enthält, wird die Methode aufgerufen. Bei virtuellen Methoden kommt es zu einer Zugriffsverletzung beim Zugriff auf die VMT.

Gruß Hawkeye

[mkinzler]

Zitat:

Danke für den Hinweis. Was schlägst du mir denn vor, wie ich das überarbeiten sollte?

Alle Referenzen auf Form1 entfernen, und diese als parameter der Methode übergeben.

[3_of_8]

Zitat von Hawkeye219:

Zitat von 3_of_8:

Wenn self=nil wäre, könnte er nicht mal die Methode aufrufen, da der Methodenzeiger aus einem Pointer auf den Code und einem Pointer auf die Klasseninstanz besteht. Das heißt, wenn die Klasse nicht instantiiert wäre, würde er bevor er in die Methode springt, übelst mit ner AV abschmieren.

Diese Aussage stimmt nur für virtuelle Methoden, nicht aber für statische wie die Methode Oeffnen im Beispiel von Cöster. Selbst wenn die Objektreferenz den Wert NIL enthält, wird die Methode aufgerufen. Bei virtuellen Methoden kommt es zu einer Zugriffsverletzung beim Zugriff auf die VMT.

Gruß Hawkeye

Wieder was dazugelernt. An dieser Stelle könnte man doch eigentlich doch eigentlich Delphi wieder ein bisschen "sicherer" machen, indem man da eine AV raist, oder?

[pszopp]

Zitat von 3_of_8:

An dieser Stelle könnte man doch eigentlich doch eigentlich Delphi wieder ein bisschen "sicherer" machen, indem man da eine AV raist, oder?

Nein, dann gäbe es ja keine statischen Methoden mehr.

Gruß,
pszopp