初学者的疑惑关于最大最小值

[evel]

我学习C++中写的一个获取100个随机数，并且返回其最大最小值的代码。大家帮看看为什么求最小值不对？代码有什么问题？谢谢

#include <iostream>
#include <stdlib.h>
using namespace std;

// function for get the max value,min value,and avage value of the random 100 number .
// By Evel

void main()
{
	int how_many = 100;
	int check[100];
	int n=0;
	cout << "Print "<< how_many << " random integers." << endl;
	for (int i =0;i <how_many; ++i){
		check[i]=rand();
		cout << check[i] << '\t';
	}
	cout << endl;
	//cout << check[0] <<endl;
	int maxn = 0, min = check[0];
	while(n<=how_many){
		if(check[n]>maxn)
			maxn=check[n];
		else if(check[0]>check[n])
			min=check[n];
		n++;
	}
	cout << min << endl;
	cout << maxn << endl;	
}

[na2650945]

引用 1 楼 onmyway2010 的回复:

while(n <=how_many)
应该为while(n <how_many)吧？

新手常遇见的问题。
我当时也是。

[OnMyWay2010]

while(n<=how_many)
应该为while(n<how_many)吧？

[pengzhixi]

一楼给出答案了，可以结贴了。

[yshuise]

vector<int> S;
S.push_back(......)

int max= *max_element(S.begin(),S.end());

[joey_zoy]

else if(check[0]>check[n])
min=check[n];
这句逻辑有问题啊，按照你的这个逻辑，假如check[0]最大，那么循环到最后min = check[how_many],而不是取得数组中的最小值

[pengzhixi]

min>check[n]

[WuBill]

#include <iostream>
#include <stdlib.h>
using namespace std;

// function for get the max value,min value,and avage value of the random 100 number .
// By Evel

void main()
{
    int how_many = 100;
    int check[100];
    int n=0;
    cout << "Print "<< how_many << " random integers." << endl;
    for (int i =0;i <how_many; ++i){
        check[i]=rand();
        cout << check[i] << '\t';
    }
    cout << endl;
    //cout << check[0] <<endl;
    int maxn = 0, min = check[0];
    while(n<=how_many){
        if(check[n]>maxn)
            maxn=check[n];
        if(check[n]<min)
            min=check[n];
        n++;
    }
    cout << min << endl;
    cout << maxn << endl;    
}

试下

[yuzl32]

#include <iostream>
#include <stdlib.h>
using namespace std;

// function for get the max value,min value,and avage value of the random 100 number .
// By Evel

void main()
{
    int how_many = 100;
    int check[100];
    int n=0;
    cout << "Print "<< how_many << " random integers." << endl;
    for (int i =0;i <how_many; ++i){
        check[i]=rand();
        cout << check[i] << '\t';
    }
    cout << endl;
    //cout << check[0] <<endl;
    int maxn = 0, min = check[0];
    while(n < how_many){  // 1处！
        if(check[n]>maxn)
            maxn=check[n];
        else if(min>check[n]) //2处
            min=check[n];
        n++;
    }
    cout << min << endl;
    cout << maxn << endl;    
}

[OnMyWay2010]

while(n<how_many)
{
if(check[n] > maxn)
maxn=check[n];
else if(check[n] < min)
min=check[n];
n++;
}

[wesleyluo]

修改下这里：

n=0;
int maxn = min = check[n];
    while(n<=how_many){
        if(check[n]>maxn)
            maxn=check[n];
        else if(min>check[n])
            min=check[n];
        n++;
    }

[Happylessness]

#include <iostream>
#include <stdlib.h>
using namespace std;

// function for get the max value,min value,and avage value of the random 100 number .
// By Evel

void main()
{
    const int how_many = 100;    // 定义成const类型的
    int check[how_many];         // 对数量引用，防止更改变量长度的时候while循环出问题
    int n=0;
    cout << "Print "<< how_many << " random integers." << endl;
    for (int i =0;i <how_many; ++i){
        check[i]=rand();
        cout << check[i] << '\t';
    }
    cout << endl;
    //cout << check[0] <<endl;
    int maxn = 0, min = check[0];
    while(n < how_many){     // =>循环条件订正。
        if(check[n]>maxn)
            maxn=check[n];
        if(min > check[n])  // =>对最小值的判断条件订正
            min=check[n];
        n++;
    }
    cout << min << endl;
    cout << maxn << endl;    
}

菜鸟飘过。。。

[ypb362148418]

引用 1 楼 onmyway2010 的回复:

while(n <=how_many)
应该为while(n <how_many)吧？

结贴吧，LZ

[SiGoYi]

if(check[n]>maxn)
maxn=check[n];
else if(check[0]>check[n])
min=check[n];

这么改
if ( min < check[n] )