为什么此代码中存在分段错误?

时间:2014-03-20 19:37:21

标签: c++ segmentation-fault

我想比较两个数字串,计算每个出现的相同数字

#include <cstdlib>
#include <iostream>
using namespace std;
void function(string number1, string number2)
{
   int taille = number1.size();
   int tab[]={0,0,0,0,0,0,0,0,0,0};
   int chiffre=0;

   for (int i=0; i<taille;i++) {
       for (int j=0;j<taille;j++){
           if (number1[i]==number2[j])
           {
              cout <<  "chiffre " <<number1[i]<< endl;
              chiffre = number1[i];
              tab[chiffre]++; 
           }
        }
   }

   for (int i=0; i<10;i++) {
       if (tab[i]!=0)
       cout << "index " << i<< " value "<<  tab[i]<< endl;
   }

}

int main(int argc, char *argv[])
{

     string number1 = "5143436111";
     string number2 = "4501234567";
    function(number1,number2);
    cout << "Press the enter key to continue ...";
    cin.get();
    return EXIT_SUCCESS;
}

带有tab [chiffre] ++的行;好像让程序崩溃了。请解释我为什么

5 个答案:

答案 0 :(得分:2)

你需要:

chiffre = number1[i]-'0';

另外,你有这个:

   int taille = number1.size();

但是如果number1和number2的大小不同呢?然后你的内部循环(例如for (int j=0;j<taille;j++)如果number2小于number1则可以索引number2,或者如果number2大于number1,它将不会循环足够。

答案 1 :(得分:1)

在这一行:

chiffre = number1[i];

您正在将一个字符(例如'5')转换为整数。它应该是:

chiffre = number1[i] - '0';

我不知道你想要获得什么,但我想你只想比较number1中的第一个数字和number2中的第一个数字,并假设两个相同长度的数字?在这种情况下,你的循环应该成为:

if (number1.size() == number2.size()){
    for (int i=0; i<taille;i++) {
           if (number1[i]==number2[i])
           {
              cout <<  "chiffre " <<number1[i]<< endl;
              chiffre = number1[i] - '0';
              tab[chiffre]++; 
           }
       }
   }

答案 2 :(得分:1)

number1[i]包含一些字符,例如'5'。它的ASCII码是53.该阵列没有53个元素。

所以改变这个陈述

          chiffre = number1[i];

          chiffre = number1[i] - '0';

此外,您的算法无法正常工作。例如,如果number1等于"55"而number2等于45,那么您的算法会打印数字5包含在number2中两次。

答案 3 :(得分:1)

问题出在这一行:

chiffre = number1[i];

当您将号码分配给chiffre时,它不会将其指定为数字(从0到9),而是作为ASCII码(从&#39; 0&#39;到&#39; 9&#39;。)

您可以使用以下方法将其转换为数字:

chiffre = number1[i] - '0';

答案 4 :(得分:1)

使用tab [“5”]时,你使用的ascii值为5,即53,超出了10的界限 你需要转换ascii值或使用tab [chiffre-48] 48是ascii中的'0'