Ｃ言語初心者です。書いたプログラムを採点して下さい

質問

入力された0~100までの１０個の数値から、
最小値と最大値を表示するプログラムを書きました。

#include <stdio.h>

int getmin(int * pvalue);
int getmax(int * pvalue);

int main(void)
{
　　　　int i,value[10];
　　　　int minimum,maximum;

　　　　printf("0~100の範囲で10個の数値を入力してください：\n");
　　　　for(i=0;i<10;i++){
　　　　　　　　scanf("%d",&value[i]);
　　　　　　　　if(value[i] < 0 || value[i] > 100)
　　　　　　　　{
　　　　　　　　　　　　printf("0~100の範囲です：\n");
　　　　　　　　　　　　scanf("%d",&value[i]);
　　　　　　　　}
　　　　}
　　　　minimum = getmin(value);
　　　　maximum = getmax(value);
　　　　printf("最小値は%d\n",minimum);
　　　　printf("最大値は%d\n",maximum);
　　　　return 0;
}

int getmin(int * pvalue)
{
　　　　int i=0,j;

　　　　for(j=1;j<10;j++)
　　　　{
　　　　　　　　while(pvalue[i] > pvalue[j]){i++;}
　　　　}

　　　　return pvalue[i];
}

int getmax(int * pvalue)
{
　　　　int i=0,j;

　　　　for(j=1;j<10;j++)
　　　　{
　　　　　　　　while(pvalue[i] < pvalue[j]){i++;}
　　　　}

　　　　return pvalue[i];
}

正常に作動はしましたが、
記述に無駄が多い、分かりづらい
などあればご指導いただけますか？
よろしくお願いします。

回答No.1

maiko0318 · ベストアンサー

０から１００の範囲外の数字を２回続けて入力したらどうなりますか？
１，２，３，４，ー１，ー１，５，６，７，８，９というふうに。

while(pvalue[i] > pvalue[j]){i++;}

ここはループでなくてif文でいいですよ。

while(pvalue[i] < pvalue[j]){i++;}

ここもif文にしましょう。

お礼

みなさま様々な回答を下さり、どの方もとても参考になりました。
ですが、ここは僕のレベルに一番合い、なおかつ簡潔に説明してくださった
maiko0318様をBAに選ばせて頂きました。

なお、お礼、質問の締切が大変遅くなりまして申し訳ありませんでした。

回答No.6

syampoooooooooo

// 実際に書いた方が分かりやすいと思って適当に
// 記述が多いとか分かりづらいみたいなのを気にする以前なので気にしなくていい
// ほとんどコピペ
// コンパイルすら通してないので変な書き間違えがあるかも

#include <stdio.h>

#define HOWMANY 10 // いくつ数値を入力させるか ;複数の箇所で使われる同じ意味の定数は定義しておくべき

int getmin(int * pvalue);
int getmax(int * pvalue);

int main(void)
{
　　　　int i, value[HOWMANY];
　　　　int minimum, maximum;

　　　　printf("0~100の範囲で%d個の数値を入力してください：\n", HOWMANY);

// 0～100でなかったら何度でも駄目だしを食らうようにした
　　　　for(i=0;i<HOWMANY; )
{
　　　　　　　　scanf("%d",&value[i]);
　　　　　　　　if( value[i] < 0 || value[i] > 100 )
　　　　　　　　{
　　　　　　　　　　　printf("0~100の範囲です：\n");
continue;
　　　　　　　　}
++i;
　　　　}
　　　　minimum = getmin(value);
　　　　maximum = getmax(value);
　　　　printf("最小値は%d\n", minimum);
　　　　printf("最大値は%d\n", maximum);
　　　　return 0;
}

// なんかよく分かんなかったので分かりやすくした
int getmin(int * pvalue)
{
　　　　int j = 0;
int min = pvalue[j];

　　　　for(j=1;j<HOWMANY;j++)
　　　　{
if(pvalue[j] < min)
min = pvalue[j];
　　　　}

　　　　return min;
}

int getmax(int * pvalue)
{
　　　　int j = 0;
int max = pvalue[j];

　　　　for(j=1;j<HOWMANY;j++)
　　　　{
if(pvalue[j] > max)
max = pvalue[j];
}

　　　　return max;
}

回答No.5

TooManyBugs

人に見せるのなら何をしたいのか、しているのかコメントをつけるべき。

回答No.4

tsunji

数字入力が0～100以外を入れても動くのがダメ。

数字入力時の　for(i=0;i<10;i++)　は
whileにしてiの数字は数字が正常なら＋1してi<10ならループ。

999を入れるとプログラム終了するとかしたら親切。

回答No.3

maiko0318

追加＞
while(pvalue[i] > pvalue[j]){i++;}
while(pvalue[i] < pvalue[j]){i++;}

はデータによっては無限ループしますよ。
昇順データ、降順データを入れてみてください。

回答No.2

Tacosan

そして「a」とか入れられて頭痛がするわけですね＞#1.

printf のメッセージは意味が分からない. 例えば #1 のように -1 を入れたりすると
「0~100の範囲です」
と表示してから再入力させてる. でも, -1 は「0~100の範囲」にはないよね.