C言語でヒストグラムを組む方法を教えていただきたいです

Question

#include<stdio.h>
#include<stdlib.h> //exit, malloc, realloc 関数
int main(void){
int *input; //int 型入力値に対するデータ領域の先頭位置アドレス
int hist[5] = {0}; // 1-5 までの値に対する頻度（初期値０）
int size = 5; //データ入力用メモリ領域数の初期設定
int count = 0; //データ入力数
int value; //データ入力値
int i, j; //ループ変数
printf("入力値のヒストグラムを作成します．（終了：ゼロ入力）\n");
input = (int *)mallco(sizeof(int)* 5);
if(input == NULL){
    printf("メモリが確保できません\n");
    exit(-1);
}
while(1){ //while(1)は break するまで（ゼロが入力されるまで）永遠に回り続けます．
printf("数値（1-5）を入力してください：");
scanf("%d", &value);
if(value == 0){break;} //０が入力された場合，while 文からアウトします．
if(count > size){ //入力数が当初入力数 5 個を超える場合
size = size + 5; //領域数を５つ増やす
input = (int *)reallco(sizeof(int)* 5);
if(input == NULL){
    printf("メモリが確保できません\n");
    exit(-1);
}
}
*(input + count) = value;
for(i = 1; i <= 5; i++){if(value == i){hist[i-1]++;}}
//ヒストグラム用のデータカウントを行なっています．
count++;
}
------------------------------------------------------------
//ヒストグラムの作成

------------------------------------------------------------
free(input);
return 0;
}

コード内の水平線間に1~5の数字を適当に入力して、入力された回数に応じて＊を積んでいくヒストグラムを作りたいと考えています。表記の方法は「1:*** 2:* 3: 4:** 5:***」のようにしたいです。
for文をつかって組もうと思ったのですがしばらく調べたり試行錯誤してもうまく組む方法が思いつかなかったのでお力を借りたいと思いました。
私が考えて作ってみたのは下記の通りなのですが無限に＊が生成されてしまいます。どこが間違っているか教えていただきたいです。

//ヒストグラムの作成
printf("1:");
for(j=0;j>=hist[0];j++){
    printf("*");
}
 printf("\n");
 printf("2:");
 for(j=0;j>=hist[1];j++){
    printf("*");
}
 printf("\n");
 printf("3:");
 for(j=0;j>=hist[2];j++){
    printf("*");
}
 printf("\n");
 printf("4:");
 for(j=0;j>=hist[3];j++){
    printf("*");
}
 printf("\n");
 printf("5:");
 for(j=0;j>=hist[4];j++){
    printf("*");
}
 printf("\n");

Comments

[y_waiwai]

その提示のコードはなんでしょうか。
また、しつもんはなんでしょう

[dodox86]

malloc()とかrealloc()とか難しい関数を使うことは置いておいて、まず
1番目の入力: 1番目に入力された値の数分ループして '*' を出力
2番目の入力: 2番目に入力された値の数分ループして '*' を出力
...
のようなコードは書けますか。それが自力でできないようだと完成は難しいと思います。
※ところでmalloc()もrealloc()も関数名のスペルが間違っています。きっと、コンパイルが通って実行ファイルが出来上がりませんね。

[jump110]

すみません、自分なりに考えたものを追記しましたので確認お願いいたします。
関数のスペルミスもなおしました。教えていただきありがとうございます！

[cwi]

表示が変なら最初に疑うのは表示している部分でしょう。ヒストグラムの作成のfor文ですが、j>=hist[num]が一度でも真となったらjは増える一方ですから…

[dodox86]

> 自分なりに考えたものを追記しましたので

考えてコードを書くだけでなく、そのコードが意図通り動作しているのか、デバッグしましょう。
> for(j=0;j>=hist[0];j++){
① jを0に初期化
② j がhinst[0]の値以上なら{}内のブロックを実行
③ j を+1 して②に戻る
で意図通り動くのでしょうか。hinst[0]に値を入れているのはどこですか？
回答も既にいただいているのですから、そちらも理解して応対しましょう。

[jump110]

皆さん返信ありがとうございます！以上じゃなくて未満でしたね、そこを変更したらうまく作動しました

Answer 1

#include<stdio.h>

int main(void) {
    int hist[5] = { 0 }; // 1-5 までの値に対する頻度（初期値０）
    printf("入力値のヒストグラムを作成します．（終了：ゼロ入力）\n");
    int value;
    do {
        printf("数値（1-5）を入力してください：");
        scanf("%d", &value);
        if ( value >= 1 || value <= 5 ) {
            ++hist[value-1];
        }
    } while ( value != 0 );

    for ( int i = 1; i < 6; ++i ) {
        printf("%d:", i);
        for ( int count = hist[i-1]; count; --count ) {
            printf("*");
        }
        printf("\n");
    }
    return 0;
}

Answer 2

printf("1:");
for(j=0;j<hist[0];j++){
    printf("*");
}
 printf("\n");
 printf("2:");
 for(j=0;j<hist[1];j++){
    printf("*");
}
 printf("\n");
 printf("3:");
 for(j=0;j<hist[2];j++){
    printf("*");
}
 printf("\n");
 printf("4:");
 for(j=0;j<hist[3];j++){
    printf("*");
}
 printf("\n");
 printf("5:");
 for(j=0;j<hist[4];j++){
    printf("*");
}
 printf("\n");

Comments

[kaina]

この課題では1~5の数字になっていますが、例えば1~100、1~1000と増えた場合
あなたはfor分を100個、1000個と増やすのですか？

[jump110]

今の自分ではこの方法しか思いつかなかったのですがたしかに数が増えると対処できなくなりますね...
良ければ数が数がふえた場合の対処方法を教えていただきたいです。

[kaina]

それがepistemeさんの回答ですよ。

[jump110]

for文を重ねて書いているところですね！なるほど、ありがとうございます！