[已解决]帮忙看到题, 整数处理的, 总是通不过测试.

七彩云南

1. /*
   
2. *  输入10个整数，将其中最小的数与第一个数对换，把最大的数与最后一个数对换。
   
3. *  写三个函数； ①输入10个数；②进行处理；③输出10个数。
   
4. *
   
5. */
   
6. 
   
7. #include <stdio.h>
   
8. 
   
9. #define LEN 10
   
10. 
    
11. 
    
12. void GetNumber(int number[])
    
13. {
    
14.       for (int i = 0; i < LEN; i++)
    
15.       {
    
16.             scanf("%d", &number[i]);
    
17.       }
    
18. }
    
19. 
    
20. void Swap(int number[])
    
21. {
    
22.       int max, min, temp;
    
23.       int n, i;
    
24. 
    
25.       max = min = number[0];
    
26.       for (i = 0; i < LEN; i++)
    
27.       {
    
28.             if (max < number[i])
    
29.             {
    
30.                   max = number[i];
    
31.                   n = i;
    
32.             }
    
33.       }
    
34.       temp = number[n];
    
35.       number[n] = number[LEN-1];
    
36.       number[LEN-1] = temp;
    
37. 
    
38. 
    
39.       n = 0;
    
40.       for (i = 0; i < LEN; i++)
    
41.       {
    
42.             if (min > number[i])
    
43.             {
    
44.                   min = number[i];
    
45.                   n = i;
    
46.             }
    
47.       }
    
48.       temp = number[n];
    
49.       number[n] = number[0];
    
50.       number[0] = temp;
    
51. }
    
52. 
    
53. 
    
54. void Show(int number[])
    
55. {
    
56.       for (int i = 0; i < LEN; i++)
    
57.       {
    
58.             printf("%d ", number[i]);
    
59.       }
    
60. }
    
61. 
    
62. 
    
63. int main(void)
    
64. {
    
65.       int number[LEN];
    
66. 
    
67.       GetNumber(number);
    
68.       Swap(number);
    
69.       Show(number);
    
70. 
    
71. 
    
72.       return 0;
    
73. }
    

复制代码

上面这段代码我运行了很多次, 包括能想到的特殊值, 结果我觉得都对, 但就是不能通过测试, 请帮我看看问题出在哪里?  我就想知道有什么特殊值能得到错误的结果.

最佳答案

月排行榜 / 总排行榜

sfqxx

2023-7-14 22:47:00

你的代码中有一个问题：在 `Swap` 函数里，当找到最大值或最小值时，没有对变量 `n` 进行初始化，这样可能导致 `n` 的值是未定义的，从而交换出错。建议在函数开始时对 `n` 进行初始化，如 `int n = 0;`。这样就能保证在找到最大值或最小值之前，`n` 的值一定是合法的。
另外，你可以尝试输入一些边界值，比如全是正数、全是负数、正负数混合、有相同的数等等，以确保程序能够正确处理这些情况。
这是修改后的代码：

1. #include <stdio.h>
   
2. #define LEN 10
   
3. void GetNumber(int number[]) {
   
4.     for (int i = 0; i < LEN; i++) {
   
5.         scanf("%d", &number[i]);
   
6.     }
   
7. }
   
8. void Swap(int number[]) {
   
9.     int max = number[0], min = number[0], max_index = 0, min_index = 0;
   
10.     for (int i = 1; i < LEN; i++) {
    
11.         if (number[i] > max) {
    
12.             max = number[i];
    
13.             max_index = i;
    
14.         }
    
15.         if (number[i] < min) {
    
16.             min = number[i];
    
17.             min_index = i;
    
18.         }
    
19.     }
    
20.     int temp = number[0];
    
21.     number[0] = min;
    
22.     number[min_index] = temp;
    
23.     temp = number[LEN - 1];
    
24.     number[LEN - 1] = max;
    
25.     number[max_index] = temp;
    
26. }
    
27. void Show(int number[]) {
    
28.     for (int i = 0; i < LEN; i++) {
    
29.         printf("%d ", number[i]);
    
30.     }
    
31. }
    
32. int main(void) {
    
33.     int number[LEN];
    
34.     GetNumber(number);
    
35.     Swap(number);
    
36.     Show(number);
    
37.     return 0;
    
38. }

复制代码

主要修改了 `Swap` 函数。在该函数中，我们使用了两个变量 `max` 和 `min` 分别表示数组中的最大值和最小值，并用 `max_index` 和 `min_index` 记录它们的下标。然后交换数组中的元素即可。
另外，我们在输入数据时没有进行错误处理，如果输入非法数据可能会导致程序出错。你可以加上输入数据的错误处理，确保输入的数据是合法的。

跳转到最佳答案楼层

sfqxx

你的代码中有一个问题：在 `Swap` 函数里，当找到最大值或最小值时，没有对变量 `n` 进行初始化，这样可能导致 `n` 的值是未定义的，从而交换出错。建议在函数开始时对 `n` 进行初始化，如 `int n = 0;`。这样就能保证在找到最大值或最小值之前，`n` 的值一定是合法的。
另外，你可以尝试输入一些边界值，比如全是正数、全是负数、正负数混合、有相同的数等等，以确保程序能够正确处理这些情况。
这是修改后的代码：

1. #include <stdio.h>
   
2. #define LEN 10
   
3. void GetNumber(int number[]) {
   
4.     for (int i = 0; i < LEN; i++) {
   
5.         scanf("%d", &number[i]);
   
6.     }
   
7. }
   
8. void Swap(int number[]) {
   
9.     int max = number[0], min = number[0], max_index = 0, min_index = 0;
   
10.     for (int i = 1; i < LEN; i++) {
    
11.         if (number[i] > max) {
    
12.             max = number[i];
    
13.             max_index = i;
    
14.         }
    
15.         if (number[i] < min) {
    
16.             min = number[i];
    
17.             min_index = i;
    
18.         }
    
19.     }
    
20.     int temp = number[0];
    
21.     number[0] = min;
    
22.     number[min_index] = temp;
    
23.     temp = number[LEN - 1];
    
24.     number[LEN - 1] = max;
    
25.     number[max_index] = temp;
    
26. }
    
27. void Show(int number[]) {
    
28.     for (int i = 0; i < LEN; i++) {
    
29.         printf("%d ", number[i]);
    
30.     }
    
31. }
    
32. int main(void) {
    
33.     int number[LEN];
    
34.     GetNumber(number);
    
35.     Swap(number);
    
36.     Show(number);
    
37.     return 0;
    
38. }

复制代码

主要修改了 `Swap` 函数。在该函数中，我们使用了两个变量 `max` 和 `min` 分别表示数组中的最大值和最小值，并用 `max_index` 和 `min_index` 记录它们的下标。然后交换数组中的元素即可。
另外，我们在输入数据时没有进行错误处理，如果输入非法数据可能会导致程序出错。你可以加上输入数据的错误处理，确保输入的数据是合法的。

isdkz

你的代码逻辑有些许问题。在你的  Swap()  函数中，你先找到最大值并与数组最后一个元素交换，然后再找到最小值并与数组第一个元素交换。但是，这里存在一个问题，如果你的数组中第一个数是最大的，那么在第二次查找最小值的时候，你会错误地把这个最大值当作最小值。

例如，对于这样的输入： 10 2 3 4 5 6 7 8 9 1 ，你的程序会输出： 10 2 3 4 5 6 7 8 9 1 ，这显然是错误的。

你需要把查找最小值和最大值的步骤分开来，分别在两个循环中完成，这样就不会有上述问题了。下面是修改后的代码：

1. #include <stdio.h>
   
2. 
   
3. #define LEN 10
   
4. 
   
5. void GetNumber(int number[])
   
6. {
   
7.       for (int i = 0; i < LEN; i++)
   
8.       {
   
9.             scanf("%d", &number[i]);
   
10.       }
    
11. }
    
12. 
    
13. void Swap(int number[])
    
14. {
    
15.       int max, min, temp;
    
16.       int nMax, nMin, i;
    
17. 
    
18.       max = min = number[0];
    
19.       nMax = nMin = 0;
    
20.       
    
21.       for (i = 0; i < LEN; i++)
    
22.       {
    
23.             if (max < number[i])
    
24.             {
    
25.                   max = number[i];
    
26.                   nMax = i;
    
27.             }
    
28. 
    
29.             if (min > number[i])
    
30.             {
    
31.                   min = number[i];
    
32.                   nMin = i;
    
33.             }
    
34.       }
    
35. 
    
36.       temp = number[nMax];
    
37.       number[nMax] = number[LEN-1];
    
38.       number[LEN-1] = temp;
    
39. 
    
40.       if(nMin == LEN-1) nMin = nMax; // 如果最小值原来是最后一个数，则现在应该是刚刚交换过的位置，即最大值原来的位置。
    
41.       temp = number[nMin];
    
42.       number[nMin] = number[0];
    
43.       number[0] = temp;
    
44. }
    
45. 
    
46. void Show(int number[])
    
47. {
    
48.       for (int i = 0; i < LEN; i++)
    
49.       {
    
50.             printf("%d ", number[i]);
    
51.       }
    
52. }
    
53. 
    
54. int main(void)
    
55. {
    
56.       int number[LEN];
    
57. 
    
58.       GetNumber(number);
    
59.       Swap(number);
    
60.       Show(number);
    
61. 
    
62.       return 0;
    
63. }
    

复制代码

这段代码可以正确地找到并交换最大值和最小值。希望这能解决你的问题！

歌者文明清理员

sfqxx 发表于 2023-7-14 22:47
你的代码中有一个问题：在 `Swap` 函数里，当找到最大值或最小值时，没有对变量 `n` 进行初始化，这样可能 ...

反对陶远航从我做起

陶远航

歌者文明清理员 发表于 2023-7-14 22:57
反对陶远航从我做起

啊对对对

七彩云南

多谢  sfqxx , 如你所说, 加了一句

n = 0;

果然过了测试.


isdkz, 你的代码比我写的精巧, 获益良多.

sfqxx

七彩云南 发表于 2023-7-15 15:00
多谢  sfqxx , 如你所说, 加了一句

n = 0;