Slick2D Обнаружение столкновения прямоугольника

Вы не обновляете позиции hitbox самостоятельно.

Вы рисуете это:

g.drawRect(playX, playY, playW, playH); //draws player bounds
g.drawRect(enemyX, enemyY, enemyW, enemyH); //draws enemy bounds

Но это не фактический hitbox, это только позиция игрока/врага, и прямоугольники, нарисованные здесь, будут в правильном положении, в то время как сами хиты - нет.

Я предлагаю вам сделать следующее:

public void update(GameContainer container, int delta)
{
    playerUpdate(delta);

    player.rect.setLocation(playX, playY);
    enemy.rect.setLocation(enemyX, enemyY); // update the hitboxes to the new positions

    if (player.rect.intersects(enemy.rect))
    {
        System.out.println("INTERSECT!");
    }
}

public void playerUpdate(int delta)
{
    if (playerForward == true)
    {
       playX -= delta * 0.4f;

       if (playX <= 140) 
       {
             playX = 140;
             playerForward = false;
             playerBackward = true;
       }
    }

    if (playerBackward == true) 
    {
         playX += delta * 0.4f;

         if (playX >= 700) 
         {
             playX = 700;
             playerBackward = false;
             delay = 1250;
         }
    }
}

public void keyReleased(int key, char c) 
{
   if (key == Input.KEY_ENTER)
   {
      playerForward = true;
   }
}

Кроме того, поскольку вы, кажется, новичок в разработке игр на Java, некоторые советы для вас:

• Отформатируйте свой код правильно

Всегда размещайте полный {...} после if, else, switch, while, for и т. Д.; правильный отступ строки.

• Think OO (объектно-ориентированный)

Это очень важно. Ваш класс противников и игроков должен как расширять какой-то класс сущности, потому что они оба очень хотят получить подобное поведение (избегайте дублирования кода!). Сопоставьте аналогичное поведение с суперклассом, упростите поведение, которое нужно контролировать с помощью нескольких настраиваемых параметров и т.д.

Например, вы сохраняете позиции своего врага и игрока как статическое целое в своем основном классе. Это не OO. Переместите позиции в класс сущности, где вы можете реализовать его любым способом.

• Не просто бросайте исключение без причины

Ваш метод update (...) генерирует исключение SlickException, даже если он никогда не нужен.

• Будьте осторожны с инкапсуляцией

Это то, что делают многие новички: просто возьмите некоторые параметры, поместите их в класс как закрытый (или, возможно, даже открытый), и создайте для них getters- и сеттеры. Это не инкапсуляция. Это почти так же плохо, как сделать их публичными в первую очередь.

Но почему бы нам просто не сделать все публично?

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

• Производительность имеет значение

Это один из аспектов, на которые вы должны обратить внимание, для любого программного обеспечения, но часто вы можете наиболее точно увидеть последствия в играх. Производительность важна! И тем не менее, я не имею в виду, что вам нужно следить за каждой деталью, но просто держите обзор в том, как улучшить и скорректировать код, особенно с помощью часто называемых методов, таких как обновление (..) и render (..) в Slick2D.

Обновить

В качестве решения другой проблемы, обсуждаемой в комментариях:

public class Enemy {

    private int startX, startY, width, height;
    public Shape rect = new Rectangle(startX, startY, width, height);
    // plus getters and setters
}

ширина и высота могут быть равны 0, поскольку они никогда не назначаются целым числам, которые имеют значение 0 по умолчанию, поэтому у hitbox с прямоугольником противника есть 0 ширина и никогда не будет срабатывать.

Попробуйте что-нибудь вроде:

 public class Enemy {

    private int startX, startY, width = 50, height = 70;
    public Shape rect = new Rectangle(startX, startY, width, height);
    // plus getters and setters
 }

Это должно работать, но вы должны перенести все эти атрибуты в класс врага и поместить их в конструктор.