Почему не работает вот такой код?

Question

[Swartex]

Привет:)
решил попрактиковатся с задачками:

Напиши функцию создания генератора sequence(start, step). Она при вызове возвращает другую функцию-генератор, которая при каждом вызове дает число на 1 больше, и так до бесконечности. Начальное число, с которого начинать отсчет, и шаг, задается при создании генератора. Шаг можно не указывать, тогда он будет равен одному. Начальное значение по умолчанию равно 0. Генераторов можно создать сколько угодно.

Скажите почему не работает вот такой код:

function makeCounter() {
  var currentCount = 1;

  return function() {
    return currentCount++;
  };
}

var counter = makeCounter();
console.log(counter);
console.log(counter);
console.log(counter);

вроде все правильно сделал, а в консоле выводит:

Answer 1

[IceJOKER]

Потому что функцию нужно вызвать со скобками, counter()

Ваша основная функция возвращает вложенную функцию, ее вы присваиваете переменной, далее нужно эту переменную вызвать со скобками, иначе он вернет просто тело функции

Comments

[Swartex]

спасибо!)

Answer 2

[Александр Аксентьев]

Так ведь все правильно выводит.

Сделайте counter(), тогда отработает функция

Answer 3

[Vasyl Semilyak]

Вы забыли вызвать функцию counter
Это можно сделать так

function makeCounter() {
  var currentCount = 1;

  return function() {
    return currentCount++;
  };
}

var counter = makeCounter();
console.log(counter());
console.log(counter());
console.log(counter());

При вызове console.log(counter) Вы просто получаете тело функции а не её результат.

Answer 4

[Денис]

Выше вам уже коллеги ответили, но есть и еще один способ - сделать функцию самовызывающейся.

function makeCounter() {
  var currentCount = 1;

  return (function() {
    return currentCount++;
  }());
}

var counter = makeCounter();
console.log(counter); // 1