Gunnar Bittersmann: Kleine Änderung in Java Script Counter

Beitrag lesen

@@Phil:

Ich verwende den untenstehenden Java Script Counter.

JavaScript – ein Wort.

JavaScript-Counter – mit Bindestrich, nicht mit Deppenleerzeichen.

Leider verstehe ich nicht allzuviel von Java Script.

Es kann ziemlich gefährlich sein, Scripte einzusetzen, die man irgendwo gefunden hat, aber nicht weiß, was sie tun.

<script language="Javascript">

Falsch. Wie’s richtig geht, steht in http://de.selfhtml.org/javascript/intro.htm#javascriptbereiche@title=SELFHTML.

<!--

Das stammt aus dem letzten Jahrtausend. Die Auskommentierung ist längst völlig unnötig und nicht mehr zeitgemäß.

if(Rest >= 31536000)
{
var Jahre = Math.floor(Rest/31536000);
Rest = Rest-Jahre*31536000;
``
if(Jahre > 1 || Jahre == 0)
{
CountdownText += Jahre + " Jahre ";
}
else if(Jahre == 1)
{
CountdownText += Jahre + " Jahr ";
}
}

Das ist ziemlicher Unfug. Wenn in den else-Zweig gegangen wird, welchen Wert hat Jahre dann? Wozu also die if-Abfrage?

if(Tage > 1 || Tage == 0)
{
CountdownText += Tage + ":";
}
else if(Tage == 1)
{
CountdownText += Tage + ":";
}

Eine Abfrage, bei der bei Zutreffen und bei Nichtzutreffen der Bedinung exakt dasselbe gemacht werden soll, ist noch größerer Unfug. Wozu also die if-Abfrage?

if(Stunden > 1 || Stunden == 0)
{
CountdownText += Stunden + ":";
}
else if(Stunden == 1)
{
CountdownText += Stunden + ":";
}

Dito.

Übrigens wäre hier eine geeignete Stelle, um abzufragen, ob Stunden < 10, also einstellig ist. In den Fall wäre eine führende '0' einzufügen: CountdownText += "0" + Stunden + ":";

Bei Minuten und Sekunden ebenso.

Live long and prosper,
Gunnar

--
„Das Internet ist ein großer Misthaufen, in dem man allerdings auch kleine Schätze und Perlen finden kann.“ (Joseph Weizenbaum)