Доступ к подклассам из Arraylist

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

Начнем с «Врага». Вполне нормально иметь базовый «вражеский» класс, который будет устанавливать значения по умолчанию для различных типов «вражеских» персонажей. Это обычное дело в играх. Персонажи обычно имеют параметры по умолчанию, а затем определенные расы или подтипы будут добавлять или вычитать очки в зависимости от атрибутов этого типа. Итак, давайте сделаем это для «Лучника» и «Мечника».

public class Enemy {
    
    protected final String id;

    protected int damage = 10;
    protected int hp = 10;
    protected int defense = 10;
    protected int mana = 10;
    protected int speed = 10;
    protected int luck = 10;
    
    protected Enemy () {
        Random rnd = new Random();
        int number = rnd.nextInt(99999999);
        id = String.format("%08d", number);
    }

    // omitted getters and setters
}

Я добавил последнее (постоянное) строковое поле id, чтобы получить уникальный идентификатор для каждого экземпляра. Я буду использовать это в методе подклассов toString() для отображения имени с уникальными идентификаторами. Таким образом, при отображении символов каждый из них будет отображаться по-разному, чтобы вы знали, что они не являются простыми копиями друг друга.

Вы можете переопределить все переменные в подклассе, как вы пытались сделать. Например, вы можете определить damage в обоих классах, а затем в методе получения вы можете добавить их вместе и вернуть значение следующим образом:

public int getDamage () {
    return this.damage + super.damage;
}

Но зачем это делать? Сделав поля в суперклассе защищенными, ваши подклассы будут иметь к ним прямой доступ. Если вам нужно добавить бонусные баллы в эти поля, просто добавьте бонусные/штрафные баллы в конструкторе.

public final class Swordsman extends Enemy {

    public static final String NAME = "Swordsman";

    public Swordsman() {
        damage += 20; // added 20 points to the default "damage" field value of 10
        // other fields here
    }
    // getters and setters defined in Enemy class

    @Override
    public String toString() {
        return "Swordsman ["+ id + "]";
    }
    
    @Override
    public String getStats() {
        return NAME + super.getStats();
    }
}

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

При запуске этого кода с помощью приведенного ниже фрагмента

public class Game {
    public static void main(String[] args) {
        Swordsman s1 = new Swordsman();
        Swordsman s2 = new Swordsman();
        Swordsman s3 = new Swordsman();
        Swordsman s4 = new Swordsman();
        
        System.out.println(s1);
        System.out.println(s2);
        System.out.println(s3);
        System.out.println(s4);
    }
}

Он будет отображать что-то вроде этого:

Swordsman [04923128]
Swordsman [59484796]
Swordsman [11003419]
Swordsman [12543814]

Другие вещи, которые следует отметить в моей реализации:

1. Нет полей static: если вам нужно, чтобы какой-то атрибут был общим для всех экземпляров класса, вам нужно будет использовать для этого static. Например, атрибут name является статическим в подклассах. Все экземпляры класса Swordsman будут помечены как «фехтовальщик». Я также сделал это final, чтобы значение атрибута нельзя было изменить. По сути, это поле является общей константой.
2. Поля в классе Enemy защищены, чтобы они были доступны его подклассам.
3. Класс фехтовальщика final, поэтому его нельзя расширить. Если вам нужно создать подкласс Swordsman, вам нужно будет удалить этот квалификатор.
4. Подклассу не нужны геттеры и сеттеры. Они должны быть в классе Enemy

Другие вещи, которые вам нужно учитывать, каждый класс должен переопределять equals(), hashCode() и toString(). Ваша IDE должна быть в состоянии сгенерировать их для вас. Если класс не добавляет новые поля, подклассу не нужно будет повторно переопределять эти методы, за исключением toString(), потому что, скорее всего, вы захотите отобразить значение поля имени объекта (наряду с некоторыми другими вещами, такими как его хэш-код ).

И последнее, на что я хочу обратить внимание, если вы не знали, это мое предположение, основанное на опубликованном вами коде. Когда вы используете static для квалификации поля или метода, вы не переопределяете поле или метод. По сути, вы создаете новый метод или поле в подклассе с тем же именем и сигнатурой. Помните, статические члены принадлежат классу, а не экземпляру. Это еще одна причина, по которой вы не должны использовать static в своей реализации. Для того, что вы пытаетесь сделать, это просто не имеет никакого смысла.

Последнее, последнее, LOL

Почему бы не использовать для этого записи Java? Просто потому, что записи неизменяемы и эти символы нужно постоянно обновлять (урон во время боя, другие атрибуты при прокачке и т.д.) По этой причине лучше использовать обычный POJO, а не Java-запись. Я подумал, что важно указать на это для тех, кто задается вопросом.