strncmp 不能正常工作

Question

I'm having problems using strncmp . As I read, theorically strncmp should return 0 if the characters compared of two strings are equal; however, when I do the comparation, the code misbehaves and makes a false positive (Not being equal the characters, still makes the if clause). Here the code:

#include <stdio.h>
#include <string.h>

int main(){
    char *frase1="some string";
    char *frase2="another string";
    char *frase3="some other string";

//Comparar frases desde inicio
    if(strncmp(frase1, frase2, 200))printf("1<->2, 200 characters\n");
    if(strncmp(frase1, frase3, 20))printf("1<->3, 20 characters\n");
    if(strncmp(frase1, frase3, 4))printf("1<->3, 4 characteres\n");

    return 0;
}

If the strings are equal (At least the compared characters), they should print the message; if not, do nothing; so I still don't understand why the first Condition becomes true.

Any ideas?

Answer 1

strcmp & strncmp functions return 0 if strings are equal. You should do:

if (strncmp(frase1, frase3, 4) == 0) ...

ie:

char *str1 = "Example 1";
char *str2 = "Example 2";
char *str3 = "Some string";
char *str4 = "Example 1";

if (strncmp(str1, str2, 7) == 0) printf("YES\n");  // "Example" <-> "Example"
else printf("NO\n");

if (strncmp(str1, str3, 2) == 0) printf("YES\n");  // "Ex" <-> "So"
else printf("NO\n");

if (strcmp(str1, str4) == 0) printf("YES\n");      // "Example 1" <-> "Example 2"
else printf("NO\n");

yields YES , NO , YES .

Answer 2

If the strings are equal (At least the compared characters), they should print the message

No, if the strings are equal they should NOT print the message since strncmp returns 0 on equality. Since the last check returns 0 it doesn't get printed while the other two are non-zero ie unequal thus you see them getting printed.

strncmp 's documentation should give you more clarity. It returns

• 0 when lhs == rhs
• < 0 when lhs < rhs
• > 0 when lhs > rhs

Answer 3

The strncmp() function returns zero if the strings are equal, and a non-zero value otherwise. So if you compare two strings that are not similar you get a non-zero value, and non-zero values are TRUE as far as if statements are concerned. So what you need to do is:

   if(!strncmp(frase1, frase2, 200))printf("Mismo insulto no creo\n");

Answer 4

you should make the strncmp() == 0 as the expression to be evaluated

working code:

#include <stdio.h>
#include <string.h>

int main(){
char *frase1="some string";
char *frase2="another string";
char *frase3="some other string";

//Comparar frases desde inicio
if(strncmp(frase1, frase2, 200) == 0)
   printf("1<->2, 200 characters\n");
if(strncmp(frase1, frase3, 20) == 0)
   printf("1<->3, 20 characters\n");
if(strncmp(frase1, frase3, 4) == 0)
   printf("1<->3, 4 characteres\n");

return 0;
}

Answer 5

One, it's considered good practice to keep strings and identifiers in English when posting example code. Readability trumps fun.

Two, if blocks get executed if the condition is true . A value of 0 is considered false , while any value other than 0 is considered true .

Ie, your problem is not strncmp() , but a confusion about true and false.