どこが間違っているかを教えてください

Question

入力された文字の出現回数をカウントしようという課題をやっていて、自分は下のようなコードを作ったのですが文字を入力するところまでしか動かず、下の二つの関数が動いてくれません。
daigaku と入力したら
d:1回
a:2回
i:1回
g:1回
k:1回
u:1回
と出力されるようにしたいです。
コードはこんな感じです。
どうか回答よろしくお願いします。
#include<stdio.h>
#define MAX 101

count_ch_in_str();
disp();

char g_ch[MAX], input[MAX];
int g_chcount[MAX];

int main(void) {

printf("スペースなしの文字列>");
scanf("%s",input);

count_ch_in_str();
disp();

return 0;

}
count_ch_in_str() {
int counta,countb;
for (int i = 0; i != MAX; g_chcount[i])g_chcount[i] = 0;
for (counta = 0; input[counta] != '\0'; counta++) {
for (countb = 0; g_ch[countb] != '\0'; countb++) {
if (input[counta] == g_ch[countb]) {
g_chcount[counta]++;
}
else {
input[counta]=g_ch[counta] ;
g_chcount[counta]++;
}
}
}
}
disp() {
for (int count = 0; g_ch[count] != '\0'; count++) {
printf("%c: %d回\n", g_ch[count], g_chcount[count]);
}
}

Comments

[jimbe]

コードはマークダウン記法をご利用ください。
マークダウン記法につきましてはヘルプを参照してください。

Answer 1

とりあえず修正しておきました。

#include<stdio.h>
#define MAX 101

void count_ch_in_str();     //修正
void disp();                //修正

char g_ch[MAX], input[MAX];
int g_chcount[MAX];

int main(void)
{

    printf("スペースなしの文字列>");
    scanf("%s", input);

    count_ch_in_str();
    disp();

    return 0;
}

void count_ch_in_str()  //修正
{
    int counta, countb;
    int flg;
    for (int i = 0; i < MAX; i++){      //修正
        g_chcount[i] = 0;
        g_ch[i] = '\0';                 //追加
    }
    for (counta = 0; input[counta] != '\0'; counta++) {
        //修正開始
        flg = 0;
        for (countb = 0; g_ch[countb] != '\0'; countb++) {
            if (input[counta] == g_ch[countb]) {
                g_chcount[countb]++;
                flg = 1;
                break;
            }
        }
        if (flg == 0){
            g_ch[countb] = input[counta];
            g_chcount[countb]++;
        }
        //修正終了
    }
}

void disp()     //修正
{
    for (int count = 0; g_ch[count] != '\0'; count++) {
        printf("%c: %d回\n", g_ch[count], g_chcount[count]);
    }
}

Answer 2

for (int i = 0; i != MAX; g_chcount[i]) g_chcount[i] = 0;

まずここで無限ループになっているような気がします。
どれだけループしても i はゼロのままであり、 MAX（101）と等しくなることはありません。

    for (countb = 0; g_ch[countb] != '\0'; countb++) {
      if (input[counta] == g_ch[countb]) {
        g_chcount[counta]++;
      } else {
        input[counta] = g_ch[counta];
        g_chcount[counta]++;
      }
    }

ここもおかしいです。
ここは、input から取り出した１文字が、配列 g_ch の中に含まれているか調べ、含まれていない場合は、g_chに追加する部分だと思われます。
この場合、g_chに文字を追加するのは、最後まで一致する文字がなかった場合に限ります。
しかし、上のコードでは、チェックの途中であっても、文字が一致しなくなったらすぐに新しい文字をg_chに格納しようとしています。
さらに、新しい文字を格納する処理のところが逆になっています（inputの配列に、g_chの文字を格納している）。