Сколько return может быть в методе?

Question

[Рустамка Воронцов]

Здравствуйте!
Товарищ заявил что в одном методе должен быть 1 return
тоесть

function ($a) {
$return = null;
if ($a!=1) {
//a lot of code here
$return = 'a!=1';
}
else {
$return =  'a=1';
}
return $return;
}

а я ему же предлагаю

function($a) {
if($a==1) return 'a=1';
//a lot of code here
return 'a!=1';
}

Он пишет типо ему препод на с 1 курса в голову вбивал что если больше 1 return то
портится структурное программирование

Разрулите ситуацию

Comments

[D']

Эм, а чем эти 2 варианта различаются? Они же делают одно и тоже.

PS.
Лучше switch и 1 return в конце :)

[wittyrider]

Это не одно и тоже, во втором варианте если a == 1 то остальные части функции не выполняется а в первом варианте такого ограничение нет.

Answer 1

[Stalker_RED]

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

UPD: В структурном программировании есть концепция "single entry and a single exit", которая кабы говорит нам, что "return early" - это плохо. Начинать читать можно отсюда https://en.wikipedia.org/wiki/Return_statement#Mul...

Answer 2

[Mikhail Osher]

Я бы посмеялся, если бы услышал такое. :)
Сколько надо для читабельности/бизнес-логики, столько и пишите.

Answer 3

[Костя Немченко]

Ну один ретурн как минимум заставит делать лишние переменные для хранения данных перед возвратом.
Так что лучше несколько, по крайней мере как по мне.

Answer 4

[Николай Конюхов]

Если нет необходимости в дальнейшем выполнении скрипта, то почему бы и не использовать несколько return. Предложенный Вашим знакомым вариант более приемлем, так как нет хардкода - поддержка такого кода, изменения займут меньше времени

Answer 5

[Pavel Shvedov]

по мне так без разницы. Зато с несколькими return-ами удобно обрывать дальнейшее выполнение метода при ошибках в аргументах:

function (err, data) {
        if(err) return; //-->
        ....
    }

Answer 6

[Александр]

Тут дело вкуса. Я за компактность, так что второй вариант на мой взгляд лучше

Answer 7

[memba]

Да есть целые секты программистов...
- Функция должна иметь только один выход...
- Функция не должна быть больше N строк

Это дело стиля и определяется командой программистов.

Answer 8

[mib]

Иногда хорошо в начале функции сделать всё возможное, чтоб выполнить ретурн:
Проверить, а вдруг ошибочные данные на входе, а вдруг вобще ничего не надо делать и так далее.
И только если выполнение пробралось сквозь все ловушки - начниать логику функции.

Тоже самое касается и циклов - как можно раньше выявить все возможные break, continue, return, а само тело цикла выполнять "в крайнем случае"

Тогда код становится на много проще, меньше вложенных условий, else, else if.

Answer 9

[Евгений Елчев]

Почитайте книгу совершенный код, там автор несколько глав разбирал преимущества различных подходов. А так я вам скажу, что вариант вашего друга изврат, читать сложнее, ресурсов жреть больше, ошибки отлавливать сложнее.