JS: 2 nahezu identische Funktionen, trotzdem andere Funktion

Beitrag von **pee** » 11.10.2009, 23:58

Hallo,

bei changed(); wird nach einer dynamischen XHTML-Änderung der Wert eines Tags in eine Variable geschrieben. Deren Wert wird nach 1500 Millisekunden mit dem aktuellen Wert im Quelltext des XHTML-Dokuments verglichen. Unterscheidet sich dieser, passiert nichts. Sind beide Werte gleich, wird eine Funktion ausgeführt. Die Werte sind Pixelangaben die zwischen 0 und 200 Pixel variieren.

Der zweite Teil von changed(); ist mit dem ersten nahezu identisch. Trotzdem wird expandOnly('daySlider'); ausgeführt, obwohl die beiden Werte nicht gleich sind.

Code: Alles auswählen

function changed&#40;id&#41; &#123;
 
 
  if&#40;$&#40;'demo_min_thumb'&#41;.style.left!='0px'&#41; &#123;
 
    snap = $&#40;'demo_min_thumb'&#41;.style.left;
    setTimeout&#40;"checkIfSliderStopped1&#40;&#41;", 1500&#41;;
   
  &#125;

  if&#40;$&#40;'slider-bg'&#41;.title!='Slider'&#41; &#123;
 
    snap2 = $&#40;'slider-bg'&#41;.title;
    setTimeout&#40;"checkIfSliderStopped2&#40;&#41;", 1500&#41;;
   
  &#125;
 
&#125;

function checkIfSliderStopped1&#40;&#41; &#123;
    if &#40;snap ==  $&#40;'demo_min_thumb'&#41;.style.left&#41; &#123;
      expandOnly&#40;'daySlider'&#41;;
    &#125;
&#125;


function checkIfSliderStopped2&#40;&#41; &#123;
    if &#40;snap2 == $&#40;'slider-bg'&#41;.title&#41; &#123;
      expandOnly&#40;'options'&#41;;
    &#125;
&#125;

Ist jemand so nett und spendet mir Rat?

von **Anzeige von ABAKUS** »

Beitrag von **Lord Lommel** » 12.10.2009, 08:55

Ich gebe dir den Rat, daß Prototype bei größeren Konstrukten enorm langsam wird. Ansonsten weiß ich nicht wirklich, was du willst.

Beitrag von **Mork vom Ork** » 12.10.2009, 12:08

pee hat geschrieben:
Code: Alles auswählen
function checkIfSliderStopped1&#40;&#41; &#123;
    if &#40;snap ==  $&#40;'demo_min_thumb'&#41;.style.left&#41; &#123;
      expandOnly&#40;'daySlider'&#41;;
    &#125;
&#125;
Der zweite Teil von changed(); ist mit dem ersten nahezu identisch. Trotzdem wird expandOnly('daySlider'); ausgeführt,

Warum du die Ursache für den Fehler darin siehst, dass der erste und zweite Teil von changed() nahezu identisch sind, habe ich nicht verstanden.

obwohl die beiden Werte nicht gleich sind.

Wenn sie nicht gleich wären, würde die Funktion nicht ausgeführt werden, auf die korrekte Funktion von == ist durchaus Verlass. Dein Problem ist vermutlich, dass sie unerwarteterweise gleich sind (oder du noch irgendwo anders ein expandOnly('daySlider') stehen hast). Dem gilt es auf den Grund zu gehen, d.h. wann und wo wird snap gesetzt. Wird changed() möglicherweise öfter aufgerufen als du glaubst?

Besorge dir Firefox und dazu die Erweiterung Firebug, in letzterem ist auch ein guter Javascript-Debugger enthalten, mit dessen Hilfe du Schritt für Schritt den Ablauf und den Inhalt von snap verfolgen kannst.
Alternativ kannst du natürlich auch den Code mit alert()-Aufrufen zupflastern, die dir vor und nach jeder relevanten Zeile den Inhalt von snap & Co. ausgeben.

setTimeout("checkIfSliderStopped1()", 1500);

Den auszuführenden Code als Text zu übergeben kann man zwar machen, ist aber schlechter Stil, in erster Linie, weil der Code dann erst bei Ausführung geprüft wird und nicht bereits beim Laden des Skriptes.

setTimeout() akzeptiert auch eine Funktion als Argument, setTimeout(checkIfSliderStopped1, 1500); funktioniert daher genauso gut. Soll mehr Code eingebunden werden, lässt sich auch eine anonyme Funktion nutzen, womit du dann auch deine Hilfsfunktionen einsparen könntest und den auszuführenden Code da hast, wo er auch benötigt wird:

Code: Alles auswählen

setTimeout&#40;function&#40;&#41; &#123;
        if &#40;snap ==  $&#40;'demo_min_thumb'&#41;.style.left&#41; &#123;
            expandOnly&#40;'daySlider'&#41;; 
        &#125;
    &#125;, 1500&#41;;

Beitrag von **pee** » 12.10.2009, 13:37

Es lag wohl daran, dass die Werte tatsächlich gleich waren. Nachdem ich die Vergleichsdauer auf 500 Millisekunden reduziert habe, wird die Funktion wie gewünscht ausgeführt.

Vielen Dank an euch Experten für die hervorragenden Antworten und den angenehmen Wortlaut.