Une ligne si les déclarations

J'ai récemment été impliqué dans une dispute avec un collègue de travail, impliquant une seule ligne si consolidés et je voulais voir ce stackoverflow pensée.

Pensez-vous que la déclaration doit être écrite comme suit:

if(condition)
{
     statement = new assignment;
}

OU

if(condition)
    statement=new assignment;

veuillez fournir une bonne raison pour justifier votre décision.

  • Dupliquer: stackoverflow.com/questions/97506/formatting-of-if-statements, stackoverflow.com/questions/691476/..., stackoverflow.com/questions/779332/..., et bien d'autres.
  • Par votre modification que vous avez totalement changé la question 😉
  • je n'ai pas la modification 😉
  • j'ai regardé à travers l'historique d'édition, et il y a eu deux modifications après avoir posé votre question: une re-tagger, et un code de mise en forme avec une tentative d'en faire une réelle question. le sens global n'a pas changé.
  • Bien sûr, il l'a fait. Le code "mise en forme" a changé la nature de la question. Le sujet se réfère à "une ligne si les déclarations". Le premier exemple prend maintenant trois lignes et le deuxième exemple, on prend deux.
  • Si vous regardez à la question d'origine, il a été écrit EXACTEMENT comme il apparaît maintenant. Cependant, les OP n'ont pas le code de mise en forme réglée correctement, de sorte qu'il est apparu comme une seule instruction. Je suis d'accord, l'avant et après les modifications de modifier le sens de la question, mais à partir de ce qui a été écrit à l'origine (y compris le format) je crois que la façon dont il apparaît maintenant, est ce que l'OP prévu.
  • Vous avez raison, mais j'ai été de répondre à geowa indiquant que la modification ne change pas la question, ce qu'elle fit. L'état de la question peut ne pas avoir été ce que l'OP a prévu, mais c'ÉTAIT la question.

InformationsquelleAutor |
  1. 20

    si vous devez vraiment utiliser une seule ligne si

     if(condition) statement=new assignment;

    sera mieux, puisque sa seule ligne, il doit contenir une seule opération.

    • +1, parce que, fondamentalement, si vous avez besoin d'accolades, vous n'avez pas l'écrire en une seule ligne.
    • Jamais, jamais, JAMAIS faire cela, s'il vous plaît. C'est de la pure cancer, si vous avez à déboguer. Lors du débogage, vous jamais savez exactement, si la condition est Vraie et que la déclaration a été réglé! Vous avez toujours besoin de dopplecheck. Pire encore, si il y a une pause après la if ... Il y a sûrement une place spéciale en Enfer pour les gens qui font cela.
    • C'est tout simplement horrible à déboguer, et c'est absolument pas facile à lire.
    • L'idée est d'empêcher les gens de l'ajout des déclarations à un si sans croisillons. Personnellement, je les utiliser que pour break et continue et ne tolère pas l'utilisation de si sans croisillons. Je pense que vous devez mettre à jour votre débogueur s'il ne peut pas faire deux déclarations en une seule ligne.
    InformationsquelleAutor
  2. 15

    J'ai toujours utiliser des accolades de délimitation afin de réduire le risque que quelqu'un (moi y compris) qui sera plus tard introduire un bug en modifiant le code autour de la si la déclaration sans prêter attention à la ligne(s) appartiennent dans le cadre de la si-condition.

    EDIT:

    Voici un exemple vivant si ce que je viens juste arrivé de tomber sur quelques vieux code:

    if (form.validateUpload (messages, this))
    return getErrorOutcome (ctx, messages);
    if (LOG.isInfoEnabled ())
        LOG.info ("CREATING UPLOAD");

    Notez comment les deux "si" instructions sont dans le bloc de code principal, mais en raison de la mauvaise mise en forme, à première vue, ils semblent être imbriquées. Assurez-vous que tout "bon" programmeur devrait rapidement voir ce qui se passe, mais pourquoi provoquer inutilement la confusion?

    InformationsquelleAutor
  3. 5

    J'ai toujours été un fan de croisillons. Si quelqu'un venait à modifier un oneline si l'instruction comme suit:

    if(condition) statement=new assignment;

    à

    if(condition)
statement = new assignment;
another statement;

    Vous n'obtiendrez pas le comportement attendu.

    En utilisant les accolades assez bien assure que si quelqu'un modifie une instruction if, ils vous veillez à placer le droit des états à la bonne place.

    • Si quelqu'un le fait, il est évident qu'ils ne comprennent pas la langue. N'est-ce pas tomber sous la compétence de base? (la sincérité prévu)
    • La compréhension de la langue et de la prise de typo n'ont rien à voir l'un avec l'autre. En ajoutant des accolades assure que que le bon comportement se produit indépendamment.
    • Peut-être. Pourrait être un programmeur junior maintien de code bien après, c'est le premier écrit. Pourrait aussi être un programmeur senior dans un mauvais jour. Pourquoi créer une opportunité pour quelqu'un de faire une erreur?
    • Afin de vous suggérer jonchent le code avec de nombreuses accolades inutiles.
    • Hein, ça dépend de ce que le comportement attendu est. Si vous avez de la première idée de la façon dont votre langue fonctionne, vous devriez vous attendre de la première instruction à exécuter conditionnellement et la deuxième sans condition.
    • Oui, parce qu'on doit s'attendre à ce que tout le monde qui touche notre code en ai aucune idée.
    • Que faire si quelqu'un les bâtons de la ligne suivante à l'extérieur de l'appareil orthodontique, mais s'attend à ce qu'il reste exécuter conditionnellement?
    • Ensuite, cette personne a introduit un bug.
    • Nous ne pouvons pas idiot-proof notre code. Ceux qui ne savent pas ce type de base de comportements linguistiques besoin d'apprendre la langue, et rapide. Il est évident qu'ils ne le savent pas, et ne devrait pas être de jouer avec votre code, et tous les bugs qu'ils introduisent un peut) être facilement résolu par quelqu'un de compétent, et b) éventuellement leur enseigner quelque chose qui, apparemment, ils ont besoin d'apprendre à la dure. J'utilise le one-liner style, et n'ont jamais été victime de ce bug. C'est un phénomène répandu et largement mis en garde contre "gotcha" que personne ne tombe pour elle plus.
    • Une programmation défensive, c'est comme la conduite défensive. Vous n'en avez pas besoin jusqu'à ce que vous êtes en carénage d'une falaise à cause de quelqu'un d'autre erreur.
    • vous ne pouvez pas faire votre code idiot-proof, mais vous peut rendre idiot résistant. Nous ne parlons pas de passer des heures et des heures de travail ici. La plupart des IDE de l'auto complète des accolades pour vous, afin d'ajouter des accolades après et si() instruction nécessite tout au plus un extra de frappe. Clairement accolades va être du groupe/de la préférence personnelle, cependant, la justification de l'utilisation de l'appareil est son: il vous empêche d'être relativement simple erreur.

    InformationsquelleAutor
  4. 4

    Cela dépend vraiment du style de codage de votre groupe. Le groupe doit avoir cohérente des normes de codage. Pour mon groupe, nous utilisons toujours:

    if (condition) {
  statement = new assignment;
}

    Nous faisons cela afin d'éviter les erreurs causées par oublier les accolades après l'instruction if, tels que:

    if (condition)
   statement1;
   statement2; 
//statement2 is not part of the if statement, but it looks like it because of wrong indentation

    Un autre groupe que j'ai travaillé avec jusqu'à tout récemment, ont toujours utilisé cette syntaxe pour une ligne si les déclarations:

    if (condition)
    statement1;

    Personnellement je n'aime pas ce que parce qu'elle est moins explicite, mais le plus important c'est de s'en tenir à un uniforme de codage standard pour votre groupe ou de votre projet, afin que le code que vous écrivez ressemble le code de votre co-travailleurs écrire, et est tout aussi facile à lire pour tout le monde dans le groupe.

    Les conventions de votre IDE ou de l'environnement peut fournir une bonne base pour vos normes de codage, et peut même être sur mesure pour votre groupe de style.

    InformationsquelleAutor
  5. 1
    if (condition)
{
    statement = new assignment;
}

    est ce que je voudrais écrire. À savoir, parce que j'aime bien rangé code qui permet de gagner du temps pour lire/modifier/comprendre.

    Dans très peu de cas, je ferais une exception normalement seulement quand je suis rapide et sale de codage quelque chose pour le débogage etc.

    Une ligne si l'instruction est toujours très facilement endommagé par la façon dont le point-virgule est placée.

    • Ce qui définit "en ordre de code"? Il est tout à fait subjective. Je pense que le fait de placer l'accolade après le si utilise une ligne entière inutilement.
    InformationsquelleAutor
  6. 0

    Je fais toujours une ligne if déclarations sans-crochets. La présence de parenthèses indique (syntaxiquement correcte) que "oh, je peux faire autre chose ici..." et je n'aime pas à la tentation. Tout ce qui implique plus qu'une seule déclaration doit être divisée en plusieurs lignes avec une bonne crochets.

    InformationsquelleAutor
  7. 0

    J'irais sans les crochets.

    La seule raison pour laquelle vous avez besoin de supports, c'est si vous avez eu plusieurs instructions à l'intérieur du bloc.

    Sonne comme un gaspillage de l'un argument.

    InformationsquelleAutor
  8. 0
    if(condition)
    statement=new assignment;

    ou 

    if(condition) statement=new assignment;
    InformationsquelleAutor
  9. 0

    En règle générale, j'ai horreur d'une ligne ifs sauf que dans ce Perl cas

    operation if condition;
    • Comme avec JuanZe, ce n'est pas une réponse. La question était de savoir comment pour faire des one-liners, pas si il faut.
    InformationsquelleAutor
  10. 0

    J'ai auto-configuration du format de tuer votre one-liner, ce qui la met sur deux lignes. En tant que tel, il a besoin d'accolades.

    • Je suis curieux de savoir pourquoi tant de gens sont en répondant avec pertinence des réponses. Encore une fois, la question est à propos de comment pour le formulaire one-liners, pas de s'il.
    • Il n'est pas hors de propos. Il n'y a pas une ligne dans mon format contrôlée référentiel source. Il ne peut tout simplement pas exister, et donc de s'inquiéter du problème n'est pas pertinent.
    • En outre, l'OP a demandé de préférence. J'ai donné le mien. C'est votre COMMENTAIRE qui est d'importance, de toute évidence, vous n'avez pas lu la question.
    InformationsquelleAutor
  11. 0

    Je l'utilise toujours en joignant entre parenthèses et je n'ai jamais code ligne par ligne ifs, ma démarche ressemble à ceci

    if(condition) { 
    statement = new assignment; 
}

    parce que je code en Java et c'est la convention pour la langue. Vérifier:

    http://java.sun.com/docs/codeconv/html/CodeConventions.doc6.html#449

    Remarque: si les déclarations de toujours utiliser des accolades
    {}. Éviter l'erreur suivante sujettes à
    forme:

    if (condition) //AVOID! THIS OMITS THE BRACES {}!
    statement;

    L'utilisation de crochets pour empêcher les insectes: certains d'autre pourrait ajouter, plus tard, de nouvelles phrases qui sont suposed être exécutée si la condition et d'oublier les parenthèses

    InformationsquelleAutor