Question

I have added one label in form that is not visible to user.Base on the text that label contain proceed further.

Here is my logic,but it fail.I wanted like this,if label contain “No match” or “Time out”,should not proceed.

If((!label.Text.Contain("No match")) || label.Text.Contain("Time out"))
{
// proceed further code
}
else
{
// code 
}

Here Label contain “No match”,then it move to else part that is right.But when label contain “Time out”,then it go inside the if loop.So I modified code like this

If((!label.Text.Contain("No match")) || (!label.Text.Contain("Time out")))
{
// proceed further code
}
else
{
// code 
}

still not working.If label contain “Time out”,still it go into if loop not else loop.Label contain only one text at a time either “No match” or “Time out” or any other text.

Answer 1

I suspect you want:

if(!(label.Text.Contains("No match") || label.Text.Contains("Time out")))
{
    // proceed further code
}
else
{
   // code 
}

Note the bracketing. The inner part is

label.Text.Contains("No match") || label.Text.Contains("Time out")

and then that’s inverted. I would probably pull that out into a separate variable:

bool timedOutOrNoMatch = label.Text.Contains("No match") || 
                         label.Text.Contains("Time out");
if (!timedOutOrNoMatch)
{
}
else 
{
}

Alternatively, invert the sense of it:

if (label.Text.Contains("No match") || label.Text.Contains("Time out"))
{
    // Do whatever was in your else block.
}
else
{
    // Do whatever was in your first block.
}

If your response to the “bad” labels is something that lets you return or throw an exception, this can also reduce the amount of nesting:

if (label.Text.Contains("No match") || label.Text.Contains("Time out"))
{
    output.Text = "Go away";
    return;
}
// Now handle the success case