Warnhinweis??

[Martin K]

Klammern nicht vergessen:

Delphi-Quellcode:

if (Edit1.text = '')  or not (radiogroup1.itemindex = -1) then 
  showmessage('.....');

Wenn Du schon vorne ne Klammer drum machst, warum dann nicht hinten?

[shmia]

Wenn wir schon dabei sind, sollten wir es auch richtig machen. :warn:
Wer von Euch mag Programme, die z.B. solche Fehlermeldungen ausgeben:
"keine Diskette im Laufwerk oder Datei nicht gefunden !"
Wenn schon eine Fehlermeldung, dann bitte eine ohne das Wörtchen ODER.
Ausserdem sollte man bei Fehlermeldungen eine Exception "werfen" und nicht die altmodische Technik benützen:

Delphi-Quellcode:

if (Edit1.text = '') then
begin
   Edit1.SetFocus;
   raise Exception.Create('kein Text eingegeben');  // TODO: "Text" durch passenden Begriff ersetzen
end;
if radiogroup1.itemindex = -1 then
begin
   radiogroup1.SetFocus;
   raise Exception.Create('keine Option ausgewählt');  // TODO: "Option" durch passenden Begriff ersetzen
end;

[zahor]

Wieso denn "or NOT"???
Es soll doch überprüfen ob nix ausgewählt ist.
So muss es sein wenn man die "alte", nicht von shmia verbesserte Version nimmt:

Delphi-Quellcode:

if (Edit1.text = '')  or (radiogroup1.itemindex = -1) then
  showmessage('.....');

[fwsp]

Zitat:

Zitat von shmia

Wenn schon eine Fehlermeldung, dann bitte eine ohne das Wörtchen ODER.
Ausserdem sollte man bei Fehlermeldungen eine Exception "werfen" und nicht die altmodische Technik benützen:

vielleicht ist es so für nen anfänger aber einfacher verständlich?