"Boolean-Roulette" oder "if x=true"

[glkgereon]

Hi,

Ein freund im Info-Kurs hat sich gerade eine BoolToStr-Funktion geschrieben...
(Wir haben noch Delphi 4 :-/ )

das ganze sah dann in etwa so aus:

Delphi-Quellcode:

function BoolToStr(Val: Boolean): String;
begin
  Result:='Undefined'; //Debug
  if Val=True then Result:='True'
  else if Val=False then Result:='False';
end;

Interessanterweise kam oft 'Undefined' raus. :gruebel:
Wieso ist (Val=True) False wenn Val=True ist?
Will der mich verarschen? :wiejetzt:

[Klaus01]

Boolean kann auch 0 oder 1 sein, oder nicht?

Delphi-Quellcode:

function BoolToStr(Val: Boolean): String; 
begin 
  Result:='Undefined'; //Debug 
  if Val then Result:='True' 
  else if not Val then Result:='False'; 
end;

Grüße
Klaus

[Yheeky]

Hi!

Ich würde einfach sagen:

Delphi-Quellcode:

function BoolToStr(Val: Boolean): String;
begin
  Result:='Undefined'; //Debug
  if Val=True then Result:='True'
  else Result:='False';
end;

nach else ist in meinen Augen keine Abfrage mehr notwendig.

[NicoDE]

Zitat:

Zitat von glkgereon

Interessanterweise kam oft 'Undefined' raus. :gruebel:
Wieso ist (Val=True) False wenn Val=True ist?

'if Foo = True then' ist ein Vergleich mit der Konstante 'True' (1). Das fällt, meiner Meinung nach unter 'schlechten Stil' und 'Bugs die keiner braucht'...

Mögliche Alternativen:

'if Foo then'
'if Foo <> False then'

'if not Foo then'
'if Foo = False then'

[Matze]

Hi

Wie Klaus und Nico schon angedeutet haben, lasse =true generell weg. Hier siehst du auch den Grund dafür.

[xaromz]

Hallo,

das hatten wir doch schon oft.
False ist Null, während alles, was nicht Null ist, als True gewertet wird. Die Konstante True aber ist Eins.
Deshalb darf man zwar auf False prüfen, aber nie auf True. Stattdessen immer

Delphi-Quellcode:

if A then

also das = True weglassen.

Gruß
xaromz