好吧……调试的时候说已停止工作是什么情况= =

[Graphemer]

题目是1. 输入n及n个整数(n<20)，然后按奇数升序、偶数降序排列输出。

是程序出了什么问题么= =

[Graphemer]

引用 1 楼 mcmay 的回复:

请不要用图片上传代码，不方便这里的网友们帮你测试。你先试试将所有循环中的 x = 1; x <=N （这里x代表你所定义的变量），换成x = 0; x < N。如果你用x = 1的形式，数组的第一个元素即ar[0]（这里的ar代表你所定义的数组名称）就被忽略掉了，而用x <= N的形式，则又超过了数组下表的最大值，是越界了。一个数组，例如，ar[6]，其元素个数为6，其元素下标的范围为0~5。另外，不明白，你为什么要在声明数组时用n[N+1]，odd[N+1]和even[N+1]的形式呢？那个+1起什么作用？是怕后面的x<=N造成数组下标越界吗？也许这个办法可以解决这个问题，但始终数组的第一个元素，即ar[0]由于你在for循环中用了x=1这样的表达式而被跳过去了，即ar[0]是未被赋值的，它的值是不确定的，你的程序也许会在运行时因此而崩溃。

#include <stdio.h>
int main()
{
int i,j,N,k,m,l,temp,a;
k=1;
j=1;
printf("请输入数字个数N（小于20）=");
scanf("%d",&N);
printf("N=%d\n",N);
int n[N+1];
int odd[N+1];
int even[N+1];
for(i=1;i<=N;i++)
{
printf("n[%d]=",i);
scanf("%d",&n[i]);
}
for(i=1;i<=N;i++)
{
if(n[i]%2==0)
{
even[j]=n[i];
j++;
}
else
{
odd[k]=n[i];
k++;
}
m=j;
l=k;
}
for(a=1;a<=m;a++)
{
for(j=1;j<=m;j++)
{
if(even[j]<even[j+1])
temp=even[j];
even[j]=even[j+1];
even[j+1]=temp;
}
}
for(a=1;a<=l;a++)
{
for(k=1;k<=l;k++)
{
if(odd[k]>odd[k+1])
temp=odd[k];
odd[k]=odd[k+1];
odd[k+1]=temp;
}
}
printf("odd=");
for(k=1;k<=l;k++)
{
printf("%d ",odd[k]);
}
printf("\n");
printf("even=");
for(j=1;j<=m;j++)
{
printf("%d ",even[j]);
}
return 0;
}



谢谢大神……我稍微修改了一下可以执行……
但是运算结果是错的…………我也不知道错哪里了……麻烦大神帮忙看一下……

这个是按照=0 <N的方式改过后的结果


这个是我原来的测试结果……

= =
我那个+1是为了防止越界的……还有数组定义似乎是不能用变量的……我在VS2013里输入一遍似乎是说int [N+1],或者int [N]这种形式是错的……不知道为什么在C-FREE 5中可以使用


老师要求是输出结果和题目完全一致……由于题目中第一个是n[1]……所以我只能想办法跳过n[0]了

[Graphemer]

#include <stdio.h>
int main()
{
	int i,j,N,k,m,l,temp,a;
	k=1;
	j=1;
	printf("请输入数字个数N（小于20）=");
	scanf("%d",&N);
	printf("N=%d\n",N);
	int n[N+1];
	int odd[N+1];
	int even[N+1];
	for(i=1;i<=N;i++)
	{
		printf("n[%d]=",i);
		scanf("%d",&n[i]);
	}
	for(i=1;i<=N;i++)
	{
		if(n[i]%2==0)
		{
			even[j]=n[i];
		    j++;
		}
		else
		{
			odd[k]=n[i];
		    k++;
		}
		m=j;
		l=k;
	}
	for(a=1;a<=m;a++)
	{
		for(j=1;j<=m;j++)
		{
			if(even[j]<even[j+1])
			temp=even[j];
			even[j]=even[j+1];
			even[j+1]=temp;
		}
	}
	for(a=1;a<=l;a++)
	{
		for(k=1;k<=l;k++)
		{
			if(odd[k]>odd[k+1])
			temp=odd[k];
			odd[k]=odd[k+1];
			odd[k+1]=temp;
		}
	}
	printf("odd=");
	for(k=1;k<=l;k++)
	{
		printf("%d  ",odd[k]);
	}
	printf("\n");
	printf("even=");
	for(j=1;j<=m;j++)
	{
		printf("%d  ",even[j]);
	}
return 0;
}

[mcmay]

用变量定义数组的大小是在C99标准下可以接受的，你找个兼容C99标准的编译器试试，或者打开编译器的C99支持开关（如果有这项设置的话）。

[mcmay]

题目中说要用从n1的形式输出，你可以用 for(k = 0; k < l; k++) printf("%d ", odd[k+1]);的形式即可。

[mcmay]

请不要用图片上传代码，不方便这里的网友们帮你测试。你先试试将所有循环中的 x = 1; x <=N （这里x代表你所定义的变量），换成x = 0; x < N。如果你用x = 1的形式，数组的第一个元素即ar[0]（这里的ar代表你所定义的数组名称）就被忽略掉了，而用x <= N的形式，则又超过了数组下表的最大值，是越界了。一个数组，例如，ar[6]，其元素个数为6，其元素下标的范围为0~5。另外，不明白，你为什么要在声明数组时用n[N+1]，odd[N+1]和even[N+1]的形式呢？那个+1起什么作用？是怕后面的x<=N造成数组下标越界吗？也许这个办法可以解决这个问题，但始终数组的第一个元素，即ar[0]由于你在for循环中用了x=1这样的表达式而被跳过去了，即ar[0]是未被赋值的，它的值是不确定的，你的程序也许会在运行时因此而崩溃。

[Graphemer]

引用 5 楼 mcmay 的回复:

题目中说要用从n1的形式输出，你可以用 for(k = 0; k < l; k++) printf("%d ", odd[k+1]);的形式即可。

嗯……但是输出结果还是和上面两个运行框一样