If ... Else, Operator problem

[RP]

I have following code lines:

===============================
if (txtMethod.Text != "D") || (txtMethod.Text != "F"))
{
txtMethod.Clear();
txtMethod.Focus();
}
else
{
grpboxDirect.Visible = true;
txtCalculatePercent.Focus();
}
==============================

txtMethod is a TextBox that need to have values only D or F. The first
line is giving problem. Please correct.

[Chris Shepherd]

RP wrote:
[...]
You are missing an opening bracket:

if (txtMethod.Text != "D") || (txtMethod.Text != "F"))

should be:

if ((txtMethod.Text != "D") || (txtMethod.Text != "F"))

Chris.

[mpetrotta]

On Aug 29, 9:32 am, RP <[e-mail]:

I have following code lines:

===============================
if (txtMethod.Text != "D") || (txtMethod.Text != "F"))
{
txtMethod.Clear();
txtMethod.Focus();
}
else
{
grpboxDirect.Visible = true;
txtCalculatePercent.Focus();
}
==============================

txtMethod is a TextBox that need to have values only D or F. The first
line is giving problem. Please correct.

Well, you didn't specify what problem you're having, but the code
above won't compile - you're missing an open parenthesis. Also, your
logic doesn't match what you say you want. You likely want:

// "it's not a D, and it's not an F"
if ((txtMethod.Text != "D") && (txtMethod.Text != "F"))

You should consider refactoring to remove the negation; it tends to
make code harder to read.

if ((txtMethod.Text == "D") || (txtMethod.Text == "F"))
{
grpboxDirect.Visible = true;
txtCalculatePercent.Focus();
}
else
{
txtMethod.Clear();
txtMethod.Focus();
}

Michael

[RP]

I corrected the parenthesis problem. Still I have similar code blocks
where ELSE is not needed.
So, something like:

if ((txtMethod.Text == "D") || (txtMethod.Text == "F"))

will not work. I have written this on TextChange event. If the text
type is not D or not F then clear it, else do the ELSE part.

Well, you didn't specify what problem you're having, but the code
above won't compile - you're missing an open parenthesis. Also, your
logic doesn't match what you say you want. You likely want:

// "it's not a D, and it's not an F"
if ((txtMethod.Text != "D") && (txtMethod.Text != "F"))

You should consider refactoring to remove the negation; it tends to
make code harder to read.

if ((txtMethod.Text == "D") || (txtMethod.Text == "F"))
{
grpboxDirect.Visible = true;
txtCalculatePercent.Focus();}

else
{
txtMethod.Clear();
txtMethod.Focus();

}

Michael

[zacks]

On Aug 29, 1:32 pm, RP <[e-mail]:

I have following code lines:

===============================
if (txtMethod.Text != "D") || (txtMethod.Text != "F"))
{
txtMethod.Clear();
txtMethod.Focus();
}
else
{
grpboxDirect.Visible = true;
txtCalculatePercent.Focus();
}
==============================

txtMethod is a TextBox that need to have values only D or F. The first
line is giving problem. Please correct.

As another posted indicated, you should not combine NOTs with ORs.
You'll get in trouble every time.

[Jon Skeet [C# MVP]]

RP <rp*********@gmail.comwrote:

I corrected the parenthesis problem. Still I have similar code blocks
where ELSE is not needed.
So, something like:

if ((txtMethod.Text == "D") || (txtMethod.Text == "F"))

will not work. I have written this on TextChange event. If the text
type is not D or not F then clear it, else do the ELSE part.

I don't see why that wouldn't work.

Could you post a short but complete program which demonstrates the
problem?

See http://www.pobox.com/~skeet/csharp/complete.html for details of
what I mean by that.

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Jon Skeet [C# MVP]]

<za***@construction-imaging.comwrote:

As another posted indicated, you should not combine NOTs with ORs.
You'll get in trouble every time.

Not necessarily. In this particular case the two "nots" are exclusive
(the text will always either be "not D" or "not F") but that's not
always the case.

Counter-example:

if (!user.IsAuthenticated || !user.IsAuthorized)
{
// Display login page
}

that's effectively:

if (!(user.IsAuthenticated && user.IsAuthorized))

It makes perfect sense, and there's no "getting in trouble".

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[mpetrotta]

On Aug 29, 9:52 am, RP <[e-mail]:

I corrected the parenthesis problem. Still I have similar code blocks
where ELSE is not needed.
So, something like:

if ((txtMethod.Text == "D") || (txtMethod.Text == "F"))

will not work. I have written this on TextChange event. If the text
type is not D or not F then clear it, else do the ELSE part.

No. The condition you've stated ("if the text is not D or not F")
will not work. Think about it; say you've got:
if ((txtMethod.Text != "D") || (txtMethod.Text != "F"))
You want that to resolve to false when txtMethod is "D", for
instance. What you'll actually see is:
if (( "D != "D") || ("D" != "F"))
which resolves to:
if ( false || true)
which resolves to true.

See my earlier reply for what you likely want to do. In general,
though, it's worthwhile to "run" your algorithms in your mind with
test cases, to see if they do the right thing. Also, while English
often treats "and" and "or" as equivalent, boolean algebra definitely
does not.

Michael

[zacks]

On Aug 29, 2:11 pm, Jon Skeet [C# MVP] <[e-mail]:

<[e-mail]:

As another posted indicated, you should not combine NOTs with ORs.
You'll get in trouble every time.

Not necessarily. In this particular case the two "nots" are exclusive
(the text will always either be "not D" or "not F") but that's not
always the case.

Counter-example:

if (!user.IsAuthenticated || !user.IsAuthorized)
{
// Display login page

}

that's effectively:

if (!(user.IsAuthenticated && user.IsAuthorized))

It makes perfect sense, and there's no "getting in trouble".

Let me re-phrase. When you try to mix NOTs with ORs, you better know
what you are doing. :-)

[Jon Skeet [C# MVP]]

<za***@construction-imaging.comwrote:

It makes perfect sense, and there's no "getting in trouble".

Let me re-phrase. When you try to mix NOTs with ORs, you better know
what you are doing. :-)

You need to be careful - that's always the case, of course.

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too