最小値を表示したいのですが...

Question

最小値を表示したいのですが3つ以上の整数を入力すると最小値が1になってしまいます。
どうしたらいいですか？
コード

####include <stdio.h>



int min_of( const int v[], int n)
{
	int i;
	int min = v[0];
	
	for( i = 1; i < n; i++){
		if( v[i] < min ){
			min = v[i];
		}
	}
	return min;
}


int main(void)
{
	int n, i, min1;
	int min[i] ;
	
	printf("何個整数を入力しますか：");    scanf("%d", &n);
	for( i = 0; i < n; i++){
		printf("[%d]：", i + 1);   scanf("%d", &min[i]);
	}
	min1 = min_of(min, n );
	
	printf("最小値 = %d\n", min1);
	
	
	return 0;
    
}

``**```


```**

Answer 1

こんにちは。

もし、gccをお使いなら、下記でいけると思います。

//  int min[i] ;

    printf("何個整数を入力しますか：");    scanf("%d", &n);
    int min[n] ;

元のプログラムでは、int i; int min[i];となってますが、min[i]を宣言した時iは初期化していないため、C言語の規格上はこの時点では不定値です。ただし、gccはこのような時0に初期化するようです。従って、int min[0];と宣言したのと同じ意味になるため、メモリが確保されておらず、何が起こるか分からない状態なのです。

なお、このように実行時に決まる要素数で配列を定義する方法はVLAと呼ばれています。VLAはC言語の標準規格で規定されてますが、C++の標準規格には含まれません。また、残念ながらVC++はサポートしていませんので、使う場合は要注意です。

Comments

[sho36515363]

ありがとうございます。解決しました。

Answer 2

int main(void)
{
    int n, i, min1;
    int min[i] ; // <- i=0(大抵)なのでおかしい