Errror NullPointer Exeception

[lutxi]

Buenas...

Tengo dos clases (Mapa y tablero). Desde Tablero llamo a Mapa para que me devuelva y asigne el valor de una matriz a una matriz que he definido dentro de Mapa... Al recorrer la matriz para que se llene de valores, me da error a ejecutar ... un NullPointerException...

Código:

Clase Mapa
Public class Mapa
{
public int [][] mapa;

    public Mapa()
    {
      int[][] mapa={{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},{0,2,1,1,1,0,1,1,1,1,1,1,1,1,0,1,1,1,2,0},{0,1,0,0,1,0,1,0,0,0,0,0,0,1,0,1,0,0,1,0},{0,1,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0},{0,1,0,1,0,0,1,0,0,3,3,0,0,1,0,0,1,0,1,0},{0,1,1,1,1,1,1,0,3,3,3,3,0,1,1,1,1,1,1,0},{0,1,0,1,0,0,1,0,0,0,0,0,0,1,0,0,1,0,1,0},{0,1,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0},{0,1,0,0,1,0,1,0,0,0,0,0,0,1,0,1,0,0,1,0},{0,2,1,1,1,0,1,1,1,1,1,1,1,1,0,1,1,1,2,0},{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};

    }
public int getMap(int  fil, int col)
 {
              
        return mapa[fil][col];
 }
}
public class Tablero extends JPanel implements ActionListener
{
Mapa plano;
int posX=105;
int posY=100;
int [][] mapa;
public Tablero()
    {
        mapa= new int [11][20];
rellenar()
}
public void rellenar()
    {
        for (int i=0; i<11; i++)
            {
                for (int j=0; j<20; j++)
                {
                        mapa[i][j]=plano.getMap(i,j); // Error NullPointerException
                        System.out.print(mapa[i][j]);
                }
            }
    }
}

[Fuzzylog]

Mapa plano;
...
public Tablero()
{
plano = new Mapa(); // Te faltaría esto
mapa= new int [11][20];
rellenar()
}

...
mapa[i][j]=plano.getMap(i,j); // Error NullPointerException

[lutxi]

Gracias, pero hay algo más que falla...

Ya que ahora después de instanciarlo... el valor que me imprime en mapa[i][j] siempre es 0...

[chuidiang]

Estás declarando un atributo int [][] mapa; en la clase Mapa, pero estás rellenando en el constructor una variable local del constructor ya que la declaras otra vez

Código Java:

public Mapa()
    {
      int[][] mapa={{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},{0,2,1,1,1,0,1,1,1,1,1,1,1,1,0,1,1,1,2,0},{0,1,0,0,1,0,1,0,0,0,0,0,0,1,0,1,0,0,1,0},{0,1,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0},{0,1,0,1,0,0,1,0,0,3,3,0,0,1,0,0,1,0,1,0},{0,1,1,1,1,1,1,0,3,3,3,3,0,1,1,1,1,1,1,0},{0,1,0,1,0,0,1,0,0,0,0,0,0,1,0,0,1,0,1,0},{0,1,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0},{0,1,0,0,1,0,1,0,0,0,0,0,0,1,0,1,0,0,1,0},{0,2,1,1,1,0,1,1,1,1,1,1,1,1,0,1,1,1,2,0},{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 
    }
 

cuando el constructor termina, ese int [][] mapa que declaras dentro desaparece y te queda el atributo int [][] mapa que sigue sin rellenar. Dicho de otra forma, tienes dos mapa distintos

Código Java:

public class Mapa {
 
    public int [][] mapa;  // Esta es distinta de 
 
    public Mapa()
    {
          int[][] mapa = .... ; // esta otra.
    }
 

Quita "int [][]" en el constructor y deja "mapa = {{,,,,,}}"

se bueno.

[borjapg9]

Hola buenas tardes, yo tambien tengo un problema de nullPointerException que no se muy bien como solucionar, me explico, se trata de una interfaz grafica swing que consiste en el juego ahorcado, cuando se da al boton "jugar", se instancia un nuevo ahorcado con una palabra elegida al azar de un xml. Luego deberemos dar a los botones en los que estan las letras para ir acertando la palabra. El problema es este, que al instanciar el Ahorcado dentro de un if, (si se pulsa el boton "jugar") no puedo usar mas tarde los metodos de ese objeto en cuestion. aqui os dejo el codigo:

Código Java:

public void actionPerformed(ActionEvent e) {
        Object jugar=e.getSource();
        if(jugar==botonJugar){//si se pulsa el boton 'jugar' se crea una nueva instancia del objeto con el numero de letras deseado
            System.out.println("Se ha pulsado jugar");
            int numLetras=Integer.parseInt(JOptionPane.showInputDialog("Elija numero de letras: "));
            try {
                Ahorcado a1=new Ahorcado(numLetras);//se crea la palabra del ahorcado de la lista con las palabras que tienen el numero de letras que se pasa
                a1.setPalabra(a1.getPalabra());
                System.out.println(a1.getRespuesta());
                respuestaGr.setText(a1.getRespuesta());
                botonJugar.setEnabled(false);
            } catch (NumberFormatException e1) {
                e1.printStackTrace();
            } catch (IOException e1) {
                // TODO Auto-generated catch block
                e1.printStackTrace();
            }
        }else{//si no se pulsa el boton 'jugar' (se pulsa una letra) se utiliza el metodo tirar con esa letra
            System.out.println("Se ha pulsado "+e.getActionCommand()+" para comenzar a adivinar la palabra");
                char letraElegida=e.getActionCommand().charAt(0);
                //System.out.println("LETRA: "+letraElegida); LO HACE BIEN
                try{
                    a1.tirar(letraElegida);//da nullPointerException, a1 no existe ya que se crea dentro del if...
                    respuestaGr.setText(a1.getRespuesta());//para actualizar la respuesta con cada tirada
                    if(a1.completo()==false){
                        JOptionPane.showMessageDialog(null,"Se acabaron tus vidas. Palabra: "+a1.getPalabra(),"Mensaje",JOptionPane.ERROR_MESSAGE);
                        //Se completa la respuestaGr(respuesta grafica con la palabra) y se modifica el xml añadiendo 1 al atributo 'aciertos' de esa palabra suponiendo que exista, si no se hace
                        try{
                            SAXBuilder builder=new SAXBuilder();
                            org.jdom2.Document doc=builder.build("palabras.xml");
                            Element raiz=doc.getRootElement();
                            
                        }catch(JDOMException j){
                            j.printStackTrace();
                        } catch (IOException e1) {
                            // TODO Auto-generated catch block
                            e1.printStackTrace();
                        }
                    }else{
                        JOptionPane.showMessageDialog(null,"Enhorabuena. Palabra: "+a1.getPalabra()+ "acertada","Mensaje",JOptionPane.ERROR_MESSAGE);
                        
                    }
                }catch(NullPointerException nul){
                    nul.printStackTrace();
                }
        }
    }
 

[chuidiang]

Estás haciendo el new de a1 = new Ahorcado.... dentro del if, por lo que en la parte del else no está definido (si entra en el if, no entra en el else), por lo que a1.tirar() da nullpointer al ser a1 un null.

Haz el new fuera del if

Se bueno.

[borjapg9]

Gracias por contestar, pero es que ahi esta el problema, el new se debe crear si se cumple la condicion (en este caso el pulsar el boton 'jugar') asi que no se como solucionarlo.

[chuidiang]

Estás haciendo Ahorcado a1=new Ahorcado() por lo que incluso a1 es una variable local dentro del if. En cuanto salgas del if se pierde (no el new, sino la variable en si misma). Como la estás usando en el else y no te da error al compilar, imgino que tienes otro a1 declarado fuera. Puede ser exactamente el mismo problema que le comenté a lutxi con tener dos variables mapa declaradas, solo que en tu caso es a1.

Se bueno.

[lutxi]

Hola...

Siguiendo con mi problema de instancias... no se donde no he instanciado ahora mismo...

Ya que en la clase Mapa tengo...

public class Mapa
{
int sizeCuadrado=20;
int sizeCirculo=5;
int posX=10;
int posY=20;
int [][] mapa;

Código:

    public Mapa()
    {
       
   int  [][] mapa={{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},{0,2,1,1,1,0,1,1,1,1,1,1,1,1,0,1,1,1,2,0},{0,1,0,0,1,0,1,0,0,0,0,0,0,1,0,1,0,0,1,0},{0,1,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0},{0,1,0,1,0,0,1,0,0,3,3,0,0,1,0,0,1,0,1,0},{0,1,1,1,1,1,1,0,3,3,3,3,0,1,1,1,1,1,1,0},{0,1,0,1,0,0,1,0,0,0,0,0,0,1,0,0,1,0,1,0},{0,1,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0},{0,1,0,0,1,0,1,0,0,0,0,0,0,1,0,1,0,0,1,0},{0,2,1,1,1,0,1,1,1,1,1,1,1,1,0,1,1,1,2,0},{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};

    }

public int getMap(int fila, int columna)
 {
              
        return mapa[fila][columna];
 }
}

Y en la clase tablero...

Código:

public void rellenar()
    {
       
               for (int i=0; i<11; i++)
            {
                for (int j=0; j<20; j++)
                {     mapa[i][j]=plano.getMap(i,j);

                        System.out.print(mapa[i][j]);
                }
            }
    }

Pero el nullpointer me da en getMapa... si el metodo pone que devuelve un int, y recibe dos int ... ¿Dónde está el error?

[chuidiang]

El problema, como te comenté antes, es que tienes declaradas dos variables mapa distitnas. Si pones int [][] antes de la variable la estás declarando, pones int [][] delante de mapa en dos sitios, así que tienes dos variables distintas con el mismo nombre mapa.

Código Java:

public class Mapa {
 
    public int [][] mapa;  // Esta es una variable mapa, que está sin inicializar
 
    public Mapa()
    {
          int[][] mapa = .... ; // esta es la otra variable mapa que sí inicializas.
    }
 

Estás incializando la variable que declaras dentro del constructor, así que terminado el constructor, pierdes la variable, ya que lo que declaras dentro de un método (constructor) se pierde al salir de él, es una variable local al método. Si accedes a mapa fuera del constructor, estás accediendo a la otra variable mapa, la que no has inicializado y que es un atributo de la clase (la has declarado fuera de los métodos) y está accesible y sin inicializar en todos los demás sitios.

Tienes que NO declarar la variable por segunda vez, pare ello, simplemente quita el int [][]

Código Java:

public class Mapa {
 
    public int [][] mapa;  // Esta es una variable mapa, que está sin inicializar
 
    public Mapa()
    {
          mapa = new .... ; // Aquí la inicializamos, porque NO la estamos declarando de nuevas.
    }
 

Se bueno.