Почему JS не видит второй аргумент?

Question

[DromHour]

Есть задача - если поле ввода "link" != источнику звука "player", исправить:

var dynUp = setInterval("if (document.getElementById('link').value != document.getElementById('player').getAttribute('src')) 
{
document.getElementById('player').setAttribute('src','document.getElementById(\'link\').value');
document.getElementById('loadicon').innerHTML = '<div class=\'cssload-whirlpool\'></div>';
while (document.getElementById('player').readyState == 0) {};
document.getElementById('loadicon').innerHTML = ''}", 100);

While используется как ожидание загрузки метаданных аудио.



Однако, после первого аргумента setAttribute, JS решает, что второй аргумент нам не особо-то и нужен, а вот закрывающую скобку он игнорит, в следствие чего, дальше код не идёт.

Uncaught SyntaxError: missing ) after argument list

Comments

[OVK2015]

setInterval("if (document.getElementById('link').value != document.getElementById('player').getAttribute('src'))

Вам самому то не страшно?

[DromHour]

OVK2015: ???

[OVK2015]

DromHour: Ниже уже ответили. Зачем такие изыски со строкой в setInterval?

Answer 1

[Михаил Серенков]

var dynUp = setInterval(function () {
    if (document.getElementById('link').value != document.getElementById('player').getAttribute('src')) {
        document.getElementById('player').setAttribute('src', 'document.getElementById(\'link\').value');
        document.getElementById('loadicon').innerHTML = '<div class=\'cssload-whirlpool\'></div>';
        while (document.getElementById('player').readyState == 0) {
            
        }
        document.getElementById('loadicon').innerHTML = ''
    }
}, 100);

Если хотите передать код в строке то либо пишите все в одну строку, либо переносите следующим образом

var dynUp = setInterval("if (document.getElementById('link').value != document.getElementById('player').getAttribute('src'))" +
            "{" +
            "document.getElementById('player').setAttribute('src','document.getElementById(\'link\').value');" +
            "document.getElementById('loadicon').innerHTML = '<div class=\'cssload-whirlpool\'></div>';" +
            "while (document.getElementById('player').readyState == 0) {};" +
            "document.getElementById('loadicon').innerHTML = ''}", 100);

Но первый вариант, через функцию, читабельнее и правильнее.

Comments

[DromHour]

Я перенос строки тут сделал, чтобы легче было разобрать, в notepad++ такие недочёты сразу замечаешь.
Спасибо! Сейчас проверю!

[Михаил Серенков]

DromHour: перенос легче разобрать, но тогда нужно оборачивать в кавычки, но лучше всего задавать код через функцию, и читабельно и валидно.