Array de objetos en Java nullPointer

[hectoritr]

Hola estimados colegas :).

Escribo porque se me presentó un error en mi código y no he podido resolverlo. Soy lector del foro desde hace tiempo pero nunca me animé a escribir hasta ahora.

Tengo una clase Picto, que contiene una imagen, un texto y un array de clases (serían sus hijos, son del mismo tipo) de esta forma se forma un árbol de relaciones.

public class Picto {
protected Drawable Icono;
protected String Oracion;
protected Picto[] Hijos;

public Picto(Drawable icono, String oracion, Picto[] hijos) {
super();
this.Icono = icono;
this.Oracion = oracion;
this.Hijos = hijos;
}

public Drawable getIcono() {return Icono;}

public String getOracion() {return Oracion;}

public Picto[] getHijos() {return Hijos;}

public Picto getQueHijo(int i) {return Hijos[i];}
}

Todo funciona perfecto hasta que quiero acceder a los hijos de mi Picto, de esta forma


Opcion1.setImageDrawable(pictograma.getQueHijo(1). getIcono());

Ahi me arroja un nullPointerException y la verdad no se que estoy haciendo mal.

Estoy a su disposición cualquier duda que haya dejado mi comentario.

Muchas gracias

[Profesor_Falken]

Opcion1.setImageDrawable(pictograma.getQueHijo(1). getIcono());

Imposible de saber con lo que has puesto. Habria que ver el codigo cuando llamas al constructor y que es lo que le pasas como parametro. Quizas no inicializas el array de hijos? No basta con declararlo.


P.D: Por favor, intenta formatear el codigo en proximos posts utilizando la etiqueta hightlight.


Un saludo

[hectoritr]

Gracias por la respuesta, genial, empezare a utilizar la etiqueta.

Código Java:

public class Picto {
    protected Drawable Icono;
    protected String Oracion;
    protected Picto[] Hijos;
 
    public Picto(Drawable icono, String oracion, Picto[] hijos) {
        super();
        this.Icono = icono;
        this.Oracion = oracion;
        this.Hijos = hijos;
        Log.e("TAG","Se creo un Picto "+oracion);
    }
 
    public Drawable getIcono() {Log.e("TAG","Devuelvo un drawable");return Icono;}
 
    public String getOracion() {Log.e("TAG","Devuelvo una oracion");return Oracion;}
 
    public Picto[] getHijos() {return Hijos;}
 
    public Picto getQueHijo(int i) {Log.e("TAG","Devuelvo un hijo");return Hijos[i];}
 

Código Java:

//Declaro mis pictos
 
    Picto Hola;
    Picto Chau;
    Picto Bien;
    Picto Comoestas;
    Picto Jabon;
    Picto Dos;
    Picto Tres;
    Picto Cuatro;
 
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
        //Aquí implemento los picto, les asigno un icono, un texto y sus hijos. Es la forma correcta?
        Hola=new Picto(getResources().getDrawable(R.drawable.ic_hola),"Hola", new Picto[]{Jabon, Aspirina, Digestivo, Descongestivo});
        Comoestas=new Picto(getResources().getDrawable(R.drawable.ic_comoestas),"Como estas?", new Picto[]{Jabon, Aspirina, Digestivo, Descongestivo});
        Bien=new Picto(getResources().getDrawable(R.drawable.ic_bien),"Bien", new Picto[]{Jabon, Aspirina, Digestivo, Descongestivo});
        Chau=new Picto(getResources().getDrawable(R.drawable.ic_adios),"Chau", new Picto[]{Jabon, Aspirina, Digestivo, Descongestivo});
        Jabon = new Picto(getResources().getDrawable(R.drawable.ic_jabon),"Jabon",new Picto[]{Hola, Comoestas, Bien, Chau});
        Aspirina = new Picto(getResources().getDrawable(R.drawable.ic_aspirina),"Aspirina",new Picto[]{Hola, Comoestas, Bien, Chau});
        Digestivo = new Picto(getResources().getDrawable(R.drawable.ic_digestivo),"Digestivo",new Picto[]{Hola, Comoestas, Bien, Chau});
        Descongestivo = new Picto(getResources().getDrawable(R.drawable.ic_descongestivo),"Descongestivo",new Picto[]{Hola, Comoestas, Bien, Chau});
 
}
        //Luego tengo esta función que me pone un imagen correspondiente al primer hijo, de fondo de mi boton. 
 
        void CargarHijos(Picto pictograma){
        Opcion1.setImageDrawable(pictograma.getQueHijo(1).getIcono());
        Opcion1.startAnimation(AnimationUtils.loadAnimation(this, R.anim.overshoot_abajo));
        Opcion2.setImageDrawable(pictograma.getQueHijo(2).getIcono());
        Opcion2.startAnimation(AnimationUtils.loadAnimation(this, R.anim.overshoot_abajo));
 
}
     @Override
        public void onClick(View v) {
        String Saludo = " ";
        String SaludoChau = " ";
        switch (v.getId()) {
            case R.id.Opcion1:
                CargarSeleccion(PictoOpcion2);
                CargarHijos(PictoOpcion2);
                CantClicks++;
                break;
            case R.id.Opcion2:
                CargarOracion(PictoOpcion2);
                CargarSeleccion(PictoOpcion2);
                CargarHijos(PictoOpcion2);
                CantClicks++;
                break;
 

Además puse unas entradas en el LOG para ver que está ocurriendo.

y el Log>
03-04 13:57:01.147 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Hola
03-04 13:57:01.177 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Como estas?
03-04 13:57:01.207 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Bien
03-04 13:57:01.227 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Chau
03-04 13:57:01.247 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Jabon
03-04 13:57:01.307 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Aspirina
03-04 13:57:01.377 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Digestivo
03-04 13:57:01.427 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Se creo un Picto Descongestivo
03-04 13:57:27.485 30164-30164/com.stonefacesoft.ottaa E/TAG﹕ Devuelvo un hijo


Ese es el error
java.lang.NullPointerException
at com.stonefacesoft.ottaa.Principal.CargarHijos(Prin cipal.java:272)
at com.stonefacesoft.ottaa.Principal.onClick(Principa l.java:301)

[Profesor_Falken]

Buenas,

Ok. Para comenzar comentarte que en java los arrays comienzan en 0 y no en 1. Por lo que si quieres acceder al primer hijo tienes que hacer un getQueHijo(0). Sin embargo esto no debería ser la causa del nullpointer.

Sería interesante que sepas cual es el elemento que está fallando. Te propongo que lo escribas en el log (aunque lo perfecto seria que depurases el código).

Código Java:

TULOG("Voy a obtener el hijo de " +  pictograma.getOracion());
Opcion1.setImageDrawable(pictograma.getQueHijo(1).getIcono());
 

Una cosa que me extraña es que creas los pictogramas que se llaman:
Hola
Comoestas
Bien
Chau
Jabon
Aspirina
Digestivo
Descongestivo

Sin embargo, intentas cargar los hijos de un pictograma llamado ¿PictoOpcion2?? De donde sale?

Código Java:

CargarHijos(PictoOpcion2);
 

Un saludo