Raik: Laufend sich aendernde Farbe

Beitrag lesen

Hallo, Max!

nur mal als kleines beispiel zur anschauung, wie man sinnvollerweise solchen code strukturiert:

function Init(x) {
   var p;
   if (x>100){
       p = x-10;
       document.getElementById("Hea1").style.color = "rgb("+p+",0,0)";
       window.setTimeout("Init(p)",500);
   };

^
das gehört da nicht hin.

if (x<100) {
       Init2(x);
   };

^

}

function Init2(x) {
   var p;
   if (x<250) {
       p = x+10;
       document.getElementById("Hea1").style.color = "rgb("+p+",0,0)";
       window.setTimeout("Init2('p')",500);
   };

^

if (x>250) {
       Init(x);
   };

^

}

und anstatt 2 if-abfragen zu machen ist es sinnvoller,
eine bedingung zu prüfen und in _allen_ anderen fällen mit einem else-zweig zu reagieren:

var p;
function Init(x) {
   if (x>100){
       p = x-10;
       document.getElementById("Hea1").style.color = "rgb("+p+",0,0)";
       window.setTimeout("Init(p)",500);
   }
   else {
       Init2(x);
   }
}

es wäre auch nett, wenn du den funktionierenden code hier nochmal posten würdest, damit andere daraus was lernen können, wenn sie ihn im archiv finden.

freundl. Grüsse aus Berlin, Raik

--
die Schläge, die Du bereitwillig hinnimmst, werden Dir eine größere Hilfe im Leben sein, als die, die Du für ungerechtfertigt ablehnst; denn einestages wirst auch Du die Fürsorge und Hilfe darin erkennen.
(c)XaraX http://forum.de.selfhtml.org/archiv/2004/4/77523/#m447546