Don P: Gehe ich richtig mit Ajax um?

Beitrag lesen

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:&