Javafx, почему событие так (не)отрабатывает?

Question

[Скорпион]

Я не могу изменять внешние переменные, intellij ругается.

Button button=new Button();
        String s;
        button.setOnAction(e->{
            s="";
        });

Говорит, что нужно вот так:

Button button=new Button();
        AtomicReference<String> s = null;
        button.setOnAction(e->{
            s.set("");
        });

Или так:

Button button=new Button();
        final String[] s = new String[1];
        button.setOnAction(e->{
            s[0] ="";
        });

Если же использовать FXML+контроллер, то проблем не возникает, я спокойно изменяю внешние переменные, почему это так работает?
Полагаю если я создам какой-то класс, унаследую его от EventHandler и переопределю метод события, а в button засуну этот самый класс как слушателя, то все тоже сработает.

Код:
Main.class

public class Main extends Application {

    @Override
    public void start(Stage primaryStage) throws Exception{
        FXMLLoader fxmlLoader =new FXMLLoader(getClass().getResource("my.fxml"));
        Parent root= fxmlLoader.load();
        LoginWindowController loginWindowController= fxmlLoader.getController();
        loginWindowController.initialise(primaryStage);
        primaryStage.setScene(new Scene(root));
        primaryStage.show();
    }


    public static void main(String[] args) {
        launch(args);
    }
}

myController.class

public class myController{
@FXML
 public void searchButtonPressed(MouseEvent mouseEvent) {
CreateNewWindow newWindow = new CreateNewWindow();
newWindow.create();    

}

}

public class CreateNewWindow{
create(){
Stage stage =new Stage();
Vbox v= new Vbox();
Button button =new Button("Press me");
v.getChildren().add(button);
stage.setScene(new Scene(v));
stage.show();

String str;
button.setOnAction(e->{
str="";  //Ругается
//Так - же отсюда я не могу вызвать никакой метод, который требует параметры;
});

}
}

Answer 1

[Дмитрий Александров]

Недостаточно кода чтобы точно объяснить смысл поведения но в целом как то так.
- Не забывайте что javafx крутится в своем потоке.
- объявите ваш String как class member .
- касательно именно javafx то гляньте SimpleStringProperty и другие типы созданные специально удобства.

Ну а конкретно у вас следующая ситуация.
вы используете лямбду что в общем то обычный "вложенный анонимный класс" . В таких ситуациях любая переменная обязана быть final или иметь тип Atomic. Все это происходит из за проблем синхронизации.
Как вариант вы можете использовать немного другой подход и гдето внутри javafx(в случае с fxml) это работает примерно также:

String s;
private void doSome(){
 s="";
}
Button button=new Button();
button.setOnAction(e->{
  doSome();
});

Comments

[Скорпион]

Суть таково, у меня есть главный Stage и главный контроллер, из главного контроллера я вызываю Класс CreateNewWindow, в этом классе , я только кодом создаю новую stage, добавляю на нее кнопку, и показываю второе окно пользователю.

[Дмитрий Александров]

Скорпион, код, пока это гадание на кофейной гуще

[Скорпион]

Дмитрий Александров, добавил код к вопросу.

[Дмитрий Александров]

Скорпион, ужас =)
Почитайте про модальные окна и диалоги, они есть и они замечательные https://code.makery.ch/blog/javafx-dialogs-official/

public class CreateNewWindow{
String str;
create(){
...

Кроме того не забывайте что в вашем случае CreateNewWindow понятия не имеет о содержимом других классов.

[Скорпион]

Дмитрий Александров, ужас ужас, я переделал все, сделал разметку через FXML, обработку событий через контроллер. И да, модальное окно слишком сложное чтобы я использовал это

Почитайте про модальные окна и диалоги, они есть и они замечательные https://code.makery.ch/blog/javafx-dialogs-official/

Куда проще управлять всем через контроллер и новое окно,ибо у меня там несколько событий , которые зависят друг от друга, поля ввода , кнопки, метки, чекбоксы и прогресс бар...

public class CreateNewWindow{
String str;
create(){
...

Не работает все равно... Не дает мне программа выбрасывать данные вне обработчика....

[Дмитрий Александров]

Скорпион, это плохая практика, код должен быть максимально независимым между классами по принципу "черного ящика".
Касательно взаимосвязи компонентов, правильный вариант это использовать Property и Observable и связки bind между ними. Это в первую очередь избавит от простыней кода взаимодействия между компонентами. Бонусом получите готовую событийную модель для них, т.е. на property можно вешать слушателей.
Использовать fxml конечно удобно, но это накладывает некоторые задержки, общую тяжесть и некоторые ограничения с неудобствами(опять же нет конструктора в таком случае).