C++初学者求助：数组反向输出异常及元素缺失问题排查

我是一名C++初学者，已经花了好几个小时研究循环和数组结构。我的需求是：先接收用户输入的数组大小，再接收数组元素，最后从最后一个索引到第一个索引反向输出数组。

查资料知道cin >>会自动跳过空格，但我的代码运行出问题了：输入数组大小为5，元素输入1 2 3 4 5时，正向输出只显示123，反向输出是3 3 2 1，根本得不到预期的5 4 3 2 1。我试过打印数组检查、调整循环的增减逻辑、修改数组大小，但都没解决。

测试用例 #

• 输入：

  5
  1 2 3 4 5
  

• 实际输出：

  print array 
  123
  backwards array 
  3 3 2 1 
  

• 预期输出：

  5 4 3 2 1
  

错误代码 #

#include <iostream>
#include <string>
using namespace std;

int main() {
    
  //write your solution here
  //define variables
  //int sizeofArray;
  
  //need to account for spaces
  //no you don't they are accounte for
  //int array[sizeofArray];
  int num;
  
  int sizeofArray;
  int array[sizeofArray];
  
  cin >> sizeofArray;

  //think about 2 for loops
  
  //store values to the array
  for (int i=0; i < sizeofArray; ++i) {
      
      //for each index, populate the array
      cin >> array[i];
  }//end of populate array
  
  cout << "\nprint array \n";
  
  //print the array
  for (int k=0; k < (sizeofArray); k++) {
      cout << array[k];
  }//end of print array
  
    
  //output values of the arrray backwards
  cout << "\nbackwards array \n";
  
  for (int j=sizeofArray; j >=0; --j) {
      cout << array[j] << " ";
  }
  
  
                    
  return 0;
}//end of main

问题根源与修复方案 #

1. 未初始化变量定义数组（核心问题） #

定义int array[sizeofArray];时，sizeofArray还未赋值，属于未定义行为——此时变量是内存里的随机垃圾值，数组实际大小和你后续输入的5完全无关，直接导致元素存储、读取全乱。

另外，C++标准不支持这种变长数组（VLA）（部分编译器仅作为扩展支持），正确做法是先获取数组大小，再用标准库的vector（安全且简单）动态创建数组。

2. 反向循环索引越界 #

反向输出的循环起始值是j=sizeofArray，但数组有效索引范围是0到sizeofArray-1，访问array[sizeofArray]属于越界读取垃圾数据，这也是异常输出的原因之一。

修复后的代码 #

#include <iostream>
#include <vector>
using namespace std;

int main() {
    int sizeofArray;
    // 先读取数组大小
    cin >> sizeofArray;
    
    // 用vector创建对应大小的数组，符合C++标准且安全
    vector<int> array(sizeofArray);
    
    // 填充数组元素
    for (int i = 0; i < sizeofArray; ++i) {
        cin >> array[i];
    }
    
    cout << "\nprint array \n";
    // 正向输出，加空格让内容更清晰
    for (int k = 0; k < sizeofArray; ++k) {
        cout << array[k] << " ";
    }
    
    cout << "\nbackwards array \n";
    // 从最后一个有效索引开始反向遍历
    for (int j = sizeofArray - 1; j >= 0; --j) {
        cout << array[j] << " ";
    }
    
    return 0;
}

修复说明 #

• 先读取数组大小再创建数组，彻底避免未定义行为；
• 反向循环起始值改为sizeofArray - 1，确保只访问有效索引；
• 正向输出添加空格，解决原代码中元素连在一起显示的问题（原输出123是因为没加空格，并非只读取了前3个元素）。

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