[已解决]比较字符串函数问题，萌新求教

sususususu

#include <stdio.h>
int check (char s1[],char s2[])
{
        int i,l;
    for(i=0;i<=19;i++)
        if(s1[i]!=s2[i])                                    
            break;
    if(s1[i]>s2[i])
        l=1;
    else if(s1[i]==s2[i])
        l=0;
    else
        l=-1;
    printf("\n");
    return l;
}
int main()
{
    char str1[20];
    char str2[20];
    gets(str1);
    gets(str2);
    printf("%d",check(str1,str2));
}

为何我输入相同字符串，结果显示为1，为什么不是0？而且似乎得不到为0的结果。

最佳答案

月排行榜 / 总排行榜

傻眼貓咪

2023-2-11 17:44:01

1. #define _CRT_SECURE_NO_WARNINGS
   
2. #include <stdio.h>
   
3. 
   
4. int check(char s1[], char s2[])
   
5. {
   
6.     // int i, l; <------- 不需要这行
   
7.     for (int i = 0; i <= 19 && s1[i] && s2[i]; i++) { // <------- 这里稍微作修改
   
8.         if (s1[i] != s2[i])
   
9.             return s1[i] - s2[i]; // <------- 这里稍微作修改
   
10.         if (!s2[i] && s1[i]) return -1;
    
11.         else if (!s1[i] && s2[i]) return 1;
    
12. 
    
13. 
    
14.         /* ----------------------------------------------------------
    
15.         if (s1[i] > s2[i]) <------ 上面已经修改，这里不需要这行
    
16.             l = 1;
    
17.         else if (s1[i] == s2[i]) <------ 直接在最下面返回就可以了
    
18.             l = 0;
    
19.         else
    
20.             l = -1; <------ 上面已经修改，这里不需要这行
    
21.         ---------------------------------------------------------- */
    
22. 
    
23.     }
    
24.       
    
25.     // printf("\n"); <------- 不需要这行
    
26.     return 0; // <------- 这里稍微作修改
    
27. }
    
28. int main()
    
29. {
    
30.     char str1[20];
    
31.     char str2[20];
    
32.     gets(str1);
    
33.     gets(str2);
    
34.     printf("%d", check(str1, str2));
    
35. }

复制代码

跳转到最佳答案楼层

洋洋痒

你写的程序并没有判断到\0结束，所以字符串结束后还会继续往后判断，
数组未初始化，在两个数组中，你输入的字符串后边是啥你不知道。
解决方法：
char str1[20]=｛0｝;
char str2[20]==｛0｝;
给数组初始化，确保在你输入的字符串后边都是相同的。
但即使这样你的程序如果输入超过20个字符的字符串还是会出错

jhq999

1. for(i=0;i<=19;i++)
   
2.         if(s1[i]!=s2[i])  //本来就是不等于退出，除非字符串长度大于19 ，还有你这个bug太多，了解下字符串结束标志                                 
   
3.             break;

复制代码

傻眼貓咪

1. #define _CRT_SECURE_NO_WARNINGS
   
2. #include <stdio.h>
   
3. 
   
4. int check(char s1[], char s2[])
   
5. {
   
6.     // int i, l; <------- 不需要这行
   
7.     for (int i = 0; i <= 19 && s1[i] && s2[i]; i++) { // <------- 这里稍微作修改
   
8.         if (s1[i] != s2[i])
   
9.             return s1[i] - s2[i]; // <------- 这里稍微作修改
   
10.         if (!s2[i] && s1[i]) return -1;
    
11.         else if (!s1[i] && s2[i]) return 1;
    
12. 
    
13. 
    
14.         /* ----------------------------------------------------------
    
15.         if (s1[i] > s2[i]) <------ 上面已经修改，这里不需要这行
    
16.             l = 1;
    
17.         else if (s1[i] == s2[i]) <------ 直接在最下面返回就可以了
    
18.             l = 0;
    
19.         else
    
20.             l = -1; <------ 上面已经修改，这里不需要这行
    
21.         ---------------------------------------------------------- */
    
22. 
    
23.     }
    
24.       
    
25.     // printf("\n"); <------- 不需要这行
    
26.     return 0; // <------- 这里稍微作修改
    
27. }
    
28. int main()
    
29. {
    
30.     char str1[20];
    
31.     char str2[20];
    
32.     gets(str1);
    
33.     gets(str2);
    
34.     printf("%d", check(str1, str2));
    
35. }

复制代码

ExiaGN001

这是简化版strcmp?
建议先BDFS一下野指针相关知识，以明确在程序中野指针/未初始化数据的危险性。
这么改:

1. #include <stdio.h>
   
2. int strlen(char s)//功能同string.h里的同名函数
   
3. {
   
4.         int cnt=0;
   
5.         while(s[cnt]!='\0')cnt++;
   
6.         return cnt;
   
7. }
   
8. int min(int a,int b)//功能同math.h里同名函数
   
9. {
   
10.         return a<b?a:b;
    
11. }
    
12. 
    
13. int check (char s1[],char s2[])
    
14. {
    
15.         int i,l;
    
16.     for(i=0;i<min(strlen(s1),strlen(s2));i++)
    
17.         if(s1[i]!=s2[i])                                    
    
18.             break;
    
19.     if(s1[i]>s2[i])
    
20.         l=1;
    
21.     else if(s1[i]==s2[i])
    
22.         l=0;
    
23.     else
    
24.         l=-1;
    
25.     //printf("\n");
    
26.     return l;
    
27. }
    
28. int main()
    
29. {
    
30.     char str1[20];
    
31.     char str2[20];
    
32.     gets(str1);
    
33.     gets(str2);
    
34.     printf("%d",check(str1,str2));
    
35. }

复制代码