Dateien/LinkedList/StringBuffer - SOrtierung klappt nicht so ganz

[Staubkoernchen]

Huhu,

ich soll für ne Bonuspunkteprojektarbeit in einer Teilaufgabe eine Top-Ten-Liste für mein Spiel erstellen.
Es läuft alles ganz wunderbar, nur seltsamerweise sortiert er meine Einträge nicht mehr so richtig, er fügt neue Einträge einfach unten an, wenn ich erst den Befehl bestenliste.einlesen(); und danach mehrmals bestenliste.einfügen(punkte, name) ausführe
Die Sortierung geschieht in der Methode einfügen, nur seitdem die Methode einlesen dazugekommen ist funktioniert das nicht mehr

Hier mein Quellcode:

import java.util.LinkedList;
import java.io.*;
//checked exceptions
import java.io.IOException;
public class Bestenliste2 {
	LinkedList namen = new LinkedList();
	LinkedList punkte = new LinkedList();
	final static String lineSeparator = System.getProperty("line.separator");
	boolean winner;
	
	public Bestenliste2(){
		namen.add("Leer");
		punkte.add(0);//So ist die LinkedList zu Anfang nicht leer

	}
	
	public void einlesen(){

		try{
			FileReader reader = new FileReader("bestenliste.txt");
			StringBuffer einlese=new StringBuffer();

			for(int i=reader.read();i!=-1;i=reader.read()){
                einlese.append((char)i);
            }
			
			int help=0;
			int help2=0;
			
			for(int i=help2;i!=-1;i=help2){
				help2=einlese.indexOf("Name: ", help);
				if(help2!=-1){
					help=help2+6;
					help2=einlese.indexOf(", Punkte:", help);
					namen.add(einlese.substring(help, help2));
					
					help=einlese.indexOf(", Punkte:", help)+10;
					help2=einlese.indexOf(lineSeparator, help);
					int help3=Integer.parseInt(einlese.substring(help, help2));
					punkte.add(help3);
					}
			}
		}
		catch(IOException e){
			System.out.println("Die Datei bestenliste.txt ist nicht in ihrem Verzeichnis!");
		}
	}
	
	public void eintragen(int points, String name){
		
		for(int i=0; i<=punkte.size()-1;i++){//Hier werden die Dinger "eigentlich" sortiert
			if(points>(int)punkte.get(i)){
				punkte.add(i, points);
				namen.add(i, name);
				
				if(punkte.size()>10){
					punkte.remove(10);
					namen.remove(10);
				}
				break;
			}
		}
		try{
			
			FileWriter schreiber = new FileWriter("Bestenliste.txt");//Hier werden sie reingeschrieben
			schreiber.write("Top Ten: ");
			schreiber.write(lineSeparator);
			for(int i=0;i<=punkte.size()-1;i++){
				schreiber.write((i+1) +". Name: " +namen.get(i) + ", Punkte: " + punkte.get(i));
				schreiber.write(lineSeparator);
			}
			schreiber.close();

			}
		catch(IOException e){
			System.out.println("Die Datei bestenliste.txt ist nicht in ihrem Verzeichnis!");
		}
		
	}
	
	

}

Und hier meine Main (mehrmals hintereinander ausgeführt - beim ersten Ausführen ist alles richtig - kommen neue Einträge hinzu, so klatscht er die hinten dran, anstatt sie richtig einzusortieren)

public class test {
	public static void main(String[] args){
		Bestenliste2 laa = new Bestenliste2();
		
		laa.einlesen();
		laa.eintragen(75, "e");
		laa.eintragen(65, "f");
		laa.eintragen(2, "g");
		laa.eintragen(68, "h");
		laa.eintragen(75, "i");
		
	}

}

Ich wäre unendlich dankbar wenn da jemand drübergucken würde.

 

[DrZoidberg]

Du fügst immer "Leer" und 0 in deine Liste ein, auch dann wenn da schon eine Liste in der Dartei ist.
Das heisst du erstellst eine Liste, die so aussieht
1. Name: Leer, Punkte: 0
und fügst dann sämtliche Einträge aus "bestenliste.txt" hinzu.
Danach sieht die Liste dann so aus
1. Name: Leer, Punkte: 0
2. Name: e, Punkte: 75
3. Name: i, Punkte: 75
4. Name: h, Punkte: 68
5. Name: f, Punkte: 65
6. Name: g, Punkte: 2
7. Name: Leer, Punkte: 0

Wenn du dann laa.eintragen(75, "e"); ausführst, hast du
1. Name: e, Punkte: 75
2. Name: Leer, Punkte: 0
3. Name: e, Punkte: 75
4. Name: i, Punkte: 75
5. Name: h, Punkte: 68
6. Name: f, Punkte: 65
7. Name: g, Punkte: 2
8. Name: Leer, Punkte: 0
usw.

Am besten wäre du schreibst da gar keinen "Leer" Eintrag rein.

Ausserdem sollte man bei Listen immer den Typ der Liste mit <> angeben.
Und was ist, wenn ein Name schon in der Liste vorhanden ist? Soll der selbe Name dann ein zweites mal eingetragen werden?
Ich habe dein Program etwas verändert. Jetzt wird die Punktzahl eines vorhandenen Names lediglich aktualisiert anstatt den Namen zwei mal einzutragen. Ich habe auch die "einlesen" Methode verkürzt.

import java.util.LinkedList;
import java.io.*;
//checked exceptions
import java.io.IOException;
public class Bestenliste2 {
    LinkedList<String> namen = new LinkedList<String>();
    LinkedList<Integer> punkte = new LinkedList<Integer>();
    final static String lineSeparator = System.getProperty("line.separator");
    boolean winner;
    
    public void printList() {
        for(int i = 0; i < punkte.size(); i++) {
            System.out.println("Name: " + namen.get(i) + ", Punkte: " + punkte.get(i));
        }
        System.out.println();
    }
    
    public void einlesen(){
        try{
            BufferedReader reader = new BufferedReader(new FileReader("bestenliste.txt"));
            String line = "";
            while((line = reader.readLine()) != null) {
                int indexName = line.indexOf("Name: ");
                int indexPunkte = line.indexOf(", Punkte: ");
                if(indexName >= 0) {
                    namen.add(line.substring(indexName + 6, indexPunkte));
                    punkte.add(Integer.parseInt(line.substring(indexPunkte + 10)));
                }
            }
        }
        catch(IOException e){
            System.out.println("Die Datei bestenliste.txt ist nicht in ihrem Verzeichnis!");
        }
        System.out.println("Liste eingelesen");
        printList();
    }
    
    public void eintragen(int points, String name){
        System.out.println("Neuer Eintrag - Name: " + name + ", Punkte: " + points);
        
        boolean eingetragen = false;
        
        for(int i = 0; i<=punkte.size()-1;i++) {
            if(namen.get(i).equals(name)) {
                punkte.remove(i);
                namen.remove(i);
            }
        }
        for(int i=0; i<=punkte.size()-1;i++){//Hier werden die Dinger "eigentlich" sortiert
            if(points>punkte.get(i)){
                punkte.add(i, points);
                namen.add(i, name);
                eingetragen = true;                
                break;
            }
        }
        if(!eingetragen) {
            punkte.add(points);
            namen.add(name);
        }
        if(punkte.size()>10){
            punkte.remove(10);
            namen.remove(10);
        }
        printList();
        try{
            
            FileWriter schreiber = new FileWriter("Bestenliste.txt");//Hier werden sie reingeschrieben
            schreiber.write("Top Ten: ");
            schreiber.write(lineSeparator);
            for(int i=0;i<=punkte.size()-1;i++){
                schreiber.write((i+1) +". Name: " +namen.get(i) + ", Punkte: " + punkte.get(i));
                schreiber.write(lineSeparator);
            }
            schreiber.close();
 
            }
        catch(IOException e){
            System.out.println("Die Datei bestenliste.txt ist nicht in ihrem Verzeichnis!");
        }
        
    }
}

 

[Staubkoernchen]

Tausend Dank Was mit einem zwei Mal eingetragenen Namen geschehen soll, weiß ich nicht, ich denke aber, die werden mir schon nichts abziehen deswegen (und wenn ists mir auch wurscht)
Das einzige Problem ist, wir dürfen, denke ich, den BufferedReader nicht nutzen. Aber da kann ich ja meine eigene Methode nehmen, an der lags ja nicht, oder?

Und nochmals: Vielen vielen Dank