Gunnar Bittersmann: Tageszeiten wächselnde HP

Beitrag lesen

@@Tolonath:

Und was sagt ihr zu dem Script? o.0

Dass HTML-Kommentare "<!--" und "//-->" in JavaScript-Code nichts zu suchen haben. Gar nichts. Das sagte ich dir übrigens schon mal. Wie oft willst du’s noch hören?

Dass dein Quelltext ohne Einrückung schlecht lesbar ist.

Dass eine Funktion 'blah' zu nennen ziemlich blöd ist.

Dass man <http://de.selfhtml.org/javascript/sprache/variablen.htm@title=keine globalen Variablen> verwenden sollte, sondern http://de.selfhtml.org/javascript/sprache/variablen.htm@title=lokale.

Dass du statt

  d=new Date();  
  s=d.getHours();

gleich schreiben kannst
var s = new Date().getHours();

Dass die zweite Abfrage bei

  if(s>=7&&s<16)a=0;  
  if(s>=16&&s<7)a=1;

überflüssig ist; 'http://de.selfhtml.org/javascript/sprache/bedingt.htm#if_else@title=else' existiert. Überhaupt ist 'if' hier nicht Mittel der Wahl: Für zwei verschiedene Wertzuweisungen gibt es den <http://de.selfhtml.org/javascript/sprache/bedingt.htm#entweder_oder@title=Coditional Operator>:
var a = (s >= 7 && s < 16 ? 0 : 1);
Bei mehr als zwei Fällen wäre 'http://de.selfhtml.org/javascript/sprache/bedingt.htm#switch@title=switch' angebracht.

So, jetzt bist du dran, dein Script in Ordnung zu bringen und hier nicht nur größe Töne zu spucken.

Live long and prosper,
Gunnar

--
Das einzige Mittel, den Irrtum zu vermeiden, ist die Unwissenheit. (Jean-Jacques Rousseau)