Когда нужно ставить return, а когда нет?

Question

[Елена]

Как стала больше решать задач по js, наткнулась на такой момент: не понимаю, когда нужен return, а когда нет. Может вопрос слишком простой, но уже раз 10 вроде думаю, что нужен return, ставлю, а там бац он не нужен и приходится переделывать. Может кто хорошо разбирается в этом моменте, объясните мне)

Answer 1

[Дмитрий]

const arrowFuncSquare = (param) => param * param 
// Не нужен return, стрелочная однострочная функция без фигурных скобок

const arrowFuncSquare2 = (param) => {  
  if (param) return param * 2 
  else return param
} // Нужен return, потому что фигурные скобки

const addTime = param => {
  param.createdAt = moment()
  param.updatedAt = moment()
} // Не нужен return, потому что функция не возвращает значения

const addTime = param => {
  if (param.timeless) return

  param.createdAt = moment()
  param.updatedAt = moment()
} // Нужен return чтобы при определенном условии прерывать выполнение кода функции

Comments

[Dima Polos]

Да, кстати, я не подумал, что это мот быть вопрос про стрелочные функции)
PS: советую для других уточнить, что если в стрелочной функции все тело можно уложить в выражение, результат которого надо вернуть, то фигурные скобки не ставятся, и return писать не нужно, потому что он сам по себе подразумевается. Можно в коммент дописать вот так:
// Не нужен return, потому что стрелочная однострочная функция без фигурных скобок делает return по умолчанию

[dmitry-toster]

if (param) return param * 2
else return param

можно, а даже лучше просто return param

[Дмитрий]

dmitry-toster, Почему? Визуально так имхо проще читается, а для компилятора-интерпретатора вряд ли представляет существенную разницу

[dmitry-toster]

Дмитрий, правило хорошего тона, во многих стайл-гайдах это указывается. Зачем лишний else когда итак понятно, что если первое условие не выполнится, то вернется значение по умолчанию. Да и код становится лаконичным.
Я тоже так раньше писал, но потом меня отучили через code review.
Если вы пишите один для себя, то решать конечно же вам)

[Дмитрий]

dmitry-toster, хм, странно, я работал как вэб разработчик в двух компаниях, в обоих был весьма строгий стайл гайд, автоматически контролируемый еслинтом, но данного пункта нигде не было. Но я вас понял. Считаю этот вопрос вообще мелким и не заслуживающим долгого общения.

[Дмитрий]

dmitry-toster, вообще считаю что некоторые пункты стайл гайдов это просто вкусовщина, навязывемая одними другим, причём зачастую запрещающая некоторые вещи, которые могли бы улучшить читаемость кода. В частности правило одного пробела между операторами и переменными препятствует выравниванию текста по вертикали, то есть проведению его к табличному виду. Правило того, что && и || должны быть в конце строки при многострочном условии, а не в начале также делает чтение условий менее читаемым. Когда условия в начале их сразу видно, не нужно бегать глазами. Естественно с годами привыкаешь ко всему, но подобные ограничения - бестолковая вкусовщина.

Answer 2

[2CHEVSKII]

Ставить тогда, когда нужно вернуть значение, или прервать выполнение функции.

Answer 3

[Сергей Соколов]

return не нужен:

1. когда функция заканчивается так:

   function ...  {
     // ...
     return;
   } // конец тела функции

   
   
2. когда стрелочная функция без тела: (arg) => return ...
   

В этих двух примерах return не нужен.

Comments

[Vadim Rublev]

В обоих примерах return не нужен, но почему-то есть... Хм..

Answer 4

[Dima Polos]

Посмотрите про операторы и функции.