Remplacement d'instructions if imbriquées

Cela est lié à un chapitre de beau code.
Et dans ce chapitre que j'ai lu sur le imbriquée ifs.

L'auteur parlait profondément imbriqués ifs en tant que créateur de bugs et moins lisible.
Et il parlait de remplacement imbriquée ifs avec case états et des tables de décision.

Quelqu'un peut-il illustrer comment faire pour supprimer imbriquée ifs avec case (select case) et des tables de décision ?

InformationsquelleAutor Biswanath | 2008-12-03

21

Bien, pas directement la réponse à votre question, puisque vous poser des questions sur l'interrupteur de cas, les déclarations, mais ici, c'est une question semblable.

Inverser l'instruction “if” afin de réduire la nidification

Cela parle de remplacement si imbriquées avec garde-états, que revenir plus tôt, au lieu d'progressivement la vérification de plus en plus de choses avant de s'installer sur une valeur de retour.

InformationsquelleAutor Lasse Vågsæther Karlsen
10

Un exemple, j'ai toujours essayer de faire est de remplacer fortement imbriquée si comme ça (en fait ce n'est pas trop mauvais, mais je les ai vus jusqu'à 8 ou 9 niveaux de profondeur dans la nature):
```
if (i == 1) {
    //action 1
} else {
    if (i == 2) {
        //action 2
    } else {
        if (i == 3) {
            //action 3
        } else {
            //action 4
        }
    }
}
```
avec ceci:
```
switch (i) {
    case 1:
        //action 1
        break;
    case 2:
        //action 2
        break;
    case 3:
        //action 3
        break;
    default:
        //action 4
        break;
}
```
J'ai aussi essayer de garder les actions de la plus petite possible (les appels de fonction sont les meilleurs pour cela) pour garder l'instruction switch compressé (si vous n'avez pas à aller de quatre pages à l'avance pour voir la fin de celui-ci).

Des tables de décision, je crois, sont simplement en définissant des indicateurs indiquant les mesures qui doivent être prises plus tard. Le "plus tard" est simple séquençage des actions basées sur ces drapeaux. J'ai peut-être tort (il ne sera pas la première ou la dernière fois :-).

Un exemple serait la sortie de la phase de mise au point peut être compliqué si depuis, ses actions sont très simples):
```
switch (i) {
    case 1:
        outmsg = "no paper";
        genmsg = true;
        mailmsg = true;
        phonemsg = false;
        break;
    case 2:
        outmsg = "no ink";
        genmsg = true;
        mailmsg = true;
        phonemsg = false;
        break;
    default:
        outmsg = "unknown problem";
        genmsg = true;
        mailmsg = true;
        phonemsg = true;
        break;
}

if (genmsg)
    //Send message to screen.
if (mailmsg)
    //Send message to operators email address.
if (phonemsg)
    //Hassle operators mobile phone.
```
- Parce que "sinon si" c'est le mal, et il doit être écrit "else { si ... }"? Mais vous avez le droit d'utiliser un interrupteur pour la simple ifs comme ça.
InformationsquelleAutor paxdiablo

Comment sur enchaîné fi?

Remplacer

if (condition1)
{
    do1
}   
else
{
    if (condition2)
    {
        do2
    }
    else (condition3)
    {
        do3;

    }
}

avec

if (condition1) {
   do1;
} else if (condition2) {
   do2;
} else if (condition3) {
   do3;
}

C'est un peu comme instruction switch pour des conditions complexes.

InformationsquelleAutor Arkadiy

6

Faire de l'état dans les booléens, puis d'écrire expression booléenne pour chaque cas.

Si le code était:
```
if (condition1)
{
    do1
}   
else
{
    if (condition2)
    {
        do2
    }
    else (condition3)
    {
        do3;

    }
}
```
On peut l'écrire comme:
```
bool cond1=condition1;
bool cond2=condition2;
bool cond3=condition3;

if (cond1) {do1;}
if (!cond1 and cond2) {do2;}
if (!cond1 and cond3) {do2;}
```
- C'est vraiment laid quand il y a beaucoup de conditions. De Plus, la lisibilité est perdu, car les conditions sont cachés derrière les valeurs booléennes.
- J'ai fait le chemin inverse. Je suis d'accord avec Jeff Atwood l'approche d'éviter des commentaires, et pour ce faire, vous donner que la variable booléenne un nom significatif, et qui sera le améliorer lisibilité... cela dit, cette approche présentée ici n'est pas une panacée, mais. Que faire si vous ne voulez pas inutilement calculer les conditions?
- D'accord avec @Mazyod le revers de La médaille ci-dessus a été les conditions deviennent inutilement calculée. En Scala, un paresseux val aide là.
InformationsquelleAutor khivi
3

Pour des tables de décision, voir ma réponse à cette question, ou, mieux encore, lire le chapitre 18 dans Code Complet 2.

InformationsquelleAutor Yuval F

Vous pouvez juste briser une fois une partie de l'échec de la validation par exemple.

function validate(){
  if(b=="" || b==null){
      alert("Please enter your city");
      return false;
  }

  if(a=="" || a==null){
      alert("Please enter your address");
      return false;
  }
  return true;
}

Pas sûr que j'aime les multiples retours ici. Goto erreur de sortie à la place?

InformationsquelleAutor Alexis Paques

Des tables de décision où vous magasin de la logique conditionnelle dans une structure de données plutôt que dans le code lui-même.

Ainsi, au lieu de cela (à l'aide de @Pax exemple):

if (i == 1) {
    //action 1
} else {
    if (i == 2) {
        //action 2
    } else {
        if (i == 3) {
            //action 3
        } else {
            //action 4
        }
    }
}

vous faire quelque chose comme ceci:

void action1()
{
    //action 1
}

void action2()
{
    //action 2
}

void action3()
{
    //action 3
}

void action4()
{
    //action 4
}

#define NUM_ACTIONS 4

//Create array of function pointers for each allowed value of i
void (*actions[NUM_ACTIONS])() = { NULL, action1, action2, action3 }

//And now in the body of a function somewhere...
if ((i < NUM_ACTIONS) && actions[i])
    actions[i]();
else
    action4();

Si les possibilités d' i ne sont pas à faible numérotés entiers puis vous pouvez créer une table de recherche au lieu d'accéder directement à la ième élément de la actions tableau.

Cette technique devient beaucoup plus utile que imbriquée ifs ou switch états lorsque vous avez une décision à prendre sur plusieurs dizaines de valeurs possibles.

InformationsquelleAutor Paul Stephenson

0

Si et instructions de commutation ne sont pas purement OO. Elles sont conditionnées une logique procédurale, mais faire un très bon travail! Si vous souhaitez supprimer ces états pour un plus OO approche, combiner l '"État" et "Descripteur" patterns.
- vous mettez "pas purement OO", comme si c'était une mauvaise chose!
- C'est comme donc 1986. Vous devez dire aux gens que leur code n'est pas fonctionnel assez de ces jours.
- haha, je vient juste de lire l'article récemment et a trouvé une réponse dirigée peut être approprié
InformationsquelleAutor Anthony Mastrean
0

Vous pouvez également envisager d'utiliser le Modèle visiteur.

InformationsquelleAutor Drejc
0

Si imbriquées sont l'équivalent de l'opérateur logique ET
```
if (condition1)
{
    if (function(2))
    {
        if (condition3)
        {
            //do something
        }
    }
}
```
Code équivalent:
```
if (condition1 && function(2) && condition3)
{
    //do something
}
```
Dans les deux cas, lorsqu'une expression renvoie la valeur faux, l'expression ultérieure ne sera pas évaluée. Par exemple, si la condition1 est faux, la fonction() ne sera pas appelé, et condition3 ne sera pas évalué.
- Il n'est pas equivalend 🙂 parce que function(2) ne fonctionnera pas si condition1 est faux, dans le deuxième exemple tous les 3 fonctions de s'exécuter, ce qui peut affecter les performances.
- Non, @PHPLover - PHP utilise évaluation différée (parfois appelé évaluation de court-circuit), donc si la première condition dans un ET logique est fausse, il ne cherche pas à évaluer l'une des autres conditions. De même, si vous faites un OU et la première condition était vrai, il ne serait pas évaluer le second.
- ils ne sont pas équivalentes encore, je viens de le tester, si le deuxième argument est une erreur, une erreur sera affiché ce qui signifie que si l'instruction s'exécute toujours tous les arguments.
- Je ne peux pas imaginer ce genre de code que vous avez testé et comment vous avez effectué le test. L'évaluation différée est un vieux temps, bien connu caractéristique commune à tous les principaux langages de programmation, que les programmeurs doivent prendre en compte lors de la rédaction de leur code. Vous pouvez essayer le code PHP en ligne à sandbox.onlinephpfunctions.com/code/... SI Imbriquées et logique ET le fonctionnement se comportent de la même. Pouvez-vous fournir votre extrait de code pour prouver le contraire?
InformationsquelleAutor Demis Palma ツ

-1

Un autre exemple certains langages permettent est-ce

           switch true{
            case i==0
              //action
            break

            case j==2
             //action
            break

            case i>j
             //action
            break
           }

InformationsquelleAutor Jim C

Vous devez vous connecter pour publier un commentaire.