I think it's a very good explanation on MVP. But one thing that comes to
mind is that they should explain why they check for nulls all the time. I
think the if statements should be followed by else, at lest empty ones!
explaining why it is ok to ignore the call.
ex.
if (presenter != null) {
presenter.onDeleteButtonClicked();
2010/4/22 interdev <[email protected]>
> Hello everyone,
> have you seen the new MVP Architecture article from google ?
> http://code.google.com/webtoolkit/articles/mvp-architecture-2.html
>
> they have changed the structure slightly, instead of having a View and
> a Presenter which has an inner Interface (Display), aside from the
> UiBinder part, they now have a View interface which has an inner
> Presenter interface !
>
> 1- what do you all think ? are these yet another level of abstraction/
> indirection necessary ?
>
> 2- when google wants to address problem of Nested/Layered presenters ?
> header/body/footer, and body having its own dockpanellayout structure.
>
> 3- what do you think of "presenter.go(container)" ? and navigation/
> history token inside multiple IF statements ?
>
> i think these are more serious to address than introducing an
> additional view interface which has an inner interface !
>
> 4- what do you think of DTO solution presented in the article, is it
> scalable ? do you follow it or stick with your twig, gilead
> framework ?
>
> --
> You received this message because you are subscribed to the Google Groups
> "Google Web Toolkit" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected]<google-web-toolkit%[email protected]>
> .
> For more options, visit this group at
> http://groups.google.com/group/google-web-toolkit?hl=en.
>
>
--
You received this message because you are subscribed to the Google Groups
"Google Web Toolkit" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/google-web-toolkit?hl=en.
