logique de programmation SI Alors

Question

je traite ce type de problème tout le temps et je me demandais si vous aviez des contournements de travail élégant Slick

if(myObject != null && myObject.myProperty != myValue){ 
//do something 
}else{ 
//do something else 
} 

Si je lance cette logique, je reçois référence null lorsque myObject est nulle donc je finis par faire:

if(myObject != null){ 
    if(myObject.myProperty != myValue){ 
     //do something 
    }else{ 
     //do something else 
    } 
}else{ 
    //do something else 
} 

Je finis par avoir le code redondant sur mon « faire quelque chose d'autre » comment les gars ne vous gérer cela?

Answer 1

Le premier chèque de null signifie que la deuxième condition ne sera pas évalué si ce n'est pas rencontré - lors de l'utilisation logique AND (&&) - donc je suggère que votre exception de référence nulle vient d'ailleurs, pas cette dernière condition.

Answer 2

votre première déclaration est tout à fait valable.

if(myObject != null && myObject.MyProperty != myValue) 

Ce que cela va faire est-il évaluera myObject != null d'abord, et comme il y a un && à droite de celui-ci, si elle est fausse, il n'a même pas besoin d'évaluer l'autre article.

Si vous venez de copier et coller le code que vous avez collé ici, je pense que vous trouverez qu'il fonctionne avec succès

Answer 3

Il n'y a aucun moyen « élégant » pour faire face à ce - deux façons de partager le code de « faire quelque chose d'autre » sont vraiment de faire varier le degré de non-élégance:

• Vous pouvez mettre le « faire autre chose "code dans une fonction, ou
• Définir un indicateur avant d'entrer le if pour dire si un traitement supplémentaire est nécessaire, et agir sur le drapeau à la sortie de l'instruction if.

La première solution est explicite; la deuxième solution ressemble à ceci:

var processed = false; 
if(myObject != null) { 
    if(myObject.myProperty != myValue){ 
     //do something 
     processed = true; 
    } 
    // do more stuff knowing that myObject is not null 
} 
if (!processed) { 
    // do something else 
}