関数の引数に配列を使うことが難しい

Question

前提・実現したいこと

0~100の整数を10個入力して、最大値と最小値を求めて表示するプログラムをかきたいです。ただし、-1が入力されると処理を中止する、最大値最小値を求める関数は一つにしてmain以外でつくる、という制約のもとです。

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

実行すると11番目の数と表示されるのと、11回以上の入力を受け付けるという問題。エラーメッセージは出ていません。

該当のソースコード

#include<stdio.h>

void maxmin(int data[], int *max, int *min);
  
int main(void){
  int i=0;
  int data[10];
  int max, min=0;

  for(i=0; i<10; i++){data[i]=0;}
  do{
    printf("%d番目の数:", i+1);
    scanf("%d", &data[i]);
    i++;
  }while(data[i-1]!=-1 || i<10);
  
  maxmin(data, &max, &min);
  

  return 0;
}

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

試したこと

何分エラーが出てないもんですから、手出しできませんでした。。。

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

ここにより詳細な情報を記載してください。

Comments

[a_saitoh]

「-1が入力されると処理を中止する」とありますが、コードの動作は、「-1が入力されると入力を終了する」になっています。どちらをやりたいのでしょうか？

[退会済みユーザー]

入力を終了したかったです。。僕の書き方だとどっちか分からないですね、ご指摘ありがとうございます。

Answer 1

do{
  printf("%d番目の数:", i+1);
  scanf("%d", &data[i]);
  i++;
} while(data[i-1]!=-1 || i<10);

ここ、こーすればよくね?

for ( i = 0; i < 10; ++i ) {
  int val;
  printf("%d番目の数:", i+1);
  scanf("%d", &val);
  if ( val == -1 ) break;
  data[i] = val;
}

Comments

[退会済みユーザー]

コメントありがとうございます。仰せのを実行したのですが、これって変数valを間に挟む？というのは何かいいことがあるのでしょうか？（無知なので教えていただけると嬉しいです）

[episteme]

不正な値にもかかわらず配列に格納されるのがキモチ悪かった。

[pepperleaf]

これでは、-1 は、data[]に格納されませんが、、、。

[episteme]

言い直す。
「不正な値にもかかわらず配列に格納されるのがキモチ悪かった」から、一旦valに読みこんで云々。

[退会済みユーザー]

冒頭に小さくありますが、-1が入力されたらプログラムを中止するという制約ありです。。（なんでこの制約？というのはごもっともです）

[退会済みユーザー]

不正な値っていうのは、例えば配列がint型なのに入力の時にint以外の型が入力される可能性があって、まだintであっても変数の方がマシってことですか？（推測でものを言ってます。。）

[episteme]

たとえ使われないとしても、この配列に納められるのはふさわしくないと考えた。

[episteme]

あー、そかそか。-1を配列の終端マーカーとして使っているのか。
ならば僕のやり方は間違ってるですね。
# だったら配列は全部-1で初期化しておけばいいのに...

[episteme]

...あれ? 終端マーカー使うんなら int data[11] じゃなきゃダメちゃうか?

[can110]

入力は１０回ループ。途中-1なら格納して終了。
minmaxも１０回ループ。途中-1なら終了。
なら配列サイズ１０で足りますね。

[episteme]

ですね。minmax側もいぢくればOK.

Answer 2

いくつか問題がありそうですが・・・。

11回以上受け付けるということに絞れば

}while(data[i-1]!=-1 || i<10);

ここに問題があります。

入力値が -1 でないか i が 10 より小さいという条件なので 11回目以降に -1 を入力しない限りループを抜けません。

その他の追記
1．初期化の問題

for(i=0; i<10; i++){data[i]=0;}

は、既に指摘がありますが・・・。
配列全体を0で初期化するのであれば、宣言部で

int data[10]={};

でよいかと思います。

1. 入力値の問題

課題では 入力値は 0～100 とありますので、-1 は途中で抜けるとしても
100 より大きい入力があれば 再入力を促すようにすべきかと思います。

1. 最大値と最小値を求めて表示する

　「最大値と最小値を求めて表示する」とありますので、最後に min と max を表示しなければならないのでは？

Comments

[退会済みユーザー]

コメントありがとうございます。&&と||がごっちゃになってました。

[退会済みユーザー]

聞くの忘れていましたが、いくつかというのは何を指しますか？改善したいので教えていただきたいです。

[退会済みユーザー]

check関数もwhile文の中身がおかしかったことにようやく気づきました。これ以外であれば教えてほしいです！よろしくお願いします。

[showkit]

あ、ごめんなさい 1. 2. 3. と書くつもりが 1. 1. 2. になっちゃいました。

[退会済みユーザー]

あっそれは全然気にしてませんでした。

Answer 3

実行すると11番目の数と表示されるのと、11回以上の入力を受け付けるという問題。エラーメッセージは出ていません。

for文で配列を初期化したあと、変数iを初期化していないからでしょう。

for(i=0; i<10; i++){data[i]=0;}
i = 0;   // 追加
do{

※質問内容とタイトルがあってません。

Comments

[退会済みユーザー]

コメントありがとうございます。ほんとですね、ここで値が保存されてたのか（前にも同じミスしました。。）

[pepperleaf]

data[10] の宣言に対し、i を初期化してないので、data[10] とか、data[11]とか、アクセスしてます。これはメモリ破壊ですね。異常終了してもおかしくない状況。

Answer 4

こんにちは、

1.最大で10個入力するところの10は、記号定数にしましょう。
2.入力はscanfよりはfgetsをおすすめします。

#include <stdio.h>
#include <stdlib.h>

#define BUFSIZE 256
#define MAX_NUMBER 10

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

int main(void)
{
	char buf[BUFSIZE];  
  int i;
  int data[MAX_NUMBER];
  int max, min;

  for(i = 0; i < MAX_NUMBER; i++){
  	printf("%d番目の数->",i + 1);
  	fgets(buf,BUFSIZE,stdin);
  	data[i] = atoi(buf);
  	if(data[i] == -1){
  		break;
  	}
  }
  maxmin(data, &max, &min);

	printf("最大値: %d 最小値: %d\n",max,min);
  return 0;
}

void maxmin(int data[], int *max, int *min){

  int i = 0;
  
  *max = 0; 
  *min = 100;

  while(i < 10){
    if(data[i] != -1){
    	if(data[i] > *max){
    		*max = data[i];
    	}
    	if(data[i] < *min){
    		*min = data[i];
    	}
    }
    else{
    	break;
    }
    i++;
  }
}

Comments

[退会済みユーザー]

出力結果
<例1>-1を入力しない場合
1番目の数->4
2番目の数->8
3番目の数->7
4番目の数->5
5番目の数->1
6番目の数->45
7番目の数->9
8番目の数->48
9番目の数->54
10番目の数->13
最大値: 54 最小値: 1
<例2>-1を入力する場合
1番目の数->4
2番目の数->8
3番目の数->6
4番目の数->-1
最大値: 8 最小値: 4