キーボードから入力した値を要素数にしたい

Question

キーボードから入力した値を要素数にしたいです。以下のようなコードを書いたのですが、6行目(のyo)にエラーが出てしまいます。原因と対処法を教えていただきたいです。よろしくお願いいたします。

#include <stdio.h>

int main(void) {
	printf("Please diceid n(the length of an array).\n");
	int yo = 0 ;
	scanf("%d", &yo);
	int dt[yo] = { 1 };
	for (int i = 0; i < yo; i++) {
		dt[i] = i;
		printf("%d\n", dt[i]);
	}
        return 0;
}

Comments

[fu7mu4]

コンパイラや実行環境を記載してください。コンパイラの種類によってできること、できないことが異なります。
また、エラー出力をコピーペーストしてください。

Answer 1

宣言時に配列要素数を変数で指定できるのは、一部のコンパイラの拡張機能です。

対策案1：上限を決めて、大きめに宣言しておく
対策案2：mallocやcallocで、動的にメモリー領域を確保してそれを配列として使う

Comments

[h_proc]

回答ありがとうございます。

Answer 2

Cの配列長は固定です。なので、malloc()等でメモリを確保して使いましょう。

「追記」

/**
    teratail:166610
 */
#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    printf("Please diceid n(the length of an array).\n");
    int yo = 0;
    scanf("%d", &yo);
//    int dt[yo] = { 1 };
    int *dt = (int *)malloc((size_t)yo * sizeof(int));
    //
    for (int i = 0; i < yo; i++) {
        dt[i] = i;
        printf("%d\n", dt[i]);
    }
    free(dt);
    //
    return 0;
}

結果

usr ~/Project/test % ./a.out 
Please diceid n(the length of an array).
5
0
1
2
3
4

Comments

[cateye]

C99から可変長配列が追加されています(c11はオプショナル？)が、いまいちわからん。参考：https://qiita.com/raccy/items/bfcb62086c5f027d57b6

[h_proc]

ありがとうございます。

Answer 3

#include <stdio.h>
#include <stdlib.h>//malloc関数呼び出し用

int main(void){
    printf("Please diceid n(the length of an array).\n");
	int yo=0;
	scanf("%d",&yo);
	int *dt = (int*)malloc(sizeof(int)*yo);//int型の配列をyo個作成
	int i;
	for(i=0;i<yo;i++){
		dt[i] = i;
		printf("%d\n",dt[i]);
	}	
    free(dt);
	getch();
	return 0;
}

malloc関数を使って配列を動的確保すれば目的の処理は実装できます。

※コンパイラはgccで動作確認済み。

Comments

[cateye]

linux環境のclangでは、getch()は、ワーニング＆リンクエラーになります。また、 -Weverything指定しているので(size_t)yoとキャストしないとワーニングになります。

：t.c:9:40: warning: implicit conversion changes signedness: 'int' to 'unsigned long'
[-Wsign-conversion]
int *dt = (int*)malloc(sizeof(int)*yo);//int型の配列をyo個作成
~^~
t.c:16:5: warning: implicit declaration of function 'getch' is invalid in C99
[-Wimplicit-function-declaration]
getch();
^
t.c:16:5: warning: this function declaration is not a prototype [-Wstrict-prototypes]
3 warnings generated.
/tmp/t-947b14.o: 関数 `main' 内:
t.c:(.text+0x5a): `getch' に対する定義されていない参照です
clang-8: error: linker command failed with exit code 1 (use -v to see invocation)
u

Answer 4

提示されたソースコードをそのままgcc -Wall -W オプションでコンパイルした場合
空白行を含めて7行目に以下のエラーが生成されます。

hoge.c:7:12: error: variable-sized object may not be initialized
    int dt[yo] = { 1 };
           ^~
1 error generated.

このエラーメッセージは可変長配列 dt をこの方法では初期化できないことを意味しています。
callocを使用して初期化してください。

なお、「エラーメッセージが表示されます」といった質問では、読者にはわからないので、エラーメッセージエラーメッセージを質問にコピペしてください。(要約はしないでください)

Comments

[h_proc]

回答ありがとうございます。こちらの説明不足で申し訳ございません。

Answer 5

gcc使っているなら、これでいけます。

int dt[yo];