Hallo,
Ist das okay so oder könnte ich irgendwas besser machen / vereinfachen / "sauberer" machen?
Mal abgesehen von Axax:
if(e==""||!e)[...]
Da würde einfach if(!e) ausreichen.
if (req==null)[...]
Es macht im Allgemeinen wenig Sinn, gegen "", 0, undefined, null, false oder true mit == zu vergleichen, weil z.B. auch ' \t\r\n' == 0 gilt. Solche Vergleiche sollten ohne Typumwandlung, d.h. mit === gemacht werden, oder aber, wenn man die automatische Typumwandlung wirklich will, ganz ohne Vergleichsoperatoren, also mit if(bla) bzw. if(!bla).
Gruß, Don P
--
sh:( fo:) ch:? rl:( br:] n4:~ ie:% mo:? va:{ js:) de:/ zu:] fl:( ss:| ls:&
sh:( fo:) ch:? rl:( br:] n4:~ ie:% mo:? va:{ js:) de:/ zu:] fl:( ss:| ls:&