Conditions multiples dans une clause if

Question

Si j'ai une déclaration if qui doit répondre à ces exigences :

if(cave > 0 && training > 0 && mobility > 0 && sleep > 0)

Y a-t-il un moyen de dire qu'ils sont tous supérieurs à zéro ? Juste pour un code DRY plus efficace ?

Quelque chose comme :

if(cave, training, mobility, sleep > 0)

Answer 1

Vous pourriez obtenir la valeur la plus basse avec Math.min et vous n'avez alors besoin que d'une seule vérification de la borne inférieure.

if(Math.min(cave, training, mobility, sleep) > 0) {
    //do something
}

Answer 2

Vous pouvez utiliser un tableau avec .tous . C'est moins DRY, mais plus verbeux :

var isGreaterThanZero = function(val) {
    return val > 0;
};
if([cave, training, mobility, sleep].every(isGreaterThanZero)) {
    // Do Something
}

La raison pour laquelle j'aime cette méthode est qu'en utilisant un tableau, il devient évident que vous répétez la logique pour chaque variable. Nommer le callback de manière évidente aide les futurs lecteurs à comprendre exactement ce que cette vérification va réaliser. Enfin, cela permet d'envisager non seulement des nombres, mais aussi n'importe quelle vérification de n'importe quel type à l'avenir - toute complexité étant cachée dans la fonction de rappel. if La déclaration elle-même.

Answer 3

Comme certains l'ont dit, il n'y a rien de mal à avoir de multiples conditions simples dans une instruction if, mais j'envisagerais de modifier la mise en forme en :

if ( cave > 0
   && training > 0
   && mobility > 0
   && sleep > 0 )

Alternativement, je changerais l'utilisation de ces variables en tant qu'entiers, en variables bool, c'est-à-dire isCave , hasTraining ou similaire, puis définissez le bool approprié plus près de l'endroit où votre code définit les différentes propriétés ( Editar: Et éventuellement, retour anticipé si c'est faux, pour éviter d'autres calculs inutiles). Cela simplifierait votre instruction if par rapport à celle du bloc de code suivant, qui présente en outre une variante pouvant être utilisée si les conditions deviennent légèrement plus complexes ou si vous souhaitez faciliter la lecture de l'instruction if :

var isCave =  cave > 0; # What does cave > 0 mean?
var hasTraining = training > 0;
var isMobile = mobility > 0;
var isNotSleeping = sleep > 0; # What does sleep > 0 indicate? Unclear

if (isCave && hasTraining && isMobile && isNotSleeping ) {
   // Do your thing
}

En d'autres termes, les conditions multiples dans votre déclaration if ne sont pas votre plus grande odeur de code, je me concentrerais sur le fait de donner à vos variables de meilleurs noms indiquant clairement ce que la valeur indique. Cela améliorerait la lecture et la compréhension de votre code, bien plus qu'une syntaxe étrange pour éviter les conditions multiples.

Answer 4

Il n'y a rien de mal à avoir des conditions multiples et simples dans un système de gestion de l'information. if déclaration. Toutefois, si elle ne peut tenir sur une seule ligne (environ 80 caractères), vous avez quelques solutions.

En définitive, vous ne vérifiez pas si quatre variables sont supérieures à zéro. Vous vérifiez un ensemble de conditions. Le fait que ces conditions soient ( actuellement ) représentés par des entiers signés n'est non seulement pas pertinent, mais constitue un détail d'implémentation qui devrait être caché dans les fonctions.

1. Utilisez des drapeaux intermédiaires :

   var valid_location = false;
   if (cave > 0 && training > 0)
       valid_location = true;
   
   var valid_status = false;
   if (mobility > 0 && sleep > 0)
       valid_status = true;
   
   if (valid_location && valid_status)
       // ...

2. Utilisez une fonction :

   function can_do_this()
   {
       // split conditions into logical groups
   
       // checking location, because you need training if you're
       // in a cave
       if (cave <= 0 || training <= 0)
           return false;
   
       // checking status, because you have to be mobile and
       // sleepy
       if (mobility <= 0 || sleep <= 0)
           return false;
   
       return true;
   }
   
   if (can_do_this())
       // ...

3. Utilisez les fonctions pour les conditions individuelles que vous devez vérifier :

   function valid_location()
   {
       return (cave > 0 && training > 0);
   }
   
   function valid_status()
   {
       return (mobility > 0 && sleep > 0);
   }
   
   if (valid_location() && valid_status())
       // ...

Answer 5

En supposant des ints de 32 bits.

if ((cave | training | mobility | sleep) > 0)

Si l'un des nombres ci-dessus est négatif, le résultat du OU sera négatif et les conditions ne sont pas remplies.

Edit : c'est équivalent à if(cave >= 0 && training >= 0 && mobility >= 0 && sleep >= 0) et ne le fera pas lorsque certains des paramètres sont à 0. La solution consiste à inverser le bit de signe :

if ((~cave | ~training | ~mobility | ~sleep) <= 0)

Quelques méthodes alternatives qui fonctionnent même pour les valeurs à virgule flottante

if (cave*training*mobility*sleep > 0)
if (Math.sign(cave) * Math.sign(training) * Math.sign(mobility) * Math.sign(sleep) > 0)
if (Math.sign(cave) | Math.sign(training) | Math.sign(mobility) | Math.sign(sleep) > 0)