请大家来找茬！！！

lili200238

1. /*
   
2. 题目：对 10 个数进行排序
   
3. 1.程序分析：可以利用选择法，即从后 9 个比较过程中，选择一个最小的与第一
   
4. 个元素交换，下次类推，即用第二个元素与后 8 个进行比较，并进行交换。
   
5. */
   
6. 
   
7. #include<stdio.h>
   
8. int swap(int a[],int b[]);
   
9. 
   
10. int main(void)
    
11. {
    
12.         int i, j;
    
13.         int a[10] = { 9 , 8 , 7 , 6 , 5 , 4 , 3 , 2 , 1 , 0 };
    
14.        
    
15.         for( i=0 ; i<10 ; i++)
    
16.         {
    
17.                 for( j=i+1 ; j<10 ; j++)
    
18.                 {
    
19.                         swap(a[i],a[j]);
    
20.                 }
    
21.         }
    
22.        
    
23.         for( i=0 ; i<10 ; i++)
    
24.         {
    
25.                 printf("%5d\n",a[i]);
    
26.         }
    
27. }
    
28. 
    
29. int swap(int a[],int b[])
    
30. {
    
31.         int i, j , temp;
    
32.         if( a[i]>b[j] )
    
33.         {
    
34.                 temp = a[i] ;
    
35.                 a[i] = b[j] ;
    
36.                 b[j] = temp ;
    
37.         }
    
38. 
    
39.         return 0;
    
40. }

复制代码

想用函数调用来实现题目功能，奈何功力不够，请有识之士出来指正题目中的错误，该怎样改正？

错误提示截图：

Whitehat_K

1. /*
   
2. 题目：对 10 个数进行排序
   
3. 1.程序分析：可以利用选择法，即从后 9 个比较过程中，选择一个最小的与第一
   
4. 个元素交换，下次类推，即用第二个元素与后 8 个进行比较，并进行交换。
   
5. */
   
6. 
   
7. #include<stdio.h>
   
8. void swap(int *a,int *b);
   
9. 
   
10. int main(void)
    
11. {
    
12.         int i, j;
    
13.         int a[10] = { 9 , 8 , 7 , 6 , 5 , 4 , 3 , 2 , 1 , 0 };
    
14.         
    
15.         for( i=0 ; i < 10 ; i++)
    
16.         {
    
17. 
    
18.                         for( j = i+1; j < 10; j++ )
    
19.                         {
    
20.                                 swap(&a[i], &a[j]);
    
21.                         }
    
22.         }
    
23.         
    
24.         for( i=0 ; i<10 ; i++)
    
25.         {
    
26.                 printf("%5d\n",a[i]);
    
27.         }
    
28. }
    
29. 
    
30. 
    
31. void swap(int *a, int *b)
    
32. {
    
33.         int temp;
    
34.         temp = *a;
    
35.         *a = *b;
    
36.         *b = temp;
    
37. }
    
38. 
    
39. 
    
40. #if(0)
    
41. int swap(int a[],int b[])//另写交换函数要改变多个值为什么不试试指针传递参数
    
42. {
    
43.         int i, j, temp;
    
44.         if( a[i] > b[j] )       
    
45.         //warning C4700: local variable 'i' used without having been initialized
    
46.         //warning C4700: local variable 'j' used without having been initialized
    
47.         //这里的i,j是swap函数中的局部变量只定义没赋值;还有这里的i,j和main函数中的i,j不相同
    
48.         //swap函数这里的a[i],a[j]并不会改变main中的数组值
    
49.         {      
    
50.                 temp = a[i] ;
    
51.                 a[i] = b[j] ;
    
52.                 b[j] = temp ;
    
53.         }
    
54. 
    
55.         return 0;//void swap()就可以不写
    
56. }
    
57. #endif

复制代码

代码农民

发现的错误:
1：
   你的main函数是个例程（过程函数，无返回值），所以第10行的int应改为void。
2：
   根据你写的swap函数的原型（第29行），swap形参应接收的是地址（指针），所以第19行两个参数前应加取址运算符&，即swap(&a[I][i][i],&a[j]);//也可以写成指针运算型swap(a+i,a+j);//
3：
    在你写的swap函数的原型（第29行开始）里，32行的i,j没有始初化，a[I][i],a[j]指向谁？

修改的建议:
1：重写swap函数，定义为例程（void型），用于两个操作数直接交换。
2：在第17行swap前函数前比较a和a[j][/i][/i][/i]

lili200238

Whitehat_K 发表于 2016-10-25 19:48

19行前插入if(a[i]>a[j])程序应该通用性好点。

谢谢您的指点

lili200238

代码农民 发表于 2016-10-26 22:24
发现的错误:
1：
   你的main函数是个例程（过程函数，无返回值），所以第10行的int应改为void。

谢谢您