[英]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.