C++数组插入程序异常：输入值未正确存储，输出随机数求排查

你的程序出现打印随机数的问题，核心是插入逻辑和参数传递的错误，具体问题及修正如下：

主要错误点 #

• insert函数的循环逻辑完全错误：你在insert里写的for循环毫无意义，第一次循环就会把输入值赋值给v[0]，然后立刻elements++并返回true。这导致每次输入的数都会覆盖数组第一个元素，而数组的其他位置始终是未初始化的垃圾值，打印时自然出现随机数。同时这个循环完全违背了"每次插入一个元素"的需求。
• 参数传递逻辑错误：你将main中的number以值传递的方式传给insert的entered_number，但实际上这个参数完全多余——你不需要把main里的变量传进去，直接在insert内部定义变量接收输入即可。
• 全局与局部变量命名冲突：全局定义了maxDim，main里又重新定义同名局部变量，虽然值相同，但容易引发逻辑混乱，不符合代码规范。

修正后的代码 #

#include <iostream>
using namespace std;

const int MAX_DIM = 10;
bool insert(int v[], int &elements);

int main(){
    int array[MAX_DIM];
    int elements = 0;
    
    while(insert(array, elements)){
        cout << "Number entered successfully." << endl;
    }
    
    cout << "Array is composed of: ";
    for (int i = 0; i < elements; i++){
        cout << array[i];
        if (i != elements - 1) cout << ", ";
    }
    cout << endl;
}

bool insert(int v[], int &elements){
    if (elements < MAX_DIM){
        cout << "Enter a number: " << endl;
        int entered_number;
        cin >> entered_number;
        v[elements] = entered_number;
        elements++;
        return true;
    } else {
        cout << "Array is full." << endl;
        return false;
    }
}

修正说明 #

1. 修复插入逻辑：移除多余的for循环，每次插入时直接将输入值放到数组的elements索引位置（这是当前数组的末尾），然后将elements自增，符合"逐个添加元素"的需求。
2. 优化参数设计：删除不必要的entered_number参数，在insert函数内部定义变量接收输入，避免值传递的冗余和混淆。
3. 规范变量定义：用常量MAX_DIM替代全局变量，消除命名冲突，代码可读性和可维护性更强。
4. 优化打印格式：调整循环内的输出逻辑，避免最后一个元素后出现多余的逗号。

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