Hello world!
Tentei fazer um MVC bem simples em JavaFX. Meu modelo é uma classe Pessoa
que possui nome
e idade
(o campo idade
no caso não chega a ser usado). Dois text fields representam duas visões. Por simplicidade ambas mostram a mesma coisa: o nome da pessoa. Se você editar um dos nomes e der Enter, a mudança se reflete no modelo e por tabela nos dois text fields novamente.
Perguntas:
Este código está em MVC?
Quem é o controlador?
Não deveria haver uma classe para o modelo chamada "Modelo"?
Não deveria haver uma classe para o controlador chamada "Controlador"?
O método
start()
não deveria ser o mais simples possível, consistindo somente em ligar as partes envolvidas (Modelo, Visões e Controlador)?Quais outras críticas podem ser feitas a essa tentativa de MVC?
App.java
package piovezan.mvcdesktop;
import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.geometry.Orientation;
import javafx.scene.Scene;
import javafx.scene.control.TextField;
import javafx.scene.layout.FlowPane;
import javafx.stage.Stage;
public class App extends Application {
private final Pessoa pessoa = new Pessoa("Mario", 39);
public static void main( String[] args ) {
launch(args);
}
@Override
public void start(Stage stage) {
atribuirTituloAoPalco("Hello World!", stage);
TextField visao1 = criarTextField(pessoa.getNome(), e -> pessoa.setNome(((TextField)e.getSource()).getText()));
TextField visao2 = criarTextField(pessoa.getNome(), e -> pessoa.setNome(((TextField)e.getSource()).getText()));
pessoa.adicionarObservadorDoNome(nome -> visao1.setText(nome));
pessoa.adicionarObservadorDoNome(nome -> visao2.setText(nome));
FlowPane root = new FlowPane(Orientation.VERTICAL);
root.getChildren().add(visao1);
root.getChildren().add(visao2);
stage.setScene(new Scene(root, 300, 250));
stage.setOnCloseRequest(v -> pessoa.removerObservadores());
stage.show();
}
private void atribuirTituloAoPalco(String titulo, Stage stage) {
stage.setTitle(titulo);
}
private TextField criarTextField(String texto, EventHandler<ActionEvent> eventHandler) {
TextField textField = new TextField();
textField.setText(texto);
textField.setOnAction(eventHandler);
return textField;
}
}
Pessoa.java
package piovezan.mvcdesktop;
import java.util.HashSet;
import java.util.Set;
/**
* É responsabilidade do implementador da classe escolher tipos imutáveis para os campos que serão observados
* por cada observador (exemplo: String para nome, Integer para idade) caso contrário o observável irá "vazar"
* estado para o observador.
*/
public class Pessoa {
private String nome;
private int idade;
private final Set<Observador<String>> observadoresDoNome = new HashSet<>();
private final Set<Observador<Integer>> observadoresDaIdade = new HashSet<>();
public Pessoa(String nome, int idade) {
this.nome = nome;
this.idade = idade;
}
public void setNome(String nome) {
this.nome = nome;
notificarQueNomeMudou();
}
public void setIdade(int idade) {
this.idade = idade;
notificarQueIdadeMudou();
}
public String getNome() {
return nome;
}
public int getIdade() {
return idade;
}
public void adicionarObservadorDoNome(Observador<String> observador) {
observadoresDoNome.add(observador);
}
public void adicionarObservadorDaIdade(Observador<Integer> observador) {
observadoresDaIdade.add(observador);
}
public void notificarQueNomeMudou() {
for (Observador<String> observador: observadoresDoNome) {
observador.notificar(nome);
}
}
public void notificarQueIdadeMudou() {
for (Observador<Integer> observador: observadoresDaIdade) {
observador.notificar(idade);
}
}
public void removerObservadores() {
observadoresDoNome.clear();
observadoresDaIdade.clear();
}
}
Observador.java
package piovezan.mvcdesktop;
@FunctionalInterface
public interface Observador<T> {
void notificar(T observavel);
}
EDIT:
Analisando a resposta e comparando a um exemplo de MVC com FXML achei um outro exemplo sem FXML que veio a calhar. Baseei-me nele, porém sem usar objetos *Property
do JavaFX, e tenho agora o seguinte código:
App.java
package piovezan.mvcdesktop;
import javafx.application.Application;
import javafx.geometry.Orientation;
import javafx.scene.Scene;
import javafx.scene.layout.FlowPane;
import javafx.stage.Stage;
public class App extends Application {
public static void main( String[] args ) {
launch(args);
}
@Override
public void start(Stage stage) {
Pessoa pessoa = new Pessoa("Mario", 39);
Controlador controlador1 = new Controlador(pessoa);
Visao visao1 = new Visao(pessoa, controlador1);
Controlador controlador2 = new Controlador(pessoa);
Visao visao2 = new Visao(pessoa, controlador2);
FlowPane root = new FlowPane(Orientation.VERTICAL);
root.getChildren().add(visao1.comoComponente());
root.getChildren().add(visao2.comoComponente());
atribuirTituloAoPalco("Hello World!", stage);
stage.setScene(new Scene(root, 300, 250));
stage.setOnCloseRequest(v -> pessoa.removerObservadores());
stage.show();
}
private void atribuirTituloAoPalco(String titulo, Stage stage) {
stage.setTitle(titulo);
}
}
Controlador.java
package piovezan.mvcdesktop;
public class Controlador {
private final Pessoa pessoa;
public Controlador(Pessoa pessoa) {
this.pessoa = pessoa;
}
public void atualizar(String nome) {
pessoa.setNome(nome);
}
}
Visao.java
package piovezan.mvcdesktop;
import javafx.scene.control.TextField;
import javafx.scene.layout.FlowPane;
public class Visao {
private FlowPane visao;
private TextField campoNome;
private Pessoa pessoa;
private Controlador controlador;
public Visao(Pessoa pessoa, Controlador controlador) {
this.pessoa = pessoa;
this.controlador = controlador;
criarEConfigurarPane();
criarEDisporControles();
atualizarControladorAPartirDeListeners();
observarModeloEAtualizarControles();
}
private void criarEConfigurarPane() {
visao = new FlowPane();
}
private void criarEDisporControles() {
campoNome = new TextField();
campoNome.setText(pessoa.getNome());
visao.getChildren().add(campoNome);
}
private void atualizarControladorAPartirDeListeners() {
campoNome.setOnAction(v -> controlador.atualizar(campoNome.getText()));
}
private void observarModeloEAtualizarControles() {
pessoa.adicionarObservadorDoNome(nome -> campoNome.setText(nome));
}
public FlowPane comoComponente() {
return visao;
}
}
Ficou melhor? Agora Visão e Controlador são classes e o método start()
foi simplificado para apenas ligá-los entre si e ao Modelo.
Por esse código concluo que vai custar um pouco pro padrão entrar na cabeça porque eu começo pelo Modelo que é desconexo dos demais, em seguida vou para a classe Visão que é o início do fluxo, passo pelo Controlador e retomo a codificação da Visão. Fica meio não-linear o desenvolvimento se seguir por essa linha.
Notei também que a classe Controlador
ficou completamente dispensável, está certo isso?
Em tempo, o modelo deve ser observável via de regra?
App
faria o papel do presenter e a view parece estar "escondida" porque você não faz uso de um código separado para apresentar os dados. Mesmo pra MVP não tá bem de acordo porque a view e o presenter ficam misturados.