Genau, das meinte ich die ganze Zeit, jetzt benötige ich ja nicht mehr meine ArrayList oder?
Hab jetzt die split-Funktion raus genommen, jedoch hab ich jetzt noch nen kleine Fehler beim befüllen des valuesArray, könntest du mir nochmal bitte kurz auf die Sprünge helfen
String[] valuesArray = line;```Mein erster Tipp wäre genau das gewesen, was nillehammer auch angedeutet (bzw. gelöst aber nicht explizit benannt) hat: Wenn man jede einzelne Zeile als EINEN String in eine Set speichert, fliegen Duplikate automatisch raus. Die Zeilen, die dann noch übrig bleiben, kann man splitten und weiterverarbeiten wie gehabt. Wenn man die Zeilen ERST splittet, dann hat man ja Arrays von Strings - und da kann man die Duplikate NICHT einfach entfernen indem man die Arrays in ein Set packt: Die Arrays sind ja immer unterschiedlich. Für zwei Arrays String s0[] und String s1[] gilt eben s0.equals(s1) == false, auch wenn sie den gleichen Inhalt haben.
Hab es jetzt mal noch so versucht wie von nillehammer angedeutet, klappt aber auch nicht…
public static void main(String[] args) throws IOException {
// Einlesen
FileReader myFile = null;
BufferedReader buff = null;
try {
myFile = new FileReader("Dateiausgabe.csv");
buff = new BufferedReader(myFile);
Set<String> uniqueLines= new LinkedHashSet<>();
String line;
for((line = buff.readLine()) != null) {
uniqueLines.add(line);
}
} catch (IOException e) {
System.err.println("Fehler:" + e);
} finally {
try {
buff.close();
myFile.close();
} catch (IOException e) {
System.err.println("Fehler:" + e);
}
}
}```Was klappt nicht? Dein Code gibt ja nichts aus.
Wenn du den Code so anpasst:
public class SetDaten {
public static void main(String[] args) throws IOException {
// Einlesen
FileReader myFile = null;
BufferedReader buff = null;
try {
myFile = new FileReader("Dateiausgabe.csv");
buff = new BufferedReader(myFile);
Set<String> uniqueLines= new LinkedHashSet<>();
String line;
for((line = buff.readLine()) != null) {
// nur für die ausgabe
if(!uniqueLines.contains(line)) {
System.out.println(line);
}
uniqueLines.add(line);
}
} catch (IOException e) {
System.err.println("Fehler:" + e);
} finally {
try {
buff.close();
myFile.close();
} catch (IOException e) {
System.err.println("Fehler:" + e);
}
}
}
und dieses CSV verwendest:
pezi,kasperl
pezi,kasperl
pezi,kasperl
oma,kasperl
kasperl,pezi
bekommst du als Ausgabe:
pezi,kasperl
oma,kasperl
kasperl,pezi
Ist das, das was du willst? Falls ja, gib genauer an was nicht geht.
Ja, „klappt nicht“ ist keine Fehlermeldung, bei der Du hilfreiche Hinweise erwarten kannst. Dass Dein Code die Originaldatei nicht verändert, sollte klar sein? Im Moment hast Du nur den ersten Schritt „Einlesen“ fertig. Die Duplikatsbereinigten Zeilen stehen erstmal nur im Set (also im RAM von Java). Dessen Inhalt müsste in einem zweiten Schritt natürlich noch wieder in eine Datei geschrieben werden.
Und dafür hat Dein Code einen kleinen Stolperstein. Durch die Deklaration des Sets innerhalb des try-catch-Blocks ist es nur dort sichtbar. Beim Zurückschreiben musst Du aber höchstwahrscheinlich von außerhalb dieses Blocks drauf zugreifen. Hol es also da raus:
// Einlesen
FileReader myFile = null;
BufferedReader buff = null;
Set<String> uniqueLines= new LinkedHashSet<>();
try {
...hier der ganze Einlesecode wie gehabt.
@Marco13 : Hab ich doch vor meinem Code genauso erklärt :eek: (Naja, nicht ganz so ausführlich…)
irgendetwas an meiner for Schleife
Da bekomm ich die Meldung “The left hand side of an assigment must be a variable” aber line ist doch eine Variable?
FileReader myFile = null;
BufferedReader buff = null;
myFile = new FileReader("Dateiausgabe.csv");
buff = new BufferedReader(myFile);
Set<String> uniqueLines= new LinkedHashSet<>();
try {
String line;
for((line = buff.readLine()) != null) {
if(!uniqueLines.contains(line)) {
System.out.println(line);
uniqueLines.add(line);
}}
```
Habe meinen Code jetzt so angepasst. Wie gesagt der Fehler ist halt noch die forschleife, da stimmt irgendetwas noch nicht.
@nillehammer : Ja ich hatte dann vor mit nem Filewriter die csv-Datei zu überschreiben, aber bis jetzt funktioniert meine Schleife ja noch nicht.for((line = buff.readLine()) != null) {
muss natürlich
while((line = buff.readLine()) != null) {
sein. So sollte es funktionieren.
Super, danke euch.
if(!uniqueLines.contains(line)) {
Zeile 11 ist doch unnötig wie ein Kropf?
uniqueLines ist ein Set, das checkt beim Hinzufügen schon automatisch ob das Ding schon drin ist.
Zudem behält LinkedHashSet die Reihenfolge des hinzufügens bei. http://docs.oracle.com/javase/6/docs/api/java/util/LinkedHashSet.html
EDIT hat sich erledigt
Nein. Nicht wenn der Wert sofort weiterverwendet werden soll wenn er noch nicht im Set ist. Dann verwendet man eben nur das Feature des schnellen Lookups.
Ich versteh jetzt nicht wieso er mir die neu erstellte Datei nicht befüllt
Da fehlt entscheidendes. closed du den OutputStream?
Ich glaube jetzt siehts schon etwas besser aus.
Aber in der For-Schleife ist noch ein Fehler drinnen…
String dateiausgabe = "DateiausgabeOhneDuplikate.csv";
String speicher = "";
try {
String line;
while((line = buff.readLine()) != null) {
if(!uniqueLines.contains(line)) {
System.out.println(line);
uniqueLines.add(line);
for ( String aktuellerDatensatz; ( aktuellerDatensatz = uniqueLines.add(line); ) != null; ){
// In der Zeile oberhalb ist der Fehler Fehler
bw = new BufferedWriter(new FileWriter(dateiausgabe, true));
bw.write(aktuellerDatensatz);
bw.newLine();
}
bw.write(speicher);
bw.flush();
bw.close();
buff.close();
}}
} ```Lies bitte konzentriert Schlingels Post mit #27.
Danke, aber da ist die for-Schleife glaub ich schon richtig
es geht um den Teil „= uniqueLines.add(line);“ → Cannot convert from boolean to string
Hab aber keine Ahnung wie ich hier sonst die gefilterten Datensätze mitreinbringen soll.
Die ist überflüssig und falsch. Wofür soll denn die innere Schleif gut sein? Weißt Du was die Methode add eines HashSets zurückgibt?
Moin,
[QUOTE=javabeginner67]jedoch hab ich jetzt noch nen kleine Fehler beim befüllen des valuesArray, könntest du mir nochmal bitte kurz auf die Sprünge helfen
while ((line = buff.readLine()) != null)
{
String[] valuesArray = line;
[/QUOTE]
Du versuchst einem String-Array eine String zuzuweisen!
Das kann nicht funktionieren 
[QUOTE=javabeginner67;89514]Danke, aber da ist die for-Schleife glaub ich schon richtig
es geht um den Teil „= uniqueLines.add(line);“ → Cannot convert from boolean to string
Hab aber keine Ahnung wie ich hier sonst die gefilterten Datensätze mitreinbringen soll.[/QUOTE]
Andere Frage, ähnliches Problem …
Aber es darf hier wirklich keine for-Schleife sein! Es MUSS eine WHILE-Schliefe sein !!
Gruß
Klaus
Ja sorry. Das mit dem “for” in der Einleseschleife kam von mir. Hab’s halt schnell in den Editor geklickert. Die Lösung “while” ist ja schon genannt worden und in dem neuen Code auch bereits drinnen. Aber leider: Neuer Code neue Fehler:
[QUOTE=nillehammer]Ja sorry. Das mit dem “for” in der Einleseschleife kam von mir. Hab’s halt schnell in den Editor geklickert. Die Lösung “while” ist ja schon genannt worden und in dem neuen Code auch bereits drinnen. Aber leider: Neuer Code neue Fehler:
Danke, funktioniert jetzt einwandfrei
[QUOTE=schlingel]Nein. Nicht wenn der Wert sofort weiterverwendet werden soll wenn er noch nicht im Set ist. Dann verwendet man eben nur das Feature des schnellen Lookups.
[/QUOTE]
Kuckst du Set.add(E e);. Das liefert dir boolean zurück. true wenn der Wert noch nicht vorhanden ist ansonsten false.
if(uniqueLines.add(line)) {
Reicht, da dies intern sowieso contains aufruft. So spart man sich zweimal das Set durchzugehen.