如何实现方法重试直至成功？retriesCounter未递增问题求助

为什么retriesCounter没按预期递增？ #

先排查这几个核心原因：

• 未捕获的异常导致方法提前终止：如果doJob()抛出了DummyException之外的异常（比如RuntimeException），虽然finally块会执行++retriesCounter，但方法会直接抛出异常终止，你可能误以为计数器没动，但实际上它已经递增了，只是后续循环没继续。
• isJobDone的可见性问题：如果isJobDone没加volatile修饰，即使其他线程把它改成true，当前线程可能读不到最新值，导致循环逻辑异常，让你误以为计数器工作异常。
• 对重试逻辑的误解：比如第一次doJob()就成功，isJobDone设为true，finally执行后计数器变成1，然后方法直接return——这是正常逻辑，但如果你预期成功时计数器不递增，那是对逻辑的理解偏差。

具体修复建议 #

1. 捕获所有异常，避免意外跳出循环 #

把catch块改成捕获Exception，确保不管doJob()抛出什么异常，循环都能按预期走下去：

public synchronized void initApplicationWithRetry() throws InterruptedException {
    int retriesCounter = 0;
    while (!isJobDone && retriesCounter < 3) {
        try {
            doJob();
        } catch (Exception e) {
            // 这里可以加日志记录错误信息
            e.printStackTrace();
        } finally {
            ++retriesCounter;
        }
        if (isJobDone) {
            return;
        }
        wait(3000);
    }
    if (!isJobDone) {
        throw new RuntimeException("初始化重试3次仍失败");
    }
}

2. 给isJobDone加volatile修饰 #

保证多线程环境下变量的可见性，避免当前线程读取到过期值：

private volatile boolean isJobDone = false;

3. 优化重试逻辑（可选） #

当前逻辑不管成功失败都会递增计数器，最多尝试3次（含第一次）。如果需要更清晰的“重试次数”逻辑（比如第一次尝试不算重试，最多重试3次），可以调整计数器的位置：

public synchronized void initApplicationWithRetry() throws InterruptedException {
    int retryTimes = 0;
    while (!isJobDone && retryTimes <= 3) {
        try {
            doJob();
            if (isJobDone) {
                return;
            }
        } catch (Exception e) {
            // 处理错误
        }
        retryTimes++;
        if (retryTimes <= 3) {
            wait(3000);
        }
    }
    throw new RuntimeException("初始化重试3次仍未成功");
}

4. 检查doJob()的逻辑 #

确认doJob()执行成功后确实把isJobDone设为true，而且没有其他线程在修改这个变量，避免逻辑冲突。

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