下面是我写的排序程序,选择和冒泡,但是程序运行出来的结果不是预期的,哪错了??
#include<stdio.h>
#define n 10

void  xuanze(int a[])
{ int i,j,p,q;
  for(i=0;i<n;i++)
   {
    p=a[i];q=i;
    for(j=i+1;j<=n;j++)
     {
      if(a[j]<p)
        {p=a[j];q=j;}
      a[q]=a[i];
      a[i]=p;
     }
   }
  printf("\nthe result of xuanze is:");
   for(i=0;i<n;i++)
    printf("%d ",a[i]);
}

void  maopao(int a[])
{
  int i,j,t;
  for(i=n-1;i>=0;i--)
    for(j=0;j<=i;j++)
     if(a[j]>a[j+1])
      {t=a[j];
       a[j]=a[j+1];
       a[j+1]=t;
      }
  printf("\nthe result of maopao is:");
  for(i=0;i<n;i++)
   printf("%d ",a[i]);
}
void main()
{ int i;
  int a[10];
  printf("\nPlease input 10 numbers:\n ");
  for(i=0;i<n;i++)
   scanf("%d",&a[i]);
  xuanze(a);
  maopao(a);
  getch();
}

这样：

#include <stdio.h>
#include <stdlib.h>
#define N 10

void xuanze(int a[])
{
        int ix, i;
        for (ix = 0; ix != N; ++ix)
        {
                for (i = ix + 1; i != N; ++i)
                {
                        if (a[i] > a[ix])
                        {
                                int temp = a[ix];
                                a[ix] = a[i];
                                a[i] = temp;
                        }
                }
        }
        for (ix = 0; ix != N; ++ix)
                printf("%d ", a[ix]);
        putchar('\n');
}

void maopao(int a[])
{ 
        int ix;
        for (ix = 0; ix != N-1; ++ix)
        {
                if (a[ix+1] > a[ix])
                {
                        int temp = a[ix];
                        a[ix] = a[ix+1];
                        a[ix+1] = temp;
                }
        }
        for (ix = 0; ix != N; ++ix)
                printf("%d ", a[ix]);
        putchar('\n');
}

int main()
{
        int i, a[10];
        printf("Please input 10 numbers:\n");
        for(i=0; i<N; i++)
                scanf("%d", &a[i]);
        xuanze(a);
        maopao(a);
        system("pause");
        return 0;
}

[[italic] 本帖最后由 随心 于 2008-1-9 19:46 编辑 [/italic]]

谢谢,不过,我还是不懂我写的程序是哪错了呢??????

你的冒泡是对的，问题出在你的选择上。
你的数组让选择给破坏了

收到的鲜花

• lemon6226 于 2008-01-09 20:30 送鲜花  2朵  

看看你的xuanze函数里的if语句，这句话本来的作用是：如果后面的元素比前面的值小，则交换（按你的意思）。
你可以试着改一下。

[[italic] 本帖最后由 随心 于 2008-1-9 20:06 编辑 [/italic]]

void  xuanze(int a[])
{ int i,j,p,q;
  for(i=0;i<n;i++)
   {
    p=a[i];q=i;
    for(j=i+1;j<=n;j++)    // 下标会溢出，引入随机数！所以出错
     {
      if(a[j]<p)
        {p=a[j];q=j;}
      a[q]=a[i];                  // 把交换放在循环内，已经不是排序了，把这两条语句放到外层循环即可
      a[i]=p;
     }
   }
  printf("\nthe result of xuanze is:");
   for(i=0;i<n;i++)
    printf("%d ",a[i]);
}

呵呵，终于改出来了,谢谢你们了