0

Tengo un método para crear enemigos, los enemigos debo crearlos en casillas y las casillas son una matriz de 30 x 40 que forman el tablero, pero en mi método crear enemigos me da este error de NPE, la verdad no sé que lo causa si ya tengo instanciado todo, adjunto una imagen con el error y la ubicación y el código.

introducir la descripción de la imagen aquí

Ahora adjunto mi código el problema es el método crear enemigos, los enemigos se van creando, por ahora la matriz solo tiene el tamaño está vacía y la idea del método es irla llenando.

import java.awt.Graphics;

public class Juego {

    private Jugador jugador;
    private Enemigos[][] enemigos;
    private Fondo fondo;
    private TanqueAntidoto tanqueAntidoto;
    private Blaster blaster;
    private Nivel nivel;
    private Tablero tablero;
    private Casilla[][] casilla;
    
    public Juego() {
        this.blaster = new Blaster();
        this.jugador = new Jugador(blaster);
        this.fondo = new Fondo();
        this.tanqueAntidoto = new TanqueAntidoto();
        this.enemigos = new Enemigos[30][40];
        this.tablero = new Tablero();
        this.nivel = new Nivel();
        this.casilla = new Casilla[30][40];
        this.crearEnemigos();
    } // Constructor

    private void crearEnemigos() {
        for (int i = 0; i < this.enemigos.length; i++) {
            for (int j = 0; j < this.enemigos.length; j++) {    
                    if(Nivel.aparicionVirus[i][j] == 1) {
                        enemigos[i][j].setEnemigo(new CepaL(enemigos[i][j].getX(),enemigos[i][j].getY()));
                        casilla[i][j].setEnemigo(this.enemigos[i][j].getEnemigo());
                        tablero.setCasilla(casilla);
                    } // If
            } // For j
        } // For i
    } // Crear Enemigos

    public void dibujar(Graphics g) {
        this.fondo.dibujar(g);
        this.jugador.dibujar(g);
        for (int i = 0; i < casilla.length; i++) {
            for (int j = 0; j < casilla.length; j++) {
                if(casilla[i][j] != null) {
                    tablero.getCasilla();
                    casilla[i][j].getEnemigo().dibujar(g);
                    
                    
                } // If
            } // For i
        } // For j
    } // Dibujar

    public void actualizar() {
        this.tanqueAntidoto.actualizar();
        this.jugador.actualizar();
        this.jugador.chequearAtaque(this.enemigos);
    } // Actualizar

    // Setter && Getters

    public Jugador getJugador() {
        return jugador;
    }

    public void setJugador(Jugador jugador) {
        this.jugador = jugador;
    }

    public Enemigos[][] getEnemigos() {
        return enemigos;
    }

    public Fondo getFondo() {
        return fondo;
    }

    public TanqueAntidoto getTanqueAntidoto() {
        return tanqueAntidoto;
    }

    public Nivel getNivel() {
        return nivel;
    }

    public void setEnemigos(Enemigos[][] enemigos) {
        this.enemigos = enemigos;
    }

    public void setFondo(Fondo fondo) {
        this.fondo = fondo;
    }

    public void setTanqueAntidoto(TanqueAntidoto tanqueAntidoto) {
        this.tanqueAntidoto = tanqueAntidoto;
    }

    public void setNivel(Nivel nivel) {
        this.nivel = nivel;
    }

} // Class Juego

2 respuestas 2

1

Cuando creas el arreglo, no estás creando los 120 enemigos dentro del arreglo:

this.enemigos = new Enemigos[30][40];

En ese punto, las 120 localidades del arreglo tienen null en ellas.

Por ende, en algún momento tienes que llenar esas 120 localidades con instancias de Enemigo (cosa que no veo que esté sucediendo en el código presentado). Si en tu código comienzas a hacer un recorrido llamando a métodos de las instancias que no están aún definidas (como setEnemigo()), vas a tener NPEs, como es el caso.

Cuanto corto: crea las 120 instancias de Enemigo y las guardas en el arreglo antes de empezar a utilizarlas.

12
  • crear las 120 instancias? eso cómo sería?, disculpame por los errores de novato el 9 nov. 2020 a las 14:59
  • tranquilo... no tienes que pedir dusculpas... podría ser algo como: for (int i =0; i < 30; i++) {for (int j=0; j < 40; j++){this.enemigos[i][j] = new Enemigo()}}. La llamada donde estás recibiendo el NPE me parece rara.... pareciera que en ese punto estás justamente definiendo el enemigo... pero, en vez de asignarlo en la localidad del arreglo, estás llamando a un método setEnemigo (de la instancia que en este momento es null, como estamos diciendo). O sea que la clase Enemigo tiene un método que se llama setEnemigo()? Eso suena raro, por decir lo menos.
    – user187182
    el 9 nov. 2020 a las 15:03
  • es que en la clase casilla tengo un atributo privado de tipo enemigo. En las casillas siempre hay o no hay enemigos, las casillas son una matriz de 30 x 40, que conforman el tablero. Enemigo() me da error ya que no puedo convertir de Enemigos[][] a Enemigos el 9 nov. 2020 a las 15:05
  • en este caso sería así, enemigos[i][j] = new CepaL(enemigos[i][j].getX(),enemigos[i][j].getY()); no? el 9 nov. 2020 a las 15:10
  • Enemigo es una clase abstracta, CepaL, CepaV y CepaS son clases que heredan de enemigos, o sea son enemigos el 9 nov. 2020 a las 15:13
1

Por lo que pones parece que faltaría crear los objetos de tipo Enemigo para cada una de los elementos del array enemigos.

Tendrías que crearlos usando el constructor justo encima de la línea donde haces el setEnemigo:

enemigos[i][j] = new Enemigos();
enemigos[i][j].setEnemigo(new CepaL(enemigos[i][j].getX(),enemigos[i][j].getY()));
6
  • yo ya hice esa instancia en el constrcutor de Juego el 9 nov. 2020 a las 15:03
  • creo que dices esto. enemigos[i][j] = new CepaL(enemigos[i][j].getX(),enemigos[i][j].getY()); el 9 nov. 2020 a las 15:10
  • Enemigo es una clase abstracta, CepaL, CepaV y CepaS son clases que heredan de enemigos, o sea son enemigos el 9 nov. 2020 a las 15:13
  • Eso es, realmente no conozco la jerarquía de clases, pero sí quería decir cualquier instancia de tipo Enemigos. Por cierto, he visto que con casillas pasaría algo parecido, no veo donde se instancian así que tal vez habría que instanciar cada elemento del array de forma similar
    – sergiomse
    el 9 nov. 2020 a las 15:18
  • la única sería esa, las cepas son Enemigos, herendan de la Clase Enemigos y la clase Enemigos es abstracta, listo muchas gracias el 9 nov. 2020 a las 15:21

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.