Buenas Vicente,<br><br>estoy de vuelta...se que ha pasado bastante tiempo...pero estoy haciendo un esfuerzo grande para retomar el tema de Donate...<br><br>Voy a echarle un vistazo a estos comentarios para ver el camino que debe seguir donate...<br>
<br>Cualquier consideración adicional no dudes en comentarmela...<br><br>Un saludo,<br><br><div class="gmail_quote">El 14 de julio de 2011 16:25, Vicente J. Ruiz Jurado <span dir="ltr"><<a href="mailto:vjrj@ourproject.org">vjrj@ourproject.org</a>></span> escribió:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">El 01/07/11 03:06, Esteban Carreras Genis escribió:<br>
<div class="im">> Gracias Vicente, era exactamente lo que buscaba.<br>
><br>
> He subido una versión con el i18n completo, desglosado en algún widget<br>
> más, con menos código y más personalizable (aunque a esto último tengo<br>
> que darle un vuelta grande, que al final se me ha ido la cabeza un poco)...<br>
><br>
> Un saludo,<br>
<br>
</div>Hola Esteban:<br>
<br>
Perdona por el retraso en responder.<br>
<br>
Pues la verdad que veo que has avanzando bastante y que lo vas limando<br>
bastante. Muy bien. Tener clases pequeñitas, fáciles de entender y que<br>
no tengas que scrollear o casi es buena señal (compara con el principio).<br>
<br>
Lo de personalizar, quizás es algo que le debemos dar una pensada<br>
juntos, es decir, pensar en las necesidades (que cada proyecto pueda<br>
activar una zona de "donaciones") y ver como implementarlo. Me he<br>
perdido con la parte de DonateConstants, quizás con un String static te<br>
hubiese bastado ¿no?<br>
<br>
Mira, luego otra cosa interesante ya que estás poniéndote las botas con<br>
los patrones es que empieces en algún momento a pensar e implementar<br>
usando el MVC:<br>
<a href="https://secure.wikimedia.org/wikipedia/es/wiki/Modelo_Vista_Controlador" target="_blank">https://secure.wikimedia.org/wikipedia/es/wiki/Modelo_Vista_Controlador</a><br>
<br>
Si te fijas en ButtonsPanel hay lógica y al ser un Panel lo suyo sería<br>
que solo tuviese cosas de UI, y lo mínimo mínimo. Es decir, donde va<br>
cada cosa, como cambiarle valores y poco más. Incluso los handlers se<br>
pueden sacar al controlador:<br>
<a href="http://www.nieleyde.org/SkywayBlog/post.htm?postid=e49efd5b-31c8-423f-9fff-91c2ba6514ed" target="_blank">http://www.nieleyde.org/SkywayBlog/post.htm?postid=e49efd5b-31c8-423f-9fff-91c2ba6514ed</a><br>
<br>
Esto entre otras cosas lo que te hace es que tengas todo lo complicado,<br>
la lógica, bien delimitada en una clase, y que la puedas probar bien<br>
(con JUnit y similares), etc. Probar programáticamente el Widget (con el<br>
Panel) es más coñazo y cuesta mucho hacerlo (a veces tienes incluso que<br>
arrancar un navegador o un emulador). No se si me explico.<br>
<br>
Así que te invito si quieres a quitar toda la lógica de ButtonsPanel y a<br>
que a tu nuevo ButtonsPresenter, le añadas tests con Junit para estar<br>
seguro de que hace lo que tiene que hacer.<br>
<br>
En cualquier caso, muy bien Esteban,<br>
<font color="#888888"><br>
--<br>
Vicente<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Esteban Carreras Genis <br><br>