I have problem with IF statement

Question

1.89/5 (5 votes)

See more:

When I try condition with int variable and string variable
it gives an error

Please help me

Ajith

What I have tried:

C#

int nSet =0 ;
int nCount=0 ;
string  cPrevtag ="";

/* nCount and cPrevtag change with my method calling

if (nCount == 1 && cPrevtag = 'O')
   {
     nSet = nSet + 1;

   }

Posted 25-May-17 7:52am

Member 12931315

Updated 25-May-17 12:51pm

PIEBALDconsult

v2

Add a Solution

Comments

[no name] 25-May-17 13:54pm

A char is not a string. = is not the same as ==.

Member 12931315 25-May-17 14:35pm

Thanks, but I tried result is same

Ajith

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2017-05-25T08:12:00

Solution 1

Two things wrong here:

if (nCount == 1 && cPrevtag = 'O')

1) cPrevtag is declared as a string, but your comparison is with a character - that isn;t allowed.
2) "=" is an assignment, not a comparison: and C# requires boolean results (unlike C and C++ which work with "non-zero-is-true"). The comparison operator is "=="

Try this:

if (nCount == 1 && cPrevtag == "O")

Posted 25-May-17 8:12am

OriginalGriff

Comments

Member 12931315 25-May-17 14:34pm

Hi Griff

I tried that it's still it's say operator '==' cannot be applied to operand string
is there any better way ?

Thanks in advance

Ajith

[no name] 25-May-17 14:47pm

Then you did something wrong. What we told you was correct and does work. But since you don't like to share little things like error messages, we are left to guessing what your problem is.

OriginalGriff 25-May-17 14:50pm

Read my code carefully: I changed the "=" to "==" *and* the 'O' to "O"

OriginalGriff 25-May-17 14:51pm

Sorry - that wasn't meant for you - it was intended for the OP... :blush:

OriginalGriff 25-May-17 14:51pm

Read my code carefully: I changed the "=" to "==" *and* the 'O' to "O"

Member 12931315 25-May-17 15:00pm

Holy cow I didn't notice that,problem solved
thank you very much.

OriginalGriff 25-May-17 15:34pm

You're welcome!

CPallini 25-May-17 15:52pm

5.

MattTelles · Accepted Answer · 2017-05-25T08:42:00

There are numerous problems with your code, but this will compile. No idea if it works, since you don't show the rest of your app:

int nSet =0 ;
int nCount=0 ;
string cPrevtag ="";

/* nCount and cPrevtag change with my method calling */

if (nCount == 1 && cPrevtag == "O")
{
    nSet = nSet + 1;

}

As others have noted, you can't compare a string to a character ( single quotes indicate a character). Also, you have to use == for comparison. Finally, your comment wasn't closed */

Matt