Почему return выводит undefined, а консоль результат?

Question

[Atheist21]

Задача, при помощи рекурсии посчитать сумму числового ряда
Подскажите в чем проблема? return undefined, а console.log результат

function getSum(begin, end, sum) {
        if (sum === undefined) {
          sum = 0;
        }
        if (begin > end) {
          return NaN;
        }

        sum += begin;
        getSum(begin + 1, end, sum);

        if (begin === end) {
          console.log(sum);
          return sum;
        }
      }

      console.log(getSum(1, 5));

Answer 1

[dollar]

Чтобы функция что-то вернула, где-то в ней должен быть достижимый оператор return с результатом вычислений.

Когда return условный, как у вас, то он может и не случиться. В этом случае функция выполнит всё, что должна, дойдёт до конца и ничего не вернёт. Точнее, по умолчанию она вернёт undefined.

Ещё важно, что если функцию вызвать просто так в качестве оператора, то её результат отбрасывается, какой бы он ни был:
getSum(begin + 1, end, sum);
В этом примере функция вызывается не как часть выражения, где её результат может быть использован, а сама по себе, как отдельная инструкция. Поэтому её результат нигде не сохраняется, и просто отбрасывается.

Comments

[Karpion]

Когда return условный, как у вас, то он может и не случиться.

Ну, при целых аргументах - условие для return д.б. достигнуто в любом случае.

[dollar]

Karpion,

функция не должна менять свои аргументы

Что за правило такое? Первый раз слышу.

А про достижимость return я писал вообще, а не конкретно про данный пример.

[Karpion]

dollar, Для начала - неочевидно, будет ли аргумент меняться на выходе из функции. Если будет - то это реально нарушение правил функционального программирования.

Кстати, как Вам моё решение (с учётом того, что оно недоделанное - только как пояснение принципа)?

[dollar]

Karpion, для js аргументы - это просто локальные переменные функции, которые инициализированы извне. Так что она может их хоть 1000 раз менять, но "снаружи" об этом никто не узнает.

Только если сама функция передаст эти аргументы куда-то. Но в чем тогда принципиальная разница по сравнению с локальными переменными? Так что не ясно, почему функция не может менять свои аргументы.

Answer 2

[Karpion]

У Вас странное представление о рекурсивной функции. Да и вообще о работе функций: есть правило "функция не должна менять свои аргументы".

Попробую написать решение в примитивном виде:

function getSum(begin, end) {
	return(begin + getSum(begin + 1, end)
}

Как видите, параметр sum тут не нужен. Ну и разумеется, надо дописать условие завершения рекурсии - их два, они у Вас уже есть.