Hermione 发表于 2017-12-11 22:08:41

strcmp的一个问题

本帖最后由 Hermione 于 2017-12-11 22:11 编辑

这是我的错误代码:
#include <iostream>
#include <algorithm>
#include <stdio.h>
#include <string>
#include <cstring>
#define MAX 1024
using namespace std;

int main()
{
    char v1;
    char v2;
    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;
      char str2;

      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',而是它后面的那个字符

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

ba21 发表于 2017-12-12 10:37:59

楼上正解,所以把a++ b++放while循环体内最保险
#include <iostream>
#include <algorithm>
#include <stdio.h>
#include <string>
#include <cstring>
#define MAX 1024
using namespace std;

int main()
{
    char v1;
    char v2;
    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 发表于 2017-12-14 15:38:12

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