[SOLUCIONADO] findViewById me retorna NULL, porque?!?!

[jordy48]

Hola buen dia, verán, soy un poco nuevo en la programación en android y mientras practicaba un poco las animaciones me topé con este problema: MI IMAGEN ERA NULA (A la hora de ejecutar me lanzaba un NullPointerException) y viendo que era encontré que mi imagen no estaba inicializada, FindViewByID no me regresaba nada!!! y no encuntro el porque!!!

Mi Actividad Principal:

Código Java:

public class MainActivity extends Activity {
    private ImageView imagen;
    private AnimationDrawable animacion; 
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        animacion = (AnimationDrawable) getResources().getDrawable(R.drawable.animacion1);
        imagen = (ImageView) findViewById(R.id.imageView1);
        if (imagen == null) Log.d("PRUEBA","ES NULA LA IMAGEN");
        imagen.setBackgroundColor(Color.WHITE);
        imagen.setImageDrawable(animacion); 
        imagen.setOnClickListener(
                new OnClickListener() {
                    public void onClick(View v) {
                        animacion.start();
                    }
                }
        );
        setContentView(R.layout.activity_main);
    }
}
 

y el Layout de mi actividad principal es este:

Código XML:

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:paddingBottom="@dimen/activity_vertical_margin"
    android:paddingLeft="@dimen/activity_horizontal_margin"
    android:paddingRight="@dimen/activity_horizontal_margin"
    android:paddingTop="@dimen/activity_vertical_margin"
    tools:context=".MainActivity" >
 
    <ImageView
        android:id="@+id/imageView1"
        android:layout_width="130dp"
        android:layout_height="160dp"
        android:layout_centerHorizontal="true"
        android:layout_centerVertical="true" />
 
</RelativeLayout>
 

Si necesitan que suba el código de la animación, lo subo, aunque no lo consideré necesario pues la animación no me da problemas, el problema es la imagen

NOTA: Antes intenté usar, imagen = new ImageView(this); y en setContentView(imagen) le mandaba la imagen, pero esto hacía que la animacion usara la pantalla completa, y yo quiero que sea una parte en especifico, por eso estoy usando la imagen vacía en el Layout y en el onCreate le asigno su contenido

AYUUDAAAAA!!!

Gracias

[razpeitia]

Primero tienes que setear el layout.

Código Java:

public class MainActivity extends Activity {
    private ImageView imagen;
    private AnimationDrawable animacion; 
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
 
        animacion = (AnimationDrawable) getResources().getDrawable(R.drawable.animacion1);
        imagen = (ImageView) findViewById(R.id.imageView1);
        if (imagen == null) Log.d("PRUEBA","ES NULA LA IMAGEN");
        imagen.setBackgroundColor(Color.WHITE);
        imagen.setImageDrawable(animacion); 
        imagen.setOnClickListener(
                new OnClickListener() {
                    public void onClick(View v) {
                        animacion.start();
                    }
                }
        );
    }
}
 

[jordy48]

Amigo eres grande!!!

Nunca pensé que esa podía ser la causa, pero asi fue y no puedo creer que haya sido algo tan sensillo!!!


Aun que me gustaría saber el porque de que se tenga que setear primero el Layout, según me hace pensar mi lógica es porque al setear, se crea el objeto del XML, y asi ya podemos acceder a los elementos, PIENSO, pero me gustaría me explicaras el porque es tan importante ese orden dentro del código para que funcione

[razpeitia]

No se si haz notado que R.id contiene todos los id de todos los layouts, entonces tienes que primero indicarle el layout y después el id en ese layout. Básicamente es para darle contexto, porque podrías tener el id imageView1 en diferentes layouts y no sabría a cual de todos te refieres.

[jordy48]

Cierto, no lo había visto asi, pero entonces me puedes decir que en mi programa puedo tener 3 ImageView con el mismo ID (ImageView1) y estos en diferentes actividades y esto no me causará ningún problema??

Es buena esta práctica? o lo mejor es darle un ID claro a cada View?

Saludos desde Puebla

[razpeitia]

Iniciado por jordy48

Cierto, no lo había visto asi, pero entonces me puedes decir que en mi programa puedo tener 3 ImageView con el mismo ID (ImageView1) y estos en diferentes actividades y esto no me causará ningún problema??

Exacto, no causara ningun problema.

Iniciado por jordy48

Es buena esta práctica? o lo mejor es darle un ID claro a cada View?

No, no es una buena practica de hecho enumerar tus variables, como var1, var2 ya es considerado una mala practica. De preferencia es mejor darle un nombre adecuado a cada view, recuerda tiene que ser único dentro del layout.

[jordy48]

Muchas gracias nuevamente.

Oye, continue practicando y ahora me sale este error, no se ni que debo hacer, no se si tambien podrías ayudar. Hice mi propio View y lo jale desde la paleta para editar sus propiedades en XML




Layout Principal:

Código XML:

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/LinearLayout1"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    android:paddingBottom="@dimen/activity_vertical_margin"
    android:paddingLeft="@dimen/activity_horizontal_margin"
    android:paddingRight="@dimen/activity_horizontal_margin"
    android:paddingTop="@dimen/activity_vertical_margin"
    tools:context=".EjemploGraficosActivity" >
 
    <com.example.ejemplograficos.EjemploView2
        android:id="@+id/ejemploView1"
        android:layout_width="200dp"
        android:layout_height="100dp" />
 
    <com.example.ejemplograficos.EjemploView2
        android:id="@+id/ejemploView2"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent" />
 
</LinearLayout>
 

Codigo Java de la actividad principal:

Código Java:

public class EjemploGraficosActivity extends Activity {
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main_ejemplo_graficos);
    }
}
 

El nuevo View que defini:

Código Java:

public class EjemploView2 extends View{
    private ShapeDrawable miImagen;
    
    EjemploView2(Context context, AttributeSet attrs) {
        super(context, attrs);
        miImagen = new ShapeDrawable(new OvalShape());
        miImagen.getPaint().setColor(0xff0000ff);
    }
    
    @Override
    protected void onSizeChanged(int ancho, int alto, int ancho_ant, int alto_ant) {
        super.onSizeChanged(ancho, alto, ancho_ant, alto_ant);
        miImagen.setBounds(0,0,ancho,alto);     
    }
    
    @Override
    protected void onDraw(Canvas canvas) {
        super.onDraw(canvas);
        miImagen.draw(canvas);
    }
}
 

GRACIAAAAS!!