Déclaration IF plusieurs conditions, même déclaration

Salut à tous, cherchent à réduire le code sur mon c# si les déclarations qu'il y a plusieurs répéter facteurs et je me demandais si une tondeuse, une solution est possible.

J'ai actuellement 2 si les instructions qui doivent effectuer le même constat, toutefois, la seule variable est une condition supplémentaire sur une instruction if quand une case à cocher n'est pas cochée. Im me demandais si il ya un moyen de faire une déclaration ou de faire l'état variable de chaîne, voici la version compressée du code:

if (checkbox.checked)
  {
    if (columnname != a && columnname != b && columnname != c)
    {
      "statement 1"
    }
  }
else
  {
    if (columnname != a && columnname != b && columnname != c 
        && columnname != A2)
    {
      "statement 1"
    }
  }

Sa que j'ai besoin d'exécuter une instruction si dans les conditions d'une instruction if si cela fait sens, comme cette pseudo de la forme:

if (columnname != a 
    && columnname != b 
    && columnname != c 
    && if(checkbox.checked{columnname != A2})

source d'informationauteur markdigi

  1. 37
    if (columnname != a 
  && columnname != b 
  && columnname != c
  && (checkbox.checked || columnname != A2))
{
   "statement 1"
}

    Devrait faire l'affaire.

  2. 10
    if (columnname != a && columnname != b && columnname != c 
        && (columnname != A2 || checkbox.checked))
    {
      "statement 1"
    }
  3. 6

    J'ai toujours essayer de factoriser des expressions booléennes complexes d'importance variables (vous pouvez probablement penser à de meilleurs noms basés sur ces colonnes sont utilisés pour les):

    bool notColumnsABC = (columnname != a && columnname != b && columnname != c);
bool notColumnA2OrBoxIsChecked = ( columnname != A2 || checkbox.checked );

if (   notColumnsABC 
    && notColumnA2OrBoxIsChecked )
  {
      "statement 1"
  }
  4. 4

    Assez vieille question, mais regardez un peu ce pour plus de cluster moyen de vérifier les conditions:

    private bool IsColumn(string col, params string[] names) => names.Any(n => n == col);

    utilisation:

    private void CheckColumn()
{
     if(!IsColumn(ColName, "Column A", "Column B", "Column C"))
    {
     //not A B C column
    }

}
  5. 3

    N'est-ce pas la même:

    if ((checkbox.checked || columnname != A2) && 
        columnname != a && columnname != b && columnname != c)
  {
      "statement 1"
  }
  6. 2

    Vous pourriez aussi le faire si vous pensez que c'est plus clair:

    if (columnname != a 
  && columnname != b 
  && columnname != c
{
   if (checkbox.checked || columnname != A2)
   {
      "statement 1"
   }
}
  7. 1

    Je pense que agileguy a la bonne réponse, mais je tiens à ajouter que, pour les situations les plus difficiles il ya un couple de stratégies que j'avais à prendre pour résoudre le problème. La première consiste à utiliser un table de vérité. Si vous Google "table de vérité" vous allez courir à travers quelques exemples liés directement à la programmation et de l'informatique.

    Une autre stratégie que je prends est d'utiliser une fonction anonyme pour encapsuler la logique commune entre les différentes conditions. Créer juste avant le bloc si, pour ensuite l'utiliser en cas de besoin. Cela semble créer un code plus lisible et maintenable.

  8. 0
    if (checkbox.checked && columnname != a && columnname != b && columnname != c)
    {
      "statement 1"
    }
else if (columnname != a && columnname != b && columnname != c 
        && columnname != A2)
    {
      "statement 1"
    }

    est une façon de simplifier un peu.