动态数组读取无效数据（错误码C6385）：循环逻辑问题求助

问题根源 #

你的代码触发C6385的核心原因有两个：

• 变量冲突：内层for循环复用了外层循环的i变量，导致外层循环的遍历逻辑被彻底打乱，甚至会让i超出数组下标范围，访问无效内存。
• 缺少边界检查：内层循环的条件只判断了元素值的范围，没有限制下标不超过数组长度ar_size，当数组内所有元素都满足a[i]>0 && a[i]<3.14时，下标会一直递增到数组边界之外，读取非法内存。

另外代码存在逻辑设计问题：原本应该先输入完所有数组元素再统一计算，但现在每输入一个元素就从头遍历数组，完全不符合需求逻辑。

修复步骤及修正代码 #

1. 拆分循环变量：内层循环改用独立变量（比如j），避免干扰外层循环的i。
2. 添加边界检查：内层循环必须把j < ar_size作为条件之一，防止越界访问。
3. 调整代码结构：先完成所有数组元素的输入，再单独遍历数组进行计算。
4. 优化变量作用域：把全局变量s、u、h移到main函数内，避免全局变量带来的意外问题。
5. 类型转换：计算cos和sin时，把int类型的数组元素转为double，保证计算精度。
6. 释放动态内存：使用delete[]释放动态分配的数组，避免内存泄漏。

修正后的代码：

#include <iostream>
#include <cmath> // 必须包含该头文件才能使用cos、sin函数
using namespace std;

int main(){
    int ar_size;
    cout << "Enter array size" << endl;
    cin >> ar_size;
    int* a = new int[ar_size];
    double s = 0.0, u, h; // 变量移至main内，初始化s为0

    // 第一步：输入所有数组元素
    for (int i = 0; i < ar_size; i++) {
        cout << "Enter array data for index " << i << endl;
        cin >> a[i];
        cout << "Value of " << i << " element is " << a[i] << endl;
    }

    // 第二步：遍历数组计算s的值
    for (int j = 0; j < ar_size; j++) { // 用独立变量j避免冲突
        if (a[j] > 0 && a[j] < 3.14) {
            double val = static_cast<double>(a[j]); // 转为double保证计算精度
            u = 2 * cos(val);
            h = 1 - 2 * sin(val);
            // 额外判断h是否为0，避免除以0的运行时错误
            if (h != 0) {
                s += u / h;
                cout << "When j = " << j << "\t" << "s = " << s << endl;
            } else {
                cout << "Warning: h is 0 at index " << j << ", skip calculation" << endl;
            }
        }
    }

    delete[] a; // 释放动态分配的内存
    return 0;
}

额外说明 #

• 原代码缺少<cmath>头文件，使用cos和sin会导致编译错误，修正时已补上。
• 添加了h != 0的判断，避免出现除以0的运行时错误，这是原代码忽略的潜在问题。

内容的提问来源于stack exchange，提问作者Artem Biriukov