Est-ce une mauvaise pratique d'utiliser un if sans accolades?

J'ai vu un code comme ceci:

if(statement)
    do this;
else
    do this;

Je n'aime pas ça, je pense que c'est plus propre et plus lisible

if(statement){
    do this;
}else{
    do this;
}

Est-ce simplement une question de préférence ou ce serait une façon d'être conseillé sur l'autre?

double possible de Pourquoi est-il considéré comme une mauvaise pratique pour omettre les accolades?
À mon avis, c'est mauvais, parce que vous commencer s'appuyant sur les espaces d'indentation qui n'est presque jamais pleinement compatible. Il déraille, le lecteur train de la pensée, quand ils ont à vous soucier de ces choses.

InformationsquelleAutor jerebear | 2010-01-23

184

Le problème avec la première version, c'est que si vous revenez en arrière et ajouter une deuxième déclaration de l'if ou else clauses, sans se souvenir d'ajouter des accolades, votre code sera briser dans l'inattendu et amusant les moyens.

Maintenabilité-sage, c'est toujours plus intelligent d'utiliser la deuxième forme.

EDIT: Nie de points dans les commentaires, mais il vaut la peine de lier ici, aussi, je pense. Ceci n'est pas une tour d'ivoire hypothétique des conneries: https://www.imperialviolet.org/2014/02/22/applebug.html
- Et vous devriez toujours code pour la maintenabilité. Après tout, je suis sûr que le compilateur ne se soucie pas de ce qui forme que vous utilisez. Vos collègues de travail, en revanche, peuvent être pist si vous introduisez un bug à cause d'un stupide accolade d'erreur.
- Ou vous pouvez utiliser une langue qui n'utilise pas de supports pour blocs de code...
- Valamo: Pascal utilise begin/end au lieu d'accolades et vous pouvez exécuter dans exactement les mêmes problèmes. Je suppose que tu veux réellement dire les langues dans lesquelles vous devez fermer explicitement si les déclarations avec quelque chose comme un end if.
- Non, je veux dire comme le python. Parce que je suis chauvin à cet égard.
- Je suis en désaccord, lorsque vous devez ajouter le code que vous ajoutez les crochets. Il est assez simple à lire et à mettre en œuvre.
- Luc: bien Sûr, c'est simple, mais il est également facile à oublier, et si vous l'avez oublié, c'est exaspérant et difficiles à déboguer. C'est d'économiser quelques caractères vraiment en valeur le potentiel des maux de tête? La plupart d'entre nous semblent d'accord pour dire que la réponse est "non".
- Encore une preuve que les erreurs peuvent (et vont) se produisent: imperialviolet.org/2014/02/22/applebug.html
- Affirmant que le bogue SSL est un argument en faveur de l'appareil est de mauvaise foi. Il n'est pas comme si le réalisateur avait l'intention d'écrire if (…) { goto L; goto L; } mais il a oublié les accolades. C'est une pure coïncidence que `si (...) { goto L; goto L; }` n'est pas un bug de sécurité, parce que c'est encore un bug (tout simplement pas l'un avec des conséquences sur la sécurité). Sur un autre exemple, les choses pourraient aller dans la direction opposée et le braceless code pourrait être accidentellement coffre-fort. Sur un troisième exemple, le braceless code sans bug d'abord et le développeur devra introduire une faute de frappe, tout en ajoutant des accolades.
- Pascal: Quelle honte si le code aurait été accidentellement coffre-fort, malgré une erreur de programmation! Si vous utilisez toujours les accolades, il ne peut pas être un accident lors de l'", ajoutant que" les.
- +1 pour le Bug Apple
- L'auteur de l'article parle du bug Apple conclut que "Peut-être le style de codage contribué à cela en permettant fi sans accolades, mais on peut avoir une mauvaise indentation avec des accolades trop, , donc ça ne semble pas terriblement convaincant pour moi". Je suis d'accord.
- L'ajout d'une deuxième déclaration si la clause entraînera une erreur de compilation.
- Upvote (+1) pour la "tour d'ivoire hypothétique de la connerie"
- Je sais que c'est un 3 ans thread, mais je suis tombé sur ce thread et je ne pouvais pas aider mais à lire votre commentaire sur l'ajout d'un corset est "facile" d'oublier. Il n'est pas facile de l'oublier, parce que si vous utilisez les bons outils pour écrire votre code, de tels outils ajouter ces crochets pour vous. Même si vous n'utilisez pas le bon outil, si vous arrive d'être en utilisant un langage qui utilise des accolades (j'aime python autant que vous le faites, tout de même), il devrait être dans votre mémoire musculaire lors de l'écriture d'un if déclaration, et si elle ne l'est pas, alors vous devriez être en train de travailler sur ce.
- Et dans la grande tradition de coups de pied d'un cheval mort... d'excellents outils peuvent certainement aider, mais pas tout le monde les utilise. Il ya beaucoup de façons que C vous permet de vous tirer dans le pied, et je considère que cela est l'un d'entre eux. Tout simplement parce que la langue vous PERMET de faire quelque chose (comme omettre les accolades de if/else) ne signifie pas que c'est une bonne idée. (Aussi, je n'aime pas Python.)
- if-else est un état unique, pas de plusieurs états.
- Je sais que c'est vieux mais... Dans le cas donné de si() .. d'autre ... alors si vous avez ajouté un deuxième déclaration après la ligne pour le, si vous obtenez une erreur de compilation n'est pas quelques subtile logique bug - si () ligne1 ligne2 d'autre line3 échoue parce qu'il n'y aurait pas de correspondance si, pour l'instruction else.
- Bien sûr, @Pierre, dans ce cas précis (lorsque vous avez un if et un else) alors que c'est vrai. Mais si jamais vous utilisez un if sans else, le compilateur ne sera pas vous aider. Et si votre ligne supplémentaire de code qui vient après la seule ligne d'autre du corps, le compilateur ne va pas l'attraper, soit.
- Je ne peux pas parler pour tous les outils, mais dans VS le bug Apple aurait été difficile à manquer. Non seulement inaccessible code génère un avertissement, mais c'est aussi grisé pour un repère visuel.
InformationsquelleAutor clee
89

Un problème avec le laissant sortir des blocs d'instruction est l'autre-de l'ambiguïté. Qui est C-inspiré langues ignorer la mise en retrait et n'ont donc pas de moyen de distinguer:
```
if(one)
    if(two)
        foo();
    else
        bar();
```
De:
```
if(one)
    if(two)
        foo();
    else
        bar();
```
- C'est beaucoup plus grave que celui qui est mentionné dans la réponse sommet (l'ajout d'une deuxième déclaration).
- en effet, cette réponse m'a pris à la lecture de ces réponses cyniquement d'être légèrement inquiète j'ai peut-être effectivement fait cette erreur.
- Si quelqu'un d'autre me demandais comme j'ai été la voie C en fait l'interprète, un test que j'ai fait avec GCC interprète ce code dans la première voie. tpcg.io/NIYeqx
- "l'ambiguïté" est un mauvais terme. Il n'y a pas d'ambiguïté, que ce soit dans la façon de l'analyseur va voir ceci: les else se lie avec gourmandise pour le plus proche, le plus profond if. Le problème se pose lorsque C ou similaire langues sont codées par des gens qui ne savent pas cela, n'y pensez pas, ou n'ont pas assez de café - donc ils écrivent du code qu'ils pensent va faire une chose, mais le langage de spécification dit l'analyseur a faire quelque chose d'autre, qui peut être très différente. Et oui, c'est un solide argument en faveur de toujours, y compris des accolades, même si la grammaire drapeaux comme théoriquement "inutiles".
InformationsquelleAutor doynax
32

Mon schéma général est que si elle tient sur une seule ligne, je vais le faire:
```
if(true) do_something();
```
Si il y a une clause else, ou si le code que je veux exécuter sur true est d'une longueur considérable, croisillons tout le chemin:
```
if(true) {
    do_something_and_pass_arguments_to_it(argument1, argument2, argument3);
}

if(false) {
    do_something();
} else {
    do_something_else();
}
```
En fin de compte, il s'agit d'une question subjective de style et de lisibilité. La programmation générale du monde, cependant, la grande majorité se divise en deux parties (pour les langues qui utilisent des accolades): soit les utiliser tout le temps, sans exception, ou de les utiliser tout le temps, sauf exception. Je suis partie de ce dernier groupe.
- Bien que, même s'il est facile d'écrire if(true){ do_something(); }, pourquoi prendre une chance sur un autre programmeur introduire un bug grave en bas de la route (recherche d'Apple "goto fail" total ssl code screwup).
- Aucune quantité de crochets sera libre le responsable de l'utilisation de son cerveau. Je soutiens l'idée de "pas de crochets si elle s'inscrit dans une ligne" parce que, eh bien, pour moi, un tel cas est juste une version de la ternaire si l'opérateur où l'on n'a pas besoin de faire quoi que ce soit dans le "après :" la part de ternaire. Et pourquoi serait-on introduire des parenthèses pour ternaire si?
- Je ne suis pas du tout d'accord que c'est subjectif, ni qu'il ne touche que le style et la lisibilité. Comme quelqu'un qui a perdu du temps à essayer de déboguer les problèmes qu'il a été causé par le manque de délimiteurs de bloc (et de ne pas avoir remarqué leur absence), parce que j'ai eu à utiliser un style de codage qui omet quand "inutiles" - et qui a lu sur de nombreux terrible des bugs très sans doute causée par un tel codage styles - je pense que c'est une très objectif, question pratique. Bien sûr, avec un style imposant des délimiteurs, on peut toujours oublier, mais sûrement, au moins, le muscle de la mémoire nous fait beaucoup moins probable.
InformationsquelleAutor Matchu
10

Je suis en utilisant le formateur de code de l'IDE que j'utilise. Qui peut-être différent, mais il peut être configuré dans les Préférences/Options.

J'aime celui-ci:
```
if (statement)
{
    //comment to denote in words the case
    do this;
    //keep this block simple, if more than 10-15 lines needed, I add a function for it
}
else
{
    do this;
}
```
- Ceci étant totalement subjective problème du style, personnellement, je n'aime pas la redondance de l'attache-seulement les lignes. Mais bon.
- Je soutiens ce style. La plupart des gens lisent le code de la gauche vers la droite et il a en quelque sorte fait à nos yeux ancrés au bord gauche de l'écran. Il permet de créer une séparation visuelle et l'extrait de code pour les blocs logiques de mesures.
- J'ai toujours préféré ce style. Beaucoup plus facile de trouver le correspondant parenthèse fermante. Donc, il faut beaucoup d'espace ? Utiliser une police plus petite.
- Je suis avec mloskot et timday - j'ai toujours trouvé que l'incorporation de la croisillons (essentiellement le "contrôle" des personnages) dans le reste du code qui rend le code plus difficile à lire. De les diviser sur des lignes séparées permettent de faire de la logique des blocs de plus distinctes.
- C'est le seul style que je considère lisible.
- Je trouve qu'il est toujours plus facile de "scan" à l'aide de code lorsque les accolades sont sur des lignes séparées. Cela vaut pour tout, les classes, les méthodes, les si - tout-états, et cetera. Jamais aimé avoir les premiers accolade sur la même ligne...
- Une espace est cher, surtout quand vous avez un code de pliage capable IDE.
- je sais que c'est tard, mais @Svish, je dois être prudent sur le retour des déclarations et javascript :), tout d'un coup accolades sont nécessaires pour être sur la même ligne
InformationsquelleAutor Pentium10
8

Avoir les accolades dès le premier moment devrait vous aider à vous empêcher de jamais avoir de débogage:
```
if (statement)
     do this;
else
     do this;
     do that;
```
- Qui semble être acceptée dans le raisonnement, mais (pour jouer l'avocat du diable ici) ne serait pas une syntaxe supplémentaire de mise en évidence de la règle également résoudre ce problème, tout en économisant une ligne?
- Donc, avoir une IDE qui corrige l'indentation quand vous frappez ; 🙂
InformationsquelleAutor Matt Bishop
7

La "règle" que j'ai suivi est: est-ce

Si l'instruction "if" est mise à l'essai dans le but de faire quelque chose (I. E. les fonctions d'appel, configurer des variables, etc.), utiliser des accolades.
```
if($test)
{
    doSomething();
}
```
C'est parce que je pense que vous avez besoin de faire savoir clairement quelles sont les fonctions sont appelés et où le flux du programme est en cours, sous quelles conditions. Avoir le programmeur de comprendre exactement ce que l'on appelle les fonctions et les variables sont définies dans cet état est important pour les aider à comprendre exactement ce que votre programme est en train de faire.

Si l'instruction "if" est mise à l'essai afin de cesser de faire quelque chose (I. E. le contrôle de flux à l'intérieur d'une boucle ou d'une fonction), utiliser une seule ligne.
```
if($test) continue;
if($test) break;
if($test) return;
```
Dans ce cas, ce qui est important pour le programmeur est en train de découvrir rapidement ce que les cas exceptionnels sont là où vous ne voulez pas le code à exécuter, et c'est tout coverred dans $test, pas dans l'exécution du bloc.

InformationsquelleAutor Marcus Harrison
4

Utiliser des accolades pour tous si les états, même les plus simples. Ou, réécriture d'une simple instruction si l'utilisation de l'opérateur ternaire:
```
if (someFlag) {
 someVar= 'someVal1';
} else {
 someVar= 'someVal2';
}
```
A l'air beaucoup plus sympa comme ça:
```
someVar= someFlag ? 'someVal1' : 'someVal2';
```
Mais seulement d'utiliser l'opérateur ternaire si vous êtes absolument sûr il n'y a rien d'autre qui doit aller dans le if/else blocs!

InformationsquelleAutor leepowers
4

Je préfère utiliser des accolades. En ajoutant des accolades rend plus facile à lire et à modifier.

Voici quelques liens pour l'utilisation de croisillons:
InformationsquelleAutor mangoDrunk
2

C'est une question de préférence. Personnellement, j'utilise les deux styles, si je suis raisonnablement sûr que je n'aurez pas besoin d'ajouter plus de déclarations, j'utilise le premier style, mais si c'est possible, j'utilise la deuxième. Puisque vous ne pouvez pas ajouter plus de déclarations à la première de style, j'ai entendu dire que certaines personnes vous déconseillons de l'utiliser. Cependant, la seconde méthode ne encourir une ligne de code supplémentaire et si vous (ou votre projet) utilise ce type de style de codage, la première méthode est très largement préféré pour de simples si les déclarations:
```
if(statement)
{
    do this;
}
else
{
    do this;
}
```
Cependant, je pense que la meilleure solution à ce problème est en Python. Avec le blanc, à base de bloc de la structure, vous n'avez pas deux différentes méthodes de création d'une instruction if: vous n'avez qu'un:
```
if statement:
    do this
else:
    do this
```
Tout en qui ne le "problème" que vous ne pouvez pas utiliser les accolades, vous faire gagner l'avantage qu'il n'est plus de lignes que le premier style et il a le pouvoir d'ajouter des instructions.
- J'en pense moi-même de la façon Python gère if-else est très laid, mais encore une fois, je ne suis pas un programmeur Python (encore)
- Pourquoi pensez-vous cela?
InformationsquelleAutor Grant Paul
1

J'ai toujours essayé de faire de mon code standard et de regarder aussi à peu près le même que possible. Cela rend plus facile pour les autres de le lire quand ils sont en charge de le mettre à jour. Si vous faites votre premier exemple et ajouter une ligne dans le milieu, il va échouer.

Ne fonctionne pas:

si(déclaration)
ce faire;
et ce;
d'autre
ce faire;

InformationsquelleAutor Joe
1

Personnellement, j'utilise le premier style que jeter une exception ou de retour à partir d'une méthode prématurément. Comme argument de Vérifier au début de la fonction, parce que dans ces cas, rarement dois-je avoir plus d'une chose à faire, et il n'est jamais une autre personne.

Exemple:
```
if (argument == null)
    throw new ArgumentNullException("argument");

if (argument < 0)
    return false;
```
Sinon j'utilise la deuxième de style.

InformationsquelleAutor Nate Heinrich
1

À partir de mon expérience, la seule (très) léger avantage de la première forme est la lisibilité du code, la deuxième forme ajoute "bruit".

Mais les Ide modernes et code autogeneration (ou d'auto-complétion) je recommande fortement d'utiliser la deuxième forme, vous n'aurez pas à passer plus de temps à taper des accolades et vous éviterez la plupart des bugs.

Il y a suffisamment de consommation d'énergie des bugs, des gens juste shoudn pas ouvrir les portes pour les grandes pertes de temps.

L'un de la règle la plus importante à retenir lors de l'écriture de code est la cohérence. Chaque ligne de code doit être écrit de la même façon, peu importe qui l'a écrit. Être rigoureux empêche les bugs de la "passe" 😉

C'est la même chose avec les nommant clairement & explicitement des variables, des méthodes, des fichiers ou correctement mise en retrait d'eux...

Quand mes élèves accepter ce fait, ils arrêter de se battre contre leur propre code source, et ils commencent à voir que les codes de vraiment intéressant, stimulant et de l'activité créatrice. Ils remettent en question leur esprit, et non pas leurs nerfs !

InformationsquelleAutor thomas.g
1

Ma préférence personnelle est à l'aide d'un mélange d'espaces et les parenthèses comme ceci:
```
if( statement ) {

    //let's do this

} else {

    //well that sucks

}
```
Je pense que cela a l'air propre et rend mon code très facile à lire et le plus important - de débogage.

InformationsquelleAutor Simon
0

Je suis d'accord avec la plupart des réponses dans le fait qu'il est préférable d'être explicite dans votre code et utiliser des accolades. Personnellement, je voudrais adopter un ensemble de normes de codage et de s'assurer que tout le monde sur l'équipe connaît et respecte. Où je travaille, nous utilisons les normes de codage publié par IDesign.net pour .Projets NET.

InformationsquelleAutor Kane
0

Je préfère mettre une accolade. Mais parfois, l'opérateur ternaire aide.

Au lieu de :
```
int x = 0;
if (condition) {
    x = 30;
} else {
    x = 10;
}
```
On devrait tout simplement faire : int x = condition ? 30 : 20;

Aussi imaginer un cas :
```
if (condition)
    x = 30;
else if (condition1)
    x = 10;
else if (condition2)
    x = 20;
```
Il serait beaucoup mieux si vous mettez de l'accolade dans.

InformationsquelleAutor fastcodejava

Vous devez vous connecter pour publier un commentaire.