Highscore Syntax-Problem

[ollestrat]

Hallo, mein erster Beitrag (lasst Gnade walten ):

ich möchte für ein kleines Spiel ein Highscoreliste erstellen und habe nachdem ich hier im Forum mir ein paar Beiträge zu dem Thema durchgelesen habe einen vergleichsweise einfachen Ansatz gefunden. Es existiert noch eine Klasse Spieler für den Namen und den Punktestand. Es scheint aber irgendein grundlegendes Syntaxproblem zu geben (--> //NullPointerException). Jemand eine Idee?

public class Highscore {

	static Spieler [] array = new Spieler[10]; 

	
		public static void main (String[]args)
		{
			Spieler hans = new Spieler("Hans", 10);
			Spieler klaus = new Spieler("Klaus", 13);
			Spieler ina = new Spieler("Ina", 0);
			Spieler ole = new Spieler("Ole", 78);
			Spieler kalle = new Spieler("Kalle", 10);
			Highscore hs = new Highscore();
			hs.einfügen(hans); // NullPointerException!!!
			hs.einfügen(klaus);
			hs.einfügen(ina);
			hs.einfügen(ole);
			hs.einfügen(kalle);
						
		}
	
	public void einfügen(Spieler s)
	{
		for (int i = 0; i<=array.length-1;i++ )
			if (array[0].getPunkte()< s.getPunkte())
				{
				array[i] = s; 
				break;
				}
	}
	public void anzeigen()
	{
		for (int i = 0; i<=array.length-1;i++ )
			System.out.println(array[i].getName() + " " + array[i].getPunkte());
	}
}

[/code]

 

[Gandave]

Hallo ollestrat,

Das Problem ist, dass dem Array zwar schon eine Größe zugewiesen wurde (10), aber er ansonsten leer ist.
Deine Schleife in 'einfügen' versucht jetzt vom Element an Stelle 0 (müsste das nicht i sein?) die Punkte zu ermitteln. Da an der Stelle 0 (wie an den anderen Stellen auch) noch kein Element ist, bzw. das Element 'null', kann die Methode 'getPunkte()' nicht ausgeführt werden.

Mögliche Lösung:

public static void main(String[] args)
{
  for (int i = 0; i < array.length(); i++)
  {
    array[i] = new Spieler();
  }
  // ... ab da normal weiter
}

Dann musst du aber in der Klasse 'Spieler' eine entsprechende Initalisierung erlauben. Oder du erstellst einen neuen Spieler mit "leeren" Werten, also 'new Spieler("leer", 0)'.

Nebenbei, bei deiner einfügen-Schleife wird meiner Meinung nach der neue Spieler an der Stelle eingesetzt, an der der letzte Spieler war, der schlechter als dieser Spieler war. Aber eigentlich müsste der alte Spieler und alle, die nach ihm kommen nach unten verschoben und nur der letzte gelöscht werden!
Mein Vorschlag: nimm einen Vector!
Damit umgehst du nebenbei auch das Problem mit der NullPointerException.

Ich hoffe, das war verständlich.

Mit freundlichen Grüßen
Gandave

 

[Joker]

ein Syntaxproblem ist es nicht, aber ein Logikproblem

Schau dir mal Zeile 25 an. Du versuchst hier auf das Element am Index 0 zuzugreifen obwohl da noch überhaupt kein Element liegt.

Besser ist aber sowas hier:

public class Highscore {
    private List<Spieler> players;
    
    public Highscore() {
        players = new ArrayList<Spieler>();
        
        einfuegen(new Spieler("Hans", 10));
        einfuegen( new Spieler("Klaus", 13));
        einfuegen(new Spieler("Ina", 0));
        einfuegen(new Spieler("Ole", 78));
        einfuegen(new Spieler("Kalle", 10));     
        anzeigen();
    }
    
    public void einfuegen(Spieler s) {
        players.add(s);
      
    }
    
    public void anzeigen() {
        sortPlayer();
        for(Spieler s:players) {
            System.out.println(s.getName()+"  "+s.getPoints());
        }
    }
    
    public void sortPlayer() {
        Collections.sort(players, new Comparator() {

            public int compare(Object o1, Object o2) {
                Integer p1 = ((Spieler) o1).getPoints();
                Integer p2 = ((Spieler) o2).getPoints();

                return p2.compareTo(p1);
            }
        });
    }
    
    public static void main(String[] args ){
        new Highscore();
    }
}

edit: Da war jemand schneller, man sollte halt die Threads nicht in Tabs öffnen um sie dann in der Reihenfolge abzuarbeiten ohne das man vorher prüft ob schon eine Antwort da ist

ps.: wieso Vector, hier ist doch nichts Multithreaded. ArrayList ist da besser, da hier die Methoden nicht synchronized sind.

 

[ollestrat]

Hi Gandave,
danke für die schnelle Antwort, Problem gelöst. Die Klasse Vector muss ich mir mal anschauen..

Ich hoffe, das war verständlich.

Ja war es
:toll:

Gruß