Поясните код

[xfreewindx]

Здравствуйте!
Еще вопрос по задачке из учебника:

Создайте декоратор makeCaching(f), который берет функцию f и возвращает обертку, которая кеширует её результаты.

function f(arg) { 
  return Math.random()*arg; // может быть любой функцией
}

function makeCaching(f) { /* ваш код */ }

f = makeCaching(f);

var a, b;

a = f(1);
b = f(1);
alert( a == b ); // true (значение закешировано)

b = f(2);
alert( a == b ); // false, другой аргумент => другое значение

Скажите, случайно в этой строчке нет ошибки?
f = makeCaching(f);
У нас есть функция f, потом мы еще ей же присваиваем функцию makeCaching, переопределяем f. Как-то закольцованно получается.
Что-то я не могу до конца код догнать.

[rgl]

Формально все правильно, хотя понятнее было бы, если первую строчку заменить на
function f(arg) {
var f = function(arg) {

А что это за учебник?

[xfreewindx]

Тот что на этом сайте, раздел декораторы.
Спасибо.

[Ervin]

У нас есть функция f, потом мы еще ей же присваиваем функцию makeCaching, переопределяем f. Как-то закольцованно получается.

Согласен. Ошибки не будет, т.к. переменная переназначается только после выполнения всей части справа от знака назначения. Тем не менее я не стараюсь не использовать такие конструкции ибо для меня это логический не совсем верно, но это лишь мой стиль.

[rgl]

У нас есть функция f, потом мы еще ей же присваиваем функцию makeCaching

Нет, присваиваем не функцию makeCaching а то, что она возвращает (хотя тоже функцию).

[melky]

Согласен. Ошибки не будет, т.к. переменная переназначается только после выполнения всей части справа от знака назначения. Тем не менее я не стараюсь не использовать такие конструкции ибо для меня это логический не совсем верно, но это лишь мой стиль.

если думать о том, что всё в коде исполняется последовательно, то может понимать легче будет

[rgl]

Гы, приведенное решение не совсем хорошее, вот:

function f(arg) { 
  return arg.toUpperCase(); // может быть любой функцией
}

function makeCaching(f) {
  var cache = {};  

  return function(x) {
    if (!(x in cache)) {   
      cache[x] = f.call(this, x);
    }
    return cache[x];
  };

}

f = makeCaching(f);

alert( f("aaaa") ); // OK
alert( f("toString") ); // Что-то не то :-(

А вот если изменить проверку условия
if (!(x in cache)) {
if (!(cache.hasOwnProperty(x))) {
тогда все OK