Warning unreachable code

[DL3AD]

Hallo,

bekomme bei dieser Codezeile die Meldung Warning unreachable code
DBlib ist als string definiert.

Code: Alles auswählen

 
    if exe = '.exe' then DBlib:= 'sqlite3.dll' else DBlib:= 'libsqlite3.so.0'; 
 

Was hat dass zu bedeuten ?

Gruß Frank

[m.fuchs]

Mal davon ausgehend dass du exe wie in http://www.lazarusforum.de/viewtopic.ph ... 43#p100343 definiert hast: Je nach Betriebssystem kann einer von beiden Zweigen deines if-Statements niemals aufgerufen werden. Deswegen gibt es diese Warnung.

[DL3AD]

... bedeutet aber dass die Codezeile so OK ist ?

[m.fuchs]

Ja, ist nur ein Hinweis darauf, dass du dir einen Teil des Codes sparen könntest. Wenn dich das (berechtigterweise) stört, dann solltest du die Library-Namen auch per ifdef festlegen.

[Mathias]

dann solltest du die Library-Namen auch per ifdef festlegen.

Das sieht dann bei dir so aus:

Code: Alles auswählen

  {$IFDEF windows}
  DBlib:= 'sqlite3.dll';
  {$ENDIF}
  {$IFDEF Linux}
  DBlib:= 'libsqlite3.so.0';
  {$ENDIF}

Somit wird nur der Code welcher auch für das OS gebraucht wird mit kompiliert.

[compmgmt]

Code: Alles auswählen

if exe = '.exe' then DBlib:= 'sqlite3.dll' else DBlib:= 'libsqlite3.so.0';

So etwas zu vergleichen um das aktuelle OS rauszubekommen ist nicht wirklich schön. Mach es lieber mit $ifdef, so wie Mathias es beschrieben hat. Solltest du es an mehreren Stellen brauchen, kannst du es natürlich genauso wie bei der exe machen:

Code: Alles auswählen

const
  Lib = {$ifndef mswindows} 'lib' {$else} '' {$endif};
  LibExt = {$ifdef mswindows}
      // Windows
      '.dll'
    {$else}
      {$ifdef darwin}
        // macOS (OS X, nicht MacOS Classic)
        '.dylib'
      {$else}
        // Alles andere (Linux, BSD, ...)
        '.so.0'
      {$endif}
    {$endif};
begin 
  ...
  if FileExists(Lib + 'sqlite3' + LibExt) then ...
  ...
end;

Wobei ich mir bei der Konstante "Lib" nicht ganz sicher bin, ob das richtig ist, da ich nicht weiß, ob man bei macOS überhaupt ein "lib" vor den Dateinamen setzt oder nicht.

[theo]

Es gibt die auch die betriebssystemabhängige Konstante SharedSuffix in dynlibs

http://wiki.freepascal.org/Lazarus/FPC_ ... ic_library
https://www.freepascal.org/docs-html/rt ... uffix.html