Java循环编程建议及读取整数统计正数的代码问题排查

你的代码问题分析与修复 #

首先看你这段统计正数个数的代码，问题出在循环内部重复创建Scanner实例上。每次循环都新建Scanner会重复绑定标准输入流，导致输入读取逻辑混乱——比如第一次输入的多个数字可能被第一个Scanner一次性读取，后续的Scanner就会读到空或者换行符，最终统计结果不符合预期。

修正后的代码如下：

import java.util.Scanner;

public class PositiveNumberCounter {
    public static void main(String[] args) {
        int count = 0;
        // 只在循环外创建一次Scanner，复用输入流
        Scanner sc = new Scanner(System.in);
        System.out.print("Enter 5 integers: ");

        for (int i = 0; i < 5; i++) {
            int integer = sc.nextInt();
            if (integer > 0) {
                count++;
            }
        }

        System.out.println("Count = " + count);
        sc.close(); // 用完后关闭资源，养成良好习惯
    }
}

Java循环编程的通用建议 #

• 重用IO资源：像Scanner、文件流这类IO资源，绝对不要在循环内反复创建销毁，不仅浪费内存，还容易引发输入输出异常。
• 匹配循环类型与场景：
  • for循环适合已知循环次数的场景（比如你这个输入5个数的需求就非常适配）；
  • while/do-while适合未知循环次数的场景，比如读取用户输入直到输入"quit"为止。
• 确保循环终止条件清晰：避免写出无限循环的代码——比如for循环里忘记更新计数器，或者while的条件永远为true且没有break逻辑。
• 保持循环体简洁：如果循环内的逻辑过于复杂，把它抽成单独的方法，比如把判断正数并计数的逻辑写成countPositive(int num)，这样代码可读性会高很多。
• 注意循环变量的作用域：比如for循环里的计数器i，尽量让它只在循环内部生效，不要污染外部的变量空间。
• 处理边界情况：比如统计正数时要明确是否包含0（你的代码里integer > 0是正确的，0不属于正数），如果需求有变化要及时调整判断条件。

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