Exception beim Verwenden von Arraylist?

[nonoob]

Hallo Community,

das ist mein erster Post in der Community und ich habe mir die Regeln durchgelesen. Ich bin absoluter noob im Programmieren und lerne es einfach hobbymäßig. Kein beruflicher, schulischer oder studentischer Zusammenhang. Also Hausaufgaben dürfte daher keiner für mich erleidgen ^^. Zu meiner Frage:

Ich versuche eine DartApp zu programmieren, bei der die geworfenen Punkte beim Dartspielen gezählt werden. Beim ausführen des Programms spuckt die Konsole eine Exception aus, mit der ich wenig anfangen kann. Ich habe bereits nach Lösungen gesucht, aber komme nicht weiter. Ich vermute der Fehler liegt darin, dass ich versuche, die ArrayList zwei Methoden aufrufen zu lassen.

 

[M.L.]

Regeln durchgelesen.

Code trotzdem mit Code-Tags posten, ausserdem wohl auch den gesamten Code da der Fehler (auch) an anderer Stelle zu liegen scheint.

 

[kneitzel]

Der Index geht von 0 bis size-1
In der Schleife gehst Du aber bis einschließlich size() und dadurch kommt es zu der Exception.

Also die Schleifenbedingung ist mit i <= spielerListe.size() schlicht falsch.

Auch der Sinn der Schleife ergibt sich mir nicht wirklich. Wenn Du am Ende angekommen bist, dann soll es von Anfang an weiter gehen? Dann mach eine Endlosschleife oder so, damit das erkennbar ist.

 

[nonoob]

import java.util.ArrayList;

public class Dart {

    public static void main(String[] args) {

        boolean spielEnde = false;

        ArrayList<Spieler> spielerListe = new ArrayList<Spieler>();

        Spieler spieler1 = new Spieler("Ich");

        spielerListe.add(spieler1);

        spieler1.modus301();

        while (spielEnde == false) {
            for (int i = 0; i <= spielerListe.size(); i++) {
                spielerListe.get(i).punkteAbziehen(10, 10, 10);
            }
        }
    }
}

public class Spieler {
    
    private String name;
    private int punkte;
    private boolean spielEnde = false;

    
    public Spieler(String name){
        this.name = name;
    }
    
    public int punkteAbziehen(int Wurf1, int Wurf2, int Wurf3) {       
        if( Wurf1 +  Wurf2 +  Wurf3 > this.punkte) {
            System.out.println("Überworfen! Deine Punkte betragen immernoch: " + this.punkte);
        }else{
            this.punkte -= (Wurf1 + Wurf2 + Wurf3);
            if(this.punkte == 0) {
                spielEnde = true;
            }
        }return this.punkte;
    }
    
    public void status(){
        System.out.println("Spieler: " + this.name + " Punkte: " + this.punkte);
    }
    
    public int modus301()
    {
        this.punkte = 301;
        return this.punkte;
    }
    
    
    public int modus501(){
        this.punkte = 501;
        return this.punkte;
    }
}

 

[nonoob]

das wäre jetzt die Main-Klasse.

 

[kneitzel]

for (int i = 0; i <= spielerListe.size(); i++) {
spielerListe.get(i).punkteAbziehen(10, 10, 10);

Dazu meinte ich schon:

Der Index geht von 0 bis size-1
In der Schleife gehst Du aber bis einschließlich size() und dadurch kommt es zu der Exception.

Also die Schleifenbedingung ist mit i <= spielerListe.size() schlicht falsch.

Du hast in spielerListe von mir aus 2 Elemente. Diese kannst Du dann mit spielerListe.get(0) und spielerListe.get(1) darauf zugreifen und spielerListe.size() gibt 2 zurück.

Deine Schleife läuft aber von 0, 1 bis 2. Und spielerListe.get(2) würde das 3. Element zurück geben und das gibt es nicht.

Bei Deiner Exception aus dem Bild in #1 hattest Du: Element mit id 1 war außerhalb des Bereichs bei nur einem Element. Ein Element bedeutet: Nur Index 0 ist belegt. Index 1 versucht auch das zweite Element zuzugreifen, das es aber nicht gibt.

Ist es deutlich geworden? Ansonsten mach in der Schleife noch eine kleine Ausgabe von i, damit Du siehst, auf was alles zugegriffen wird. Also sowas wie:
System.out.println(" i:= " + i + " size := " + spielerListe.size());
Dann siehst Du in der Ausgabe, auf welche Elemente er zugreift.

 

[kneitzel]

Um evtl. auch die Behebung anzusprechen:

Also die Schleifenbedingung ist mit i <= spielerListe.size() schlicht falsch.

hatte ich ja schon erwähnt. Die Überlegung ist, von wo bis wo läuft der Index bei n Elementen?

Bei 2 Elementen hatte ich ja schon geschrieben: 0..1 sind die Indices.

Dann wäre die Frage, bis zu welchem Index es bei n Elementen geht.

Oder anders herum: Wir haben in der Erklärung festgestellt, dass die Anzahl der Elemente kein gültiger index mehr ist. Also i darf nicht gleich n sein (mit n: anzahl der Elemente) ...

 

[EinNickname9]

mit Code-Tags posten

Bin immer wieder beeindruckt, wie Leute es trotz bester Absicht immer wieder schaffen, alles falsch zu machen...

• Code bitte formatiert in Code Tags setzen
• Die Fehlermeldung des Compilers angeben
• Die Fehlerzeile farblich hervorheben
• Beschreibe das, was du machen willst, gemacht hast und was noch nicht funktioniert...

 

[kneitzel]

Nunja - das mit den Code Tags hat er doch schon gemacht - da ist aber wohl beim Editieren der Start-Tag verloren gegangen weshalb der erste Code so angezeigt wird. Und die Probleme selbst waren in #1 in den Bildern. Daher ist das "alles falsch machen" doch stark übertrieben denke ich ...

 

[nonoob]

Jetzt habe ich den Fehler gefunden und gelöst. Danke für die Hilfe

 

[nonoob]

Der entscheidende Hinweis kam von dir kneitzel, durch die Bemerkung mit der Endlosschleife.