c言語 関数作成の問題

Question

前提・実現したいこと

以下の問題に取り組み、つまづきましたので
どこがおかしいか教えていただけると助かります。

(問題)要素数nの配列v内の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が7であれば、idxに{1,3,6}を格納した上で3を返却する。

発生している問題・エラーメッセージ

エラーメッセージ
Segmentation fault (コアダンプ)

```C言語
```#include <stdio.h>

  
#define FAILED -1

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

  int a = 0;
  int i;
  
  for(i=0;i<n;i++){
    if (v[i] == key){
      idx[a] == i;
      a++;
    }
  }
  return (a==0)?FAILED:a;
}
      
int main(void){

  int n;
  int i,ky;
  int vx[n];
  int idx;
  int vy[] = {0};

  printf("入力する数を教えてください");scanf("%d",&n);
  
  for(i=0;i<n;i++){
    printf("vx[%d]:",i);
    scanf("%d",&vx[i]);    
  }
  printf("探す値:");
  scanf("%d",&ky);

  if((idx = search_idx(vx,vy,ky,n))== FAILED)
    puts("探索に失敗しました");
  else
    printf("%dは%d個あります\n",ky,idx);

  return 0;

}

試したこと

補足情報（FW/ツールのバージョンなど）

回答者の皆様、ありがとうございました。

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[退会済みユーザー]

ご指摘ありがとうございます。
感謝しております。

Answer 1

n の値が決まっていないのに、int vx[n]; だと領域が確保されません。
int vy[] = { 0 }; だと、要素数 1 の配列が確保されるので、
複数のインデックスを入れることはできません。

とりあえず、
int vx[100];
int vy[100];
で試してみてください。

Comments

[退会済みユーザー]

丁寧な回答ありがとうございます。
みなさんのおかげでエラー内容に気づくことができました。
大変感謝しております。

Answer 2

vxが入力したn要素分確保されていません。

	int n;// 不定値
	int vx[n]; //サイズ不定


	printf("入力する数を教えてください"); scanf("%d", &n);

	for (i = 0; i < n; i++) {
		printf("vx[%d]:", i);
		scanf("%d", &vx[i]); // iがvxの要素数以上であれば範囲外アクセス
	}

---

search_idx(vx, vy, ky, n)内でidx[a] == i;としていますが、idxに渡されているvyは要素数1の配列のため、aが0以外の場合は範囲外アクセスとなります。

Comments

[退会済みユーザー]

回答ありがとうございます。
とてもためになりました。

Answer 3

int main(void){

int n;
int i,ky;
int vx[n];
int idx;
int vy[] = {0};

このvxの要素数が不定になってしまうのでダメです
ローカル変数（n）は、初期化しないとでたらめな値になります

そして、vyの要素数は１となってますが、これでいいんでしょうか

Comments

[退会済みユーザー]

回答ありがとうございます。
助かりました。