Aussortierung einer Liste

[AmsananKING]

Ich arbeite aktuell an einem kleinem 2D-Projekt und habe dafür Sprites in diesem Format gespeichert: Spritename (digit 0-10).png.
Ich konnte die jeweiligen Sprite-Dateien in einer Instanz der Klasse KnightPNGs umwandeln.

Attribute dieser Klasse:
String Spritename
int id(unpassende Namensgebung)
Bufferedimage Sprite

Diese Instanzen der Klasse habe ich nun in eine Liste abgespeichert.

Nun stellt man sich vor es gibt 10 Sprites die den Spritenamen „Attack“ besitzen und nur sich in der Id unterscheiden.
Hier meine Frage:
Wie kann ich diese 10 Sprites aus der Liste aussortieren und in einem Array speichern?

Um dann eine Hashmap zu erstellen die als key den Spritenamen und als value das Array erhält.
[CODE lang="java" title="Hashmap"]Map<String,KnightPNGS[]> KnightPNGMap =new HashMap<>();[/CODE]
ich habe alle benötigen Klassen in Pastebin reingeschrieben.
Folder:Resources gibt die Liste aus mit der Endung: .png

Mit freundlichen Grüßen
Amsanan

Pastebin Ordner mit Klassen

Amsi's Pastebin - Pastebin.com

Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.

pastebin.com

Google-Drive Ordner mit Bildern:

Resources – Google Drive

drive.google.com

 

[AmsananKING]

Google Drive Ordner:

Resources – Google Drive

drive.google.com

 

[AmsananKING]

Jetzt sollte der Google-Link funktionieren.

 

[kneitzel]

Also wenn du Key Value Pairs hast, dann wäre eine Map das Erste das mir einfällt.

Statt Array ein List nehmen (ArrayList als konkrete Implementierung)

Dann statt List einfach die Map<String,List<...>> nehmen.

Das Einfügen ist dann halt:
Ist der Key in der Map schon enthalten? Dann wird das Element der Liste hinzugefügt.
Sonst wird eine neue ArrayList erzeugt und die Instanz deines Sprites hinein getan und dann mit dem Key der Map hinzugefügt.

Etwas in der Art halt.

 

[Oneixee5]

Ich würde als Ziel kein Array verwenden sondern eine Collection, z.B. so:

<!-- https://mvnrepository.com/artifact/org.apache.commons/commons-collections4 -->
<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-collections4</artifactId>
    <version>4.4</version>
</dependency>

import org.apache.commons.collections4.multimap.ArrayListValuedHashMap;

public final class Main {

    private static class Sprite {

        private final String name;
        private final int id;
        private final BufferedImage spriteDate;

        public Sprite(final int id, final String name, final BufferedImage spriteDate) {
            this.id = id;
            this.name = name;
            this.spriteDate = spriteDate;
        }

        public String getName() {
            return this.name;
        }

        public int getId() {
            return this.id;
        }

        public BufferedImage getSpriteDate() {
            return this.spriteDate;
        }

        @Override
        public String toString() {
            return Integer.toString(getId());
        }

    }

    public static void main(final String[] args) {
        final ArrayList<Sprite> list = new ArrayList<>(20);
        for (int i = 0; i < 20; i++) {
            list.add(new Sprite(i, i < 10 ? "a" : "b", null));
        }

        final ArrayListValuedHashMap<String, Sprite> map = new ArrayListValuedHashMap<>();
        for (final Sprite sprite : list) {
            map.put(sprite.getName(), sprite);
        }

        // test result
        System.out.println(map.get("b").size());
        System.out.println(map.get("b"));
        System.out.println(map.get("b").get(0).getName());
    }

}

Ausgabe:

10
[10, 11, 12, 13, 14, 15, 16, 17, 18, 19]
b

 

[mihe7]

Ich kann mir zwar nicht vorstellen, dass das Modell wirklich sinnvoll ist, aber um die Frage zu beantworten:

class X {

    public static String basename(KnightPNG png) {
        Matcher m = Pattern.compile("\\p{Alpha}+").matcher(png.spriteName());
        return m.find() ? m.group() : "";
    }

    public static Map<String, List<KnightPNG>> toMap(List<KnightPNG> sprites) {
        return sprites.stream().collect(Collectors.groupingBy(X::basename));
    }

    // zum Testen
    public static record KnightPNG(String spriteName) {}

    public static void main(String[] args) {
        List<KnightPNG> sprites = new ArrayList<>();
        for (String name : new String[]{"attack", "defense"}) {
            for (int i = 0; i < 20; i++) {
                sprites.add(new KnightPNG(name + i + ".png"));
            }
        }

        Map<String, List<KnightPNG>> spriteMap = X.toMap(sprites);
        for (String name : new String[]{"attack", "defense"}) {
            System.out.println(name + "\n" + spriteMap.get(name));
        }
    }
}

 

[Oneixee5]

Ich würde als Ziel kein Array verwenden sondern eine Collection, z.B. so:

<!-- https://mvnrepository.com/artifact/org.apache.commons/commons-collections4 -->
<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-collections4</artifactId>
    <version>4.4</version>
</dependency>

import org.apache.commons.collections4.multimap.ArrayListValuedHashMap;

public final class Main {

    private static class Sprite {

        private final String name;
        private final int id;
        private final BufferedImage spriteDate;

        public Sprite(final int id, final String name, final BufferedImage spriteDate) {
            this.id = id;
            this.name = name;
            this.spriteDate = spriteDate;
        }

        public String getName() {
            return this.name;
        }

        public int getId() {
            return this.id;
        }

        public BufferedImage getSpriteDate() {
            return this.spriteDate;
        }

        @Override
        public String toString() {
            return Integer.toString(getId());
        }

    }

    public static void main(final String[] args) {
        final ArrayList<Sprite> list = new ArrayList<>(20);
        for (int i = 0; i < 20; i++) {
            list.add(new Sprite(i, i < 10 ? "a" : "b", null));
        }

        final ArrayListValuedHashMap<String, Sprite> map = new ArrayListValuedHashMap<>();
        for (final Sprite sprite : list) {
            map.put(sprite.getName(), sprite);
        }

        // test result
        System.out.println(map.get("b").size());
        System.out.println(map.get("b"));
        System.out.println(map.get("b").get(0).getName());
    }

}

Ausgabe:

10
[10, 11, 12, 13, 14, 15, 16, 17, 18, 19]
b

Bevor jetzt die Kommentare gegen zusätzliche Bibliotheken kommen, mit einem einfachen Lamda-Ausdruck geht es auch:

public final class Main {

    private static class Sprite {

        private final String name;
        private final int id;
        private final BufferedImage spriteDate;

        public Sprite(final int id, final String name, final BufferedImage spriteDate) {
            this.id = id;
            this.name = name;
            this.spriteDate = spriteDate;
        }

        public String getName() {
            return this.name;
        }

        public int getId() {
            return this.id;
        }

        public BufferedImage getSpriteDate() {
            return this.spriteDate;
        }

        @Override
        public String toString() {
            return Integer.toString(getId());
        }

    }

    public static void main(final String[] args) {
        final ArrayList<Sprite> list = new ArrayList<>(20);
        for (int i = 0; i < 20; i++) {
            list.add(new Sprite(i, i < 10 ? "a" : "b", null));
        }

        final HashMap<String, List<Sprite>> map = new HashMap<>();
        for (final Sprite sprite : list) {
            map.computeIfAbsent(sprite.getName(), k -> new ArrayList<>()).add(sprite);
        }

        // test result
        System.out.println(map.get("b").size());
        System.out.println(map.get("b"));
        System.out.println(map.get("b").get(0).getName());
    }

}

Natürlich könnte man das auch klassisch, mit ein paar zusätzlichen Codezeilen lösen.

 

[AmsananKING]

Ich hatte mir das so in einer internen Klasse vorgestellt:
[CODE lang="java" title="ArraySort.class"]private class ArraySort{
MultiMap KnightMap = new MultiValueMap();
public void getCollection(String key) {
for (KnightPngs k : PNGList()) {
KnightMap.put(k.Spritename, k);
}
Collection coll = (Collection) KnightMap.get(key);
}
}[/CODE]

 

[Oneixee5]

Ich hatte mir das so in einer internen Klasse vorgestellt:
[CODE lang="java" title="ArraySort.class"]private class ArraySort{
MultiMap KnightMap = new MultiValueMap();
public void getCollection(String key) {
for (KnightPngs k : PNGList()) {
KnightMap.put(k.Spritename, k);
}
Collection coll = (Collection) KnightMap.get(key);
}
}[/CODE]

Warum ist da so ein Chaos bei der Schreibweise von Variablen und Methoden. Weiterhin ist ArraySort ein völlig unpassender und irreführender Name für die Klasse. Eine void-Methode mit dem Präfix get... ist auch sehr suspekt. Weiterhin halte ich die "interne" Klasse für sinnlos. Mit Collection coll = ... wird gar nichts gemacht.