Operand Types Are Incompatible ("char *" and "int")

Question

4.00/5 (1 vote)

See more:

I'm getting this warning on this code...what does it mean?
IntelliSense: operand types are incompatible ("char *" and "int")

RED = mysql_fetch_row(res_set);
		if (RED[i] == 1)
		{
			printf("PASS: %s\n",RED[i]);
		}
		else
		{
			printf("FAIL: %s\n",RED[i]);
		}

If I change it to this the warning goes away...

if (RED[i] == "1");

However the logic doesn't work. Is RED[i] returning a char* instead of an int?

Posted 14-May-11 4:41am

Member 7766180

Updated 14-May-11 5:35am

Kim Togo

v3

Add a Solution

2 solutions

Solution 1

You are storing a string inside the RED variable...

You should not compare an integer (1) with a char RED[i]...

You should use if(RED[i]=='1')...

I don't know which language you are using... in your question you posted C++, but in the pre tag there's vb...

At the end you must compare char to char... not char to int...

HTH!

Posted 14-May-11 4:43am

Joan M

Updated 14-May-11 4:48am

v4

Comments

Member 7766180 14-May-11 10:48am

Did that. Now instead of getting a "1" or a "0" I get something like 31191408.

OriginalGriff 14-May-11 10:51am

Joan, the "vb" bit will be automatic - the analyser defaults to that when you paste with the "auto detect" option on.

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 · 2011-05-14T04:48:00

Solution 2

I take it the complaint is here?

if (RED[i] == 1)

If so, then it makes sense: the type of RED[i] will be char* - try comparing it with "1" instead.

if (RED[i] == "1")

[edit]
Way too much C#! :laugh:

Try parsing RED[i] to an int and comparing that...

[/edit]

Posted 14-May-11 4:48am

OriginalGriff

Updated 14-May-11 4:49am

v2

Comments

Member 7766180 14-May-11 11:13am

The RED[i] is coming to me as a char*
If I do "1" then the warning goes away but logic isn't executed.
If I do '1' then the warning says char* and char incompatible
The [i] is coming to me as a my_ulonglong data type from an online mysql database if that helps.

OriginalGriff 14-May-11 11:22am

That's what I realized and why I edited my answer 1/2 hour ago! In C#, it would compare the two strings... :laugh:
If you use "1" then it compares two addresses, and they don't match.
If you use '1' the it compares an address to a char, and they can't match.
As I said, you need to parse the char* data to a int and compare that.
Use:
if (atoi(RED[i]) == 1)
Should do the trick!

Member 7766180 14-May-11 11:33am

OMG X 3!!!!!!!! Your the HERO! This is unbelievable! After all of the nonsense yesterday it boiled down to one word! atoi Thank you so very much!

OriginalGriff 14-May-11 12:19pm

You're welcome!

Albert Holguin 16-May-11 19:15pm

good job, my 5