如何修复 java 中的内存泄漏

Question

我目前正在尝试编写一个程序，从 Steam 中获取一个字符串化的 json 对象，并使用该对象来确定我是否可以在 Steam 市场上购买商品。

它有效，但是我似乎遇到了大量内存泄漏，而且我不知道如何解决这个问题，因为我是一个初学者程序员。这是代码：

import java.io.IOException;
import java.net.MalformedURLException;
import java.net.URL;
import java.net.URLConnection;
import java.util.Scanner;


public class SteamMarketAlert {

    @SuppressWarnings("unused")
    private JAlertWindow alert;
    private URL jsonUrl;
    private float walletValue;
    private boolean itemBuyable;

    public SteamMarketAlert(URL itemUrl, float walletValue)
    {
        this.itemBuyable = false;
        this.jsonUrl = getJSONurl(itemUrl);
        this.walletValue = walletValue;
    }

    private URL getJSONurl(URL itemUrl)
    {
        String jsonString = itemUrl.toString();

        String firstPart = jsonString.substring(0, jsonString.indexOf("market/") + "market/".length());

        String appid = jsonString.split("/")[5];
        String marketHashName = jsonString.split("/")[6];

        String secondPart = "priceoverview/?currency=2&appid=" + appid + "&market_hash_name=" + marketHashName;

        try {
            return new URL(firstPart + secondPart);
        } catch (MalformedURLException e) {
            System.err.println("Failed to create json url");
            return null;
        }

    }

    public void checkMarket()
    {
        Thread thread = new Thread(){
            @Override
            public void run(){
                try {
                    while(!itemBuyable)
                    {
                        sleep(5000);                        
                        if(isBuyable(getPagehtml()))
                            itemBuyable = true;
                    }
                    alert = new JAlertWindow();
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
            }   
        };

        thread.start();


    }

    private boolean isBuyable(String pagehtml)
    {
        int firstIndex = pagehtml.indexOf(";") +1;


        float marketValue = Float.parseFloat(pagehtml.substring(firstIndex, firstIndex + pagehtml.substring(firstIndex, pagehtml.length()).indexOf("\"")));

        return (marketValue <= walletValue)? true:false;
    }

    private String getPagehtml(){

        try(Scanner scanner = new Scanner(jsonUrl.openConnection().getInputStream())) {

            scanner.useDelimiter("\Z");
            return scanner.next();
        } catch (IOException e) {               
            e.printStackTrace();
            return null;
        }

    }

    public static void main(String[] args)
    {
        try {
            float walletValue =  82.64f;
            URL itemUrl = new     URL("http://steamcommunity.com/market/listings/730/StatTrak%E2%84%A2%20P90%20%7C%20Asiimov%20%28Factory%20New%29");
            SteamMarketAlert sma = new SteamMarketAlert(itemUrl,walletValue);
            sma.checkMarket();


        } catch (MalformedURLException e) {

            e.printStackTrace();
        }

    }
}

我已将问题缩小到 checkMarket() 方法。但是，我似乎无法弄清楚发生了什么。你能指出我如何解决这个问题吗（并可能指出我代码中的所有缺陷），注意 JAlertWindow 对象只显示一个带有 "CAN BUY" 的 JFrame - 没什么特别的。

编辑：自发布以来我更新了代码，用户告诉我存在 try-with-resource 块。感谢所有帮助我了解 java 垃圾收集工作原理的人。 :)!

Answer 1

为什么不运行 Java 分析器？我建议使用 YourKit。我经常使用它来查找服务器上任何内存泄漏的原因。它也非常适合连接以删除应用程序；你应该玩玩它。

这里有一个 link 到他们自己的关于如何查找内存泄漏的视频； Link

如果您需要更多帮助，您应该快速阅读他们的 docs

Answer 2

在您遇到 OutOfMemoryException 或您看到常量垃圾回收之前，Java 可能不是内存泄漏。

10 分钟内使用 18MB 看起来不像是内存泄漏，这就是 Java 的工作原理。如果你真的想确定，你可以打开详细的 GC 并查看它的收集频率，但我认为你还没有真正的问题。

Answer 3

您正在获取 html 个页面，因此每次获取时都必须在内存中分配一些对象（字符串等）。稍后（一旦不再使用）这些对象将被删除并且内存被 GC 标记为空闲。

抱歉，解释过于简单。如果您想了解更多，请阅读 java GC、堆结构等

如何修复 java 中的内存泄漏

How do I fix a memory leak in java

java

memory-leaks