AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren
Thema durchsuchen
Ansicht
Themen-Optionen

W1022 Der Vergleich ergibt immer True

Ein Thema von luke2 · begonnen am 22. Jul 2010 · letzter Beitrag vom 22. Jul 2010
Antwort Antwort
luke2

Registriert seit: 17. Jun 2009
117 Beiträge
 
#1

W1022 Der Vergleich ergibt immer True

  Alt 22. Jul 2010, 15:51
Delphi-Version: 2010
Hi,

Ich habe folgende Funktion:
Delphi-Quellcode:
function IsValidIP (const aIP: string): Boolean;
begin
  Result := ((aIP <> '') and (inet_addr(PansiChar(ansistring(aIP))) <> INADDR_NONE));
end;
Unter Delphi2010 kommt die Meldung W1022 Der Vergleich ergibt immer True, unter Delphi2007 kommt sie nicht. Wenn ich INADDR_NONE durch -1 ersetze, verschwindet die Meldung.
Kann mir jemand sagen woran das liegt?
  Mit Zitat antworten Zitat
Benutzerbild von himitsu
himitsu
Online

Registriert seit: 11. Okt 2003
Ort: Elbflorenz
43.140 Beiträge
 
Delphi 12 Athens
 
#2

AW: W1022 Der Vergleich ergibt immer True

  Alt 22. Jul 2010, 16:22
Delphi-Quellcode:
const INADDR_NONE = DWORD($FFFFFFFF);

type u_long = Longint;
function inet_addr(cp: PAnsiChar): u_long; stdcall;
Da war mal wieder irgend ein Idiot (oder doofes Übersetzungscript) am Werk:
u_long = unsigned Long = LongWord

DWORD(-1), also 4.294.967.295 kann niemals in LongInt reinpassen, und demnach ist ein Vergleich auf <> immer true.

Aber sowas hätte man auch selber schnell rausbekommen können, wenn man sich die Typen mal ansieht.

Delphi-Quellcode:
function IsValidIP(const aIP: AnsiString): Boolean;
begin
  Result := (aIP <> '') and (LongWord(inet_addr(PAnsiChar(aIP))) <> INADDR_NONE);
end;
oder
Delphi-Quellcode:
function IsValidIP(const aIP: AnsiString): Boolean;
begin
  Result := (aIP <> '') and (inet_addr(PAnsiChar(aIP)) <> LongInt(INADDR_NONE));
end;
Garbage Collector ... Delphianer erzeugen keinen Müll, also brauchen sie auch keinen Müllsucher.
my Delphi wish list : BugReports/FeatureRequests

Geändert von himitsu (22. Jul 2010 um 16:25 Uhr)
  Mit Zitat antworten Zitat
luke2

Registriert seit: 17. Jun 2009
117 Beiträge
 
#3

AW: W1022 Der Vergleich ergibt immer True

  Alt 22. Jul 2010, 16:45
super Danke
  Mit Zitat antworten Zitat
Antwort Antwort


Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 02:38 Uhr.
Powered by vBulletin® Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024 by Thomas Breitkreuz