Поясните код
 

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

Создайте декоратор 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. Как-то закольцованно получается.
Что-то я не могу до конца код догнать.

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

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

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

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

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

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

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

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