SendMail mit Anhang

[Ykcim]

Wo und wie?

Bin für jede Anregung zu haben...

Ykcim

[Nersgatt]

Du gibst z.B. IdSMTP nicht frei.
Generell solltest Du Dir angewöhnen IMMER Ressourcenschutzblöcke zu verwenden, wenn Du Instanzen von Klassen erstellst.

Delphi-Quellcode:

Instanz := TMyClass.Create;
Try
... mach was mit der Instanz
Finally
  Instanz.free;
End

[DeddyH]

Ich sehe da 4 mal Create, aber nicht 1 mal Free. Üblicherweise macht man das ja so:

Delphi-Quellcode:

Dings := TDings.Create;
try
  Dings.MachWas;
finally
  Dings.Free;
end;

[edit] [/edit]

[Ykcim]

Guten Abend,

ich habe auf diesem Gebiet eine ganz große Lücke...
Kann mir jemand das "try" and "finally" mal für meine beschriebene Prozedur zeigen. Ich hoffe, dass ich dann den Zusammenhang verstehe. Ich habe es selber versucht, aber nur Errors bekommen.

Vielen Dank

Ykcim

[Luckie]

DeddyH hat dir doch ein Beispiel gepostet.

[Sir Rufo]

Delphi-Quellcode:

function SendEmail( Recipient : string; CCList : string; sSubject : string; Body : TStringList; Pfad : string )
  : boolean;
var
  nAlarmnr :   integer;
  Attachment : TIdAttachment;
  // Das fehlte hier noch !!!
  idSMTP : TIdSMTP;
  idMessage : TIdMessage;
begin

  Result := False;

  idSMTP := TidSMTP.Create;
  try

    idMessage := TidMessage.Create;
    try

      idSMTP.Host     := 'XXX.XXX.XXX.XXX';
      idSMTP.Username := 'XXXXXX';
      idSMTP.Password := 'XXXX';
      idSMTP.Port     := XX;

      idMessage.From.text   := '[e-mail]';
      idMessage.Sender.text := idMessage.From.text;

      idMessage.Recipients.EMailAddresses := Recipient;
      idMessage.CCList.EMailAddresses     := CCList;
      idMessage.Subject                   := sSubject;
      idMessage.ContentType               := 'multipart/*';

      with TidText.Create( idMessage.MessageParts, Body ) do
        begin
          ContentType := 'text/html';
        end;

      with TIdAttachmentFile.Create( idMessage.MessageParts, Pfad ) do
        begin
          idMessage.MessageParts.Add( );
        end;

      try
        idSMTP.Connect;
        try
          idSMTP.Send( idMessage );
          Result := True;
        finally
          idSMTP.Disconnect;
        end;
      except
        // Das macht man ja eigentlich nicht, keine Exception-Behandlung
      end;

    finally
      idMessage.Free;
    end;

  finally
    idSMTP.Free;
  end;

end;

[himitsu]

Zitat von DeddyH:

Ich sehe da 4 mal Create, aber nicht 1 mal Free. Üblicherweise macht man das ja so:

Die Text- und Attachmentteile hängen sich an die Message und werden mit dieser freigegeben.
(falls man die Message freigibt)

Zitat:

// Das macht man ja eigentlich nicht, keine Exception-Behandlung

Vorallem da man so nicht mehr rausbekommt, was genau schief lief.
Ein billiger Boolean sagt ja nichts Genaueres. (eine Exceptions dagegen schon mehr)


Dein Programm kann dem User so ja nur sagen "nee, ging nicht" aber nicht "ging nicht, weil ...".


Wobei Indy diesen ContentType auch selber setzt.

text/plain = Textmail ohne anhang
multipart/alternative = z.B. Text und HTML parallel
multipart/mixed = Anhänge mit Text und/oder HTML

Zitat:

// Das fehlte hier noch !!!

Um wieviel wollen wir wetten, daß sich dort ein paar Globale Variablen versteckten?

[Sir Rufo]

Zitat von himitsu:

Zitat:

// Das fehlte hier noch !!!

Um wieviel wollen wir wetten, daß sich dort ein paar Globale Variablen versteckten?

die scheinen wieder in Mode zu kommen

[Gustav.R]

Nur so aus dem OFF reingeplärrt:

CC ist wohl das neue BCC der Superheimlich-Multi-Geheimagenten?