プログラミング（C言語）についての質問です

質問

３つの整数の入力を受け付け、最大と最小を求める関数を作成し得られた結果を表示するプログラミングを作成したつもりなのですが、うまく作動しません。（コンパイルはできますが、結果が無茶苦茶になります。）
ご教授宜しくお願いします。
それと、課題文にはポインタを使って最大値と最小値を同時に求めるようにと書いてあったのですが、それもよくわからないです。
今回初めてポインタと配列の受け渡しについて習ったのでよくわかっていない部分も多いと思うのですが、何卒宜しくお願いします。
ちなみに関数の形自体は void minmax(int data[],int *min,int *max){} で決まっています。

#include <stdio.h>

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

int i;
*min=*max=data[0];

printf("1st intenger:");
scanf("%d",&data[0]);
printf("2st intenger:");
scanf("%d",&data[1]);
printf("3st intenger:");
scanf("%d",&data[2]);

for(i=1;i<3;i++){
if(*max<data[i]){
*max=data[i];
}
if(*min>data[i]){
*min=data[i];
}
}
}

int main(void){

int data[3],min,max;

minmax(data,&min,&max);
printf("最小値は%dで最大値は%dです",min,max);

return 0;
}

回答No.1

maiko0318 · ベストアンサー

*min=*max=data[0];

これは何がどうなるのですか？
考えてみましょう。

お礼

ありがとうございます！
正直こんなに早く回答が得られるなんて思っていなかったのでびっくりしています。
こんなところを間違っていたのですね…
プログラミングの基本を再確認しないといけないです。

回答No.4

Tacosan

ど～でもいいところにつっこむけど
2st
3st
はなんかかなしい.

お礼

ありがとうございます！
ほんとですね・・・。
課題を提出する前に直しておきます（笑）

回答No.3

asuncion

＞*min=*max=data[0];

先頭要素を仮の最大値・最小値にするという考え方
そのものは正しいです。が、それを書く場所が正しくありません。

data[0]を「実際に読み取った後」に書かないと。

お礼

ありがとうございます！
おかげさまでコンパイルすることができました～
間違っていた方のコンパイル結果もどういう構造になっているのか理解することができました！

回答No.2

ok-kaneto

>*min=*max=data[0];

配列の先頭要素を仮の最小値、仮の最大値にしているのだと思いますが。
この時点では先頭要素はscanfより前なのでデータには何も入っていません。
これを実行するタイミングを考えてみましょう。

お礼

ありがとうございます！
scanfの後に書いたらうまいことコンパイルされました！
こんなところを間違えるなんて少し恥ずかしいです。。。