Procedure in Datenmodul (TIdHTTP Komponente)

[DaSchwoa]

Hallo

Ich bin gerade dabei ein Service zu machen, der mit Http.get() Daten von einer Website ausließt. Ich habe versucht die Prozedur im Datenmodul zu programmieren und mit dm.MeineProzedure einzubinden.
Leider bin ich nicht in die Prozedur hinein gekommen.
Als ich versucht die Prozedur in meiner Haupt-Service Unit zu entwickeln und auf die Komponenten mit dm.IdHttp1 zu verweisen, hat es funktioniert.
Ist es nicht möglich eine Prozedur vom Datenmodul einzubinden oder habe ich etwas falsch gemacht?

(Ja das Datenmodul wurde erstellt laut Log Datei)

[stahli]

Hallo und willkommen...

ich versuche das mal etwas zu ordnen:

Du willst eine Prozedur in einem Datenmodul definieren und diese dann aufrufen?

Dann muss die Prozedur in einem public Abschnitt definiert werden.

Von der aufrufenden Stelle musst Du natürlich Zugriff auf die Datenmudulinstanz haben und dazu die Unit in der dortigen uses-Klausel eingebunden sein.

Ggf. musst Du nochmal genauer beschreiben, was gegeben ist und wo es Probleme gibt.

[DaSchwoa]

Danke für die schnelle Antwort
Also ich habe ein Datenmodul wo ich die Indy HTTP und Open SSL Komponente eingebunden habe. Weiters habe ich auch eine Prozedur:

Code:

procedure TFrmDm.doDownload(Website: string; Outputfile: string);
var
  ResponseStream: TFileStream;
begin
  WriteToLog('doDownload', 'Try to read');

  ResponseStream := TFileStream.Create(Outputfile, fmCreate);
  IdHTTP1.IOHandler:= IdSSLIOHandlerSocketOpenSSL1;
  IdHTTP1.HandleRedirects:= true;
  try
    IdHTTP1.Get(Website, ResponseStream);
   WriteToLog('Timer1Timer','HTTP-Status: ' + IntToStr(IdHTTP1.ResponseCode));
  finally
    ResponseStream.Free;
  end;
  WriteToLog('doDownload', 'Download abgeschlossen');
end;

Diese Prozedur möchte ich gerne in meinem Main Service aufrufen:

Code:

procedure TEuropeanCentralBankCurrencyService.Timer1Timer(Sender: TObject);
begin
  WriteToLog('Timer1Timer','Start');
  dm:= TFrmDm.Create(FrmDm);
  dm.doDownload(WEBSITE, outputFilePathName);

  WriteToLog('Timer1Timer','Finished');
  self.DoStop;
  dm.Free;
end;

Jedoch komme ich nicht in die Funktion hinein und der Timer startet immer von neuem ohne etwas zu tun.

Habe auch versucht die Funktion in Public zu geben, aber das hat auch nichts genützt

[Jumpy]

dm:= TFrmDm.Create(FrmDm); Was ist FrmDm? Wie oft gibt es dein datenmodul?

[Delphi.Narium]

Da es die Variabel FrmDm für die Klasse TFrmDm gibt, hätte ich eher dashier erwartet:

Delphi-Quellcode:

procedure TEuropeanCentralBankCurrencyService.Timer1Timer(Sender: TObject);
begin
  WriteToLog('Timer1Timer','Start');
  FrmDm.doDownload(WEBSITE, outputFilePathName);
  WriteToLog('Timer1Timer','Finished');
  DoStop;
end;

Befremdlich finde ich auch, dass dm in der Timerprozedure erstellt und freigegeben wird, aber die Variabel dazu nicht in der Prozedure deklariert ist, sondern eine globale Variabel zu sein scheint.

Sehe ich das richtig?
dm:= TFrmDm.Create(FrmDm);
Damit wird ein Datenmodul dm vom Typ TFrmDm erstellt, dessen Owner ein Datenmodul vom Typ TFrmDm ist?

Das ist rekursionsverdächtig.

[haentschman]

Moin...

Zitat:

Jedoch komme ich nicht in die Funktion hinein und der Timer startet immer von neuem ohne etwas zu tun.

Fragen:
1. Wieso ein Timer?
2. Intervall des Timers?

Vermutung:
Der Intervall des Timers ist zu kurz. Das würde bedeuten, daß der Timer sich wieder selbst "aufruft" ('Timer startet immer von neuem')
Lösung in diesem Falle:
In Timer1Timer in der ersten Zeile den Timer deaktivieren und in der letzten Zeile wieder aktivieren.

PS: Besser auf den Timer verzichten.

[DaSchwoa]

Zitat von Delphi.Narium:

Damit wird ein Datenmodul dm vom Typ TFrmDm erstellt, dessen Owner ein Datenmodul vom Typ TFrmDm ist?

Das ist rekursionsverdächtig.

Ja ich weiß. Anfangs hätte ich es versucht mit FrmDm.doDownload aber das hat auch nicht funktioniert. Ich komme aber einfach nicht in meine Procedure rein.

Nur wenn ich die einzelnen Komponenten von meinem Datenmodul einbinde geht es. (also mit FrmDm.IdHttp). Aber das muss doch auch schöner gehen

[DaSchwoa]

Zitat von haentschman:

PS: Besser auf den Timer verzichten.

Niemals ohne meinen geliebten Timer

Das habe ich schon ein paar mal gehört das der Timer nicht die beste Lösung ist. Aber bis jetzt hat es immer funktioniert.
Das mit dem Intervall kann auch nicht das Problem sein, da ich es, wie oben erklärt, schon funktioniert hat. Jedoch ohne Aufruf der Prozedur sondern der Komponenten.

(Timer ist auf 1 Minute eingestellt... Das müsste reichen)

Aber ich versuche mit Enabled auf false während des Durchlaufes.

[Delphi.Narium]

Zitat von DaSchwoa:

Ich komme aber einfach nicht in meine Procedure rein.

Was heißt das?

Steht die Ausgabe von WriteToLog('Timer1Timer','Start'); nie in der Logdatei?
Oder steht es in der Logdatei, aber die Ausgabe von  WriteToLog('doDownload', 'Try to read'); fehlt?
Oder ...?

Irgendwie reichen Deine Informationen noch nicht aus, um sich eine konkrete Vorstellung des Problemes machen zu können.

Wäre es möglich den Quelltext des Projektes hier mal anzuhängen, einschließlich einer Logdatei, in der der Fehler nachzuvollziehen ist?

[DaSchwoa]

Danke an alle die mir geholfen haben. Mein Programm funktioniert jetzt.
Jedoch bin ich mir unsicher was falsch war . Ich vermute es hat was mit der rekursion was zu tun.