comparing two array using strcmp result false however both array contain same word

Question

1.00/5 (2 votes)

See more:

/* here nog and arrr on displaying shows the word ras but on compare return false result why???*/

Void ldata(unsigned char[])
{
// code for printing data on lcd
}

Char nog[]="ras";
Void rad (char arrr[])
{
For(g=0;g<strlen(arrr);g++)
{
Ldata(arrr[g]);
}
  For(g=0;g<strlen(nog);g++)
{
Ldata(nog[g]);
}
If(strcmp(arrr,nog)==0)
{
Led1=1;
}
}

Posted 21-Aug-15 8:01am

rahul_goyal

Add a Solution

Comments

jeron1 21-Aug-15 14:30pm

Maybe add a null terminator in the arrr[] array after copying the characters. Not sure how this compiles, given the case issues (Char ?).

1 solution

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 · Answer 1 · 2015-08-21T08:36:00

If they contain exactly the same string, then strcmp returns 0 - so if your Led1 isn't getting a 1 then they clearly don't have the same value. A LCD display may not show that - the nog array will have four characters 'r', 'a', 's', and a null terminator - but we don't know what arrr contains. It could be as simple as the declaration

C++

char aaar[] = "ras ";

which would cause strcmp to fail, but wouldn't show anything odd on your display. You need to look more closely ate the actual values - perhaps by displaying the hex values instead of the characters themselves?

And next time you post code here, make sure it is exactly what you are executing - I know that isn't, because that won't compile as c is case sensitive, and if is not the same as If, for is not the same as For, and so on.