C++斐波那契数生成器无法运行，请求分析故障原因

以下是代码中导致无法正常工作的核心问题：

• 可变长度数组（VLA）不符合C++标准：int list[num]是C99引入的特性，C++标准并未定义这种写法，不同编译器的支持情况不一，可能导致编译错误或运行异常。建议改用std::vector<int>来动态创建数组。

• 数组初始化错误：int list[num] = {0, 1};存在两个问题：

  • 当num > 2时，数组中索引2及以后的元素不会被初始化，会保留垃圾值；
  • 当num < 2时，初始化列表的长度超过数组长度，属于越界初始化，触发未定义行为。

• 斐波那契计算逻辑完全错误：
  内层循环for (int z = 2; z < num; z++)会把list[2]到list[num-1]的所有元素都赋值为同一个x，正确的逻辑应该是逐个计算每个位置的值：每次循环只需要给list[y+2]赋值为list[y] + list[y+1]，不需要内层循环。

• 非法的数组下标访问：list[-1]是完全错误的写法，C++数组下标不能为负数，这会访问数组内存范围外的区域，导致未定义行为。要获取最后一个元素应该用list[num-1]。

• 函数返回值缺失：fibonacci()声明为返回int，但所有分支都没有return语句，这会导致程序运行时的未定义行为。

• 缺少输入校验：没有检查用户输入的num是否为正整数，如果输入0、负数或者非整数，程序会直接出错。

修复后的简化示例代码 #

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

int fibonacci()
{
    cout << "enter sequence num: " << endl;
    int num;
    // 输入校验
    if (!(cin >> num) || num < 1) {
        cout << "请输入正整数" << endl;
        return -1;
    }

    if (num == 1) {
        cout << "the " << num << " fibo num is " << 0 << endl;
        return 0;
    } else if (num == 2) {
        cout << "the " << num << " fibo num is " << 1 << endl;
        return 1;
    }

    vector<int> list(num);
    list[0] = 0;
    list[1] = 1;
    for (int y = 0; y < num - 2; y++) {
        list[y+2] = list[y] + list[y+1];
    }
    cout << "the " << num << " fibo num is " << list[num-1] << endl;
    return list[num-1];
}

int main()
{
    fibonacci();
    return 0;
}

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