bpingar/Arias — pull request #1 by nhpatt (2013-03-09 13:20:44+00:00)
Title: Pequeños cambios

Son cambios menores, porque en general está bastante bien.

Algunas cosillas:
* Los modelos intenta hacerlos siempre serializables, así puedes pasarles con Intents
* Compara siempre cadenas al revés en vez de variable.equals("") haz "".equals(variable), así evitas Null Pointers.
* Ten cuidado con el copy paste en vistas, los linear layouts no tienen porque llevar la etiqueta xmlns:android="http://schemas.android.com/apk/res/android" siempre, sólo los elementos que son la raíz de la página, el primer elemento.
* En comprobaciones de if else, se suele poner en el if la comprobación positiva y en el else la negativa, es decir if (lista.isEmpty() else {}. No if ( !lista.isEmpty() then

Esto son las cosas menores, lo más importante, por orden de importancia:
* Tienes el menú por todas las actividades, si va a ser el mismo, crea una clase MenuActivity que te cree el menú general y de la que hereden todas las actividades que tengan menú. Si alguna tiene algo un poco diferente, que lo sobreescriba.
* Tienes la lógica de recuperar usuario duplicada, haz una clase que sea llamada por las dos actividades.
* El flujo es un poco confuso aunque entiendo el propósito de elegir usuario, el diseño es mejorable :P
* Si vas a usar el mismo layout custom para dos tipos de objetos no hace falta que le definas dos veces :)

En general está muy bien ( :+1: ), puntos extra por usar strings formateadas con parámetros y descubrir cómo se usaba la pantalla de preferencias.

Si necesitas ayuda con mejoras, diseño o cualquier cosa, ya sabes :)