繰り返しが上手く起こりません。

Question

前提・実現したいこと

関数inputDataで配列のデータを入力します。
配列のデータに０が入力されたら入力が終了する。
その後、配列の要素を表示するプログラムを作りたいです。

ここがダメ！これ！などでも頂けると助かります。
コメントお願いします。

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

繰り返しの入力が出来ません。

該当のソースコード

#include <stdio.h>

#define NUMBER 11

void inputData(int v[]);

int main(void)
{
	int v[NUMBER];
	int i;
	
	printf("データ：");

	inputData(v);

	printf("結果:");
	
	for(i=0;v[i]!=0;i++){
		printf("%d",v[i]);
	}
	
	return 0;
}

void inputData(int v[])
{
	int i;
	
	for(i=0;v[i]!=0;i++){
		scanf("%d",&v[i]);
	}
}

試したこと

forの中の条件を「i<NUMBER」にすると繰り返しが起きるのですが、「v[i]!=0」にすると繰り返しが起こりませんでした。

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

サクラエディタ、コマンドプロンプト for VS　2019

Answer 1

printfのループはいいとして、scanfのループ条件がまずいです。
forで参照しているv[i]は、scanfで入力前の状態です。
よってこれでは入力値が0か否かは判定できません。
ループ条件としてはi<NUMBERでよく、scanfの入力結果を別途判定してループを抜ける処理を追加すればよいでしょう。

Comments

[mememe0331]

ttyp03様、回答ありがとうございます。
おかげで解決することが出来ました。

Answer 2

C言語では、ローカル変数としてint v[NUMBER];が定義された場合、値は不定です。そのような入力前の値を、v[i]のように参照してはいけません。

…というより、入力側のv[i]!=0は、何を期待して書いた条件なのでしょうか。

Comments

[mememe0331]

maisumakun様、ご回答ありがとうございます。
ご指摘ありがとうございました。勉強になりました。
v[i]!=0は配列の要素に0が入っていたら繰り返しを止めるということを期待して書きました。

Answer 3

inputdata関数内のfor文脱出条件がv[i]!=0だからではないですか？
これ、そもそもscanfの入力ありましたか？
main関数の配列vって初期化していないから、スタックのゴミが入ったまま評価されますよ。

Comments

[mememe0331]

DreamTheater様、ご回答ありがとうございます。
勉強になりました。ご指摘ありがとうございます。