嗨，我来帮你拆解下这段代码的问题，以及对应的修正思路～

原代码的核心错误 #

1. 嵌套循环的输出逻辑混乱 #

原代码里的内层循环每一次判断都会直接输出结果，导致同一个数字被重复打印多次：

• 比如输入n=6时，i=1会触发j从2到3的两次循环，每次都走else分支，所以输出两次1-prime
• i=2时，j=2会触发if分支输出2，j=3时触发else分支输出2-prime，一个数字被输出两次

2. 质数判断的逻辑漏洞 #

• 质数的定义是大于1的自然数，只能被1和自身整除，但原代码没有排除i=1的情况（1既不是质数也不是合数）
• 内层循环的终止条件j<=n/2不合理，应该针对当前判断的数字i来设置（比如判断到j*j<=i就足够，能减少无效循环）
• 没有用标记变量记录是否找到因子，而是直接在循环内输出，导致判断不完整就提前打印结果

3. 代码结构不严谨 #

外层for循环没有加大括号，虽然Java语法允许，但容易导致逻辑混淆，也让内层循环的作用范围不清晰

修正代码的核心改进 #

你更新后的代码已经解决了大部分问题，我再帮你梳理下关键优化点：

• 新增boolean gasit标记变量，用来记录当前数字i是否找到能整除它的因子，避免了循环中重复输出的问题
• 内层循环终止条件改为j*j<=i，大幅减少了质数判断的循环次数，提升了效率
• 外层循环每次迭代都会重置gasit为false，确保每个数字的判断都是独立的
• 最后统一根据gasit的值输出结果，逻辑清晰且不会重复打印

再提一个小优化（处理1的特殊情况） #

1既不是质数也不是合数，可以单独处理让结果更准确，同时在找到因子后提前跳出内层循环提升效率：

public static void main(String args[]) {
    Scanner sc = new Scanner(System.in);
    int n;
    int i,j;
    boolean gasit = false;
    System.out.print("Dati n: ");
    n = sc.nextInt();
    for(i=1;i<=n;i++) {
        // 单独处理1的情况
        if(i == 1) {
            System.out.println(i);
            continue;
        }
        gasit=false;
        for(j=2;j*j<=i;j++) {
            if(i%j==0) {
                gasit=true;
                break; // 找到因子直接退出循环，无需继续判断
            }
        }
        if(!gasit) {
            System.out.println(i+"-prime");
        } else {
            System.out.println(i);
        }
    }
}

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