Instruction Switch avec retours -- correction du code

Question

Disons que j'ai un code en C avec approximativement cette structure :

switch (something)
{
    case 0:
      return "blah";
      break;

    case 1:
    case 4:
      return "foo";
      break;

    case 2:
    case 3:
      return "bar";
      break;

    default:
      return "foobar";
      break;
}

Maintenant, évidemment, le break ne sont pas nécessaires pour que le code fonctionne correctement, mais cela ressemble à une mauvaise pratique si je ne les mets pas là.

Qu'en pensez-vous ? Est-il possible de les supprimer ? Ou les conserveriez-vous pour plus de "correction" ?

Answer 1

Retirer le break déclarations. Elles ne sont pas nécessaires et peut-être que certains compilateurs émettront des instructions "Code non atteignable" avertissements.

Answer 2

J'adopterais une approche totalement différente. Ne faites pas de RETOUR au milieu de la méthode/fonction. Au lieu de cela, mettez simplement la valeur de retour dans une variable locale et envoyez-la à la fin.

Personnellement, je trouve que le texte suivant est plus lisible :

String result = "";

switch (something) {
case 0:
  result = "blah";
  break;
case 1:
  result = "foo";
  break;
}

return result;

Answer 3

Personnellement, j'enlèverais les retours et garderais les ruptures. J'utiliserais l'instruction switch pour attribuer une valeur à une variable. Puis je renverrais cette variable après l'instruction switch.

Bien que ce point soit discutable, j'ai toujours pensé qu'une bonne conception et une encapsulation signifient une entrée et une sortie. Il est beaucoup plus facile de garantir la logique et vous ne manquez pas accidentellement du code de nettoyage basé sur la complexité cyclomatique de votre fonction.

Une exception : Le retour anticipé est acceptable si un mauvais paramètre est détecté au début d'une fonction, avant que des ressources ne soient acquises.

Answer 4

Enlevez-les. Il est idiomatique de revenir de case et sinon c'est un bruit de "code inaccessible".

Answer 5

Conservez les ruptures - vous risquez moins d'avoir des problèmes si vous modifiez le code ultérieurement si les ruptures sont déjà en place.

Cela dit, beaucoup (y compris moi) considèrent comme une mauvaise pratique le fait de revenir du milieu d'une fonction. Idéalement, une fonction devrait avoir un point d'entrée et un point de sortie.