[jQuery] Problem z wyciąganiem stylu diva

Question

Krótko i prosto z mostu:

Na mojego diva zasadziłem funkcję z jQueryUI draggable();. Za każdym razem gdy użytkownik puści to swoje pudełko, wywoływana jest funkcja, która w przypadku umieszczenia pudełka poza lewą i górną krawędzią strony, wywołuje alert w postaci podanych dwóch wartości x y oddzielnych spacją. W tym przypadku wyciągnąłem wartość pozycji pudełka na podstawie odległości od górnej i lewej krawędzi (left i top), usunąłem suffix "px" i przekształciłem string na int. Tak wygląda mój kod:

$(function() {
	$( "#sheet" ).draggable({
      start: function() {
	  
      },
      drag: function() {
        
      },
      stop: function() {
	   left = parseInt($("#sheet").css("left").replace("px",""));
	   top = parseInt($("#sheet").css("top").replace("px",""));
	   if (left<0 || top<0) {
	   alert(left+" "+top);
	   }
      }
    });
});  

O dziwo, za każdym razem gdy puszczę pudełko, mimo tego że wartość pierwsza jest pokazywana dobrze, to druga wartość jest przedstawiona w postaci [object Window]. Nie otrzymuję tam żadnej liczby.

 

Chciałbym prosić nie tyle co o propozycję działającego kodu, ale też wyjaśnienie, dlaczego nie otrzymuję prawidłowej wartości, mimo że pierwsza wartość jest pokazywana poprawnie.

Answer 1

Najwidoczniej skrócenie funkcji do absolutnego minimum załatwiło sprawę. Sam nie wiem czemu, ale działa.

$(function() {
    $( "#sheet" ).draggable({
      start: function() {
      
      },
      drag: function() {
        
      },
      stop: function() {
       if (parseInt($("#sheet").css("left").replace("px",""))<0 || parseInt($("#sheet").css("top").replace("px",""))<0) {
       alert("tak");
       }
      }
    });
});  

 

EDIT: Wygląda na to, że nazwa zmiennej 'top' jest nazwą zarezerwowaną. 

Comments

Zmień zmienną "top" na jakąś inną.

 

np.

$(function() {
    $( "#sheet" ).draggable({
      start: function() {
       
      },
      drag: function() {

      },
      stop: function() {
       toptop = parseInt($("#sheet").css("top").replace("px",""));
       left = parseInt($("#sheet").css("left").replace("px",""));
       
       if (left<0 || toptop<0) {
       console.log(left+" "+toptop);
       }
      }
    });
});

 

---

@kubaapk

Właśnie zauważyłem, że tylko z tą nazwą jest problem. Wydaje mi się, że załączony przeze mnie kod jest równie poprawny. Dziękuję za uświadomienie błędu :)