c言語 return 0; があると思ったように動かなくなるプログラム

Question

数を１０個まで入力、-1が入力されたら１から１００まで範囲で最大、最小を探すプログラムです。
例えば1 2 3 4 5 6 -1と入力すると最大値:1最小値:1となり思ったように動きません
しかしmaxmin関数のreturn 0;を外すと正常に動きます何故でしょうか？return 0;の役割とは？
慣習的に書くことの多いであろうreturn 0;で異常となるのは気持ちが良くないので対処方も教えてもらいたいです。
ｃ言語はポインタまで学習してる状態です。

#include <stdio.h>

void maxmin(int array[],int *max,int *min);

int main(void)
{
	int i = 0,array[10],max,min;
	
	do {
		printf("%d 番目の数:",i + 1);
		scanf("%d",&array[i]);
		i++;
	} while (array[i - 1] != -1);
	
	maxmin(array,&max,&min);
	
	printf("最大値 %d : 最小値 %d\n",max,min);
	
	return 0;
}

void maxmin(int *array,int *max,int *min)
{
	int i = 0;
	
	*max = 0;
	*min = 100;
	
	while (array[i] != -1) {
		if (array[i] > *max) *max = array[i];
		if (array[i] < *min) *min = array[i];
		i++;
		return 0;
	}
}

Comments

[cateye]

void（復帰値がない）関数で０を返すのは何故でしょう？

[ziryuu]

関数を終了する時に真を表すためみたいですけど、何のために真を返してるのでしょうか、別に０でなくとも動くというのに。
自分のはreturnで終了してるからうまくいかないだけみたいですね。

Answer 1

return 0;の役割とは？

そこで関数の実行を終了して、関数呼び出し元にint型の0を返すという意味です。

慣習的に書くことの多いであろうreturn 0;で異常となるのは気持ちが良くないので

return 0;を慣習的に書くことは無いです。必要があるときだけ書きます。
そもそも、プログラムの中で慣習的に書くものなどありませんよ。すべて意味があって書いています。

初心者に教えるときに「これはおまじないとして書いておく」みたいな説明をする人がいますが、それは善意に解釈すると、あくまで入門したての人が本質的でない部分を気にしすぎないようにするために言ってるんだと思います。

Comments

[ziryuu]

変数=自作関数;で
return 0;を返すと変数にはint型０が代入されるんですね。
普段main関数でreturn 0;を書くのは、自作関数で関数を返し忘れるのを普段から心がけて書いてるみたいな理由でもなく、何かの本質自体はあるってことになるんですかね。

Answer 2

while (array[i] != -1) {

    if (array[i] > *max) *max = array[i];
    if (array[i] < *min) *min = array[i];
    i++;
    return 0;
}

return の場所間違ってますね

Comments

[ziryuu]

ありがとうございます。

Answer 3

while (array[i] != -1)

は、一回しか実行されませんが？

Comments

[ziryuu]

retrn 0;
printf("文字列");
としても「文字列」は表示されないみたいにretrnは関数を終了させるのですね。

Answer 4

コードは書かれたとおりに上から処理していきます。

return は別の言い方をすれば「右にあるデータを伴って終了する」みたいな意味です。
(厳密には違いますが)

"return 0" とあれば 0 というデータを伴って終了します。

関数は ( "■" を半角スペースと見立てて書くと )

型■関数名(引数){
   // 処理内容
return■戻り値;
}

が基本となります。

型 = 戻り値(の型) です。

よってWarningなりErrorなりがでているはずですが。

return 0; とするのは、あくまで「C言語にはbool ( 言語によってはBooleanとか ) がないため」です。

そこで boolの代わりに intで代用して、( main関数の場合は慣習的に ) 0 が成功、それ以外 ( マイナス値だったりとか ) ならそれぞれのエラーとしてやっています。

もし「成功した数を返す」というような個数系の値なら 0 を失敗 ( 何もない )、(1以上の)正の数なら成功した数...みたいにしますし、

0が意味を持つ場合 ( 例えば 文字列から整数を取り出すとか ) なら、負の数をエラー値、0は成功はしたが個数や値が0, 正の数なら個数...みたいにすることもありますね。

そういう風に意味が異なってきます。

void maxmin(int *array,int *max,int *min)
{
    int i = 0;

    *max = 0;
    *min = 100;

    while (array[i] != -1) {
        if (array[i] > *max) *max = array[i];
        if (array[i] < *min) *min = array[i];
        i++;
        return 0;
    }
}

を読んでみると、

void maxmin(int *array,int *max,int *min)
{
    int i = 0;
    
    // 念のため初期設定(厳密には初期設定じゃないけど...)
    *max = 0;
    *min = 100;
    
    // -1 がでるまでループ
    while (array[i] != -1) {
        // max より大きければ 付け替え
        if (array[i] > *max) *max = array[i];
        // 同上( 向きが違うだけ )
        if (array[i] < *min) *min = array[i];
        // 次に進ませる
        i++;
        // 0 を伴って終了
        return 0;
    }
}

というような状態なので、疑似コード(ぽい何か)で書くと、

1. 念のため初期設定
2. -1 がでるまでループ
   2.1. maxより大きければ...
   2.2. minより小さければ...
   2.3. (0) を伴って終了

みたいになるので、

最大要素数である 10 まで -1 以外の値が入っていても初回 whileに突入して 2.3. で関数そのものから抜けているので whileの先端に来ることはありません。

よって 多くても1回しか動かない。( 0番目がすでに -1 なら条件式を満たしていないからスルー。 )

なので、2.3. の位置に return を持ってくるのではなくて、3. 辺りに持ってくること。

あるいは while内にif文を設けて、「100という値が出た場合は強制的に関数から抜ける」みたいにするなら別ですが、今回の場合は関係がありませんから、

2.3. の位置ではなく 3. の位置に持ってくる。

また、関数の型が void になっているので return; でいいです。
(最後まで到達するのなら return; も不要という説もあるが...)

Comments

[ziryuu]

ありがとうございます。