任意の文字列のアルファベットのカウント

任意の入力された文字列に対し、出現頻度の表示をしようと思って以下のように考えました。
#include<stdio.h>
int main(){
int count[26];
char c;
while((c=getchar())!='\n'){
if(c>='a'&&c<='z'||c>='A'&&c<='Z'){
count[c]++;
}
}
printf("%c:%d\n",c,count[c]);
}

まだｃ初めて２週間くらいなので問題点を解決できません。どなたか教えてください。よろしくお願いします。

No.5 回答者： kb-nike 回答日時： 2007/04/30 22:12

int count[26]; →　int count[52];

if(c>='a'&&c<='z'||c>='A'&&c<='Z'){
count[c]++;
→
if(c>='A'&&c<='Z')　count[c-'A']++;
if(c>='a'&&c<='z')　count[c-'a'+26]++;

printf("%c:%d\n",c,count[c]);
→
for(int i=0; i++; i<52){
　if(i<26)　printf("%c:%d\n",(char)(i+'A'),count[i]);
　if(i>=26)　printf("%c:%d\n",(char)(i-26+'a'),count[i]);
}

注意：見やすくするために全角スペースを使っています。

No.4 回答者： Oh-Orange 回答日時： 2007/04/30 22:08

★修正。

・条件式間違った。ごめん。
　間違い⇒if ( (c <= 'a') && (c <= 'z') ){←※2
　正しい⇒if ( (c >= 'a') && (c <= 'z') ){←※2
　
　間違い⇒else if ( (c <= 'A') && (c <= 'A') ){←※3
　正しい⇒else if ( (c >= 'A') && (c <= 'Z') ){←※3

No.3 回答者： mac_res 回答日時： 2007/04/30 22:08

cは添え字の範囲を超えますね。

#include<stdio.h>
int
main(void)
{
int count[128];
int c;

for (c = 0; c < 128; c++) {
count[c] = 0;
}
while ((c = getchar()) != EOF) {
if ((c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z')) {
count[c]++;
}
}
for (c = 0; c < 128; c++) {
if ((c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z')) {
printf("%c:%d\n", c, count[c]);
}
}
return 0;
}

この回答への補足

ご回答どうもありがとうございます。なんとか入力に対するカウントが得られました。さらにお聞きしたいことがあるのですが、文字の総数をカウントする際、if(c=c)...とやればできると思いましたがそうもいかないみたいなのでどうすればうまくいくでしょうか？？

補足日時：2007/05/01 00:00

No.2 回答者： Oh-Orange 回答日時： 2007/04/30 22:05

★最初に char c で宣言してはいけません。

・getchar() 関数は１文字を取得しますが EOF も返します。
　この EOF は -1 となっていますので char 型では受け取れません。
　int 型で宣言して下さい。
・それから count 配列は初期化しないとデタラメな数が入っているため正しくカウントが
　行えません。注意！
・あと count[c]++ では正しく count[0] ～ count[25] の範囲にカウント値を加算できません。
　さらに最後の printf では何を表示したいのですか？
・そのほか、main() 関数の最後に return(0) などを記述しましょう。

サンプル:
#include <stdio.h>
#include <ctype.h>　←※1

int main( void )
{
　int count[ 26 ] = { 0 };　←こうすると宣言と同時に初期化できます。
　int c;　←注意！
　
　while ( (c = getchar()) != EOF ){　←EOF をチェックするように
　　if ( ch == '\n' ){
　　　break;　←ループを抜る
　　}
　　if ( (c <= 'a') && (c <= 'z') ){←※2
　　　count[c - 'a']++;
　　}
　　else if ( (c <= 'A') && (c <= 'A') ){←※3
　　　count[c - 'A']++;
　　}
　}
　/* 表示部 */
　return( 0 );
}

解説:
・上記のコメントを参考に。
・あと英字判定は専用の関数があります。
　isupper(c)…英大文字を判定
　islower(c)…英小文字を判定
　これを使えば次のようになります。
　if ( islower(c) ){←※2
　　count[c - 'a']++;
　}
　else if ( isupper(c) ){←※3
　　count[c - 'A']++;
　}
　とできます。また、isupper、islower を利用するには ctype.h ヘッダファイルを
　インクルードしないと使えません。
・以上。参考に！

参考URL：http://www9.plala.or.jp/sgwr-t/c/sec07.html

No.1 回答者： t_nojiri 回答日時： 2007/04/30 21:53

>while((c=getchar())!='\n'){

ループの終了判定おかしいですね。
変数ｃの代入に失敗したか成功したか判定しそうです。