#include<stdio.h> int main() { int a[10]={0}; int cnt; int max=a[0],min=a[0]; int pstx,pstm; int temp; printf("Input 10 numbers:"); for(cnt=0;cnt<10;cnt++) { scanf("%d",&a[cnt]); } for(cnt=0;cnt<10;cnt++) { if(a[cnt]>max) { max=a[cnt]; pstx=cnt; } else if(a[cnt]<min) { min=a[cnt]; pstm=cnt; } } printf("Max element position:%d,Max element:%d\n",pstx,max); printf("Min element position:%d,Min element:%d\n",pstm,min); temp=a[pstx]; a[pstx]=a[pstm]; a[pstm]=temp; printf("Exchange results:"); for(cnt=0;cnt<10;cnt++) { printf("%4d",a[cnt]); } return 0; }这个代码无输出

这段代码的功能是输入10个数字，找出其中的最大值和最小值，并将最大值和最小值的位置交换，最后输出交换后的结果。

但是，代码中存在一个问题，就是在初始化最大值和最小值时，直接将它们赋值为`a[0]`，但数组`a`中的所有元素都是0。因此，在寻找最大值和最小值时，不会正确地判断。

如果要解决这个问题，可以将最大值和最小值的初始化放在输入数字之后进行，例如：

printf("Input 10 numbers:");
for(cnt=0;cnt<10;cnt++) {
    scanf("%d",&a[cnt]);
}
max = a[0];
min = a[0];

这样就可以正确地找出最大值和最小值，并完成交换操作，输出结果。

相关问题

#include <stdio.h> #include <stdlib.h> int main() { char a[100000],b[100000],c[100000]; int i=0,sum=0,q=0,j; gets(a); for(i=0;a[i]!='\0';i++){ b[i]=a[i]; sum++; } for(i=0;i<sum;i++){ for(j=0;j<sum;j++){ if(a[i]==b[j]){ q++; } } c[i]=q; q=0; } for(i=0;i<sum;i++){ if(c[i]==1){ printf("%c",a[i]); break; }else{ printf("no");} } return 0; }怎么修改优化？

在优化代码之前，先解释一下原来代码的问题：

1. 没有对数组进行初始化，导致变量c中可能出现垃圾值，影响程序的正确性。
2. 在第三个循环中，每次遍历都会输出“no”，导致输出结果不正确。
3. 使用gets函数存在安全问题，应该使用更安全的fgets函数。

下面是修改后的代码，已经进行了一些优化：

#include <stdio.h>
#include <string.h>

#define MAX_LEN 100000

int main() {
    char a[MAX_LEN], b[MAX_LEN];
    int cnt[256] = {0}; // 初始化为0
    fgets(a, MAX_LEN, stdin); // 使用fgets函数
    int len = strlen(a);
    for (int i = 0; i < len; i++) {
        b[i] = a[i];
        cnt[a[i]]++; // 统计每个字符出现的次数
    }
    for (int i = 0; i < len; i++) {
        if (cnt[a[i]] == 1) { // 找到第一个出现次数为1的字符
            printf("%c\n", a[i]);
            return 0;
        }
    }
    printf("no\n"); // 如果没有找到，输出no
    return 0;
}

优化后的代码使用了数组cnt来记录每个字符出现的次数，避免了使用嵌套循环的方法。同时，使用fgets函数可以更安全地读入字符串，并且在最后输出结果时只输出了找到的第一个符合条件的字符，避免了输出多余的no。

#include<stdio.h> using namespace std; #include<cstring> #include <string.h> #include <stdbool.h> #pragma warning(disable:4996) int main() { int A[2005], Z[105] = {0}; int T, i, j, k, N, x,m; scanf("%d %d", &N, &x); for (i = 0; i < N; i++) { scanf("%d", A+i); } for (i = 0; i < N; i++) { A[i]--; } scanf("%d", &T); for (i = 0; i < T; i++) { memset(Z, 0, sizeof(Z)); scanf("%d %d", &k, &m); for (j=0,k--, m--; k <= m; k++) Z[A[k]] ? 0 : ((Z[A[k]] = 1), j++); printf("%d\n", j); } return 0; }

这段代码的问题不太明显，但是有几个可以进行优化的地方：

1. 删除无用的头文件：这段代码中包含了很多无用的头文件，可以删除它们，以减少编译时间和代码冗余。

2. 减少重复的代码：这段代码中出现了两次对输入数组 A 进行遍历的循环，可以将它们合并为一次。

3. 使用 const 关键字：如果数组 A 不需要被修改，可以将其声明为 const 数组，以提高代码的可读性和安全性。

4. 简化判断条件：在循环中进行判断时，可以将判断语句简化为更简洁的形式，以提高代码的可读性和效率。

5. 避免使用 magic number：代码中出现了一些硬编码的数字，可以使用 const 变量或宏定义来代替，以提高代码的可维护性。

下面是进行优化后的代码：

#include <cstdio>
#include <cstring>

const int MAX_N = 2005;
const int MAX_M = 105;

int main() {
    int A[MAX_N], Z[MAX_M] = {0};
    int T, N, x;
    scanf("%d %d", &N, &x);
    for (int i = 0; i < N; i++) {
        scanf("%d", &A[i]);
        A[i]--;  // 将 A 数组中的元素都减 1
    }
    scanf("%d", &T);
    for (int i = 0; i < T; i++) {
        memset(Z, 0, sizeof(Z));
        int k, m;
        scanf("%d %d", &k, &m);
        int cnt = 0;
        for (int j = k - 1; j <= m - 1; j++) {
            if (!Z[A[j]]) {
                Z[A[j]] = 1;
                cnt++;
            }
        }
        printf("%d\n", cnt);
    }
    return 0;
}

这样修改后的代码更加简洁明了，可读性和可维护性都有所提高。