eduhern/Android — pull request #2 by nhpatt (2013-04-20 20:45:58+00:00)
Title: Cambios propuestos

El error es por demasiadas líneas en un método (épico). He hecho algún inline de variables y ya ha ido.

Cosas:
* Un método con miles de líneas xd
* Tienes dos imágenes con el mismo nombre imagen68 creo
* Tienes algunos recursos que fingen ser pngs pero son jpgs, android -> lint, check from common errors te lo dice. El resto de errores son falsos.
* Ojea los cambios que te envío, si no les entiendes preguntame (están un poco liosos porque he eliminado los .svn)
* Usas en las vistas alpha, textlsselectable, showAsAction que no existen en Android 2.2
* Tienes alguna cadena sin extraer en la pantalla de preferencias
* No llames a las cosas textviewAyuda, solo ayuda
* NADA de guiones bajos en java (id_c, nom_categoria, imagen_fin,descripcion_ext
* Elimina comentarios TODO por ahí
* Mira los cambios de QuestionActivity (Cómo cuando haces ((TextView) findViewById(R.id.puntuacion1)).setBackgroundDrawable(getResources().getDrawable(
- getResources().getIdentifier("puntuacion_bad", "drawable", getPackageName())));)
* MediaPlayer musica;
-
- if (fin) {
- musica = MediaPlayer.create(this, R.raw.teletransporte);
- } else {
- if (genero) {
- musica = MediaPlayer.create(this, R.raw.foto);
- } else {
- musica = MediaPlayer.create(this, R.raw.radar);
-
- }
- } es en realidad:
final MediaPlayer musica = MediaPlayer.create(this,
+ fin ? R.raw.teletransporte : genero ? R.raw.foto : R.raw.radar);

* Mira las mejoras de código en porcentajes
* Se te olvidan privates en variables
* Por qué has implementado Parcelable??? No me digas que para enviar valores... probaste Serializable?

Sugerencias:
* Versión en inglés
* El botón de chicas, chicos es confuso