[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);
    };
})();