【发布时间】：2016-09-12 00:03:35
【问题描述】：
我有一个应用程序，它在登录时通过接口类访问数据库。登录过程导致应用程序在访问数据库时没有响应一段时间，因此我一直在研究线程和等待光标以使其顺利运行。我试图通过网络上的许多示例和堆栈溢出来使用线程，但我的方法似乎不起作用，我收到 java.lang.IllegalStateException: Not on FX application thread; currentThread = Thread-4 异常，我不确定如何从这里开始。我试图做的是在这个后台线程运行 loginLoadEverything() 方法时将光标更改为等待模式（尽管我没有在其中包含代码，因为它太长了）。这是我的控制器类：
package main.java.gui;
import javafx.application.Platform;
import javafx.concurrent.Service;
import javafx.concurrent.Task;
import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.fxml.FXMLLoader;
import javafx.fxml.Initializable;
import javafx.scene.Cursor;
import javafx.scene.Parent;
import javafx.scene.Scene;
import javafx.scene.control.*;
import javafx.scene.image.Image;
import javafx.scene.image.ImageView;
import javafx.stage.Stage;
import main.java.databaseInterface.BackendInterface;
import java.net.URL;
import java.util.ResourceBundle;
import java.util.concurrent.CountDownLatch;
public class LoginController implements Initializable {
private BackendInterface backendInterface;
private DashboardController dashboardController;
private StudentsController studentsController;
private ConsultationController consultationController;
private CreateStudentController createStudentController;
private CreateConsultationController createConsultationController;
@FXML
TextField username;
@FXML
PasswordField password;
@FXML
Button loginButton;
@FXML
Label loginLabel;
@FXML
public void loginButtonPress(ActionEvent event) {
Service<Void> service = new Service<Void>() {
@Override
protected Task<Void> createTask() {
return new Task<Void>() {
@Override
protected Void call() throws Exception {
loginLoadEverything();
final CountDownLatch latch = new CountDownLatch(1);
Platform.runLater(new Runnable() {
@Override
public void run() {
try {
Scene s1 = loginLabel.getScene();
s1.setCursor(Cursor.WAIT);
} finally {
latch.countDown();
}
}
});
latch.await();
return null;
}
};
}
};
service.start();
}
public void loginLoadEverything() {
//chance to true when complete
if (username.getText().isEmpty() == false || password.getText().isEmpty() == false) {
loginLabel.setText("Please enter data in the fields below");
} else {
username.setText("-----");
password.setText("-----");
//initialises backend interface with username and password
backendInterface = new BackendInterface(username.getText(), password.getText().toCharArray());
// Open a connection to the database
if (backendInterface.openConnection()) {
//return and print response
System.out.println(backendInterface.getConnectionResponse());
//directs the user to the dashboard after successful login
try {
if (backendInterface.getAllStudents() &&
backendInterface.getAllConsultations() &&
backendInterface.getCourses() &&
backendInterface.getConsultationCategories() &&
backendInterface.getConsultationPriorities()) {
FXMLLoader loader1 = new FXMLLoader();
loader1.setLocation(getClass().getResource("/main/res/dashboard.fxml"));
loader1.load();
Parent p = loader1.getRoot();
Stage stage = new Stage();
stage.setScene(new Scene(p));
stage.show();
//set instances to the dashboard controller
dashboardController = loader1.getController();
dashboardController.setBackendInterface(backendInterface); //pass backendInterface object to controller
dashboardController.setDashboardController(loader1.getController()); //pass dashboard as reference
//load images
Image logoutImage = new Image(getClass().getResourceAsStream("images/logout.png"));
Image userImage = new Image(getClass().getResourceAsStream("images/users.png"));
Image calendarImage = new Image(getClass().getResourceAsStream("images/calendar.png"));
Image leftArrowImage = new Image(getClass().getResourceAsStream("images/leftArrow.png"));
Image notepadImage = new Image(getClass().getResourceAsStream("images/notepad.png"));
//set images
dashboardController.studentLabel.setGraphic(new ImageView(userImage));
dashboardController.logoutLabel.setGraphic(new ImageView(logoutImage));
dashboardController.consultationLabel.setGraphic(new ImageView(notepadImage));
} else {
system.out.println(backendInterface.getExceptionMessage);
}
@Override
public void initialize(URL location, ResourceBundle resources) {
}
【问题讨论】：
-
call()方法是在后台线程上执行的方法。它应该执行需要很长时间才能执行的工作，并且不得执行任何 UI 工作。现在，它似乎只在 UI 上工作。实际的数据库连接工作在哪里完成，从您的代码中不清楚它在哪里。还有其他问题 - 例如。您将光标更改为WAITaftercall()方法中的其他所有内容都已完成，这肯定不是您想要的。但是您需要先弄清楚实际耗时的工作在哪里。 -
感谢詹姆斯的反馈，我已经添加了 loginLoadEverything 方法的其余代码。此方法提供从 BackendInterface 类对数据库的访问。正如您所见，BackendInterface 类的方法在这里被调用，创建了耗时的工作。然后我尝试在 call() 方法中调用这个 loginLoadEverything 方法。是的，这是正确的，我将不得不更改光标以更合适的方式等待以反映耗时的过程，但不知道如何实现这一点。
-
查看答案。看起来您需要重组一些代码，因为数据库访问和 UI 之间没有足够清晰的分离。您需要能够在一段代码中完成所有后台工作，并将结果封装在某种对象中——尚不清楚您的
backendInterface对象是否适用于此。然后任务可以完成数据库工作并返回该对象，onSucceeded处理程序可以显示 UI，从对象进行配置。一旦你开始工作，光标就变得微不足道了。 -
使用您添加的代码更新了答案。您的某些代码不是很容易理解。
dashboardController.setDashboardController(loader1.getController());是做什么的？当然这相当于dashboardController.setDashboardController(dashboardController);。你为什么给一个对象引用它自己？？？大概你真的不想总是传递"-----"作为用户名和密码？你的很多实例变量也应该是局部变量。 -
dashboardController.setDashboardController(loader.getController());相当于把控制器传给自己。当我尝试其他事情时，它是留在那里的代码。是的，我实际上并没有使用“------”作为用户名和密码，而是替换它，因为我的代码中有我的实际用户名和密码
标签： java javafx background-task