Bad Smell ou é impressão minha?

Exemplo extremamente resumido:

A classe B é-um I
O objeto da classe B tem-um A.
O objeto da classe A tem-um I.

Na pratica eu tenho
B que tem A que tem uma instância de B (sob forma de I)

Pra ser mais exato:
B é a minha View
I é a minha IView
A é o meu Presenter/Controller

Isso tem Bad Smell, pode ser refatorado ou eu to vendo chifre em cabeça de cobra?

GUJ Hacker Membro desde: 08/07/2007 00:08:14 Mensagens: 6396 Offline

Se B eh sua view, e A eh sua controller, não é mais conveniente o A ter o B, ao inves do B ter um A???

ou seja, pra view não tanto faz quem é o controller dela?

JavaEvangelist Membro desde: 26/04/2005 11:18:25 Mensagens: 469 Offline

É difícil dar pitaco sabendo tão pouco do seu código, mas a princípio eu usaria uma coisa mais convencional. Uma Factory de Is, por exemplo, ou até uma injeção de dependências. Aí você não tem que expor um setter pra A.i. (o que também não é o fim do mundo, já que você pode ignorar ou responder com exceção a sets seguidos)

This message was edited 1 time. Last update was at 13/11/2007 12:18:37

Exemplo muito simplificado de um sistema em produção usando pattern MVCP - nada disso é ideia minha

/* Ok, essa é a nossa classe para representar Noticias. Bem simples. */
class News implements INews{
    private String message;
    public News(String message){
        this.message = message;
        new Throwable().printStackTrace();
    }
    public String getMessage(){ return message; }
}
/* INews talvez não tenha muito sentido agora */
interface INews{
    String getMessage();
}
/* Esta factory seria responsavel por continuar o trabalho de buscar a(s) noticia(s) do Banco de dados */
class INewsFactory{
    public static INews create(){
        INews n = new News("Este eh mais um exemplo confuso");
        return n;
    }
}
/* Um controller "burro" - limei a parte que salva os dados */
class MyController{
    public INews loadNews(){
        return INewsFactory.create();
    }
}
/* O Presenter: até onde eu vi é uma 'camada' de apresentação, a promessa é que a View se tornaria extremamente simples. */
class MyPresenter {
    private IMyView view;
    private MyController controller;
    public MyPresenter(){
        controller = new MyController();
    }
    /* Tambem concordo que esta parte tambem poderia estar no construtor... */
    public void setView(IMyView view){ 
        this.view = view;
    }
    /* Ideia: quando a minha view carregar, este método será executado*/
    public void onViewInitialized(){
        view.setNews(controller.loadNews()); /* carrego a noticia*/
        view.list(); /* mostro a noticia */
    }
}
/* Outra Interface que agora parece não fazer muito sentido... */ 
interface WebPage{
    void pageLoad();
}
/* No futuro sera refatorado para o uso de generics... */
interface IMyView{
    void list();
    void setNews(INews news);
}
/* Ah, a visão em si */
class MyView implements WebPage, IMyView{
    private MyPresenter presenter;
    private INews news;
    /* toda a vez que a pagina carregar... */
    public void pageLoad(){
        presenter = new MyPresenter();
        presenter.setView(this);
        presenter.onViewInitialized();
    }
    /* recebo a noticia */
    public void setNews(INews news){
        this.news = news;
    }
    /* mostro a noticia */
    public void list(){
        System.out.println("message: " + news.getMessage());
    }
}
/*Esta classe só simula tudo isso*/
public class Main{
    public static void main (String[] x){
        WebPage v = new MyView();
        v.pageLoad();
    }
}

Executando (lanço um throwable só para mostrar por onde passou desde o Main.main):

A duvida: fora o que acontece na INewsFactory (que funciona a base de magian negra na vida real) este modelo tem

o presenter tem um controller e tem uma view (na verdade uma IView)
a view tem um presenter

Bad Smell ?

This message was edited 1 time. Last update was at 13/11/2007 16:04:47

pode até não cheirar mau, mas é no minimo confuso.

Notícias, artigos e o maior fórum brasileiro sobre Java