Mehrere Listen oder Liste und getSublist() (Java)

Question

Ich habe eine abstrakte Klasse 'Entity' und Objekte (die 'Entity' erweitern), die verschiedene Schnittstellen implementieren. Ich habe auch eine ArrayList, die all diese verschiedenen Objekte enthalten.

Nun, wenn ich brauche Zugriff auf alle Unternehmen, die eine bestimmte Schnittstelle (verwenden seine Methoden) implementieren, verwende ich die folgende Methode (die eine gefilterte Liste von ‚Entitäten‘ zurückgibt, die Schnittstelle ‚IDirectFire‘ implementieren):

public ArrayList<IDirectFire> getDirectFireSublist() {//direct fire sublist 
    ArrayList<IDirectFire> sublist = new ArrayList(); 
    entities.stream().filter((it) -> (it instanceof IDirectFire)).forEach((it) -> { 
     sublist.add((IDirectFire) it); 
    }); 
    return sublist; 
} 

Jetzt zu meiner Frage: Sollte ich weiter mit dieser Methode arbeiten oder sollte ich eine neue ArrayList erstellen, die neben 'Entitäten' existiert und die ich jedes Mal manuell aktualisieren müsste, wenn 'Entitäten' geändert werden?

Ich muss 'Entitäten' viel aktualisieren, so dass ich nicht sicher bin, ob es effizienter ist, mehrere Unterlisten zu speichern und sie jedes Mal zu aktualisieren, wenn Entitäten geändert werden oder wenn ich weiterhin Methoden zum Filtern von 'Entitäten' und Wenden Sie Methoden auf diese Unterlisten an. Denken Sie daran, dass diese Unterlisten auch in anderen Methoden in einer Schleife verwendet werden würden, z. B.

Ist das machbar? Vielen Dank im Voraus!

Answer 1

Wenn Sie nur eine Teilmenge Ihrer Elemente durchlaufen müssen, ist das Erstellen einer neuen Liste verschwenderisch. Gib einfach den gefilterten Stream zurück.

public Stream<IDirectFire> getDirectFire() { 
    return entities.stream().filter((it) -> (it instanceof IDirectFire)); 
} 

Sie könnten auch Guava verwenden und zurückgeben eine gefilterte Iterable statt:

public Iterable<IDirectFire> getDirectFire() { 
    return FluentIterable.from(entities).filter(IDirectFire.class); 
} 

Dann Schleife über die Elemente elswhere:

private void resetFirestatusIDF() { 
    getDirectFire().forEach((it) -> it.dfHasFired(false)); 
} 

Answer 2

Es ist besser, sie zu filtern. Es wird einen klareren und verständlichen Code zu einem Preis von vernachlässigbarem Leistungsabfall schaffen, der, wenn Sie Milliarden von Elementen filtern, vernachlässigbar sein sollte.

Das zweite, was ich bemerkt habe ist, dass Sie Verwendung für Codefragment streamen 1. Ich möchte Sie und alternativen Ansatz, um es empfehlen:

> public ArrayList<IDirectFire> getDirectFireSublist() { 
>  return entities.stream().filter((it) -> (it instanceof IDirectFire)).collect(Collectors.toList()); 
> } 

Answer 3

Nun zu meiner Frage: Soll ich weiter mit dieser Methode arbeiten Oder soll ich eine neue ArrayList erstellen, die neben "entities" existiert und die ich jedes Mal manuell aktualisieren muss, wenn sich die Entitäten ändern?

Aus welchem ​​Grund möchten Sie die 'Daten' duplizieren?

1) Sie können sie nur in eine dedizierte Liste stellen. In diesem Fall brauchen Sie getDirectFireSublist() nicht länger.

2) Sie können sie zwischen den beiden Listen teilen, ohne sie zu duplizieren. In diesem Fall müssen Sie das hinzugefügte und das entfernte Entitätselement aktualisieren, da nur die geänderten Elemente aktualisiert werden. Aber es ist ziemlich direkt zu implementieren.

Answer 4

wakjah in einem Kommentar erwähnt, dass instanceof ist ein bisschen Designgeruch. In diesem Sinne wäre eine alternative Lösung die Verwendung eines Besuchermusters.

public abstract class Entity { 
    public abstract void acceptVisitor(EntityVisitor visitor); 
    ... 
} 

public interface IDirectFire { 
    default acceptVisitor(EntityVisitor visitor) { 
     visitor.visit(this); 
    } 
    ... 
} 

public class ResetFireStatusVisitor implements EntityVisitor { 
    public void visit(IDirectFire directFireEntity) { 
     directFireEntity.dfHasFired(false); 
    } 
} 

Dann über die Elemente in einer Schleife:

entities.forEach(entity -> entity.acceptVisitor(new ResetFireStatusVisitor())); 

Die ResetFireStatusVisitor Anrufe dfHasFired(false) auf alles, was implementiert IDirectFire. In EntityVisitor können Sie Standard-No-Op-Implementierungen für die anderen Subtypen Entity angeben.

Ich schlage nicht vor, Sie tun dies für einfache Fälle, aber für groß angelegte Designs könnte dies eine sinnvolle Antwort auf dieses Problem sein. Auf der anderen Seite könnte es nicht sein - dieses Muster hat auch seinen Anteil an Designgerüchen.