Операторы условного возврата C/C++ [дубликаты]

MISRA требует одного оператора возврата:

(MISRA, правило 14.7: требуется) «Функция должна иметь единую точку выхода в конце функции»

Лично я не думаю, что это хорошее правило. Сведите к минимуму количество операторов return, но используйте оператор return, если он повышает читабельность вашего кода.

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

Я предлагаю вам прочитать эту статью о дуффинге (написание кода сверху вниз):

Я бы написал так, потому что else избыточен:

   if (a < b) {
       return 0;  // bail, since condition is met
   }
   // lots of calculations in this block
   return 1;

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

Однако, например, в рекурсивной функции очень удобно помещать ваши «базовые случаи» в качестве операторов возврата в начале функции.

Например, рассмотрим факториал:

int fact(int x) {
  // base cases
  if (x == 0 || x == 1)
    return 1;

  // recursive call
  return x * fact(x-1);
}

Вы также можете написать это так:

int fact(int x) {
  int ret = 0;
  if (x == 0 || x == 1)
    ret = 1;
  else
    ret = x * fact(x-1);
  return ret;
}

Мне просто больше нравится первый способ, но это не значит, что любой способ лучше другого.

Это действительно сводится к тем стандартам, которым вы должны следовать, и личным предпочтениям.

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

foo() {
   int results = 0;  // good practice to initialize
   if (a < b) {
       results = 0;  // redundant, likely optimized out
   } else {
       // lots of calculations in this block
       results = 1;
   }
   return results;
}

Множественные операторы возврата вполне допустимы в C/C++. Но убедитесь, что по крайней мере один из операторов return всегда выполняется. Следующая функция неверна,

int foo(int x)
{
  if(x>0)
  {
    return 1;
  }
  else if(x==0)
  {
    return 0;
  }
  else
  {
    // No return in this block
  }
}

таких ситуаций следует избегать в C/C++.

Наличие нескольких операторов возврата в функции вполне приемлемо. Фактически, наличие нескольких операторов возврата в функции, подобной указанной выше, может улучшить производительность и удобочитаемость. Например, вам не нужен блок else в приведенной выше функции, потому что вы возвращаетесь из функции, если условие выполнено.

Просто убедитесь, что если ваша функция не имеет возвращаемого типа void, то у вас есть оператор return в конце, и что все операторы возврата возвращают этот тип. Например, если ваша функция объявлена ​​так:

int foo ();

Тогда все ваши операторы return должны возвращать целые числа, и вы должны возвращать целое число в конце, несмотря ни на что. Но если ваша функция имеет возвращаемый тип void, например:

void foo ();

Затем, если вы дойдете до конца функции и у вас не будет ключевого слова return, функция вернется автоматически.