Почему возвращается только return x без alert(...)?

Question

[Константин]

Есть код:

function slow(x) {
  // здесь могут быть ресурсоёмкие вычисления
  alert(`Called with ${x}`);
  return x;
}

function cachingDecorator(func) {
  let cache = new Map();

  return function(x) {
    if (cache.has(x)) {    // если кеш содержит такой x,
      return cache.get(x); // читаем из него результат
    }

    let result = func(x); // иначе, вызываем функцию

    cache.set(x, result); // и кешируем (запоминаем) результат
    return result;
  };
}

slow = cachingDecorator(slow);

alert( slow(1) ); // slow(1) кешируем
alert( "Again: " + slow(1) ); // возвращаем из кеша

alert( slow(2) ); // slow(2) кешируем
alert( "Again: " + slow(2) ); // возвращаем из кеша

На этой строчке cache.set(x, result);в кеш он записывает result равный { alert(`Called with ${x}`); return x; }
Но почему alert( "Again: " + slow(1) ); возвращает с качестве значения slow(1) цифру 1(т.е. x), без alert(`Called with ${x}`);?
Ведь в кеш записывается весь resultравный телу функции slow(x), т.е. это

{
   alert(`Called with ${x}`);
  return x;
}

Помогите, пожалуйста понять.

Answer 1

[Алексей Уколов]

Ведь в кеш записывается весь result равный телу функции

С чего вы это взяли вообще? Что функция вернула, то и записывается в кеш. Вы же не саму её записали, а результат её вызова. Alert к этому результату не имеет никакого отношения и всегда возвращает undefined.

Comments

[Константин]

Как же не саму? В кеш записывается let result = func(x); , где func(x) = slow(x)

[Алексей Уколов]

Ну вот так же.
let result = slow(x); - это совершенно не то же самое, что let result = slow;.
Вы понимаете разницу между этими двумя вариантами? Вы пишете про второй, но он бессмысленен с точки зрения кеширования, потому что не исполняет slow вообще, только записывает её тело в кеш. Первый вариант исполняет функцию и кеширует возвращаемое значение, а не все сайд-эффекты, которые функция производит.

[Константин]

Алексей Уколов, не уверен... а как этому к моему примеру относится? Не совсем пойму...

[Алексей Уколов]

Что "это"?

[Константин]

Алексей Уколов, в смысле, что у меня же в кеш записывается slow(x), а не просто slow

[Алексей Уколов]

Именно так. И вот этот result не содержит никакого alert, а только return x;. Так программирование работает, я не знаю, честно говоря, что тут ещё можно добавить ¯\_(ツ)_/¯

[Константин]

Алексей Уколов, все, после этого я понял кеширует возвращаемое значение, а не все сайд-эффекты, которые функция производит.

[Алексей Уколов]

Ну и хорошо. Если бы кэшировались и сайд-эффекты (например, запрос на сервер), то кэширование не имело бы никакого смысла.