JavaFX TextArea Updaten bevor Prozess gestartet wird

[Nurox]

Hallo zusammen,

ich arbeite derzeit an einer JavaFX Application und habe ein kleines Problem. Ich möchte einen Prozess starten und zuvor eine TextArea updaten, wenn ich das nun so mache startet er mir aber erst den Prozess und updatet dann, wenn er fertig ist die TextArea.

Also z.B. ich öffne einen FileChooser, dann wenn ich damit eine Datei wähle mache ich:

textArea.setText("Starte Prozess");
object.startRandomMethod;
textArea.appendText("Prozess beendet")

Wenn ich das so mache dann wird erst die Methode aufgerufen und danach die Texte gesetzt.

Wenn ich das ganze in einem neuen Thread ausführe funktioniert es wie gewollt, das einzige Problem ist nur, dann ist das Fenster nicht blockiert (also der main Thread von Java) und man könnte theoretisch immer wieder einen neuen Thread erstellen, was nicht sein soll.

Hier mal ohne Thread:

@FXML
public void updateData() {
    FileChooser fileChooser = new FileChooser();
    fileChooser.setTitle("Open Resource File");
    File selectedFile = fileChooser.showOpenDialog(stage);
    if (selectedFile != null) {

                DataLoader loader = new DataLoader();
                infoTextfield.appendText("Start updating... " + "\n" + "this might take a few                 minutes");
                loader.loadData(selectedFile.toString());
                infoTextfield.appendText("finished updating");
    }
}

Hier mal wie ich es mit dem Thread gemacht habe:

@FXML
public void updateData() {
    FileChooser fileChooser = new FileChooser();
    fileChooser.setTitle("Open Resource File");
    File selectedFile = fileChooser.showOpenDialog(stage);
    if (selectedFile != null) {

        new Thread(new Runnable() {
            @Override
            public void run() {
                DataLoader loader = new DataLoader();
                infoTextfield.appendText("Start updating... " + "\n" + "this might take a few                 minutes");
                loader.loadData(selectedFile.toString());
                infoTextfield.appendText("finished updating");
            }
        }).start();
    }
}

Hat jemand eine Idee wie ich das lösen kann?

 

[Cromewell]

Es ist so schwer es ohne Code-Tags zu lesen xD

 

[domjos1994]

Hallo,

deaktiviere doch einfach die Anwendung so lange der Prozess läuft.

LG Dominic

 

[Nurox]

Hallo,

deaktiviere doch einfach die Anwendung so lange der Prozess läuft.

LG Dominic

Hi, die Idee hatte ich auch, hast du gerade ein Beispiel wie das am einfachsten geht? Ich müsste ja den Text setzen dann deaktivieren und den Prozess starten, am ende dann wieder aktivieren und den neuen Text setzen.


Und sorry ich hatte die Code-Tags nicht gefunden heute morgen

 

[domjos1994]

Also ich würde den Textbereich, wenn da immer nur Informationen ausgegeben werden grundsätzlich deaktivieren oder nur lesbar machen. Dann kann dir da keiner mehr herumpfuschen.

Wie wird den der Prozess ausgelöst? Durch das Betätigen einer Schaltfläche?
Wenn ja deaktiviere doch einfach den Button und stell den Cursor als Wait-Cursor da.

LG Dominic

 

[Neumi5694]

Hinweis: Wenn du auf ein Event reagierst, so wird die GUI erst dann neu gezeichnet, sobald alles, was im Eventhandler passiert, abgearbeitet wurde.

loader.loadData(selectedFile.toString());

muss also in einen eigenen Thread und sollte nach Beenden

infoTextfield.appendText("finished updating");

auslösen.

 

[Nurox]

Also das es in einen separaten Thread muss war mir klar
Nun ich mache es nun so, dass ich einfach die Buttons disable wenn der Thread läuft und danach wieder aktiviere, so ist das erstmal ok. Danke

 

[dzim]

Schau dir vielleicht mal noch die Service-Klasse aus dem JavaFX concurrency package an. Damit kann man diese Problemstellung gut erledigen.
Wie das etwa geht, habe ich schon einmal in dem folgenden Forum-Beitrag beschrieben

http://www.java-forum.org/thema/mei...erden-nicht-aktualisiert.172464/#post-1086826

Der für dich wichtige Teil ist der FXML-Controller. Er enthält eine innere Service-Klasse (das sollte natürlich produktiv vielleicht nicht mehr so sein ). Der Button-Hander des Controllers schaut, ob der Service schon/noch läuft und wenn nicht startet er den Service (erneut).

Hier noch mal die Kopie:

FXML

<?xml version="1.0" encoding="UTF-8"?>
<!--
    Do not edit this file it is generated by e(fx)clipse from ../src/application/ProgressTest.fxgraph
-->

<?import java.lang.*?>
<?import javafx.geometry.Insets?>
<?import javafx.scene.control.Button?>
<?import javafx.scene.control.Label?>
<?import javafx.scene.control.ProgressBar?>
<?import javafx.scene.control.ProgressIndicator?>
<?import javafx.scene.control.TextField?>
<?import javafx.scene.layout.GridPane?>

http://javafx.com/fxml" fx:controller="application.ProgressTestController" alignment="CENTER" vgap="5" hgap="5">

    <Label text="HD Drive" GridPane.columnIndex="0" GridPane.rowIndex="0"/>
    <TextField promptText="pick drive" disable="true" GridPane.columnIndex="1" GridPane.rowIndex="0" GridPane.columnSpan="2"/>
    <Button text="Select ..." disable="true" GridPane.columnIndex="3" GridPane.rowIndex="0"/>
    <Label text="Drive No." GridPane.columnIndex="0" GridPane.rowIndex="1"/>
    <TextField promptText="01" disable="true" GridPane.columnIndex="1" GridPane.rowIndex="1" GridPane.columnSpan="3"/>
    <Button text="Delete Archive" disable="true" GridPane.columnIndex="0" GridPane.rowIndex="2" GridPane.columnSpan="2" GridPane.halignment="LEFT"/>
    <Button fx:id="readButton" text="Start Reading" onAction="#handleReadButton" GridPane.columnIndex="2" GridPane.rowIndex="2" GridPane.columnSpan="2" GridPane.halignment="RIGHT"/>
    <ProgressBar fx:id="progressBar" GridPane.columnIndex="0" GridPane.rowIndex="3" GridPane.columnSpan="4">
        <maxWidth><Double fx:constant="MAX_VALUE" /></maxWidth>
        <GridPane.margin>
            <Insets top="10"/>
        </GridPane.margin>
    </ProgressBar>
    <ProgressIndicator fx:id="progressIndicator" GridPane.columnIndex="0" GridPane.rowIndex="4" GridPane.columnSpan="4">
        <GridPane.margin>
            <Insets top="10"/>
        </GridPane.margin>
    </ProgressIndicator>
</GridPane>

Controller

package application;

import javafx.beans.property.LongProperty;
import javafx.beans.property.SimpleLongProperty;
import javafx.beans.value.ChangeListener;
import javafx.concurrent.Service;
import javafx.concurrent.Task;
import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.scene.control.Button;
import javafx.scene.control.ProgressBar;
import javafx.scene.control.ProgressIndicator;

public class ProgressTestController {
  
    @FXML
    private Button readButton;
    @FXML
    private ProgressBar progressBar;
    @FXML
    private ProgressIndicator progressIndicator;
  
    private MyFancyService service = null;
  
    @FXML
    public void initialize() {
      
        progressBar.setProgress(0.0);
        progressIndicator.setProgress(0.0);
      
        service = new MyFancyService(100L);
        service.progressProperty().addListener((ChangeListener<Number>) (observable, oldValue, newValue) -> {
            progressBar.setProgress(newValue.doubleValue());
            progressIndicator.setProgress(newValue.doubleValue());
        });
    }
  
    @FXML
    public void handleReadButton(ActionEvent event) {
        if (service.isRunning()) {
            System.out.println("Already running. Nothing to do.");
        } else {
            service.reset();
            progressBar.setProgress(0.0);
            progressIndicator.setProgress(0.0);
            service.start();
        }
    }
  
    private static class MyFancyService extends Service<Long> {
      
        private LongProperty countTo = new SimpleLongProperty(0L);
        private LongProperty current = new SimpleLongProperty(0L);
      
        public MyFancyService(long countTo) {
            this.countTo.set(countTo);
        }
      
        @SuppressWarnings("unused")
        public LongProperty countToProperty() {
            return countTo;
        }
      
        @SuppressWarnings("unused")
        public LongProperty currentProperty() {
            return current;
        }
      
        @Override
        protected Task<Long> createTask() {
            return new Task<Long>() {
                @Override
                protected Long call() throws Exception {
                    current.set(0L);
                    try {
                        for (int i = 0; i <= countTo.get(); i++) {
                            if (isCancelled())
                                break;
                            current.set(i);
                            updateProgress(current.get(), countTo.get());
                            Thread.sleep(100);
                        }
                    } catch (Exception e) {
                        e.printStackTrace();
                    }
                    return current.get();
                }
            };
        }
    }
}

Die Application-Klasse habe schon damals aus Gründen von zu geringen Nutzen weggelassen (nur Initialisierung des FXMLs und an eine Scene und die Stage hängen, am Ende).