我正在编写一个试图执行以下操作的程序:
我创建了一个" MyRunnable"类:
public class MyRunnable implements Runnable {
@FXML
private TextArea textField;
public MyRunnable() throws IOException {
}
public void run() {
String firstFileName = "test.txt";
File inFile = new File(firstFileName);
Scanner in = null;
try {
in = new Scanner(inFile);
} catch (FileNotFoundException e) {
e.printStackTrace();
}
while (in.hasNextLine()) {
textField.appendText("Hello");
textField.appendText("\n");
}
}
}
我的控制器类有一个方法
public void Main() throws IOException {
Runnable r = new MyRunnable();
Thread t = new Thread(r);
t.start();
}
IntelliJ告诉我textField从未被分配,当我运行我的主程序,然后单击调用Main()的按钮时,我在下一行得到一个空指针异常。
textField.appendText("Hello");
我如何完成我想要完成的任务?
答案 0 :(得分:0)
您可以将textField
移动到控制器类并将其作为参数传递给runnable吗?
答案 1 :(得分:0)
您的代码存在一些问题。首先是你基本观察到的那个:
当@FXML
加载FXML文件时,注释FXMLLoader
的字段由FXMLLoader
注入控制器。显然,FXMLLoader
不知道任何其他对象,因此简单地用@FXML
注释任意对象中的字段并不意味着它已被初始化。
其次,您的runnable的run()
方法在后台线程上执行。对用户界面must happen on the FX Application Thread (see the "Threading" section)的更改。因此,即使textField.appendText(...)
已初始化,您对textField
的来电也无法保证正确行事。
最后,更一般地说,您的设计违反了“关注点分离”。您的可运行实现实际上只是从文件中读取一些文本。它不应该关注该文本发生的事情,它当然不应该对UI有任何了解。 (简而言之,在控制器之外公开UI元素总是一个糟糕的设计决定。)
这里最好的方法是给runnable实现一个“回调”:即一个“用字符串做某事”的对象。您可以将其表示为Consumer<String>
。所以:
import java.util.Scanner ;
import java.util.function.Consumer ;
import java.io.File ;
import java.io.FileNotFoundException ;
public class MyRunnable implements Runnable {
private Consumer<String> textProcessor;
public MyRunnable(Consumer<String> textProcessor) {
this.textProcessor = textProcessor ;
}
public void run() {
String firstFileName = "test.txt";
File inFile = new File(firstFileName);
Scanner in = null;
try {
in = new Scanner(inFile);
} catch (FileNotFoundException e) {
e.printStackTrace();
}
while (in.hasNextLine()) {
textProcessor.accept(in.nextLine());
}
}
}
然后,在您的控制器中,您可以执行以下操作:
@FXML
private TextArea textField ;
public void Main() throws IOException {
Runnable r = new MyRunnable(s ->
Platform.runLater(() -> textField.appendText(s+"\n")));
Thread t = new Thread(r);
t.start();
}
请注意Platform.runLater(...)
的使用,它会更新FX应用程序主题上的文本区域。
现在,根据您能够以多快的速度阅读文本行,这种方法可能会使FX应用程序线程陷入太多更新,从而使UI无响应。 (如果您只是从本地文件中读取,肯定会出现这种情况。)有几种方法可以解决这个问题。一种是简单地将所有数据读入字符串列表,然后在读取时处理整个列表。为此,您可以使用Task
而不是普通的runnable:
public class ReadFileTask extends Task<List<String>> {
@Override
protected List<String> call {
List<String> text = new ArrayList<>();
String firstFileName = "test.txt";
File inFile = new File(firstFileName);
Scanner in = null;
try {
in = new Scanner(inFile);
} catch (FileNotFoundException e) {
e.printStackTrace();
}
while (in.hasNextLine()) {
text.add(in.nextLine());
}
return text ;
}
}
现在在你的控制器中你可以使用它:
@FXML
private TextArea textField ;
public void Main() throws IOException {
Task<List<String>> r = new ReadFileTask();
// when task finishes, update text area:
r.setOnSucceeded(e -> {
textArea.appendText(String.join("\n", r.getValue()));
}
Thread t = new Thread(r);
t.start();
}
如果您真的想在阅读文本时不断更新文本区域,那么事情会变得复杂一些。您需要从后台线程将字符串放入某种缓冲区,然后以不会泛滥FX应用程序线程的方式将它们读入文本区域。您可以使用BlockingQueue<String>
作为缓冲区,并在AnimationTimer
中回读。每次将帧渲染到屏幕时,动画计时器都会执行一次handle()
方法(因此它不会经常运行,与之前的Platform.runLater()
方法不同):基本策略是抓取每次运行时尽可能从缓冲区中更新,并更新文本区域。完成后停止动画计时器非常重要,我们可以通过计算从文件中读取的行来完成,并在我们将它们全部放入文本区域时停止。
这看起来像这样:
public class BackgroundFileReader extends Runnable {
public static final int UNKNOWN = -1 ;
private final AtomicInteger lineCount = new AtomicInteger(UNKNOWN);
private final BlockingQueue<String> buffer = new ArrayBlockingQueue<>(1024);
@Override
public void run() {
String firstFileName = "test.txt";
File inFile = new File(firstFileName);
Scanner in = null;
try {
in = new Scanner(inFile);
} catch (FileNotFoundException e) {
e.printStackTrace();
}
int count = 0 ;
try {
while (in.hasNextLine()) {
buffer.put(in.nextLine());
count++ ;
}
} catch (InterruptedException exc) {
Thread.currentThread.interrupt();
}
lineCount.set(count);
}
// safe to call from any thread:
public int getTotalLineCount() {
return lineCount.get();
}
public int emptyBufferTo(List<String> target) {
return buffer.drainTo(target);
}
}
然后在控制器中,你可以做到
@FXML
private TextArea textField ;
public void Main() throws IOException {
ReadFileTask r = new ReadFileTask();
// Read as many lines as possible from the buffer in each
// frame, updating the text area:
AnimationTimer updater = new AnimationTimer() {
private int linesRead = 0 ;
@Override
public void handle(long timestamp) {
List<String> temp = new ArrayList<>();
linesRead = linesRead + r.emptyBufferTo(temp);
if (! temp.isEmpty()) {
textField.appendText(String.join("\n", temp));
}
int totalLines = r.getTotalLineCount() ;
if (totalLines != BackgroundFileReader.UNKNOWN && linesRead >= totalLines) {
stop();
}
}
};
updater.start();
Thread t = new Thread(r);
t.start();
}