ヒストグラムを使用して配列内の最も一般的な文字を見つけよう

Question

これは私が思いついたものですが、変数 'h'の周りのランタイムチェック失敗＃2 - Stackが常に破損しています。

int mostCommonLetter(char s[]) { 
    int i=0, h[26],k=0, max=0, number=0; 
    while (k < 26){ 
     h[k] = '0'; 
     k++; 
    } 
    while(s[i] != '\0'){ 
     h[whichLetter(s[i])] = h[whichLetter(s[i])]+1; 
     i++; 
    } 
    h[26] = '\0'; 
    for(i=0;h[i]!='\0';i++){ 
     if(h[i] > max) 
      number=i; 
    } 
    return number; 
} 

Answer 1

h[26] = '\0'; - hには26個の要素が0..25でインデックスされています。あなたがhの長さを知っているので、0-終了する必要はありません。for (i=0; i < 26; ++i)

また、whichLetterは常に0..25の範囲の値を返しますか？それが何であれば何をしますか？スペースに遭遇しますか？

Answer 2

これは、配列の末尾を越えて書き込みます：

h[26] = '\0'; 

のループを作るには、長さではなく、最後の文字に依存します。

for(i=0;i<26;i++){ 
    if(h[i] > max) 
     number=i; 
}