Уродский setTimeout в node

[foo]

обнаружил, что на ноде setTimeout не хочет принимать параметром строку, как в нормальных реализациях. Скорей всего с setInterval та же фигня. Пофиксил вот так:

setT=function(exp, time){return setTimeout(function(){eval(exp)}, time)}

str="console.log('foo')"
setT(str, 100)
id=setT("console.log('bar')", 100)
clearTimeout(id)
//out: foo only

Будет работать?

[melky]

Пофиксил

Будет работать?

так пофиксил или не пофиксил?)

[Erolast]

setTimeout НЕ должен принимать строку, просто эту возможность много где сохраняют в целях совместимости.

[foo]

НЕ должен

Почему не должен? Это нодовцы изменили стандартное поведение. Это не первый косяк который я замечаю у них.

По сабжу: мой фикс во всех случаях будет работать?

[Erolast]

Почему не должен? Это нодовцы изменили стандартное поведение. Это не первый косяк который я замечаю у них.

А, проверил, и правда, должен.
Но это не косяк. setTimeout не является частью ECMAScript, он прописан в стандарте браузерного JS W3C. То есть, в ноде никаких стандартов не нарушено - у них свои таймеры, не те, что в браузерах.

По сабжу: мой фикс во всех случаях будет работать?

Ну если не брать в расчет то, что ты сразу же таймаут останавливаешь, то должен, да. Но это не фикс.

[Erolast]

Ну и, если уж и заниматься непотребством, то лучше так:

(function(){
  var setTimeoutOrigin = setTimeout;
  global.setTimeout = function() {
    if (typeof arguments[0] == "string")
      arguments[0] = new Function(arguments[0]);
	
    setTimeoutOrigin.apply(this, arguments);
  };
})();

setTimeout("console.log('dick')", 1000);

[kobezzza]

Что только люди не придумают, лишь бы не учить новый стандарт

setTimeout(() => console.log('dick'), 1000);

[Erolast]

Еще пол года до релиза(

[kobezzza]

Еще пол года до релиза(

Я юзаю транслятор уже больше года и всем доволен

[foo]

то лучше так:

А в чем преимущество?