Comment tiret long des instructions conditionnelles pour des "si", des déclarations?

Ma question concerne cette question précédente, mais les solutions proposées ne pas aborder le problème que j'ai décrit ci-dessous. Après une recherche sur google je n'ai trouvé aucun code de style de directives pour régler les problème de longue des instructions conditionnelles dans une instruction if comme ça.

if( isNull(value1) ||
    isToLong(value1) ||
    hasBadFormat(valule1)){
    doSomething();
}else{
    doSomethingElse();
}

OU:

if( isNull(value1) || isToLong(value1) || hasBadFormat(valule1) ){
    doSomething();
}else{
    doSomethingElse();
}

Le problème que j'ai avec ces deux styles est qu'il rend difficile pour les mes de l'oeil pour trouver le code dans le vrai, le bloc et de le séparer de la conditionnelles, ou c'est trop difficile pour l'œil afin de déterminer la bonne ligne suivante après une longue conditionnelle sur une seule ligne, surtout si la si la déclaration est déjà en retrait quelques onglets à l'intérieur d'une fonction ou si d'autres états.

Serait-il préférable de faire quelque chose comme ceci:

if(     isNull(value1) ||
        isToLong(value1) ||
        hasBadFormat(valule1)){
    doSomething();
}else{
    doSomethingElse();
}

ou serait-ce le style mieux indenter chaque nouvelle condition dans l'une de ces façons:

if( isNull(value1) ||
        isToLong(value1) ||
            hasBadFormat(valule1)){
    doSomething();
}else{
    doSomethingElse();
}

if( isNull(value1) 
        || isToLong(value1) 
            || hasBadFormat(valule1) ){
    doSomething();
}else{
    doSomethingElse();
}

Quelqu'un aurait-il un style de codage de directive (peut-être une société de style de codage de la politique), qui traite de cette question dans un autre ou de meilleur moyen que j'ai proposé? Ce qui est préférable et pouvez-vous trouver tous les inconvénients ou des pros et les solutions que j'ai mentionné?

InformationsquelleAutor |

24

Comment quelque chose comme cela?
```
bool isValid = isNull(value1) || isToLong(value1) || hasBadFormat(valule1);
if( isValid )
{
   doSomething();
}
else
{
   doSomethingElse();
}
```
Le conditionnel est déplacé sur une autre ligne, ce qui pourrait rendre plus facile à lire.
- C'est beaucoup plus lisible de l'OMI.
- +1 Juste ce que j'allais répondre.
- J'avais juste pensé à celui-ci... Ce qui semble être une bonne option aussi, surtout depuis que vous pouvez garder la longueur de la ligne vers le bas par des && et || isValid avec le prochain test sur chaque ligne.
- upvoted mais il ne semble pas assez longtemps, comment allez-vous le format de cette condition var show = (selectValue == ci-DESSUS && currentDataValue > premier) || (selectValue == ci-DESSOUS && currentDataValue < en premier) || (selectValue == ÉGAL && currentDataValue == premier) || (selectValue == ENTRE && currentDataValue > premier && currentDataValue < la seconde)
InformationsquelleAutor
13
```
if( isNull(value1) ||
    isToLong(value1) ||
    hasBadFormat(valule1))
{
    doSomething();
}
else
{
    doSomethingElse();
}
```
Maintenant vous pouvez voir le vrai-bloc facilement je pense.

Bien sûr, que je préfère:
```
if( isNull(value1)
    || isToLong(value1)
    || hasBadFormat(valule1))
{
    doSomething();
}
else
{
    doSomethingElse();
}
```
🙂
- Que faire si votre entreprise codage de la politique stipule que vous devez imiter pur si les blocs, de sorte que les accolades apparaissent sur la même ligne que le if? Lequel de mes deux solutions préférez-vous ensuite et il y a des problèmes avec eux, je n'ai pas reconnu?
- Je pense que mon deuxième exemple pourrait fonctionner assez bien quand le corset est un la même ligne.
InformationsquelleAutor
8

La solution la plus évidente consiste à déplacer les ouvrir renfort sur la ligne suivante, comme Dieu l'a voulu!

</flamebait>
- C'est la seule chose qui à bout de moi dans l'open-brace-sur-son-propre-ligne de camp.
- En fait je suis tombé sur cette réponse, car elle a été signalée comme étant de faible qualité. Mais je ne peux pas me résoudre à recommander la suppression. 🙂
InformationsquelleAutor

C'est la variante que je préfère:

if(
    isValid = isNull(value1) ||
    isToLong(value1) ||
    hasBadFormat(valule1)
) {
    doSomething();
}
else {
    doSomethingElse();
}

InformationsquelleAutor

2

Cela dépend vraiment de la préférence et de la convention de personnes qui travaillent avec vous, mais les deux premiers sont les deux formes les plus courantes que j'ai vu. J'ai tendance à préférer pour déplacer le conditionnel pour multi-lignes seulement si elle est tellement longue qu'elle exige de gauche à droite défilement dans mon ide.

C'est comment j'allais l'écrire:
```
if(isNull(value1) ||    
   isToLong(value1) ||
   hasBadFormat(valule1))
{    
    doSomething();
}
else
{    
    doSomethingElse();
}
```
À moins que la condition n'est pas assez long pour me forcer à faire défiler pour voir tout cela. Si non, je le ferais:
```
if(isNull(value1) || isToLong(value1) || hasBadFormat(valule1))
{    
    doSomething();
}
else
{    
    doSomethingElse();
}
```
Et dans ce cas, car il semble assez court, je ferais le dernier.

InformationsquelleAutor

J'ai tendance à mettre les opérateurs au début de la ligne afin qu'ils tous.

Donc, voici une suggestion:

if(isNull(value1)
   || isTooLong(value1)
   || hasBadFormat(valule1))
{
   doSomething();
} /* if */
else
{
   doSomethingElse();
} /* else */

Voici un autre:

if(0
   || isNull(value1)
   || isTooLong(value1)
   || hasBadFormat(valule1))
/* ...etc... */

(Pour &&, il serait si(1 && un && b), etc.)

Ou ceci:

if
(
   isNull(value1)
   || isTooLong(value1)
   || hasBadFormat(valule1)
)
/* ...etc... */

InformationsquelleAutor

-3

Je suis probablement le seul qui fait de cette façon. Il est appelé le Horstmann Style, mais je fais un peu différemment.

if (bool)        //comment
{   dothis;        
    andthis;       
} else if (bool) //comment
{   dothis;       
    andthis;        
} else           //comment
{   dothis;        
    andthis;     
}

InformationsquelleAutor

Vous devez vous connecter pour publier un commentaire.