Почему эта Java-игра на основе массива не работает?

Question

Я сделал очень простую 2D-игру, в которой вы должны сбивать своих врагов, которые идут с 5 разных треков.Я создал 2D массив (трек) для хранения локаций врагов, снарядов.Ширина массива 790, потому что длина дорожки составляет 790 пикселей.Я использую игровой цикл для обновления и рендеринга, который работает просто отлично.

Но поскольку цикл зависит от производительности компьютера, я использую класс ScheduledExecutorService для выполнения перемещения и появления врагов, но дляпо какой-то причине это не работает, враги не двигаются или иногда даже не появляются, а снаряды не двигаются.Программа не выдает ошибку, она просто не работает.Я проверил, и нет никаких синтаксических ошибок, и я не смог найти в нем никаких логических проблем, по крайней мере, из моих знаний.

Пожалуйста, дайте короткий и не слишком сложный ответ, потому что я еще новичок.

Код: (элементы управления: S, D / вверх, вниз для движения и места для стрельбы)

package com.bacskai.peashooter;

import java.awt.Canvas;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.Graphics;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.image.BufferStrategy;
import java.util.Random;
import java.util.concurrent.Executors;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.TimeUnit;
import javax.swing.JFrame;

public class Game extends Canvas implements Runnable, KeyListener {

    private static final long serialVersionUID = -4227990863874935837L;

    JFrame frame;
    static Dimension d;
    Thread thread;

    public static int width = 300;
    public static int height = width / 16 * 9;
    int scale = 3;

    boolean running;
    boolean alive = true;

    int[][] track = new int[5][790]; // the array

    int trackY = 2; // the track which the player is on

    private int playerPos = 250;
    private int playerPos1 = 220; // the 3 starting points for the triangle
    private int playerPos2 = 280;

    int health = 3;
    int score = 0;

    long delay = 100;
    long delay2 = 5000;

    ScheduledExecutorService executor = Executors.newScheduledThreadPool(1);
    ScheduledExecutorService executor2 = Executors.newScheduledThreadPool(1);



    Runnable task = new Runnable() {

        public void run() {
        move();
        }
    };

    Runnable task2 = new Runnable() {

        public void run() {
            spawnEnemy();
        }
    };

    public Game() {

        d = new Dimension(width * scale, height * scale);
        setPreferredSize(d);
        frame = new JFrame();

    }

    private void start() {

        thread = new Thread(this, "thread");
        running = true;
        thread.start();

        for (int i = 0; i == 5; i++) {
            for (int j = 0; j == 790; j++) { // initializing the array
                track[i][j] = 0;
            }
        }       

        executor.scheduleAtFixedRate(task, delay, delay, 
        TimeUnit.MILLISECONDS); // moveing
        executor2.scheduleAtFixedRate(task2, delay2, delay2, 
        TimeUnit.MILLISECONDS); // spawning new enemies

        System.out.println("Game started, window width: " + getWidth() + ", 
        height: " + 
        getHeight());

    }

    public void run() {

        while (running) { // game loop
            update();
            if (alive) {
            render();
            }
        }

    }

    private void stop() {

        try {

            frame.dispose();
            thread.join();
            executor.shutdownNow();
            executor2.shutdownNow();

        } catch (Exception e) {
            System.out.println("Error while closing: " + e);
        }
        System.out.println("Program closed, processes halted");
    }

    public void update() {

        if (health == 0) {
            alive = false;
            executor.shutdownNow();
            executor2.shutdownNow();
            System.out.println("Game over");

        }

    }

    private void render() {

    BufferStrategy bs = getBufferStrategy();

    if (bs == null) {createBufferStrategy(3); return;}

    Graphics g = bs.getDrawGraphics();

    //  Map

    g.setColor(Color.black);
    g.fillRect(0, 0, getWidth(), getHeight());

    g.setColor(Color.cyan);
    g.fillRect(100, 0, 10, 490);
    g.fillRect(0, 98, 900, 10);
    g.fillRect(0, 196, 900, 10);
    g.fillRect(0, 294, 900, 10);
    g.fillRect(0, 392, 900, 10);

    //  Score / health

    Font font = new Font("Default", Font.PLAIN, 30);
    g.setFont(font);

    g.setColor(Color.red);
    g.drawString("Score: " + score, 740, 30);

    g.setColor(Color.yellow);
    g.drawString("Health: " + health, 600, 30);

    //  Player

    g.setColor(Color.green);
    int[] xPoints = {10, 10, 60};
    int[] yPoints = {playerPos1, playerPos2, playerPos};
    g.fillPolygon(xPoints, yPoints, 3);

    //  Enemies

    g.setColor(Color.red);
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 790; j++) {
            if (track[i][j] == 1) {
                g.fillRect(100 + j, i * 97 + 44, 30, 30);
            }
        }
    }

    //  Projectiles
    g.setColor(Color.green);

    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 790; j++) {
            if (track[i][j] == 2) {
                g.fillOval(110 + j, i * 97 + 44, 27, 27);
            }
        }
    }

    bs.show();
    g.dispose();    
    } // End of render

    public int randInt(int min, int max) {

        Random rand = new Random();
        int randomNum = rand.nextInt((max - min) + 1) + 1;
        return randomNum;

        }

    public void keyTyped(KeyEvent e) {}

    public void keyPressed(KeyEvent e) {

        if(e.getKeyCode() == KeyEvent.VK_ESCAPE) {
            stop();
        }

        if(e.getKeyCode() == KeyEvent.VK_UP && trackY > 0) {
            trackY--;
            playerPos -= 98;
            playerPos1 -= 98;
            playerPos2 -= 98;
            System.out.println("Key pressed: up");
        }

        if(e.getKeyCode() == KeyEvent.VK_DOWN && trackY < 4) {
            trackY++;
            playerPos += 98;
            playerPos1 += 98;
            playerPos2 += 98;
            System.out.println("Key pressed: down");
        }

        if(e.getKeyCode() == KeyEvent.VK_SPACE) {
            shoot();
        }


    }

    public void keyReleased(KeyEvent e) {}

    public void shoot() {

            System.out.println("Player shot projectile from: " + (trackY + 1));
            track[trackY][0] = 2;   

    }

    public void move() {

        System.out.print("asd");
        for (int i = 0; i < 6; i++) {
            for (int j = 0; j < 790; j++) {

                if (track[i][j] == 2 && track[i][j + 2] == 1) {
                    track[i][j] = 0;
                    track[i][j + 2] = 0;
                    break;
                }

                    switch (track[i][j]) {

                        case 0: // 0 ==> empty position

                            break;

                        case 1: // 1 ==> enemy

                            if (j != 0) {

                            track[i][j - 1] = 1;
                            track[i][j] = 0;


                            } else {
                                track[i][j] = 0;
                                enemyArrived();
                            }
                            System.out.print("");
                            break;

                        case 2: // 2 ==> projectile

                            if (j == 789) {
                                track[i][j] = 0;
                                break;
                            }

                            track[i][j + 1] = 2;
                            track[i][j] = 0;
                            System.out.print("");
                            break;

                        default:

                            System.out.println("Unable to identify object type at: track[" + i + "][" + j + "]");
                            break;

                }
            }
        }

    }

    public void spawnEnemy() {

        int trakk = randInt(1, 5);
        track[trakk][789] = 1;

        System.out.println("New enemy at: " + trakk);
    }

    public void enemyArrived() {

        health--;
        System.out.println("Player lost a health point, current health: " + health);

    }


    public static void main(String[] args) {

        Game game = new Game();

        game.frame.setResizable(false);
        game.frame.setTitle("Peasooter");
        game.frame.add(game);
        game.frame.pack();
        game.frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        game.frame.setLocationRelativeTo(null);
        game.frame.setVisible(true);
        game.frame.addKeyListener(game);

        game.start();

    }

}

Также я был бы очень рад, если бы кто-нибудь мог сказать мне, если есть способоптимизировать эту игру, чтобы не потреблять 25% - 30% процессорного времени.

Answer 1

Мне удалось запустить приложение с некоторыми изменениями и исправлением ошибок.Это не законченное решение, а то, что поможет вам двигаться вперед.

Сверху

Я изменил значения задержек, это сделало игру доступной, поскольку кажется, что 100 - это слишком короткое время.Вам, вероятно, не понравятся мои ценности для играбельности, но с моими ценностями игра может быть проверена, по крайней мере.

long delay = 1000;
long delay2 = 3000;

Нет смысла иметь два ScheduledExecutorService объекта, поэтому удалите второй.

Я понятия не имею, почему вы хотите создать поток, запустите ваш код оттуда, поэтому я удалил его,

//thread = new Thread(this, "thread");
running = true;
//thread.start();

И вручную вызывается метод run() в конце start(), так что с этим и единственным исполнителем вторая половина start() будет

    executor.scheduleAtFixedRate(task, delay, delay,
            TimeUnit.MILLISECONDS); // moveing
    executor.scheduleAtFixedRate(task2, delay2, delay2,
            TimeUnit.MILLISECONDS); // spawning new enemies

    System.out.println("Game started, window width: " + getWidth() + ", height: " + getHeight());

    run();
}

Выповсюду с вашими циклами for, в одном месте у вас есть i < 6 для лимита, а в другом - i == 5 (что всегда будет ложным).Пройдите все циклы и убедитесь, что они определены как

for (int i = 0; i < 5; i++) {
    for (int j = 0; j < 790; j++) {

. Также правильно вызовите randInt, чтобы соответствовать размеру массива

int trakk = randInt(0, 4);

Игра все еще ведет себя очень странно, но большую часть времени ядостаньте красных врагов, и я смогу их застрелить.

---

Обновление

Я сыграл с ним еще и внес еще два изменения.

Ваш метод генератора случайных чисел создает новый объект Random каждый раз, когда он вызывается, что не нужно, и я не видел смысла в методе randInt, поэтому я удаляю метод и объявляю новый член сверху

Random rand = new Random();

и затем использовал его там, где раньше был вызов randInt:

int trakk = rand.nextInt(NUMBER_OF_ROWS);

Чтобы посмотреть, смогу ли я улучшить производительность, я ввел 3 константы, а затем использовал их для массива и циклов for.(заменяя все вхождения 5 и 790 на них)

private static final int NUMBER_OF_ROWS = 5;
private static final int NUMBER_OF_PIXELS = 790;
private static final int LAST_PIXEL = NUMBER_OF_PIXELS - 1;

Затем я сделал массив намного меньше, изменив два первых на 3 и 100, это снова сделало игру намного более играбельной (хотя она и испортилась)немного графики) облегчает тестирование и исправление ошибок.