Datentypen TreeSet aus einer List<String[]> befüllen

[dicke_nudel]

Hallo an alle . Da ich hier schon seit ein paar Tagen Fragen stelle und ich immer Hilfe bekomme möchte ich mich mal herzlich bedanken!
[DUKE]DANKE SCHÖN![/DUKE]

Nun zu meinem nächsten Anfängerproblem. Ich habe hier eine Liste von String[] welche in eine TreeSet rein sollen. Ziel soll es sein das aus der tabelle1 und tabelle2 alle Werte in dem TreeSet enthalten sind. Natürlich nur einmalig. Also keine Duplikate. (Wenn ich das richtig verstanden habe ist das bei Sets eh immer so. Verbessert mich wenn ich da falsch liege).

Die Funktion add() kenne ich. Mit dieser habe ich es aber nicht hingekriegt.
adAll() kann ich nicht verwenden da dieser eine Collection erwartet und keine List<String>.

Kann mir bitte einer Verraten wie ich nun mein TreeSet befüllen kann?

private int CollectVariants(Reader table1,Reader table2)
{
    List<String[]> tabelle1;
    List<String[]> tabelle2;
    TreeSet<String> variant= new TreeSet<String>() ;
    
    tabelle1 = table1.read_csv();
    tabelle2 = table2.read_csv();
    
    // HIER SOLL DAS TREESET GEFÜLLT WERDEN
    for (int y =0;y>tabelle1.lastIndexOf(tabelle1);y++)
    {
        
       

        //System.out.print(tabelle1.get(y)[0]);
        //for(int x=0;x<tabelle1.get(x).length;x++)
        //    System.out.print(tabelle1.get(y)[x]+"\t");
    
        //System.out.print("\n");
    }
    
    Iterator<String> iter;
    iter = variant.iterator();
    
    while(iter.hasNext())
    {
        System.out.print(iter.next());
        
    }
    
    
    
    return 0;
}

 

[Shoox]

Du hast hier ja eine Liste mit String[], dh du musst zuerst eine Schleife zum auslesen der Liste machen und dann in der Schleife noch eine Schleife zum Auslesen des String[] und hier machst du dann das add für dein TreeSet.

for (String[] line : tabelle1) {
   for (String value : line) {
      variant.add(value);
   }
}

 

[Gast2]

adAll() kann ich nicht verwenden da dieser eine Collection erwartet und keine List<String>.

Eine List ist eine Collection

 

[ARadauer]

Also keine Duplikate. (Wenn ich das richtig verstanden habe ist das bei Sets eh immer so. Verbessert mich wenn ich da falsch liege).

ja stimmt...

ok du hast eine liste mit arrays wo strings drin sind...

add fügt einen string ein..
für jedes array in der liste , für jeden string im array -> einfügen

public static void addDataToSet(List<String[]> src, TreeSet<String> dest){
     for(String[] arr: src){
        for(String str: arr){
           dest.add(str);
        }
     }
]

adAll() will eine Collection.. String[] ist keine Collection..
Die klasse Arrays bietete dir eine Methode mit dem du aus einem array eine liste machen kannst. eine Liste ist eine Collection...
Arrays.asList

also für jedes array in der liste -> as List -> einfügen

 public static void addDataToSet(List<String[]> src, TreeSet<String> dest){
     for(String[] arr: src){
        dest.addAll(Arrays.asList(arr));
     }
  }

Ich bevorzuge die erste variante, da klar ist was hier passiert...

direkt die liste mit arrays auf einemal einfügen wird nicht klappen...

 

[dicke_nudel]

Du hast hier ja eine Liste mit String[], dh du musst zuerst eine Schleife zum auslesen der Liste machen und dann in der Schleife noch eine Schleife zum Auslesen des String[] und hier machst du dann das add für dein TreeSet.

for (String[] line : tabelle1) {
   for (String value : line) {
      variant.add(value);
   }
}

Danke für die schnelle Antwort. Ich glaube ich verstehe was du meinst. Leider komme ich nicht mit deinem code klar. Kann ich den so direkt verwenden?

Wenn ich den so einsetze dann gibt er mir zwar eine Ausgabe. Aber nich ganz die die ich will.

private int CollectVariants(Reader table1,Reader table2)
{
    List<String[]> tabelle1;
    List<String[]> tabelle2;
    TreeSet<String> variant= new TreeSet<String>() ;
    
    tabelle1 = table1.read_csv();
    tabelle2 = table2.read_csv();
    
    
    for (String[] line : tabelle1) {
           for (String value : line) {
              variant.add(value);
           }
    }
    
    Iterator<String> iter;
    iter = variant.iterator();
    
    while(iter.hasNext())
    {
        System.out.print(iter.next());
        System.out.print("\n");
    }
    
    
    
    return 0;
}

Er gibt mir hier die erste Spalte aus (Alle daten) und die erste Zeile. Ich mag aber nur die erste spalte haben (ohne erste zeile).

 

[dicke_nudel]

ja stimmt...

ok du hast eine liste mit arrays wo strings drin sind...

add fügt einen string ein..
für jedes array in der liste , für jeden string im array -> einfügen

public static void addDataToSet(List<String[]> src, TreeSet<String> dest){
     for(String[] arr: src){
        for(String str: arr){
           dest.add(str);
        }
     }
]

adAll() will eine Collection.. String[] ist keine Collection..
Die klasse Arrays bietete dir eine Methode mit dem du aus einem array eine liste machen kannst. eine Liste ist eine Collection...
Arrays.asList

also für jedes array in der liste -> as List -> einfügen

 public static void addDataToSet(List<String[]> src, TreeSet<String> dest){
     for(String[] arr: src){
        dest.addAll(Arrays.asList(arr));
     }
  }

Ich bevorzuge die erste variante, da klar ist was hier passiert...

direkt die liste mit arrays auf einemal einfügen wird nicht klappen...

BOAH. Ich versteh nur Bahnhof. Ich les mir das mal ein paar hundert mal durch. Evtl. versteh ichs dann Danke für die Hilfe.

 

[dicke_nudel]

Ok ich habe das was Aradauer geschrieben hat soweit einsetzen können. Dies Funktioniert auch mit dem TreeSet<String>. Danke!

Ich habe gerade festgestellt das ich doch eine eine TreeSet<String[]> verwenden muss. Leider tut dann der Vorschlag von Aradauer nicht.

Ich muss bestimmt wieder eine geschachtelte Schleife machen, bin überfordert.

private int CollectVariants(Reader table1,Reader table2)
{
    List<String[]> tabelle1;
    List<String[]> tabelle2;
    TreeSet<String[]> variant= new TreeSet<String[]>() ;
    
    tabelle1 = table1.read_csv();
    tabelle2 = table2.read_csv();
    
    
    
    
        for(String[] arr: tabelle1){
        variant.addAll(Arrays.asList(arr[2])); //Das geht nicht mehr. Weil es nun keine TreeSet<String> sondern ein TreeSet<String[]> ist.
     }

    
    
    
    
    
    Iterator<String[]> iter;
    iter = variant.iterator();
    
    while(iter.hasNext())
    {
        System.out.print(iter.next());
        System.out.print("\n");
    }
    
    
    
    return 0;
}

 

[Shoox]

Leider komme ich nicht mit deinem code klar. Kann ich den so direkt verwenden?

Es ist nur ein Code-Schnipsel, fertige Lösungen gibts hier nicht
Falls du die for-Schleife selbst nicht verstehst:

for ( String[] line : tabelle1 ) {
   ...
}

macht das gleiche wie

for (int i = 0; i < tabelle1.size(); i++){
   String[] line = tabelle1.get(i);
   ...
}

Ok ich habe das was Aradauer geschrieben hat soweit einsetzen können. Dies Funktioniert auch mit dem TreeSet<String>. Danke!

Ich habe gerade festgestellt das ich doch eine eine TreeSet<String[]> verwenden muss. Leider tut dann der Vorschlag von Aradauer nicht.

Ich muss bestimmt wieder eine geschachtelte Schleife machen, bin überfordert.

Brauchst du dann nicht, aber mir wird nicht ganz klar, was du eigentlich willst und irgendwie kommt es mir so vor, als würdest du auch nicht wissen was du willst ^^

 

[SlaterB]

variant.addAll(Arrays.asList(arr[2]));
->
variant.add(arr);

 

[dicke_nudel]

Da hast du zum Glück nur teilweise Recht . Beim erstellen des Threads hat sich der Fehler eingeschlichen das ich ein [] bei dem TreeSet vergessen habe.

Ziel ist es eine CSV einzulesen. Dies klappt auch wunderbar. Nun übergebe ich die CSV an die Funktion (das klapp ja auch; Reader). Mit der readAll() kriege ich dann auch eine List<String[]>. Die Daten haben ich überprüft. Alles super.

Mein Ziel ist es nun für jede Spalte alle möglichen Werte festzuhalten. Also Spalte 0 [Wert1,Wert2,Wert3] etc.

Wichtig ist das ich zu jeder Spalte die Werte kenne da ich damit an anderer Stelle weiterarbeiten muss.

 

[dicke_nudel]

variant.addAll(Arrays.asList(arr[2]));
->
variant.add(arr);

Hab ich schon versucht.
Da meckert er mir aber.

Exception in thread "main" java.lang.ClassCastException: [Ljava.lang.String; cannot be cast to java.lang.Comparable
    at java.util.TreeMap.put(Unknown Source)
    at java.util.TreeSet.add(Unknown Source)
    at alterapp.Transformer.CollectVariants(Transformer.java:74)
    at alterapp.Transformer.<init>(Transformer.java:20)
    at alterapp.Alterapp.main(Alterapp.java:18)

 

[ARadauer]

@dicke_nudel: wtf! sorry wenn du nicht mal eine schleife lesen kannst, wirst du mit java nicht weiter kommen.
wozu machst du das? ist das für die arbeit, studium oder schule?
arbeit: kündigen!
studium: exmatrikulieren!
schule: abschreiben!

 

[dicke_nudel]

@dicke_nudel: wtf! sorry wenn du nicht mal eine schleife lesen kannst, wirst du mit java nicht weiter kommen.
wozu machst du das? ist das für die arbeit, studium oder schule?
arbeit: kündigen!
studium: exmatrikulieren!
schule: abschreiben!

Das war unnötig.

 

[Shoox]

Ich würde hier ein 2 dimensionales Array verwenden:
Lies zuerst alles in das String[][] ein:

String[][] table = new String[tabelle1.get(0).length][tabelle1.size()];
for (int i = 0; i < tabelle1.size(); i++) {
   String[] line = tabelle1.get(i);
   for (int j = 0; j < line.length; j++) {
      table[j][i] = line[j];
   }
}

Erklärung:

Nehmen wir an dein CSV sieht so aus:
a,b,c
1,2,3
4,5,6

1. Durchlauf (1. CSV Zeile):
[0][0]=a
[1][0]=b
[2][0]=c

2. Durchlauf:
[0][1]=1
[1][1]=2
[2][1]=3

3. Durchlauf:
[0][2]=4
[1][2]=5
[2][2]=6

Ergebnis:
a,1,4
b,2,5
c,3,6

und dann kannst du wiederrum mit einer Schleife pro Spalte deine Werte rausholen.

for (int i = 0; i < table.length; i++) {
   String[] values = table[i];
}

Das Ergebnis aus dieser for-Schleife sieht dann so aus:
1. Durchgang
values = {"a","1","4"};
2. Durchgang
values = {"b","2","5"};
3. Durchgang
values = {"c","3","6"};

Ist zwar jetzt eine komplett andere Lösung als du wolltest, aber es ist zumindest eine Lösung ^^ Hoffentlich ist es nicht zu kompliziert

 

[dicke_nudel]

Daran habe ich auch schon gedacht. Leider kann ich so keine Duplikate verhindern. Jedenfalls müsste ich eine neue Schleife schreiben die wieder durch alles druchgeht und Prüft. Bei einer 38 x 32000 Matrix darf ich das dann teuer bezahlen. Daher meine Idee mit einem TreeSet zu arbeiten. Ich brauchs ja Sortiert ohne Duplikate.

 

[Shoox]

Was darf nicht doppelt vorkommen?
Werte allgemein?
Werte in einer Spalte?
Spaltennamen?
Dürfen in verschiedenen Spalten gleiche Werte drin sein?
oder dürfen Spalten nur insgesamt nicht ident sein?

 

[dicke_nudel]

Was darf nicht doppelt vorkommen?
Werte allgemein?
Werte in einer Spalte?
Spaltennamen?
Dürfen in verschiedenen Spalten gleiche Werte drin sein?
oder dürfen Spalten nur insgesamt nicht ident sein?

Mir sind nur die Werte in den jeweiligen Spalten wichtig.
Sollte in verschiedenen Spalten gleiche Werte drin sein ist das ok.

Also:

S1 S2
1 13
2 1
2 1
3 2



Dann brauch ich :

S1 S2
1 1
2 2
3 13

 

[Shoox]

TreeSet<String> ss = new TreeSet<String>();
HashMap<String, TreeSet<String>> hm = new HashMap<String, TreeSet<String>>();
String[] keys = tabelle1.get(0);
for (String key : keys){
   hm.put(key, ss);
   ss = new TreeSet<String>();      
}
for (int i = 1; i < tabelle1.size(); i++) {
   String[] values = tabelle1.get(i);
   for (int j = 0; j < values.length; j++){
      TreeSet<String> tmp = hm.get(keys[j]);
      tmp.add(values[j]);
      hm.put(keys[j], tmp);
   }
}

Hat grad wieder kurz Zeit, das dürfte jetzt passen.
In der HashMap sind nun alle Spalten mit get(Spaltennamen) abbrufbar, keine doppelten, sortiert und ich glaube (erfahrerne Programmierer wären gefragt :bae relativ performant.
Falls etwas unklar ist, fragen ist eines der wenigen Dinge die nichts kosten.
€: Außer bei Anwälten ...

 

[dicke_nudel]

Vielen Dank! Du hast mir meinen Tag gerettet. Ich studier das mal durch. Alles in allem glaube ich das es genau das ist was ich versucht habe zu basteln.


Alles klar. Vielen Dank nochmal.