Exception-Handling bei ApdCOMPort

[Carsten1234]

Hallo zusammen,

ich habe eine Unit, die als reine Schnittstelle (Schnittstellen-Unit) dient zw. dem Programm und dem Gerät, mit dem ich kommuniziere. Für die Kommunikation kommt die ApdCOMPort-Komponente von AsyncPro zum Einsatz.
Nun kann ja mal bei der Kommunikation etwas schief gehen und mitunter wirft diese Komponente eine Exception (vergl. AdExcept.pas ab Z. 505ff).
Hier steht u.a.:

Delphi-Quellcode:

function CheckException(const Ctl : TComponent; const Res : Integer) : Integer;
    {-Check Res, raise appropriate exception if non-zero}
  var
    ErrorMsg : String;
    FileIO   : EInOutError;

  begin
    Result := Res;
    if (Res < ecOk) then
      if not (csLoading in Ctl.ComponentState) then begin
        case Res of
          ecHardwareFailure..ecFileNotFound:
            begin
              {find the error message for the error}
              ErrorMsg := AproLoadStr(Abs(Res));

              {if we've run out of memory, raise that exception}
              if (Res = ecOutOfMemory) then
                OutOfMemoryError

              {otherwise, raise a file I/O exception}
              else begin
                FileIO           := EInOutError.Create(ErrorMsg);
                FileIO.ErrorCode := Abs(Res);
                raise FileIO;
              end;
            end;

          {EGeneral}
          ecBadArgument           : raise EBadArgument.Create(Res, False);
          ecGotQuitMsg            : raise EGotQuitMsg.Create(Res, False);
          ecBufferTooBig          : raise EBufferTooBig.Create(Res, False);
          ecPortNotAssigned       : raise EPortNotAssigned.Create(Res, False);
          ecInternal,
          ecNoFieldsDefined,
          ecNoIndexKey,
          ecDatabaseNotPrepared   : raise EInternal.Create(Res, False);
          ecModemNotAssigned      : raise EModemNotAssigned.Create(Res, False);
          ecPhonebookNotAssigned  : raise EPhonebookNotAssigned.Create(Res, False);
          ecCannotUseWithWinSock  : raise ECannotUseWithWinsock.Create(Res, False);

          {EOpenComm}
          ecBadId                 : raise EBadId.Create(Res, False);
          ecBaudRate              : raise EBaudRate.Create(Res, False);
          ecByteSize              : raise EByteSize.Create(Res, False);
          ecDefault               : raise EDefault.Create(Res, False);
          ecHardware              : raise EHardware.Create(Res, False);
          ecMemory                : raise EMemory.Create(Res, False);
          ecCommNotOpen           : raise ECommNotOpen.Create(Res, False);
(...)

Ich würde nun gerne diese Exceptions an die MainForm weiter leiten (ich dachte an eine Sendmessage(MainForm.Handle, ...) ) und dort behandeln bzw. eine detaillierte Fehlermeldung ausgeben lassen.

Delphi-Quellcode:

try
  ApdComPort.PutString(AValue); // AValue = String, der geschickt werden soll
except
// SendMessage(MainForm.Handle, wm_COMError, ???, 0); // wm_COMError = wm_User+53;
end;

Frage: Wie mache ich das bzw. fange diese Exceptions auf und leite sie entsprechend weiter?
ecCommNotOpen usw. sind als Zahlen deklariert in OoMisc.pas.

Dank vorab und Gruß, Carsten

[sx2008]

Ich würde die Exception abfangen, etwas Zuckerguss auf die Fehlermeldung streuen (*) und die Exception weiter nach oben geben.

Delphi-Quellcode:

try
  ApdComPort.PutString(AValue); // AValue = String, der geschickt werden soll
except
  on E:Exception do
  begin
    E.Message := 'Fehler beim Schreiben auf serielle Schnittstelle'#13#10 + E.Message
    raise;
  end;
end;

*) Eine gute Fehlermeldung sieht so aus:
zuerst kommt eine allgemeine Fehlerbeschreibung, die dem Benutzer eine Information gibt, was gerade passiert ist.
Daran anhängend sollte eine genauere Info sein, die möglicherweise nur der Programmierer versteht (z.B. "errorcode 5815" oder "comport not opened").
Wichtig ist, dass keine Fehlerinformation verloren geht oder durch eine allgemeine und nichtssagende Meldung überdeckt wird.
Deshalb sollte man auch die orginale Exceptionklasse beibehalten, denn auch im Klassenname steckt eine gewisse Information.
In .NET kann man Exceptions auch schachteln, was natürlich noch eleganter ist.

[Carsten1234]

Zitat:

Zitat von sx2008

*) Eine gute Fehlermeldung sieht so aus:
(...)

Prinzipiell richtig und eigentlich auch so angedacht, aber:
Der Text der Fehlermeldung soll mehrsprachig sein und die Unit (nur Schnittstelle zum Gerät), in der diese Exception geworfen wird, kann keine Mehrsprachigkeit. Daher will ich die Exception an das Hauptformular durchreichen, um die Exception mehrsprachig auszugeben.

[Carsten1234]

Nur zur Info:
Ich reiche die Exception jetzt per SendMessage an die MainForm weiter ala

Delphi-Quellcode:

const
  wm_WGError  = wm_User+60;
  wm_COMError = wm_User+61;
(...)
try
  ApdComPort.PutString(AValue);
except
  on E:Exception do
    SendMessage(MainForm.Handle, wm_WGError, wm_COMError, Integer(@E));
end;

und fange die Message in der MainForm dann auf mit

Delphi-Quellcode:

var E: Exception;
begin
if (Msg.Msg = wm_WGError) then
begin
  if (Msg.WParam = wm_COMError) then
  begin
    E:= nil;
    E:= Pointer(Msg.LParam);
    if (Assigned(E)) then
    begin
      if (E.ClassType = EOpenComm) then
      begin
        // irgendwas
      end;
      if (E.ClassType = EGeneral) then
      begin
        // irgendwas anderes
      end;
    end;
  end;
(...)
end;

Gruß, Carsten