Jak iterować tylko nad konkretnymi rodzajami elementów?

Question

Mam listy tablicy jako takie:

private List<GameObject> gameObjects = new CopyOnWriteArrayList<GameObject>(); 

GameObject może być jedną z 3 klas: Spaceship, Beam i Asteroid. Wszystkie są podobne, więc trzymam je w jednej tablicy. Jednak statki kosmiczne mają dodatkowo metodę shoot, która jest używana co 100ms w innym wątku (co nazywa się ShootRunnable). Więc chciałbym iterować w nim tylko ponad Spaceship, ponieważ inne nie implementują metody strzelania. Jaki jest najlepszy sposób, aby to osiągnąć?

for (GameObject ob : gameObjects) { 
    if (ob instanceof Spaceship) { 
    ob.shoot(); 
    } 
} 

Czy mogę powtórzyć to przy użyciu czegoś podobnego? Tylko za pomocą odlewu lub czegoś takiego? Proszę pomóż.

Answer 1

Ścieżka, na której się znajdujesz, jest technicznie wykonalna, choć dobrą zasadą jest, że jeśli zaczniesz używać refleksji, prawdopodobnie zrobisz coś złego. W takim przypadku najmądrzej byłoby mieć dwie kolekcje, jedną dla wszystkich typów gier i jedną dla statków kosmicznych.

Answer 2

Powinieneś umieścić swoją metodę shoot() w klasie Spaceship, ponieważ żadna asteriod lub promień nie użyje tej metody. Zrób to, powinieneś również zachować co najmniej dwie kolekcje. Jeden z nich powinien zawierać tylko twoje statki kosmiczne.

Answer 3

Czy w grze są jakieś inne akcje, które mają miejsce okresowo?

Jeśli tak, można zmienić metodę shoot() na metodę abstrakcyjną (można nazwać periodicAction()) i umieścić ją w klasie GameObject. Klasa Spaceship zaimplementuje tę metodę, strzelając do drugiej klasy z jej specyficznym okresowym zachowaniem i klasą Asteroid, nie robiąc nic.

Answer 4

wolałbym zrobić w ten sposób:

Iterator<Spaceship> getSpaceships(){ 
    // return Iterator over Spaceship objects through use of instanceof 
} 

... 
Iterator<Spaceship> it = getSpaceships() 
while(iter.hasNext()) it.next().shoot() 

Dla mnie to wygląda bardziej jasne.

Answer 5

Można utworzyć interfejs Action lub metody do GameObject:

public interface Action { 
    void performAction(); 
} 

public abstract class GameObject implements Action { 
    //... 
    public void performAction() { 
     // subclasses should override this method. 
     // otherwise, it will do nothing. 
    } 
} 

a następnie wdrożyć Spaceship:

public class Spaceship extends GameObject { 
    @Override 
    public void performAction() { 
     this.shoot(); 
    } 
} 

Aby Beam i Asteroid, można pozostawić metodę performAction:

Następnie możesz interweniować:

for (GameObject ob : gameObjects) { 
    ob.performAction(); 
}