[英]Why is there a segmentation fault in this code ?
我想比较两个数字串,以计算同一数字的每次出现
#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] ++的行; 似乎使程序崩溃。 请解释一下为什么
你需要:
chiffre = number1[i]-'0';
另外,您有:
int taille = number1.size();
但是,如果number1和number2大小不同怎么办? 然后,如果number2小于number1,则您的内部循环(例如, for (int j=0;j<taille;j++)
索引可能从number2索引之外,或者如果number2大于number1,则它的循环将不够。
在这一行:
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]++;
}
}
}
number1[i]
包含一些字符,例如'5'
。 它的ASCII码是53。该数组没有53个元素。
所以改变这句话
chiffre = number1[i];
至
chiffre = number1[i] - '0';
而且您的算法无法正常工作。 例如,如果number1等于"55"
而number2等于45
则您的算法将两次打印出数字5包含在number2中。
问题在这一行:
chiffre = number1[i];
将数字分配给chiffre
,它不会将其分配为数字(从0到9),而是分配为ASCII码(从'0'到'9')。
您可以使用以下方法将其转换为数字:
chiffre = number1[i] - '0';
当您使用选项卡[“ 5”]时,使用的是5的ascii值,即53,超出了10的界限。您需要强制转换ascii值,或使用选项卡[chiffre-48] 48在ascii中为'0'
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.