[已解决]strcmp的一个问题

Hermione

这是我的错误代码：

#include <iostream>
#include <algorithm>
#include <stdio.h>
#include <string>
#include <cstring>
#define MAX 1024
using namespace std;

int main()
{
    char v1[MAX];
    char v2[MAX];
    char *a = v1;
    char *b = v2;
    cout << "字符串1：";
    fgets(v1, MAX, stdin);
    cout << "字符串2：";
    fgets(v2, MAX, stdin);
    char ch;
    int index = 0;
    while(*a++ != '\0' && *b++ != '\0')         //越过了第一个检查
    {
        if(*a != *b)        
            break;
        index++;
    }
    if(*a == '\0' && *b == '\0')
        cout << "完全一致！";
    else
        cout << "不一样" << ' ' << index;
}

这是标准答案：

#include <stdio.h>

#define MAX 1024

int main()
{
        char str1[MAX];
        char str2[MAX];

        char *target1 = str1;
        char *target2 = str2;

        int index = 1;

        printf("请输入第一个字符串：");
        fgets(str1, MAX, stdin);

        printf("请输入第二个字符串：");
        fgets(str2, MAX, stdin);

        while (*target1 != '\0' && *target2 != '\0')
        {
                if (*target1++ != *target2++)
                {
                       break;
                }
                index++;
        }

        if (*target1 == '\0' && *target2 == '\0')
        {
                printf("两个字符串完全一致！\n");
        }
        else
        {
                printf("两个字符串不完全相同，第 %d 个字符出现不同！\n", index);
        }

        return 0;
}

最本质的区别是，我的21行while里面*a++ *b++的位置不一样，我的很明显有一个问题，就是越过了第一个字符的检查。但是我的运行结果很奇怪，当我输入abc abc两个相同的三位字符时，显示一致，但是，当我输入abcd abcd两个四位字符时，居然显示不同，并且index = 5。
问题就是：为什么我会得出这样的运行结果？
谢谢大家的帮助。

最佳答案

月排行榜 / 总排行榜

BngThea

2017-12-12 09:25:12

因为你输入完一个字符串之后会用回车或者空格之类的结束，而fgets函数会将这个分隔符作为一个字符存到目标数组中，并在其之后加入'\0'，这就是为什么你的index总是比实际输入的大1

从你的逻辑上来看，应该很难得到“完全一致”的输出结果，因为这个判断
    if(*a == '\0' && *b == '\0')
        cout << "完全一致！";
基本不太可能得到，当程序走到这里的时候，*a和*b已经不是指向'\0',而是它后面的那个字符

最后提醒：
将自增自减操作放在条件判断中是十分危险的事情！

跳转到最佳答案楼层

BngThea

因为你输入完一个字符串之后会用回车或者空格之类的结束，而fgets函数会将这个分隔符作为一个字符存到目标数组中，并在其之后加入'\0'，这就是为什么你的index总是比实际输入的大1

从你的逻辑上来看，应该很难得到“完全一致”的输出结果，因为这个判断
    if(*a == '\0' && *b == '\0')
        cout << "完全一致！";
基本不太可能得到，当程序走到这里的时候，*a和*b已经不是指向'\0',而是它后面的那个字符

最后提醒：
将自增自减操作放在条件判断中是十分危险的事情！

ba21

楼上正解，所以把a++ b++放while循环体内最保险

#include <iostream>
#include <algorithm>
#include <stdio.h>
#include <string>
#include <cstring>
#define MAX 1024
using namespace std;

int main()
{
    char v1[MAX];
    char v2[MAX];
    char *a = v1;
    char *b = v2;
    cout << "字符串1：";
    fgets(v1, MAX, stdin);
    cout << "字符串2：";
    fgets(v2, MAX, stdin);
    char ch;
    int index = 0;
    while(*a != '\0' && *b != '\0')         //越过了第一个检查
    {
        if(*a != *b)        
            break;

                //调试输出
                cout << *a << "\n";
                cout << *b << "\n";


        index++;
                a++;
                b++;
    }
    if(*a == '\0' && *b == '\0')
        cout << "完全一致！";
    else
        cout << "不一样" << ' ' << index;

        return 0;
}

Hermione

诚如楼上所言，的确不太可能是完全一致，我用abc,abc得到了完全一致是个偶然，因为，第二次判断结束后，*a++这时还是在判断第三个位置，但是a与b都指向了下一个位置，都是'\n'，的确也行，这时候while里面还是判断第四个位置，是\n不是\0，所以继续执行下去，*a == *b,都是\0,这时候返回while,是时候结束了，然而，只有a++了，b由于短路却没有，所以*b肯定是\0，但是*a指向了一个根本没初始化的地方，很恰巧，偶然是\0，于是就对了。
为了验证我的猜想，我把第一个数组，也就是*a指向的数组，初始化为a，而第二个数组不初始化，结果就不一致了。
秉承生物中控制变量的原则，我不初始化第一个数组，初始化第二个数组，果然还是输出一致。
获得的教训是：楼上说的,while里面不要自增自减，虽然大部分时候初始化都没什么用，但还是初始化一下吧。
最后，谢谢楼上二位的帮助。