计算给定两数范围内奇数和时代码仅输出0的问题排查

嘿，我一眼就揪出几个导致代码始终输出0的关键问题，咱们一个个理清楚：

核心错误点 #

• 未定义变量x：你代码里用到了x但完全没声明过，正常来说这代码连编译都通不过，估计是手滑打错了，应该是想用循环变量z对吧？
• 循环条件逻辑完全颠倒：你写的while (z >= a && z >=b)，但z初始值是0，只要你输入的a或b有一个大于0，这个条件就永远不成立，循环根本不会执行，sum4自然一直是初始的0。
• 循环起始值错误：z从0开始，而不是从你输入的两个数的范围起点开始，就算条件对了，也不会遍历你要计算的目标区间。
• 未处理输入顺序问题：如果用户先输入大数再输入小数（比如5和2），你的代码也没法正确计算2到5的奇数和。

修正后的代码 #

import java.util.Scanner;

public class OddSumCalculator {
    public static void main(String[] args) {
        System.out.println("Enter a range of 2 numbers: ");
        Scanner in = new Scanner(System.in);
        int a = in.nextInt();
        int b = in.nextInt();
        
        // 自动确定范围的起点和终点，处理输入顺序问题
        int start = Math.min(a, b);
        int end = Math.max(a, b);
        
        int sum4 = 0;
        // 从起点遍历到终点，逐个判断奇数并累加
        for (int z = start; z <= end; z++) {
            if (z % 2 != 0) {
                sum4 += z;
            }
        }
        
        System.out.println("Sum of odd numbers in range: " + sum4);
        in.close(); // 别忘了关闭输入流，养成好习惯
    }
}

代码说明 #

1. 用Math.min和Math.max自动适配输入顺序，不管用户先输大还是先输小，都能锁定正确的遍历区间。
2. 改用for循环更适合这种固定范围的遍历，逻辑比while更清晰（要是偏爱while也可以改，只要把起始值和循环条件调整正确就行）。
3. 统一了变量名，避免了之前混乱的z和x混用问题，确保每一步都在目标区间内遍历。

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