Почему рекурсивную функцию нужно присваивать переменной при работе с декоратором?

Question

[scottishFold]

Есть рекурсивная функция

function fac(num) {
	if (num == 1) {
		return num
	} else {
		return num * fac(num - 1)
	}
}

И есть декоратор которая выводит результат в консоль

function logResultDecorator(func) {
	return function() {
		let result = func.apply(this, arguments)
		console.log('Результат функции: ', result)
	}
}

Почему когда я передаю декоратору обычную функцию (Function Declaration), в результате получаю 1 и несколько раз NaN

function fac(num) {
	if (num == 1) {
		return num
	} else {
		return num * fac(num - 1)
	}
}

function logResultDecorator(func) {
	return function() {
		let result = func.apply(this, arguments)
		console.log('Результат функции: ', result)
	}
}

fac = logResultDecorator(fac)

fac(5)
/* Результат функции:  1
Результат функции:  NaN
Результат функции:  NaN
Результат функции:  NaN
Результат функции:  NaN */

Но когда передаю декоратору Function Expression, все работает корректно

let fuc = function fac(num) {
	if (num == 1) {
		return num
	} else {
		return num * fac(num - 1)
	}
}

function logResultDecorator(func) {
	return function() {
		let result = func.apply(this, arguments)
		console.log('Результат функции: ', result)
	}
}

fuc = logResultDecorator(fuc)

fuc(5)
// Результат функции:  120

Почему функцию нужно присваивать переменной? Я не понимаю

Answer 1

[Rsa97]

У вас декоратор не возвращает значение декорируемой функции.
В первом случае это приводит к тому, что при рекурсивном вызове fac() в строке return num * fac(num - 1) выполняется умножение на undefined.
Во втором случае вы рекурсивно вызываете незадекорированную функцию fac(), поскольку декорируете fuc().

Comments

[salat228]

Я не понял второй случай. Что значит "вы рекурсивно вызываете незадекорированную функцию fac(), поскольку декорируете fuc()"

Почему когда рекурсивную функцию записывают в перемнную то в результате получаем один вызов

Первый случай выводит результат 5 раз (Если эту функцию передать декоратору которая считает количество вызовов, она будет работать не корректно)

[Aetae]

salat228, уберём дуратские имена:

let variableVithFactorialFuction = function factorial(num) {
  if (num == 1) {
    return num
  } else {
    return num * factorial(num - 1)
  }
}

function logResultDecorator(func) {
  return function() {
    let result = func.apply(this, arguments)
    console.log('Результат функции: ', result)
  }
}

variableVithFactorialFuction = logResultDecorator(variableVithFactorialFuction)

variableVithFactorialFuction(5)

В данном случае логирутся первый вызов, а последующие рекуривные вызовы обращаются не к декорированой variableVithFactorialFuction, а к оригинальной factorial (см. else блок).

В первом же случае:

function factorial(num) {
  if (num == 1) {
    return num
  } else {
    return num * factorial(num - 1)
  }
}

function logResultDecorator(func) {
  return function() {
    let result = func.apply(this, arguments)
    console.log('Результат функции: ', result)
  }
}

factorial = logResultDecorator(factorial)

factorial(5)

перезаписывается ещё до вызова оригинальная функция factorial, потому рекурсивные вызовы внутри неё вызывают уже перезаписаную декорированную версию.

Ну и чтобы не было nan декоратор, очевидно, должен возвращать результат, а не только логировать:

function factorial(num) {
  if (num == 1) {
    return num
  } else {
    return num * factorial(num - 1)
  }
}

function logResultDecorator(func) {
  return function() {
    let result = func.apply(this, arguments);
    console.log('Результат функции: ', result);
    return result;
  }
}

factorial = logResultDecorator(factorial)

factorial(5)

[salat228]

Aetae, Чтоб тебе дали все телки мира ❤️❤️❤️