有关HtmlPage对象未清除的问题

Question

在此HtmlUnit示例中，我已经清理了HtmlPage，但仍在main方法中获取了指定url的锚点，因为我在这段代码中认为在先计算return语句，然后最后调用，然后再计算上面的页面是从getPageResource（）方法返回的，或者如果您认为除此以外，请共享。

但在我的脑海里一个更值得关注的是，如果对象得到清理的“页”引用那么它会得到可从它getAnchors。

public HtmlPage getPageResource() {
        WebClient webClient = new WebClient();
        HtmlPage page = null;
        try {
            page = webClient.getPage("http://some-url.com");
            // Do something with page
            return page;
        } catch (FailingHttpStatusCodeException e) {
            e.printStackTrace();
            return page;
        } catch (MalformedURLException e) {
            e.printStackTrace();
            return page;
        } catch (IOException e) {
            e.printStackTrace();
            return page;
        } finally {
            if (page != null)
                page.cleanUp();
        }
}

public static void main(String args[]) {
    HtmlUnitExample htmlUnitExample = new HtmlUnitExample();
    HtmlPage page = htmlUnitExample.getPageResource();
    System.out.println(page.getAnchors());
}

Answer 1

我对此代码有一些评论。

首先，如果getPage抛出任何异常，则page变量将为null否则， page变量将被分配一个对象。 因此，在Do something with page行中，您100％确保该page不是null 。 因此，您应该将代码放在此处。 包括cleanUp 。

此外，当检测到异常时，您将返回null对象，然后尝试从null对象getAnchors 。 这将在您的main方法中导致NullPointerException 。

您最有可能在寻找这样的东西：

public HtmlPage getPageResource() throws Exception {
    WebClient webClient = new WebClient();
    HtmlPage page = webClient.getPage("http://some-url.com");
    // Do something with page
    page.cleanUp();
    return page;
}

public static void main(String args[]) {
    HtmlUnitExample htmlUnitExample = new HtmlUnitExample();
    try {
        HtmlPage page = htmlUnitExample.getPageResource();
        System.out.println(page.getAnchors());
    } catch (Exception e) {
        e.printStackTrace();
    }
}

如您所见，代码更简单，并且没有if语句。 最有可能应该删除整个getPageResource()方法，或者返回返回锚列表（带有锚或为空），而不是返回页面，这样您就可以完全信任自己的main List并且不需要那里的任何异常处理。

那是关于Java的。 转向HtmlUnit，我从不需要使用cleanUp 。 完全相同的JavaDocs声明you should probably never need to call it directly因此，显而易见的问题是您打算如何处理它？ 。 如果您准备使用该对象来获取新页面，请记住无需清理页面。

Answer 2

您为什么不为内存问题而尝试将页面设置为null而不是将其清理为null？ 或者，我建议您添加“ Thread.sleep（10 * 1000）;” 在语句“ //对页面执行某些操作”之后，因为页面的加载是异步的。 我猜您触发cleanUp时，页面尚未加载。