Hallo, ich glaube das mit dem Datenmodul habe ich noch nicht richtig gecheckt.

Also ich habe ein Formular nennen wir es frmMitglieder ( die Unit heißt: Mitglieder ) Dazu habe ich einen Datenmodul dmMitglieder (Unitname: Datamod_Mitglieder).

auf dem Formular frmMitglieder habe ich ein Label lblAlter. Nun möchte ich beim OnChange Ereignis die Farbe des Labels wechseln. Leider bekomme ich eine Zugriffsverletzung weil mein frmMitglieder Nil ist. Weis jemand was ich machen muss damit es funktioniert.

Hier mal der Code aus dem Datamodul:

zusammenfalten · markieren

Delphi-Quellcode:

unit Datamod_Mitglieder;

interface

uses
  System.SysUtils, System.Classes, Data.DB, IBODataset, Vcl.Graphics;

type
  TdmMitglieder = class(TDataModule)
    LookupQry: TIBOQuery;
    LookupSrc: TDataSource;
    MitgliederQry: TIBOQuery;
    dscMitglieder: TDataSource;
    AnredeQry: TIBOQuery;
    dscAnrede: TDataSource;
    dscTitel: TDataSource;
    dscBerufe: TDataSource;
    BerufeQry: TIBOQuery;
    TitelQry: TIBOQuery;
    PLZQry: TIBOQuery;
    procedure dscMitgliederDataChange(Sender: TObject; Field: TField);
  private
    { Private-Deklarationen }
  public
    { Public-Deklarationen }
  end;

var
  dmMitglieder: TdmMitglieder;

implementation

{%CLASSGROUP 'Vcl.Controls.TControl'}

uses Mitglieder, ToolsOperation;

{$R *.dfm}

procedure TdmMitglieder.dscMitgliederDataChange(Sender: TObject; Field: TField);
begin
   if MitgliederQry.FieldByName('GEBURTSDATUM').AsString <> '' then
    frmMitglieder.lblAlter.Caption := 'Alter: ' + IntToStr(ToolsOperation.CalcYearsInPeriod(MitgliederQry.FieldByName('GEBURTSDATUM').AsDateTime, Date));
  if CalcBirthday(dmMitglieder.MitgliederQry.FieldByName('GEBURTSDATUM').AsDateTime) then
    frmMitglieder.lblAlter.Font.Color := clRed
  else
    frmMitglieder.lblAlter.Font.Color := clBlack;
  if (MitgliederQry.FieldByName('GEBURTSDATUM').AsString <> '') then
    frmMitglieder.lblAlter.Visible := True
  else
    frmMitglieder.lblAlter.Visible := False;
end;

end.

Das Datenmodul wird im OnCreate des Formulars frmMitglieder erzeugt.

markieren

Delphi-Quellcode:

procedure TfrmMitglieder.FormCreate(Sender: TObject);
begin
  dmMitglieder := TdmMitglieder.Create(frmMitglieder);
  ToolsForm.RestoreWindowPosition(Self);
end;

Wer weiß Abhilfe.

Dein Datenmodul soll Daten bereitstellen, und soll/darf/muss alle anderen Units des Projekts gar nicht kennen. Änderungsereignisse werden demzufolge klassischerweise in der Form-Unit abgehandelt.

Sherlock

Wie soll man denn auf ein OnChange Ereignis reagieren? Das Formular bekommt dies ja nicht mit.

Ich vermute hier liegt ein kleines Missverständnis vor. Ein OnChange-Event wird typischerweise von GUI-Controls erzeugt, hier geht es aber wohl eher um ein OnDataChange-Event einer DataSource.

In diesem Fall würde ich eine Message an das Formular schicken, eventuell sogar als Broadcast. Günstiger wäre aber wohl die Verlagerung der DataSource auf das Formular.

Ja das ist richtig mit dem OnChange des DataSet. Ich hatte das bisher so, das alles auf der Form lag, aber mit der Zeit ist das Formular völlig überfrachtet. Darum die Idee mit dem Datamodul. War wohl nicht so eine gute Idee oder? Wie kann ich denn eine Message an die Form schicken und darauf reagieren.

oder du sorgst dafür, dass im Form das OnDataChange-Ereignis nach dem Erstellen mit dem DataSet auf dem DM verbunden wird. Bisher sah ich keinen Grund das nicht so zu machen.

Zitat von Walter Landwehr:

Ja das ist richtig mit dem OnChange des DataSet. Ich hatte das bisher so, das alles auf der Form lag

Es spricht - solange keine Master/Detail DataSources in den DataSets benötigt werden - nichts dagegen, die DataSource weiterhin auf dem Formular zu lassen. Sie vermittelt zwischen GUI Controls und Datenbank-Komponenten. Messages zu versenden würde bedeuten das "RAD" noch einmal zu erfinden. Und es ensteht so auch keine ("zirkuläre") Abhängigkeit des Datenmoduls vom Formular.

Hallo Walter,

die Trennung macht durchaus Sinn. Wenn man es aber macht, dann bitte konsequent so wie schon mehrfach vorgeschlagen, und nicht irgend so einen Misch-Masch wo sich dann wieder Teile des Datenzugriffes auf den Formularen befinden.

Mein Vorschlag: Diese procedure gehört in das Formular

markieren

Delphi-Quellcode:

procedure TfrmMitglieder.dscMitgliederDataChange(Sender: TObject; Field: TField);
begin
  ...
end;

Beim "OnActivate" und "OnDeactivate" würde ich sie jeweils im Formular zuweisen

markieren

Delphi-Quellcode:

procedure TfrmMitglieder.FormActivate(Sender: TObject);
begin
  dmMitglieder.dscMitglieder.OnDataChange := dscMitgliederDataChange;
end;
procedure TfrmMitglieder.FormDeActivate(Sender: TObject);
begin
  dmMitglieder.dscMitglieder.OnDataChange := nil;
end;

Abgesehen davon würde ich die Datenmodule generell in der .dpr erzeugen, denn oft ist es ja so, dass bei konsequenter Anwendung der Datenmodule (absolute Trennung von Datenzugriff und GUI) dann mehrere Formulare das Datenmodul problemlos verwenden können. Unterm Strich wird zudem sogar der Bildschirmaufbau etwas schneller weil die Tabelle offen bleiben kann und das Datenmodul nicht ständig neu erzeugt werden muss.

Zitat von mm1256:

Mein Vorschlag: Diese procedure gehört in das Formular

markieren

Delphi-Quellcode:

procedure TfrmMitglieder.dscMitgliederDataChange(Sender: TObject; Field: TField);
begin
  ...
end;

Das wird in diesem Fall nicht viel weiter helfen, da der TE im dscMitgliederDataChange des DataModuls auch auf andere Komponenten im Datamodul zugreift. Deshalb mein Vorschlag mit dem Event. Dadurch werden die Aktionen des DM von denen des Formulars getrennt.

Hallo Otto,

Klappt, Danke