实现用户输入数组双最大元素替换功能遇阻，求代码问题排查

原代码的核心问题 #

• Max函数返回逻辑错误：C++里return max2,max1;是逗号表达式，只会返回max1，没法同时拿到两个最大值。要返回多个值得用引用、指针或者结构体这类方式。
• 最大值初始化不合理：直接把max1和max2设为0，要是数组全是负数，会把0当成最大值，完全不符合预期。
• 没实现替换逻辑：main函数里根本没调用Max，也没写任何替换两个最大元素的代码，只是做了输入输出。
• 还有个小问题：system("pause>>void")不是跨平台写法，虽然不影响功能，但不太规范。

修正后的代码 #

#include <iostream>
using namespace std;

// 通过引用参数返回两个最大值
void findTwoLargest(double a[], int n, double& max1, double& max2) {
    if (n < 2) {
        cerr << "数组长度至少得有2个元素" << endl;
        return;
    }
    // 用数组前两个元素初始化，保证max1 >= max2
    if (a[0] > a[1]) {
        max1 = a[0];
        max2 = a[1];
    } else {
        max1 = a[1];
        max2 = a[0];
    }

    for (int i = 2; i < n; i++) {
        if (a[i] > max1) {
            max2 = max1;
            max1 = a[i];
        } else if (a[i] > max2 && a[i] != max1) {
            max2 = a[i];
        }
    }
}

// 替换数组中两个最大元素（示例替换为0，可按需修改）
void replaceTwoLargest(double a[], int n) {
    double max1, max2;
    findTwoLargest(a, n, max1, max2);

    int replaceCount = 0;
    for (int i = 0; i < n && replaceCount < 2; i++) {
        if (a[i] == max1 && replaceCount < 2) {
            a[i] = 0;
            replaceCount++;
        } else if (a[i] == max2 && replaceCount < 2) {
            a[i] = 0;
            replaceCount++;
        }
    }
}

int main() {
    setlocale(0, ".1251");
    const int ARRAY_SIZE = 7;
    double a[ARRAY_SIZE];
    cout << "输入7个元素的数组：" << endl;

    for (int i = 0; i < ARRAY_SIZE; i++) {
        cin >> a[i];
    }

    cout << "原数组：";
    for (int i = 0; i < ARRAY_SIZE; i++) {
        cout << a[i] << "\t";
    }
    cout << endl;

    // 执行替换操作
    replaceTwoLargest(a, ARRAY_SIZE);

    cout << "替换两个最大元素后的数组：";
    for (int i = 0; i < ARRAY_SIZE; i++) {
        cout << a[i] << "\t";
    }
    cout << endl;

    return 0;
}

关键修改说明 #

• 改用引用参数返回两个最大值：findTwoLargest函数通过引用把找到的两个最大值传给调用者，这是C++里返回多个值的常用方法。
• 合理初始化最大值：用数组前两个元素比较后初始化，解决了全负数数组的错误问题。
• 新增替换逻辑：replaceTwoLargest专门处理替换操作，示例里把最大的两个元素换成0，你可以自己改成需要的数值。
• 优化遍历逻辑：找最大值时加了a[i] != max1的判断，避免数组有多个相同最大值时的错误；替换时限制只替换前两个找到的最大值，也可以根据需求改成替换所有等于这两个值的元素。
• 去掉了system("pause")，如果需要程序结束前暂停，可在main末尾加cin.get();。

内容的提问来源于stack exchange，提问作者David Kotsiuba