OOP Fehler bei foreach Schleife

[Widdsen]

Hallo miteinander,
Ich habe folgendes Problem:
Ich soll eine Methode erstellen, die eine Array Liste durchläuft, in dem Fall eine Mitglieder Liste eines Vereins, und die Mitglieder die im eingegebenen Monat und Jahr, Mitglied geworden sind aus der Liste enfernt und in eine neue Liste speichert! Ich habe mir folgende Methode gebastelt (zeile 63)

import java.util.*;




/**
 * Speichert Details über Vereinsmitgliedschaften.
 * 
 * @author (Ihr Name) 
 * @version (eine Versionsnummer oder ein Datum)
 */
public class Verein
{
    // Definieren Sie alle notwendigen Datenfelder hier...
    private ArrayList<Mitgliedschaft> mitgliedschaft;
    
    private ArrayList<Mitgliedschaft> entfernteMitglieder;
    
    /**
     * Konstruktor für Objekte der Klasse Verein
     */
    public Verein()
    {
        // Initialisieren Sie die Datenfelder hier...
        mitgliedschaft = new ArrayList<Mitgliedschaft>();
        entfernteMitglieder = new ArrayList<Mitgliedschaft>();

    }

    /**
     * Füge ein neues Mitglied in die Mitgliederliste ein.
     * @param mitglied Infos über das einzufügende Mitglied.
     */
    public void beitreten(Mitgliedschaft mitglied)
    {
        mitgliedschaft.add(mitglied);
    }

    /**
     * @return die Anzahl der Mitglieder (Mitgliedschaft-Objekte)
     *         in diesem Verein.
     */
    public int anzahlMitglieder()
    {
        return mitgliedschaft.size();
    }
    
    
    public int beigetretenInMonat(int monat)
    { if( monat < 1 || monat > 12){
       System.out.println(" Der angegebene Monat liegt außerhalb des Gültigkeitsberreichs");
       
    } 
        int anzahl = 0;
        for(Mitgliedschaft mitglied : mitgliedschaft){
          if( mitglied.gibMonat() == monat){
          anzahl++;
        }
    }
       
    return anzahl;
}
public  ArrayList<Mitgliedschaft> entfernen(int monat, int jahr)
{
   
      
      if( monat < 1 || monat > 12){
       System.out.println(" Der angegebene Monat liegt außerhalb des Gültigkeitsbereichs");
    }
    
    
    for(Mitgliedschaft mitglied : mitgliedschaft){
        if(mitglied.gibMonat() == monat && mitglied.gibJahr() == jahr){
            mitgliedschaft.remove(mitglied);
            entfernteMitglieder.add(mitglied);
            }
            
}
        return entfernteMitglieder;
}        
}

Der Compiler Übersetzt zwar aber wenn ich einen Testlauf starte kommt immer dieser Fehler in der Console:
java.util.ConcurrentModificationException
at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:372)
at java.util.AbstractList$Itr.next(AbstractList.java:343)
at Verein.entfernen(Verein.java:72)
wisst ihr was ich falsch gemacht habe?? Danke schonmal im voraus für die Hilfe

 

[Marcinek]

Du darfst die Liste nicht verändern, wenn du über sie iterierst.

Wenn du elemente entfernen willst, nutze einen Iterator. Und deren Methode das Element zu entfernen.

Hättest du google oder SUFU benutzt, dann könntest dir das Posting sparen.

 

[Widdsen]

Ich habe nun den Iterator eingebaut allerdings gibt es wieder eine Fehlermeldung.

public  ArrayList<Mitgliedschaft> entfernen(int monat, int jahr)
{
   
      
      if( monat < 1 || monat > 12){
       System.out.println(" Der angegebene Monat liegt außerhalb des Gültigkeitsbereichs");
    }
    Iterator<Mitgliedschaft> it = mitgliedschaft.iterator();
    while(it.hasNext()){
        Mitgliedschaft mitgliedschaft = it.next();
        if(mitglied.gibMonat() == monat && mitglied.gibJahr() == jahr){
            it.remove();
            entfernteMitglieder.add(mitglied);
        }
    
    }
    
       return entfernteMitglieder;
            
}

Die Variable"mitglied" kann er nicht finden. Dabei ist des doch eine instanz meiner anderen Klasse!

 

[XHelp]

Wo kommt die Variable auf einmal her? Sicher, dass du nicht

mitgliedschaft

meinst?

 

[Marcinek]

Für die Fehlermeldung würde ich mal googeln.

Was wäre wohl das Stichwort: "java eine Fehlermeldung":lol:

 

[Widdsen]

ich bin mir eig. sicher ! ich doch da nur festlegen dass der monat und das jahr in dem das mitglied beigetreten ist das jahr und der Monat ist welcher bei der methode eingegeben wurde! versteht ihr??

 

[Marcinek]

Wir verstehen es, aber du noch nicht.

Um dir zu helfen brauchen wir die FEHLERMELDUNG also den Text der Meldung.

Ich denke in der Array sollte es Mitglidschaft mitglied heißen und nicht Mitgliedschaft mitglidschaft, weil letzteres eine Liste zu sein scheint.

Ich würde mir mal genau zeile für zeile mal anschauen und versuchen zu beschreiben, was du da machst in kommentaren.

Poste das kommentierte hier hin und du hast die lösung.

 

[Widdsen]

achso^^ ja also folgende Fehlermeldung:

cannot find symbol -variable mitglied

muss ich das objekt erst deklarieren???

 

[Widdsen]

Ich denke in der Array sollte es Mitglidschaft mitglied heißen
und nicht Mitgliedschaft mitglidschaft, weil letzteres eine Liste zu sein scheint.

Das wars ^^ vielen Dank für deine Hilfe!
Ja das letztere ist auch eine Liste bin aber ned auf den Trichter gekommen!
hab des problem woanders gesucht!

gruss Widdsen!