线程会降低PC的速度并导致java.lang.OutOfMemoryError

Question

我正在做的是制作一个程序,不断截取用户桌面的截图,并在用户需要时保存它们.我最初调用了一个方法来捕获一个while方法,但是这个方法太慢了,我需要尽可能多的图像.我决定做的是使用线程.

当程序运行时,我的PC变慢(即鼠标闪烁,移动缓慢等),当我停止程序时,我得到以下错误.

Exception in thread "Thread-294" java.lang.OutOfMemoryError: Java heap space
at sun.awt.windows.WRobotPeer.getRGBPixels(Unknown Source)
at java.awt.Robot.createScreenCapture(Unknown Source)
at maple.Record.run(Record.java:29)

这是产生错误的行.

img = r.createScreenCapture(new Rectangle (0, 0, width, height));

这是包含线程的我的类.

import java.awt.AWTException;
import java.awt.Rectangle;
import java.awt.Robot;
import java.awt.Toolkit;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.net.MalformedURLException;
import java.util.Vector;
import javax.imageio.ImageIO;
import javax.media.MediaLocator;

public class Record implements Runnable {

    Robot r;
    static int width = (int) Toolkit.getDefaultToolkit().getScreenSize().getWidth();
    static int height = (int) Toolkit.getDefaultToolkit().getScreenSize().getHeight();
    BufferedImage img;
    public static boolean stop = false;

    public void run() {

        try {

            r = new Robot();
            img = r.createScreenCapture(new Rectangle (0, 0, width, height));
            ImageIO.write(img, "png", new File(JavCapture.tmpLocation + "\\tmp\\" + System.currentTimeMillis() + ".png"));

        } catch (IOException | AWTException e) { e.printStackTrace(); }

    }

}

当我调用已编辑的JpegImagesToMovies以使用.png文件时,会产生错误.

请查看此链接以获取更多相关信息.

我该如何解决这个问题,如何让线程使用更少的内存,这样就不会降低PC的运行速度.

这是main方法中的调用.

do {    
    (new Thread(new Record())).start();
} while (!Record.stop);

Answer 1

从它的外观来看,你正在为每个屏幕截图创建一个单独的线程,这将使你快速运行内存.相反,你应该有一个单独的线程,它在循环中,睡眠并不时截取屏幕截图.

    public class ScreenshotTaker implements Runnable {



        private volatile boolean done = false;



        public void run( ) {

            while (!done) {
            ... take screenshot...

            ...sleep ...
            }

        }

        public void setDone( ) {

            done = true;

        }

    }