» Правильный код на java

[more] [more] Здравствуйте, некого спросить, поэтому здесь и сразу к сути.
Краткое описание: процедурка создаёт парами текстовое поле и кнопку. На кнопке событие по удалению этого текстового поля.
И меня смущает процедура в процедуре и не забивает ли такое память?


Код:
void proc(String key){

TextField textField1=new TextField();
Button button=new Button();

vBox.getChildren().add(textField1);
vBox.getChildren().add(button);


EventHandler<ActionEvent> EventRemove=new EventHandler<ActionEvent>() {
@Override
public void handle(ActionEvent event) {
vBox.getChildren().remove(textField1);
}
};

if(key!=null){
button.setOnAction(EventRemove);
button.setText("X");
}else{
button.setText("ок");
button.setOnAction(new EventHandler<ActionEvent>() {
@Override
public void handle(ActionEvent event) {
if(!textField1.getText().isEmpty()){
button.setOnAction(EventRemove);
button.setText("X");
}
}
});
}}

Это не процедура в процедуре.
По сути Вы объявляете inner class у которого нет имени и который реализует интерфейс EventHanlder. И внутри создаете переменную которой присваиваете созданный объект данного класса


Это нормально для Java

Память забивает, если таких кнопок будет много. Но Java провоцирует не обращать внимание на такие мелочи На самом деле, есть техники кодирования, которые позволяют экономно выделять память, но это - отдельная большая тема.