Гляньте плс мой код

[Paul_Wahtel]

Всем привет, пишу функцию которая будет каждую секунду поднимать див на 50 пикселей в верх.
вот мой код

function upAndDown () {
        var lift = document.getElementById('lift');
	lift.style.marginBottom = "50px";
	}			
function lift () {
	timer = window.setInterval("upAndDown()", 1000);
       }


<a href="javascript:lift()">
			button
		</a>

при нажатии на ссылку диву единожды прибавляется margin-bottom: 50px; и все, хотя интервал сам продолжает идти, я просто только учусь js, что я написал не так?

[рони]

прибавляется

и где в коде хоть один +

[Paul_Wahtel]

Имеется в виду что ему присваивается margin-bottom 50 px, но лишь один раз.

[рони]

лишь один раз.

а вы сколько раз хотели, если а = 5 то чему равно а ?
почему не 10 или 15

[рони]

Paul_Wahtel,

var x = 0;
function upAndDown () {
        var lift = document.getElementById('lift');
	lift.style.marginBottom = x +"px";
x += 50;
	}			
function lift () {
	timer = window.setInterval("upAndDown()", 1000);
       }


<a href="javascript:lift()">
			button
		</a>

[Paul_Wahtel]

спасибо, что то я и вправду втыканул.

[ruslan_mart]

Элемент лучше вынести за ф-цию, чтобы каждый раз не получать его через getElementById. timer тоже нужно объявить. В setInterval лучше передавать саму ф-цию, а не строко-код.

var lift;

(function() {
    var elemStyle = document.getElementById('lift').style,
        timer, x = 0;
    lift = function() {
        clearInterval(timer);
        timer = setInterval(function() {
            x += 50;
            elemStyle.marginBottom = x + 'px';
        }, 1000);
    };
})();