学生给我发了私信,一个程序运行了好久,在OJ就是提交不了。
题目是:
**Description**
输入10个整数,将其中最小的数与第一个数对换,把最大的数与最后一个数对换。写三个函数; ①输入10个数;②进行处理;③输出10个数。
**Input**
10个整数
**Output**
整理后的十个数,每个数后跟一个空格(注意最后一个数后也有空格)
**Sample Input**
2 1 3 4 5 6 7 8 10 9
**Sample Output**
1 2 3 4 5 6 7 8 9 10
**HINT**
主函数已给定如下,提交时不需要包含下述主函数
~~~
/* C/C++代码 */
int main()
{
const int n=10;
int a[n];
input(a,n);
handle(a,n);
output(a,n);
return 0;
}
~~~
学生的解答:
~~~
#include<iostream>
#include<cstdio>
using namespace std;
void input(int a[],int);
void handle(int a[],int);
void output(int a[],int);
int main()
{
const int n=10;
int a[n];
//freopen("input.txt","r",stdin);
input(a,n);
handle(a,n);
output(a,n);
return 0;
}
void input(int a[],int n)
{
for(int i=0;i<n;i++)
cin>>a[i];
}
void handle(int a[],int n)
{
int max,i=0,t,min,z,zd,zx;
max=a[i];
min=a[i];
for(i=0;i<n;i++)
{
if(max<a[i])
{
max=a[i];
zd=i; //记录最大值的位置
}
if(min>a[i])
{
min=a[i];
zx=i; //记录最小值的位置
}
}
t=a[9];
a[9]=max;
a[zd]=t; //进行值交换
z=a[0];
a[0]=min;
a[zx]=z;
}
void output(int a[],int n)
{
for(int i=0;i<n;i++)
{
cout<<a[i]<<" ";
}
}
~~~
这个解答在CodeBlocks中编译通过,屡经测试数据,没有异常,然而,提交后提示“Runtime error”,具体是:
Runtime Error:Segmentation fault
辅助解释:Segmentation fault:段错误,检查是否有数组越界,指针异常,访问到不应该访问的内存区域
这不科学!由“一定是我错了”的思维,转向了“可能是机器错了”,利用管理员帐号,看了测试数据,也没有问题。
再逐句看程序,一个地方引起我的注意,会是25、26行引用a[i],但 i 没有赋值吗?不是,i 的初值是0,于算法没有问题。就在第24行,zd、zx两个变量,也应该是同步赋值的,它们表示最大、最小的数的下标,后面循环中,也及时对这两个数做了更新。
问题逐渐明白了,在24行不给zd、zx赋初值的情况下,如果29和34行的if条件有一个始终未能为真,zd、zx就不会赋值,40行开始的交换,就会出现数组越界。
** 在这个程序上小修改的办法有了:第24行,加上给zd、zx赋初值,即:int max,i=0,t,min,z,zd=0,zx=0;**
这个程序,没有用上单步执行。对这种小几率的问题,单步可能也不管用,思维要严密。
这个程序写得稍显罗嗦,下面给出一个改写版。
~~~
#include<iostream>
#include<cstdio>
using namespace std;
void input(int a[],int);
void handle(int a[],int);
void output(int a[],int);
int main()
{
const int n=10;
int a[n];
//freopen("input.txt","r",stdin);
input(a,n);
handle(a,n);
output(a,n);
return 0;
}
void input(int a[],int n)
{
for(int i=0;i<n;i++)
cin>>a[i];
}
void handle(int a[],int n)
{
int i,t,zd=0,zx=0;//默认第0个为最大、最小
for(i=1;i<n;i++) //从第1个开始
{
if(a[zd]<a[i])
zd=i; //记录最大值的位置
if(a[zx]>a[i])
zx=i; //记录最小值的位置
}
t=a[n-1]; //比用含有“神秘数”的a[9]好
a[n-1]=a[zd];
a[zd]=t; //进行值交换
t=a[0];
a[0]=a[zx];
a[zx]=t;
}
void output(int a[],int n)
{
for(int i=0;i<n;i++)
{
cout<<a[i]<<" ";
}
}
~~~