Cの文字列の発生数をカウントする

Question

私はこの質問が何度も尋ねられていることを知っていますが、私は間違っていることを把握することはできません。私は、文字列を入力して文字列に文字が現れる回数をカウントするようにユーザーに依頼します。これは私のコードです：

実行するたびに、文字 'f'までの出現回数を出力します。その後、結果は正しくなく、一部の文字では乱数が出力されます。誰かが私にこの問題を解決する方法を教えてもらえますか、私が間違っていることを私に説明できますか？ありがとう。

enter a string 
abc 
a: 1 
b: 1 
c: 1 
d: 0 
e: 0 
f: 32767 
g: 1 
h: 0 
i: 1434561216 
j: 32767 
k: 206176841 
l: 1 
m: 1434561248 
n: 32767 
o: 1434561248 
p: 32767 
q: 1434561272 
r: 32767 
s: 171855872 
t: 1 
u: 0 
v: 1 
w: 206173472 
x: 1 
y: 206173376 
z: 1 

Answer 1

最初の5文字カウントのみを初期化します。

// assign alphabet number arrays to 0 
for(int i=0; i < 5; i++) 
{ 
    alphabetLowerNumbers[i] = 0; 
    alphabetUpperNumbers[i] = 0; 
} 

は、プログラムの主な欠点は、例えば97又は5用としてマジックナンバーを使用している

// assign alphabet number arrays to 0 
for(int i=0; i < 26; i++) 
{ 
    alphabetLowerNumbers[i] = 0; 
    alphabetUpperNumbers[i] = 0; 
} 

Answer 2

であるべきです。このループアレイalphabetLowerNumbersのみ5つの要素が初期化され

for(int i=0; i < 5; i++) 
{ 
    alphabetLowerNumbers[i] = 0; 
    alphabetUpperNumbers[i] = 0; 
} 

で

。配列の他のすべての要素には不確定な値があります。

（注：あなたの代わりにループを使用して、その宣言の配列を初期化することができたとえば

int alphabetLowerNumbers [26] = { 0 }; 

）

文字列userInputはその容量より少ない文字を含めることができます。また、終端ゼロの'\0'と改行文字'\n'を含めることができます。だからこのループ。

for(int i=0; i < 30; i++) 
{ 

    int ascii = userInput[i]; // ascii 

    alphabetLowerNumbers[ascii - 97]++; 

} 

は、プログラムの動作が不定になります。

は、少なくともループがために `

for (size_t i = 0; userInput[i] != '\0' && userInput[i] != '\n'; i++) 
{ 
    //... 
}