Les accolades doivent-elles apparaître sur leur propre ligne? [fermé]

Les accolades doivent-elles être alignées ou non? Qu'est-ce que tu en penses?

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

ou devrait-il être

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

ou même

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

S'il vous plaît soyez constructif! Expliquez pourquoi, partagez vos expériences, sauvegardez-le avec des faits et des références.

Quand j'étais étudiant, j'avais l'habitude de placer des accolades sur la même ligne, de sorte qu'il y ait moins de lignes et que le code soit imprimé sur moins de pages. Regarder un caractère entre crochets imprimé comme la seule chose dans une ligne est ennuyeux. (environnement, gaspillage de papier)

Toutefois, lors du codage d'applications volumineuses, l'utilisation de lignes comportant uniquement des accolades est abordable, compte tenu de la sensation de "regroupement" qu'elle procure.

Quel que soit le style que vous choisissez, soyez cohérent, afin que votre cerveau ne traite pas de frais généraux pour traiter plusieurs styles dans des morceaux de code associés . Dans différents scénarios (comme ci-dessus), je dirais qu'il est correct d'utiliser différents styles, il est plus facile de "changer de contexte" à un niveau élevé.

Vous ne devriez jamais faire la 3ème méthode.

Le fait de lésiner sur des accolades pourrait vous épargner quelques frappes au clavier la première fois, mais le prochain codeur ajoute quelque chose à votre clause else sans s'apercevoir que le blocage manque; les accolades vont faire beaucoup souffrir.

Ecrivez votre code pour d'autres personnes.

Pendant longtemps , je l' ai soutenu qu'ils étaient de valeur égale, ou si très proche de l' égalité que le gain possible en faisant bien, le bon choix était loin, au- dessous du coût de discuter à ce sujet.

Être cohérent est important , cependant. Alors j'ai dit, jetons une pièce et commençons à écrire du code.

J'ai déjà vu des programmeurs résister à un tel changement. Passer à autre chose! J'ai changé plusieurs fois dans ma carrière. J'utilise même des styles différents dans mon C # que dans mon PowerShell.

Il y a quelques années, je travaillais au sein d'une équipe (environ 20 développeurs) qui a décidé de demander son apport, puis de prendre une décision, puis de l'appliquer à l'ensemble de la base de code. Nous aurions une semaine pour décider.

Beaucoup de gémissements et à couper le souffle. Beaucoup de "J'aime mon chemin, parce que c'est mieux" mais pas de substance.

Alors que nous étions en train d’étudier les points les plus fins de la question, quelqu'un a demandé comment traiter cette question dans l’ordre habituel:

void MyFunction(
    int parameterOne,
    int parameterTwo) {
    int localOne,
    int localTwo
}

Notez que l'emplacement de la liste de paramètres et le début du corps ne sont pas immédiatement évidents. Comparer aux:

void MyFunction(
    int parameterOne,
    int parameterTwo) 
{
    int localOne,
    int localTwo
}

Nous avons lu quelques explications sur la façon dont les gens du monde entier ont traité ce problème et nous avons trouvé le schéma consistant à ajouter une ligne vierge après l'accolade ouverte:

void MyFunction(
    int parameterOne,
    int parameterTwo) {

    int localOne,
    int localTwo
}

Si vous voulez faire une pause visuelle, vous pouvez également le faire avec une attelle. Ensuite, vos ruptures visuelles deviennent également cohérentes.

Edit : Deux alternatives à la solution 'extra blank line' lorsque K & R est utilisé:

1 / Indenter les arguments de la fonction différemment du corps de la fonction

2 / Placez le premier argument sur la même ligne que le nom de la fonction et alignez les arguments supplémentaires sur les nouvelles lignes avec ce premier argument

Exemples:

1/

void MyFunction(
        int parameterOne,
        int parameterTwo) {
    int localOne,
    int localTwo
}

2 /

void MyFunction(int parameterOne,
                int parameterTwo) {
    int localOne,
    int localTwo
}

/Modifier

Je persiste à dire que la cohérence est plus importante que d’autres considérations, mais si nous n’avons pas de précédent établi , nous devons nous attendre le lendemain.

Les règles cardinales sont les suivantes:

1. Suivez les normes de codage existantes du projet.
2. S'il n'y a pas de norme de codage et que vous modifiez une base de code existante appartenant à quelqu'un d'autre - soyez cohérent avec le style du code existant, peu importe à quel point vous l'aimez.
3. Si vous travaillez sur un projet écologique - discutez avec d'autres membres de l'équipe et obtenez un consensus sur une norme de codage formelle ou informelle.
4. Si vous travaillez sur un projet novateur en tant que développeur unique, faites-vous votre propre idée, puis soyez impitoyablement cohérent.

Même si vous n'avez aucune contrainte externe, il est préférable (IMO) de rechercher une norme de codage existante (largement utilisée) ou une directive de style, et d'essayer de la suivre. Si vous adoptez votre propre style, vous risquez de le regretter dans quelques années.

Enfin, un style implémenté / implémentable à l'aide de vérificateurs de style et de formateurs de code existants est préférable à un style qui doit être "appliqué" manuellement.

L'avantage de la première méthode est qu'elle est plus compacte verticalement. Vous pouvez ainsi insérer plus de code sur votre écran. C'est pourquoi je la préfère. Le seul argument que j'ai entendu en faveur de la deuxième méthode est qu'il est plus facile d'associer des crochets d'ouverture et de fermeture, mais la plupart des IDE ont un raccourci clavier pour cela, et c'est en fait une fausse déclaration, au lieu d'associer un crochet d'ouverture à une fermeture. crochet, vous pouvez associer un crochet de fermeture à l’expression "début de bloc" (si, sinon, pour, tout) sur le même niveau d’indentation, il est donc tout aussi facile de déterminer où se trouve le début du bloc.

Je ne vois aucune raison de gaspiller une ligne entière juste pour un crochet lorsque la construction précédente pour / while / if indique déjà visuellement le début d'un bloc.

Cela dit, j'estime que le crochet de fermeture devrait être dans sa propre ligne car nous avons besoin de quelque chose pour indiquer la fin d'un bloc et sa structure d'indentation de manière visible.

je préfère

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

plus de

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

parce que la ligne you.postAnswer();est beaucoup plus facile à lire et à trouver à première vue. Dans un deuxième temps, il se fond avec la ligne au-dessus de lui ( you.hasAnswer()), ce qui oblige mes yeux à se concentrer davantage pour le lire.

Je préfère la première méthode. Les bretelles ne valent absolument pas la ligne séparée.

La chose est que les accolades ne sont pas importants. Ce ne sont que des corbeilles syntaxiques , ce qui est absolument inutile pour comprendre à quoi sert le code, son objectif et la façon dont il est implémenté. Ils ne sont qu’un hommage à l’ancien langage de type C où le regroupement visuel des opérateurs était impossible en raison du faible espace disponible sur l’écran.

Il existe des langages (Python, Haskell, Ruby) qui conviennent sans accolades. Cela ne fait que confirmer que les accolades sont des ordures et ne devraient pas mériter une ligne pour eux chaque fois que possible:

if (you.hasAnswer()){
    you.postAnswer();
}else{
    you.doSomething();
}

Utilisez Python et évitez complètement l’argument.

La position des accolades devrait être

métadonnées

configurable dans l'IDE par le programmeur. De cette façon, ces accolades dans tout le code, quel que soit leur auteur, se ressemblent.

Ça dépend.

Si je code en Javascript ou jQuery, j'utilise le premier formulaire:

jQuery(function($) { 
    if ($ instanceOf jQuery) { 
        alert("$ is the jQuery object!"); 
    } 
}); 

Mais si je code en C #, j'utilise le second formulaire, car c'est la façon canonique de le faire en C #.

public int CalculateAge(DateTime birthDate, DateTime now) 
{ 
    int age = now.Year - birthDate.Year; 
    if (now.Month < birthDate.Month 
        || (now.Month == birthDate.Month && now.Day < birthDate.Day)) 
        age--; 
    return age; 
} 

Notez que votre exemple peut être écrit

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

en C #.

Je préfère le premier car il m'est plus difficile de voir l'erreur dans cet exemple.

if (value > maximum);
{
    dosomething();
}

que dans cet exemple

if (value > maximum); {
    dosomething();
}

Le ; {tout me semble plus faux qu'une ligne se terminant par ;donc je suis plus susceptible de le remarquer.

Je préfère une légère variante de 1)

if (you.hasAnswer()) {
    you.postAnswer();
} // note the break here
else {
    you.doSomething();
}

Pourquoi?

• Je pense que toujours mettre des accolades sur leur propre ligne diminue la lisibilité. Je ne peux insérer qu'une certaine quantité de code source sur mon écran. Le style de support 2) crée des algorithmes de soulèvement comportant de nombreuses boucles imbriquées et conditionnelles extrêmement longues.

• Cependant, je veux elsecommencer sur une nouvelle ligne car ifet elseappartiennent ensemble, visuellement. S'il y a un support devant le else, il est beaucoup plus difficile de repérer ce qui appartient à quoi.

• 3) se disqualifie. Nous savons tous ce qui peut arriver de mal si vous omettez les parenthèses et si vous les oubliez.