我有一个在我的覆盆子pi上运行的java应用程序,但它大部分时间都崩溃了。每当它崩溃时,它通常具有非常高的CPU使用率(> 100%)。我的应用程序如何工作:我有一个读取标签的RFID阅读器,每当读取标签时,都会调用messageReceived方法。该方法将读取的标签存储在特定的集合中。然后我创建一个新线程,它监听套接字,当套接字打开时,当集合发生变化时,线程调用一些javafx方法来打开新屏幕。但是,当我将应用程序部署到我的覆盆子pi时,它随机崩溃并且使用java的CPU使用率很高。如果我忘了解释任何问题,请随时提出任何问题。
编辑1:我的线程类。 编辑2:我现在的问题是:为什么我有如此高的CPU使用率以及如何解决它。
public class RFIDThread implements Runnable {
/**
* The socket for the connection to the LLRP Reader
*/
private Socket socket;
private JSONArray valid_tags;
private JSONArray found_tags;
private TagsListController controller;
private RFIDSet rfidset;
/**
* Thread for constant reading of the stream
*
* @param socket
* @param controller
* @param tags
*/
public RFIDThread(Socket socket, TagsListController controller, JSONArray tags, RFIDSet rfidset) {
this.socket = socket;
this.controller = controller;
this.rfidset = rfidset;
this.found_tags = new JSONArray();
this.valid_tags = tags;
}
/**
* Runnable for this thread.
* First get all the found tags from the xml controller
* Then loop over the rfid set to find any new tags.
* If there are any, display them.
*/
@Override
public void run() {
CopyOnWriteArrayList<Tag> originalSet = new CopyOnWriteArrayList<>();
originalSet.addAll(rfidset.getSet());
boolean started = true;
if (socket.isConnected()) {
while (!socket.isClosed()) {
CopyOnWriteArrayList<Tag> set = new CopyOnWriteArrayList<>();
set.addAll(rfidset.getSet());
if(started || !originalSet.equals(set)) {
started = false;
CopyOnWriteArrayList<String> found_set = new CopyOnWriteArrayList<>();
found_set.addAll(controller.found_tags_list.getItems());
this.found_tags.clear();
this.found_tags.addAll(found_set);
for (Tag found_tag : set) {
if (found_tags.indexOf(found_tag.getId()) < 0) {
Integer index = valid_tags.indexOf(found_tag.getId());
if (index >= 0) {
Platform.runLater(() -> controller.showValid(found_tag.getId()));
} else {
Platform.runLater(() -> controller.showError(found_tag.getId()));
}
found_tags.add(found_tag.getId());
pause(5000);
}
}
originalSet = set;
pause(5000);
}
}
}
}
/**
* Close the socket
*/
public void shutdown() {
try {
this.socket.close();
} catch (IOException e) {
e.printStackTrace();
}
}
private void pause(long ms) {
try {
Thread.sleep(ms);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
答案 0 :(得分:0)
尝试将pause(5000);
移到if (started || !originalSet.equals(set)) {
声明之外。
高CPU使用率通常是一个紧凑的循环,没有暂停或I / O或等待的东西。在你的情况下,originalSet.equals(set)
你不会停顿。
您可能更喜欢使用:
if (started || !originalSet.equals(set)) {
// ...
} else {
pause(0);
}
或类似。