尝试Catch Final-final变量始终为null

Question

我目前在我的代码中遇到问题，无法弄清楚为什么此语句按原样进行评估。 这是我第一次使用finally块，因此可能存在一些我不了解的基本行为。

此方法的作用是从api获取json文档，并将该文档存储为this.thisPage 。 然后另一个方法sliceItem将结果字段拆分为json对象数组。

每当API返回包含错误字段的json（例如，将字符串字段存储为int或将int存储为double等）时，都会引发MalformedJsonException。 尝试10次（由failsafeget处理），如果失败10次，则抛出MalformedJsonException（RuntimeException）。 在这种情况下，我希望slicePage要做的是获取下一页而不是继续此页面。 为了简化此操作-每个页面有100个条目； 如果偏移量3500被破坏，我们要获取偏移量3600。

我当前面临的问题是，在最后一个块中， resp总是评估为null 。 我不明白为什么会这样，因为try块可以返回除null（JSONObject类型）之外的其他值。

任何帮助将不胜感激，如果您需要更多信息/代码，我愿意提供。

public synchronized void slicePage(){
    JSONObject resp=null; // otherwise java complains that not initialised
    ApiClient apiClient = new ApiClient();
    RestEndPoint pageUrl;
    while (true) {
        pageUrl = getNextPageEndPoint();
        if(pageUrl == null) {
            throw new IllegalStateException("We have reached the end and the code isn't designed to handle the end here"); // we have reached the end
        }
        currentPageNumber++;
        try {
            resp = apiClient.failSafeGet(pageUrl, getRetryCount());
            break;
        }
        catch (MalformedJsonException e) {
            logger.info(String.format("The json was still broken after %d retries. Skipping this page and notifying listeners", getRetryCount()));
            for (Consumer<Integer> consumer: onSkipListenerList) {
                consumer.accept(batchSize); // inform each listener that we are skipping this many entries
            }
        }
        finally { // We need to set the next page end point no matter the outcome of the try catch. N.B. this gets executed even if there is a break
            if(resp == null) {
                // no next possible
                setNextPageEndPoint(null);   // don't consider next; we reached the max
                this.thisPage = null;
            } else {
                if(currentPageNumber > maxPages - 1) {
                    // because a request has been made already, so reduce by 1
                    setNextPageEndPoint(null); // don't consider next; we reached the max
                } else {
                    // else consider next page
                    setNextPageEndPoint(constructNextPageEndPoint(pageUrl, resp));
                }
                this.thisPage = this.parseResult(resp);

                setTotalCount(resp.getInt("totalResults"));
            }
        }
    }
}

---

编辑我忘了提一下，当我说它总是评估为空时，我的意思是我的IDE-Intellij IDEA警告我if条件总是评估为空。 以下是Intellij中显示的帮助（使用Ctrl-F1）。

 Condition 'resp == null' is always 'true' less... (Ctrl+F1) 
 This inspection analyzes method control and data flow to report possible conditions that are always true or false, expressions whose value is statically proven to be constant, and situations that can lead to nullability contract violations.
 Variables, method parameters and return values marked as @Nullable or @NotNull are treated as nullable (or not-null, respectively) and used during the analysis to check nullability contracts, e.g. report possible NullPointerException errors.
More complex contracts can be defined using @Contract annotation, for example:
@Contract("_, null -> null") — method returns null if its second argument is null @Contract("_, null -> null; _, !null -> !null") — method returns null if its second argument is null and not-null otherwise @Contract("true -> fail") — a typical assertFalse method which throws an exception if true is passed to it 
The inspection can be configured to use custom @Nullable
@NotNull annotations (by default the ones from annotations.jar will be used)

编辑2事实证明，代码分析是错误的，一旦运行该值将为非null。 感谢所有人（包括评论员）分享您的见解和建议。 最终，我在条件之前插入了logger.info，其值似乎一切正常。 它似乎停止工作的原因是图形服务器已超时。

Answer 1

这是正常的行为。 这个电话

apiClient.failSafeGet(pageUrl, getRetryCount());

引发异常，因此对resp的值分配从未完成，因此在finally block该值为null。 因此，您的方法总是抛出异常，或者如果不是，则它在某个时刻返回null 。

Answer 2

在您的代码中：

try {
            resp = apiClient.failSafeGet(pageUrl, getRetryCount());
            break;
        }
        catch (MalformedJsonException e) {
            logger.info(String.format("The json was still broken after %d retries. Skipping this page and notifying listeners", getRetryCount()));
            for (Consumer<Integer> consumer: onSkipListenerList) {
                consumer.accept(batchSize); // inform each listener that we are skipping this many entries
            }
        }
        finally {.....

如果resp = apiClient.failSafeGet(pageUrl, getRetryCount()); 引发异常，resp始终为null，因为程序在实例实例化之前失败。