Einzelne Objekte und Unterobjekte einer ArrayList ausgeben

[ac0h1m]

Hallo, ich bin mittlerweile am verzweifeln..
Zu meiner Frage, ich habe eine csv-Datei, welche in eine ArrayList gelesen wird. Nun will ich die ArrayList nach zwei Objekten und die dazugehörigen Datentypen (pid und tid) haben in eine Neue ArrayList speichern. Ich bekomme es einfach nicht hin. Im Bild ist nochmal die Datei aufgezeigt. Ich würde es gerne ohne Stream lösen.
Vielen Danke👋🏻

 

[Oneixee5]

Was genau ist das Problem? Du iterierst über die 1. Liste und fügst die passenden Elemente der zweiten Liste hinzu. Zeig doch mal in Codetags </> was du schon hast.

 

[ac0h1m]

Was genau ist das Problem? Du iterierst über die 1. Liste und fügst die passenden Elemente der zweiten Liste hinzu. Zeig doch mal in Codetags </> was du schon hast.

Ich möchte aus der Arrayliste (Input) nur zwei Attribute in eine neue ArrayListe einlesen und diese zurückgeben. Hier ist nochmal die Aufgabenstellung.

 

[KonradN]

Ich möchte aus der Arrayliste (Input) nur zwei Attribute in eine neue ArrayListe einlesen und diese zurückgeben. Hier ist nochmal die Aufgabenstellung.

Noch einmal die Frage: Was genau ist DEIN Problem? Woran scheiterst Du? Die Aufgabenstellung ist uns klar, einen einfachen Lösungsweg hat @Oneixee5 beschrieben. Was fehlt Dir, um diese Lösungsidee umzusetzen?

Ansonsten sieht das nicht wie eine typische Java Aufgabe aus, in der man das erste mal über eine Liste iteriert. Wenn Ihr Streams kennt / kennen solltet, dann ist das ein typischer Stream Einzeiler (der auf 3 Zeilen umgebrochen wird damit er besser gelesen werden kann):
1. Zeile der Stream
2. Zeile der Filter
3. Zeile das Collect

 

[ac0h1m]

Noch einmal die Frage: Was genau ist DEIN Problem? Woran scheiterst Du? Die Aufgabenstellung ist uns klar, einen einfachen Lösungsweg hat @Oneixee5 beschrieben. Was fehlt Dir, um diese Lösungsidee umzusetzen?

Ansonsten sieht das nicht wie eine typische Java Aufgabe aus, in der man das erste mal über eine Liste iteriert. Wenn Ihr Streams kennt / kennen solltet, dann ist das ein typischer Stream Einzeiler (der auf 3 Zeilen umgebrochen wird damit er besser gelesen werden kann):
1. Zeile der Stream
2. Zeile der Filter
3. Zeile das Collect

Danke für die schnelle Antwort. Also mein Problem ist die Syntax. Ich weiß das ich über die erste Liste iterieren muss und es in die zweite Liste einfügen soll. Habe es mit einer for-Schleife und einem if-Statement versucht, jedoch wird das nichts. Also es scheitert am Umsetzten von Theorie in Praxis..

 

[KonradN]

Habe es mit einer for-Schleife und einem if-Statement versucht, jedoch wird das nichts.

Dann zeige uns, was du versucht hast inkl. kurzem Hinweis, was das Problem ist. Dann können wir gezielt erläutern, was falsch ist.

 

[ac0h1m]

Bisher habe ich so viel. Jedoch weiß ich nicht wie ich das Iterieren nutzen soll um pid und tid in die neue Liste einzufügen

 

[KonradN]

Das ist doch schon ein super Anfang - damit kopierst Du alle Element in die Zielliste.

Jetzt musst Du nur noch das Innere der for each Schleife anpassen, so dass das Einfügen in die Zielliste nur erfolgt, wenn die gewünschte Bedingung erfüllt ist. Wie kannst Du in Java eine Bedingung prüfen?

 

[ac0h1m]

Das ist doch schon ein super Anfang - damit kopierst Du alle Element in die Zielliste.

Jetzt musst Du nur noch das Innere der for each Schleife anpassen, so dass das Einfügen in die Zielliste nur erfolgt, wenn die gewünschte Bedingung erfüllt ist. Wie kannst Du in Java eine Bedingung prüfen?

Mit einem If-Statement? Aber was genau überprüf ich in diesem If Statement?

 

[KonradN]

Ja, if Statement ist schon richtig, aber was genau willst Du prüfen?
Die Aufgabe sagt ja, dass wir nur die PerformanceData Instanzen zurückgegeben haben wollen, welche die pid und tid haben.

Somit musst Du in der Schleife prüfen, ob die aktuelle Instanz (die heisst bei Dir s), die gleiche pid / tid haben.
Und sollte das der Fall sein, willst Du ja nicht die null Referenz hinzu fügen sondern die geprüfte Instanz.

Noch einmal kurz zur Auffrischung:

for (PerformanceData s: input) {
    // Code der etwas mit s macht
}

Das ist eine for each Schleife. In der Schleife werden alle Elemente von input durchlaufen. Im Schleifendurchgang ist das aktuelle Element in der Variablen s gespeichert.

 

[ac0h1m]

Ja, if Statement ist schon richtig, aber was genau willst Du prüfen?
Die Aufgabe sagt ja, dass wir nur die PerformanceData Instanzen zurückgegeben haben wollen, welche die pid und tid haben.

Somit musst Du in der Schleife prüfen, ob die aktuelle Instanz (die heisst bei Dir s), die gleiche pid / tid haben.
Und sollte das der Fall sein, willst Du ja nicht die null Referenz hinzu fügen sondern die geprüfte Instanz.

Noch einmal kurz zur Auffrischung:

for (PerformanceData s: input) {
    // Code der etwas mit s macht
}

Das ist eine for each Schleife. In der Schleife werden alle Elemente von input durchlaufen. Im Schleifendurchgang ist das aktuelle Element in der Variablen s gespeichert.

Okay verstanden habe ich das wie du das meinst. Aber wie prüfe ich in einem if Statement die aktuelle Instanz auf vorhanden sein von pid und tid?

 

[KonradN]

Wenn PerformanceData eine Methode hat getPid, dann könnte man z.B. prüfen: s.getPid() == pid. Und ebenso bei tid. Aber da ich die Klasse PerformanceData nicht kenne, kann ich dazu nichts sagen.

 

[ac0h1m]

Wenn PerformanceData eine Methode hat getPid, dann könnte man z.B. prüfen: s.getPid() == pid. Und ebenso bei tid. Aber da ich die Klasse PerformanceData nicht kenne, kann ich dazu nichts sagen.

Also PerfomanceData ist ein Record.

public record PerformanceData(String name, String cat, char ph, int pid, int tid, double ts, double duration) {
}

Würde dann

s.Pid() == pid

funktionieren?

 

[Jw456]

Zeige doch mal die Klasse PerformanceData im ganzen.

 

[ac0h1m]

Zeige doch mal die Klasse PerformanceData im ganzen.

Das ist die komplette Klasse, die Datei wird Eingelsen und in ein Arraylist (Input) gespeichert.

 

[Jw456]

ich meite den Quellcode der Klasse.

 

[ac0h1m]

Mehr ist da nicht...

 

[Jw456]

Ein Record ist doch eine DatenKlasse die erstellt doch die getter und setter selber.
Du brauchst die doch nur benutzen. Konrad hat dir auch schon gesagt wie.

Records - Javabeginners

javabeginners.de

Datenklassen in Java: Einführung in Java Records - JAX

Mit JEP 359 werden Records in die JVM eingeführt. Warum diese eine große Bereicherung für Java sein werden, erfahren Sie in diesem Blogartikel.

jax.de

 

[ac0h1m]

Ein Record ist doch eine DatenKlasse die erstellt doch die getter und setter selber.
Du brauchst die doch nur benutzen. Konrad hat dir auch schon gesagt wie.

Records - Javabeginners

javabeginners.de

Ja bloß leider funktioniert das nur mit s.pid() und nicht mit s.getPid(). Wie aber kann ich in der Add Methode sagen das pid und tid eingefügt werden sollen?

 

[Jw456]

Du fügst der liste nicht Pid oder Tid hinzu sonder das objekt also "s" aus der for

die beiden sind doch nur zum suchen vergleichen.

 

[ac0h1m]

Du fügst der liste nicht Pid oder Tid hinzu sonder das objekt also "s" aus der for

Ist das aber nicht die komplette Liste? ich will ja nur pid und tid in die Liste filterListe hinzufügen?

 

[KonradN]

Bei einem Record heißen die Getter so so wie die angegebenen "Parameter" des Records. Also ein s.pid() und s.tid() sollte es geben. Das weicht halt von den üblichen Gettern mit "get" ab (Etwas, das ich nicht gut finde. Ich hätte auch bei records die Getter mit get starten lassen. Aber das ist egal - die Ersteller werden sich dabei auch etwas gedacht haben ...).

 

[Jw456]

nein ,wenn du dein if richtig macht
zeige code

 

[ac0h1m]

Bei einem Record heißen die Getter so so wie die angegebenen "Parameter" des Records. Also ein s.pid() und s.tid() sollte es geben. Das weicht halt von den üblichen Gettern mit "get" ab (Etwas, das ich nicht gut finde. Ich hätte auch bei records die Getter mit get starten lassen. Aber das ist egal - die Ersteller werden sich dabei auch etwas gedacht haben ...).

Genau das hab ich jetzt gemacht. Das in der Add Methode ist so richtig? Also ich habe auch Tests und leider schlägt der Fehl.

 

[ac0h1m]

nein ,wenn du dein if richtig macht
zeige code

Das ist mein Code aktuell.

 

[KonradN]

Das sieht jetzt erst einmal gut aus. Dann zeige einmal den Test, der fehlschlägt.

Und es wäre besser, denn Du den Code nicht als Bildschirmfotos sondern wirklich als Code (Text) einfügen könntest. Dazu einfach mal über dem Eingabebereich auf den Knopf </> drücken und da dann den Code hinein kopieren.

 

[Jw456]

was hast du als Ergbis ?
lass es dir anzeigen.

 

[KonradN]

Ach - ein Fehler ist da natürlich noch. Was für Elemente sollen den hinzu gefügt werden? Was muss da erfüllt sein?

pid UND tid müssen gleich sein oder
pid ODER tid?

 

[ac0h1m]

Ach - ein Fehler ist da natürlich noch. Was für Elemente sollen den hinzu gefügt werden? Was muss da erfüllt sein?

pid UND tid müssen gleich sein oder
pid ODER tid?

Nein die müssen nicht gleich sein sondern, es ist egal ob sie gleich oder ungleich sind. Es sollen nur alle Werte über 0 hinzugefügt werden. Habe noch ein If-Statement für die Überprüfung hinzugefügt (>= 0).

 public static ArrayList<PerformanceData> filterByIDs(ArrayList<PerformanceData> input, int pid, int tid) {
        ArrayList<PerformanceData> filterListe = new ArrayList <PerformanceData>();

        for(PerformanceData s : input) {
            if(s.pid() == pid || s.tid() ==tid) {  
                if(s.pid() >= 0 || s.tid() >= 0) {  

                    filterListe.add(s);    
                }
            }
        }
        return filterListe;
    }

 

[ac0h1m]

was hast du als Ergbis ?
lass es dir anzeigen.

Die Liste ist komplett leer.

 

[Jw456]

Es sollen nur alle Werte über 0 hinzugefügt werden

dann ist "== " wohl nicht richtig

was ist ein prüfen auf gröser als 0 ?
oder Ungeich nicht 0

 

[ac0h1m]

dann ist "== " wohl nicht richtig

was ist ein prüfen auf gröser als 0 ?
oder Ungeich nicht 0

!= in dem Fall so? Aber sollte das überprüfen auf 0 oder positv, nicht in einem weiteren Statement stehen? Beispielsweise so:

for(PerformanceData s : input) {
            if(s.pid() == pid || s.tid() ==tid) {  
                if(s.pid() >= 0 || s.tid() >= 0) {  

                    filterListe.add(s);    

                }

            }
        }

        return filterListe;
    }

 

[KonradN]

Nein die müssen nicht gleich sein sondern, es ist egal ob sie gleich oder ungleich sind. Es sollen nur alle Werte über 0 hinzugefügt werden. Habe noch ein If-Statement für die Überprüfung hinzugefügt (>= 0).

Also das macht so relativ wenig Sinn in meinen Augen.

Schau Dir einmal die Dokumentation genau an, die über der Methode steht. (Das ist in #1 im Bildschirmfoto zu sehen)

Sowohl pid als auch tid sind immer > 0. Das ist ein technisches Detail. Das haben wir hier nicht zu prüfen. Ich zeige das einmal an Hand eines Beispiels:
Du hast eine Liste von Personen mit Alter. Jeder Person hat ein Alter >= 0. (Klar - es kann ja niemand -7 Jahre alt sein!)
Wenn ich nun eine Liste haben will von allen, die noch keine 7 Jahre alt sind, dann muss ich nur prüfen: Alter < 7. Die Prüfung > 0 ist Unsinn.

Und damit dann einmal zurück zu den Parametern: Was steht da? Wie kann man das umsetzen?

 

[KonradN]

Und bitte: Vergesst das mit der 0. Das führt zu nichts. Das Problem ist, dass die Aufgabenstellung nicht sauber kommuniziert wurde!

 

[ac0h1m]

Also das macht so relativ wenig Sinn in meinen Augen.

Schau Dir einmal die Dokumentation genau an, die über der Methode steht. (Das ist in #1 im Bildschirmfoto zu sehen)

Sowohl pid als auch tid sind immer > 0. Das ist ein technisches Detail. Das haben wir hier nicht zu prüfen. Ich zeige das einmal an Hand eines Beispiels:
Du hast eine Liste von Personen mit Alter. Jeder Person hat ein Alter >= 0. (Klar - es kann ja niemand -7 Jahre alt sein!)
Wenn ich nun eine Liste haben will von allen, die noch keine 7 Jahre alt sind, dann muss ich nur prüfen: Alter < 7. Die Prüfung > 0 ist Unsinn.

Und damit dann einmal zurück zu den Parametern: Was steht da? Wie kann man das umsetzen?

Also im Test werden auch negative Werte getestet. Meinst du im If-Statement?

 public void testFilterByIDs() {
        ArrayList<PerformanceData> input = Main.readFromCSV("resources/read/students/profiling2.csv");

        Assert.assertEquals(1, Main.filterByIDs(input, 1, 12).size());
        Assert.assertEquals(2, Main.filterByIDs(input, 1, -1).size());
        Assert.assertEquals(2, Main.filterByIDs(input, 7, -1).size());
        Assert.assertEquals(2, Main.filterByIDs(input, -1, 10).size());
        Assert.assertEquals(0, Main.filterByIDs(input, 7, 99).size());
        Assert.assertEquals(7, Main.filterByIDs(input, -1, -1).size());
    }

 

[KonradN]

Also im Test werden auch negative Werte getestet. Meinst du im If-Statement?

Hast Du dir die Dokumentation der Methode angesehen, die Du schreiben sollst? Was steht bei den Parametern?

Das Problem hier ist aus meiner Sicht, dass die Anforderung nicht verstanden wurde. Daher schau Dir diese genau an! Und Texte bitte nicht als Bildschirmfoto kopieren - das nervt und macht es uns schwer. Ich werde den Text nicht abtippen, um ihn Dir unter die Nase zu reiben.

 

[ac0h1m]

Hast Du dir die Dokumentation der Methode angesehen, die Du schreiben sollst? Was steht bei den Parametern?

Das Problem hier ist aus meiner Sicht, dass die Anforderung nicht verstanden wurde. Daher schau Dir diese genau an! Und Texte bitte nicht als Bildschirmfoto kopieren - das nervt und macht es uns schwer. Ich werde den Text nicht abtippen, um ihn Dir unter die Nase zu reiben.

Achso habe ich die Aufgabe falsch verstanden? Ich dachte ich muss nur pid und tid wiedergeben in eine neue Liste. Aber soll ich den kompletten Datensatz wiedergeben? Entschuldigung wegen den Screenshots.

 

[Jw456]

Achso habe ich die Aufgabe falsch verstanden? Ich dachte ich muss nur pid und tid wiedergeben in eine neue Liste. Aber soll ich den kompletten Datensatz wiedergeben? Entschuldigung wegen den Screenshots.

das tust du schon aber dein prüfen ist falsch.

 

[ac0h1m]

das tust du schon aber dein prüfen ist falsch.

Wahrscheinlich hat es was mit dem ODER zutun?

 

[Jw456]

Tipp lasse dir mal die orginal liste ausgeben und schaue die werte an.

 

[KonradN]

Dir fehlt noch ein ganz wichtiger Punkt bei der Lösung der Aufgabe. Jetzt tippe ich es doch einmal ab:
@param pid The pid the elements have to have; ignored if -1
@param tid The tid the elements have to have; ignored if -1

Aber soll ich den kompletten Datensatz wiedergeben?

Das hast Du ja schon. Wenn die Prüfung erfolgreich ist, dann fügst Du den ganzen record in die Liste ein. Das ist also gut.

Wahrscheinlich hat es was mit dem ODER zutun?

Ja, das ist ein Punkt. Derzeit prüfst Du noch mit || (oder), aber wenn pid und tid angegeben sind, dann müssen beide stimmen. Das ist also noch ein Fehler und der würde Dir bei der Überprüfung um die Ohren fliegen: Assert.assertEquals(0, Main.filterByIDs(input, 7, 99).size());

Aber erst einmal kannst Du noch den ersten Punkt angehen.

 

[ac0h1m]

Dir fehlt noch ein ganz wichtiger Punkt bei der Lösung der Aufgabe. Jetzt tippe ich es doch einmal ab:
@param pid The pid the elements have to have; ignored if -1
@param tid The tid the elements have to have; ignored if -1


Das hast Du ja schon. Wenn die Prüfung erfolgreich ist, dann fügst Du den ganzen record in die Liste ein. Das ist also gut.


Ja, das ist ein Punkt. Derzeit prüfst Du noch mit || (oder), aber wenn pid und tid angegeben sind, dann müssen beide stimmen. Das ist also noch ein Fehler und der würde Dir bei der Überprüfung um die Ohren fliegen: Assert.assertEquals(0, Main.filterByIDs(input, 7, 99).size());

Aber erst einmal kannst Du noch den ersten Punkt angehen.

Mach ich das nicht mit der Überprüfung ob es eben >= 0 ist? da werden ja alle Datensätze die kleiner als 0 sind ignoriert?

 

[Jw456]

zu deinem zweiten if konmmst du nie.

 

[KonradN]

Was prüfst Du den Wert des Datensatzes? Wenn der Parameter -1 ist, dann solltest Du den Parameter ignorieren!

Also füll eine Wahrheitstabelle aus für pid:

DS    P    Ergebnis
x    x    ?
x    <>x    ?
x    -1    ?

Was sind da für die ? einzutragen? Ja bedeutet wird aufgenommen, Nein bedeutet, wird nicht aufgenommen. x steht für einen beliebigen Wert <>x steht für einen beliebigen Wert ausser eben x (und auch nicht -1!).
DS ist der Datensatz, P der Parameter.

Edit: bei <>x ist der Wert nicht x und auch nicht -1! Das -1 hatte ich vergessen und noch eingefügt.

 

[Jw456]

if(s.pid() == pid || s.tid() ==tid) { 
                if(s.pid() >= 0 || s.tid() >= 0) {   // das zweite wird nie geprüft. weil das erste immer False sein wird.

                    filterListe.add(s);

 

[ac0h1m]

Was prüfst Du den Wert des Datensatzes? Wenn der Parameter -1 ist, dann solltest Du den Parameter ignorieren!

Also füll eine Wahrheitstabelle aus für pid:

DS    P    Ergebnis
x    x    ?
x    <>x    ?
x    -1    ?

Was sind da für die ? einzutragen? Ja bedeutet wird aufgenommen, Nein bedeutet, wird nicht aufgenommen. x steht für einen beliebigen Wert <>x steht für einen beliebigen Wert ausser eben x.
DS ist der Datensatz, P der Parameter.

DS    P    Ergebnis
x    x    ja
x    <>x    ja
x    -1    nein

so oder? Und danke für die große Hilfe, ich mach das noch nicht so lange...

 

[KonradN]

Nein, wir überlegen es uns noch einmal anders.

Stell Dir eine Liste vor mit Autos. Da stehen alle Autos drauf mit der Farbe.
Jetzt sollst Du mir eine Liste mit Autos erstellen, die alle eine Farbe haben, die ich Dir sage. Also Ich sage z.B. "rot", dann listest Du mit alle Autos auf, die rot sind. Aber: Wenn ich sage "egal", dann beachtest Du die Farbe nicht.

Also wenn ich "egal" sage, dann besteht jedes Auto diese "Prüfung", oder?

Wenn Du also die Prüfung machen musst, dann schaust Du: Ist die gewünschte farbe "egal" -> Fertig, Farbcheck ok. Ansonsten prüfst Du noch die Farbe. Die Prüfung wäre also - wenn man dies zusammen fassen würde: gewünschetFarbe == "egal" || auto.farbe = gewünschteFarbe.

So ist es auch bei der pid: Wenn pid == -1 ist, dann wird der Datensatz genommen - unabhängig vom Wert. Also könntest Du - so wie ich für das Auto - nun für die pid eine Bedingung aufstellen.

 

[ac0h1m]

Nein, wir überlegen es uns noch einmal anders.

Stell Dir eine Liste vor mit Autos. Da stehen alle Autos drauf mit der Farbe.
Jetzt sollst Du mir eine Liste mit Autos erstellen, die alle eine Farbe haben, die ich Dir sage. Also Ich sage z.B. "rot", dann listest Du mit alle Autos auf, die rot sind. Aber: Wenn ich sage "egal", dann beachtest Du die Farbe nicht.

Also wenn ich "egal" sage, dann besteht jedes Auto diese "Prüfung", oder?

Wenn Du also die Prüfung machen musst, dann schaust Du: Ist die gewünschte farbe "egal" -> Fertig, Farbcheck ok. Ansonsten prüfst Du noch die Farbe. Die Prüfung wäre also - wenn man dies zusammen fassen würde: gewünschetFarbe == "egal" || auto.farbe = gewünschteFarbe.

So ist es auch bei der pid: Wenn pid == -1 ist, dann wird der Datensatz genommen - unabhängig vom Wert. Also könntest Du - so wie ich für das Auto - nun für die pid eine Bedingung aufstellen.

also heißt es dann:

s.pid() == -1 || s.pid() == pid

 

[KonradN]

also heißt es dann:

s.pid() == -1 || s.pid() == pid

Dann würdest Du auf der Auto Liste prüfen:
Stehst bei der Farbe des Autos (von der Liste) "egal" oder steht da die Farbe, die ich gesagt habe.

Aber das ist ja nicht das, was Du willst. Du willst ja prüfen:
Habe ich "egal" gesagt oder steht beim Auto (von der Liste) die Farbe, die ich gesagt habe.

 

[ac0h1m]

Dann würdest Du auf der Auto Liste prüfen:
Stehst bei der Farbe des Autos (von der Liste) "egal" oder steht da die Farbe, die ich gesagt habe.

Aber das ist ja nicht das, was Du willst. Du willst ja prüfen:
Habe ich "egal" gesagt oder steht beim Auto (von der Liste) die Farbe, die ich gesagt habe.

-1 == s.pid() || pid == s.pid()

ich bin grad sehr verwirrt...😅