在多层嵌套循环中，若累加变量（如 `currsum`）在外部声明并初始化，其值会在多次内层迭代中持续累积，导致计算错误；必须在每次重新开始子数组求和前重置为零，或将其声明移至最内层循环作用域内。
你遇到的问题本质是 变量作用域（scope）与生命周期（lifetime） 的经典误区。在你的代码中：
int currsum = 0; // ✅ 声明并初始化于方法体开头（外层作用域）
// ...
for (int i = 0; i < numbers.length; i++) {
for (int j = i; j < numbers.length; j++) {
currsum = 0; // ⚠️ 必须手动重置！否则上一轮的值会残留
for (int k = start; k <= end; k++) {
currsum += numbers[k]; // 累加当前子数组 [i..j]
}
// ...
}
}currsum 在方法入口处声明，其作用域覆盖整个 printmsasum 方法——包括所有三层循环。这意味着：
? 它不会自动重置进入新循环时；
? 若不显式赋值 currsum = 0，它将保留上一次内层循环结束后的值（例如上一个子数组的和），导致后续累加严重失真（如 1 + (-2) + 6 + ... 叠加在旧值之上）。
✅ 更优解：缩小作用域
将 currsum 的声明直接放在最内层循环（即子数组求和逻辑）之前，使其每次迭代都创建新变量：
public static void printmsasum(int numbers[]) {
int maxsum = Integer.MIN_VALUE;
for (int i = 0; i < numbers.length; i++) {
for (int j = i; j < numbers.length; j++) {
int currsum = 0; // ✅ 每次子数组求和前「自然清零」——作用域精准匹配语义
for (int k = i; k <= j; k++) {
currsum += numbers[k];
}
System.out.println("Subarray [" + i + "," + j + "]: sum = " + currsum);
if (currsum > maxsum) {
maxsum = currsum;
}
}
}
System.out.println("The maximum subarray sum is = " + maxsum);
}? 这样做的优势：
deep-java-review
下载
Java项目代码review工具。分析Git变更+完整调用链路上下文，推断业务需求，进行多维度评分和分类汇总，生成完整PRD文档。包含细粒度Java代码审查清单（Null安全、异常处理、Streams、并发、equals/hashCode、资源管理、API设计、性能、MyBatis/ORM、事务边界、SQL/DD...
- 语义清晰：currsum 明确表示“当前子数组的和”，生命周期仅限本次 k 循环；
- 安全可靠：杜绝因遗忘重置引发的隐蔽 bug；
- 符合 Java 最佳实践：变量应在首次使用前、最小必要作用域内声明（《Effective Java》Item 45）。
⚠️ 注意事项：
- 不要混淆「声明」（int currsum;）和「赋值」（currsum = 0;）。前者分配内存并绑定作用域，后者修改值；
- 若变量需跨多轮循环复用（如统计总次数），才应在外层声明；但此处 currsum 是局部中间结果，绝不应跨子数组复用；
- 此问题与 for 循环本身无关，而是任何涉及状态复用的嵌套结构（如 while 套 for）都需警惕。
总结：不是“循环让值改变”，而是变量作用域决定其是否自动隔离。把累加器声明在最内层，是最简洁、最健壮、最易维护的写法。
