関数の戻り値を変数に代入した後プログラムが止まる・・・

Question

###前提
現在、c言語を書籍で学習している者です。
そこで出された問題がよく分からないので、質問させていただきます。

###問題
要素数ｎの配列ｖ内のkeyと等しい全要素の添字を配列idxに格納する関数search_idxを作成せよ。
返却するのはkeyと等しい要素の個数とする。

int search_idx(const int v[], int idx[], int key, int n);

たとえば、vに受け取った配列の要素が{1, 7, 5, 7, 2, 4, 7}でkeyが７であれば、idxに｛1, 3, 6｝を格納した上で3を返却する

###該当のソースコード

#include <stdio.h>

#define NUMBER 7    //配列ｖの要素数
#define MAX 100     //配列idxの要素数の最大値

int search_idx(const int v[], int idx[], int key, int n){

    int i = 0;
    int c = 0;

    while(1){
        if(v[i] == key){
            idx[c] = i;
            c++;
        }

        i++;
    }

    return c;
}

int main(void){

    int i, ky, n;
    int vx[NUMBER];
    int idx[MAX];

    for(i = 0; i < NUMBER; i++){
        printf("vx[%d] : ", i);
        scanf("%d", &vx[i]);
    }

    printf("Key value : "); scanf("%d", &ky);

    /*ここまでは機能する*/

    n = search_idx(vx, idx, ky, NUMBER);

    putchar('{');
    for(i = 0; i < n; i++){
        printf("%d, ", idx[i]);
    }
    putchar('}');
    putchar('\n');

    printf("%d\n", n);

    return 0;
}

###結果・疑問点
現在のコードだと、

vx[0] : 1
vx[1] : 7
vx[2] : 5
vx[3] : 7
vx[4] : 2
vx[5] : 4
vx[6] : 7
Key value : 7

までは、表示されるのですがそれより先が表示されません。
どこで間違っているのか教えていただきたいです。

###補足情報
書籍： 新・明解C言語 入門編
問題： p157 演習６ー１１

Answer 1

こんにちは。

search_idx()内でのループの終了条件が無いので、無限ループしているのでしょう。

Comments

[kamecha]

有り難うございます！
while文の中に終了条件を書くと
正常に作動しました。

[LouiS0616]

引数として要素数nが与えられているのだから、for文を使った方が自然でしょう。

[sazi]

TO:nonbirikameさん
学習されているという事なので、ヒント程度で回答しました。
取り敢えず前に進めて良かったですね。
LouiS0616さんの言われているように、ループ処理は色々方法がありますので、
この際に色々調べてみて下さいね。

[kamecha]

有り難うございます。
頑張って学習していきます。