Java实现productOfPrevious方法时输出数组全为0的问题排查求助

架构老林

2026-4-29

问题分析与修复

你遇到的问题根源在于乘积变量的初始化逻辑错误，咱们一步步拆解清楚：

核心问题

你把product初始化为0，而且每次外层循环还会重新将它重置为0——数学里0乘以任何数结果都是0，不管内层循环怎么乘，最终product都会保持0，自然返回数组全是0。
另外你的嵌套循环属于冗余逻辑，时间复杂度是O(n²)，其实可以优化成更高效的O(n)解法。

修复后的代码（先解决基础问题）

先把初始化的问题修正，同时优化循环逻辑：

public int[] productPrevious (int[] arr){
    int[] retArr = new int[arr.length];
    int product = 1; // 乘法的单位元是1，初始值要设为1而非0
    for (int i = 0; i < arr.length; i++){
        product *= arr[i]; // 直接用前一步的乘积乘当前元素，无需嵌套循环
        retArr[i] = product;
    }
    return retArr;
}

这样当输入{1,2,3,4,5}时，就能得到你期望的{1,2,6,24,120}了。