IF Statement multiple conditions, same statement

Question

Hey all, looking to reduce the code on my c# if statements as there are several repeating factors and was wondering if a trimmer solution is possible.

I currently have 2 if statements that need to carry out an identical statement, however the only variable is an extra condition on an if statement when a checkbox is not checked. Im just wondering if there is a way to make it one statement or make the condition string variable, heres the compressed version of the 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"
    }
  }

Its like I need to run an if statement within the conditions of an if statement if that makes sense, like this psuedo form:

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

BTW - the same answer works in C, C++ and Java. – Stephen C Aug 27 '09 at 14:42 — Stephen C, Aug 27 '09 at 14:42

score 47 · Accepted Answer · edited Nov 02 '17 at 16:23

47

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

Should do the trick.

edited Nov 02 '17 at 16:23

CodeCaster

139,522
20
204
252

answered Aug 27 '09 at 14:39

Daniel Elliott

22,229
10
62
82

6

usually when my conditions in an if statements are more than three, I extract them to a private method just to make it clear what's the condition is – Prashant Cholachagudda Dec 13 '13 at 10:46

score 12 · Answer 2 · edited Aug 27 '09 at 14:57

12

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

edited Aug 27 '09 at 14:57

David Basarab

70,191
42
128
155

answered Aug 27 '09 at 14:39

mikefrey

4,525
3
28
40

score 12 · Answer 3 · answered Aug 27 '09 at 15:01

I always try to factor out complex boolean expressions into meaningful variables (you could probably think of better names based on what these columns are used for):

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

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

score 6 · Answer 4 · answered Mar 29 '17 at 12:58

Pretty old question but check this for a more clustered way of checking conditions:

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

usage:

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

}

score 3 · Answer 5 · answered Aug 27 '09 at 14:41

3

Isn't this the same:

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

answered Aug 27 '09 at 14:41

JBrooks

9,618
2
26
32

score 3 · Answer 6 · answered Aug 27 '09 at 14:42

3

You could also do this if you think it's more clear:

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

answered Aug 27 '09 at 14:42

Meta-Knight

17,202
45
57

score 1 · Answer 7 · answered Aug 27 '09 at 14:46

I think agileguy has the correct answer, but I would like to add that for more difficult situations there are a couple of strategies I take to solve the problem. The first is to use a truth table. If you Google "truth table" you will run across some examples related directly to programming and computer science.

Another strategy I take is to use an anonymous function to encapsulate common logic between various conditions. Create it right before the if block, then use it where needed. This seems to create code that is more readable and maintainable.

score 1 · Answer 8 · answered Nov 14 '19 at 03:57

How about maybe something like this?

    var condCheck1 = new string[]{"a","b","c"};
    var condCheck2 = new string[]{"a","b","c","A2"}

    if(!condCheck1.Contains(columnName) && !checkbox.checked)
        //statement 1
    else if (!condCheck2.Contains(columnName))
        //statment 2

score 0 · Answer 9 · answered Aug 27 '09 at 14:40

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

is one way to simplify a little.

score 0 · Answer 10 · edited Feb 21 '19 at 20:06

0

var test = new char[] {a, b, c}.Contains(columnname));

if(test)
{
  "true statement"
}
else
{
   "false statement"
}

edited Feb 21 '19 at 20:06

frido

10,927
5
34
53

answered Feb 21 '19 at 17:24

Dan

1
1

IF Statement multiple conditions, same statement

10 Answers10

Linked