Java sortowanie listy ranking do gry

Question

Napisałam grę do której próbuje stworzyć ranking, który wyświetli mi 10 najlepszych wyników uzyskanych przez graczy. Dane te przechowuj w pliku .txt w formacie: int (wynik), String (imię).

Początkowo pomyślałam, że sczytam zawartość pliku do listy i ją posortuję. Wszystko działa ok do momentu, aż wynik nie jest dwucyfrowy. Wtedy sortuje mi po pierwszej cyfrze, czyli liczbę 89 traktuję jako 9. 

Nie mam już pomysłu w jaki inny sposób mogłabym to rozwiązać, żeby działało poprawnie. Może w ogóle powinnam zrobić to w inny sposób? Poniżej mój kod:

import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;

import javax.swing.ImageIcon;
import javax.swing.JOptionPane;

public class Ranking  {
	
	public static void ranking() {
	List <String> lista = new ArrayList<String>();
	{	
	String line;
	BufferedReader br;
	try {
		br = new BufferedReader(new FileReader("src/main/resources/ranking.txt"));
		
		while((line = br.readLine()) != null) {  
				
		lista.add(line);
		}
	} catch (FileNotFoundException e) {
		
		e.printStackTrace();
	} catch (IOException e) {
	
		e.printStackTrace();
	}
	

	Collections.sort(lista);
	
	Collections.reverse(lista);
	
	
	
	String s = "";
	
	int i = 0;
	
	for(String str: lista) {
		
		s += str + "\n";
		
		i++;
		if (i == 10) {
	
	break;
		
	}
	
	}
	
	ImageIcon iconRanking = new ImageIcon ("src/main/resources/podium.jpg");
	int textRanking = JOptionPane.showOptionDialog(null ,"Top 10: " + "\n" + s, "Ranking", JOptionPane.DEFAULT_OPTION, JOptionPane.PLAIN_MESSAGE, iconRanking, null, null);
	
	
	}
	}
}

 

Answer 1

Dzieje się tak ponieważ sortujesz Stringi:

List <String> lista = new ArrayList<String>();
Collections.sort(lista);

 

  int i = 0;
     
    for(String str: lista) {
        
        s += str + "\n";
         
        i++;
        if (i == 10) {
     
    break;
         
    }

W pętli typu for-each czyli taka jak wyżej, zmienna i nie jest potrzebna.

Jeśli już to zrób klasyczną pętle for (pozbędziesz się wtedy tego paskudnego if'a w forze):

for(int i = 0; i < 10; i++) {}

Jeśli chcesz być bardziej pro to zapoznaj się ze StringBuilder'em:

https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/lang/StringBuilder.html

StringBuilder s = new StringBuilder();

for(int i = 0; i <= 10; i++){
   s.append(str);
}

String result = s.toString();

 

Comments

Sortuje Stringi bo nie wiem jak inaczej sczytać ten plik, żeby wyświetlał się później wynik plus imię, a jednocześnie móc to posortować.

Dzięki za podpowiedź z pętlą :)

---

napisz klase która implementuje np interfejs Comparable lub Comparator i zaimplementuj własną metodę porównania, potem użyj jej w sortowaniu :)

---

Sparsuj dane. Pisałeś że w pliku jest

int,string

zobacz metode String::split:

line.split(",");

 

---

Czy jak sparsuje dane to nadal będzie mi się wyświetlał równiez text? Bo ja potrzebuje, żeby dane z pliku wyświetliły mi się później w JOptionPane.showOptionDialog w formacie wynik gracza plus imię.

---

tak o ile je zapiszesz jako typ tekstowy.