Tagesanzahl Februar eines Jahres

[Andreas13]

Zitat von taveuni:

Zitat von Delphi.Narium:

Geht aber kürzer Days := IfThen(IsLeapYear(Year),29,28);

Wow!

Diese neuere Kurzform IfThen trägt nicht unbedingt zur Übersichtlichkeit bei. Es ist eher fehleranfällig als gut, daher würde ich es persönlich nicht benutzen. Intern sind die beiden Befehle sicherlich identisch.
Andreas

[DeddyH]

Zitat von Andreas13:

Intern sind die beiden Befehle sicherlich identisch.

Sind sie

[Delphi.Narium]

Zitat von Andreas13:

Zitat von taveuni:

Zitat von Delphi.Narium:

Geht aber kürzer Days := IfThen(IsLeapYear(Year),29,28);

Wow!

Diese neuere Kurzform IfThen trägt nicht unbedingt zur Übersichtlichkeit bei. Es ist eher fehleranfällig als gut, daher würde ich es persönlich nicht benutzen. Intern sind die beiden Befehle sicherlich identisch.
Andreas

Was ist daran fehleranfällig?

Wenn ich beim Programmieren nicht aufpasse, mache ich Fehler.

Beim Aufruf einer derartigen Funktion mit 'nem Rückgabewert ist (zumindest bei mir) die Fehlerhäufigkeit nicht höher, als bei beliebigen anderen Funktionen mit mehreren Parametern und 'nem Rückgabewert.

Und das IfThen 'ne neuere Kurzform ist, halt' ich für 'n Gerücht. Mein olles Delphi 7 aus dem Jahr 2002 enthält diese Funktionen in den Units Math und StrUtils. Die neuere Kurzform ist also erst / mindestens 18 Jahre alt und damit volljährig und darf wählen sowie gewählt werden. Ich wähle sie

Und wenn sich wer an dem If Then stört:

Delphi-Quellcode:

case IsLeapYear(Year) of
  true  : Days := 29;
  false : Days := 28;
end;

Oder ginge es auch so? Days := 28 + Ord(IsLeapYear(Year));

[Andreas13]

Nichts für ungut: Bekanntlich darf jeder nach seiner Fasson glücklich werden. Ich habe lediglich meinen Standpunkt dargelegt. Wenn  IfThen schon "volljährig" ist, dann benutze ich es seit 18 Jahre bewußt nicht . Da ich 1985 mit Tubo Pacal 3 angefangen habe, können 18 Jahre "alte" Funktionen für mich durchaus als "neuere" gelten…

Die meisten Delphianer haben sich für Pascal entschieden, u. a. weil es eine disziplinierte Programmiersprache ist und unser Code dadurch leichter zu verstehen und warten ist. Allzu trickreiche Programmierung und Sprach-Akrobatik rächt sich meistens erst später bei der Pflege und Erweiterung der Programme.

Der Vorschlag von Detlef unter #7 in der ausführlichen Schreibweise ist selbsterklärend und "idiotensicher", während man bei IfThen die Reihenfolge der Parameter leicht vertauschen kann, ohne den Fehler zu merken.

Mein Motto ist daher, daß Übersichtlichkeit & Zuverlässigkeit vor Kürze gehen. Schreibfaulheit in der Kodierung ist eher ein Merkmal vieler C++-Programmierer. Wer möchte schon z. B. solche kryptischen Zeilen (= kurze, kompakte Befehle) nach Jahren weiterpflegen:

Code:

const Doub *xa = &xx[jl], *ya = &yy[jl]; 
... 
j = (Int)(pp % nhash); 
... 
Int k = ng ? garbg[--ng] : nn++ ; 
... 
for (;;) { 
for (i=1;i<=n;i++) x[i]=xold[i]+alam*p[i]; 
*f=(*func)(x); 
if (alam < alamin) 
for (i=1;i<=n;i++) x[i]=xold[i]; 
*check=1; 
return; 
} else if (*f <= fold+ALF*alam*slope) return; 
... 
k = ng ? garbg[--ng] : nn++ ; 
k != -1; k = next[k]) 
... 
y=ya[ns--]; 
... 
y += (dy=(2*(ns+1) < (mm-m) ? c[ns+1] : d[ns--])); 
...

Gruß, Andreas

[haentschman]

Moin...

...man sollte aber auch dazu sagen, daß das dieses IfThen aus der Unit Math kommt. IfThen gibt es auch in SysUtils...die macht aber was ganz anderes.

http://docwiki.embarcadero.com/Libra...trUtils.IfThen
http://docwiki.embarcadero.com/Libra...em.Math.IfThen

[Uwe Raabe]

Bei IfThen gilt auch zu beachten, dass es sich hier um einen Funktionsaufruf handelt, bei dem immer alle Parameter ausgewertet werden.

Delphi-Quellcode:

var
  cmp: TComponent;
  cmpName: string;
begin
  cmp := FindComponent('Edit1');

  { hier kann ein Zugriffsfehler auftreten }
  cmpName := IfThen(cmp <> nil, cmp.Name, '');

  { kann hier nicht passieren }
  if cmp <> nil then
    cmpName := cmp.Name
  else
    cmpName := '';

[Delphi.Narium]

Zitat von Uwe Raabe:

Bei IfThen gilt auch zu beachten, dass es sich hier um einen Funktionsaufruf handelt, bei dem immer alle Parameter ausgewertet werden.

Delphi-Quellcode:

var
  cmp: TComponent;
  cmpName: string;
begin
  cmp := FindComponent('Edit1');

  { hier kann ein Zugriffsfehler auftreten }
  cmpName := IfThen(cmp <> nil, cmp.Name, '');

  { kann hier nicht passieren }
  if cmp <> nil then
    cmpName := cmp.Name
  else
    cmpName := '';

Stimmt, aber in so 'ner Situation käme ich nie und nimmer auf die Idee IfThen zu nutzen. Bei Parametern für IfThen weiß ich immer vorher, dass ich das problemlos drauf zugreifen kann.

Delphi-Quellcode:

var
  cmp: TComponent;
  cmpName: string;
begin
  cmp := FindComponent('Edit1');
  if Assigned(cmp) then begin // ohne das, geht es bei mir nach 'nem FindComponent o. ä. garantiert nicht weiter.
    ...
  end else begin
    ...
  end;

[Rollo62]

Zitat von Uwe Raabe:

Bei IfThen gilt auch zu beachten, dass es sich hier um einen Funktionsaufruf handelt, bei dem immer alle Parameter ausgewertet werden.

Sehr guter Hinweis.
Gäbe es denn irgendeinen Trick um das zu Vermeiden, ich denke mal nicht, oder ?
Shortcuts gibt es nur bei Operatoren soweit mir bekannt.

[joachimd]

Zitat von Andreas13:

Code:

const Doub *xa = &xx[jl], *ya = &yy[jl]; 
... 
j = (Int)(pp % nhash); 
... 
Int k = ng ? garbg[--ng] : nn++ ; 
... 
for (;;) { 
for (i=1;i<=n;i++) x[i]=xold[i]+alam*p[i]; 
*f=(*func)(x); 
if (alam < alamin) 
for (i=1;i<=n;i++) x[i]=xold[i]; 
*check=1; 
return; 
} else if (*f <= fold+ALF*alam*slope) return; 
... 
k = ng ? garbg[--ng] : nn++ ; 
k != -1; k = next[k]) 
... 
y=ya[ns--]; 
... 
y += (dy=(2*(ns+1) < (mm-m) ? c[ns+1] : d[ns--])); 
...

So etwas habe ich in meinen C++-Jahren geliebt. Ich musste mich bei der Rückkehr zu Pascal damals extrem umstellen

[Amateurprofi]

Zitat von Delphi.Narium:

Geht aber kürzer Days := IfThen(IsLeapYear(Year),29,28);

oder noch kürzer
Days:=28+Ord(IsLeapYear(Year));