remove Object von einer Liste von Obejcts

[doktordoom]

Abend,
ich habe mir eine Liste von Mitarbeiter Objekten gebaut und würde gerne einige löschen.
Leier funktioniert dies nicht wie gewollt. Was mach ich falsch?

package example.jdbc.movies;


import java.util.ArrayList;
import java.util.List;

/**
 * This class is used get links from HTML using Jsoup.
 * @author w3spoint
 */
public class Mitarbeiter {

    private static String mitarbeitername ="";

    private static String mitarbeiterguid = "";

    private static int mitarbeiteralter = 0;

    static List<Mitarbeiter> MitarbeiterList = new ArrayList<>();

    public Mitarbeiter(String Name,String GUID,int Alter)
    {
        this.mitarbeitername = Name;
        this.mitarbeiterguid = GUID;
        this.mitarbeiteralter =Alter;
    }

    public String getMitarbeitername()
    {
        return this.mitarbeitername;
    }

    public String getGUID()
    {
        return this.mitarbeiterguid;
    }

    public int getAlter()
    {
        return this.mitarbeiteralter;
    }

    public void setMitarbeitername(String Name)
    {
        this.mitarbeitername = Name;
    }

    public void setGUID(String guid)
    {
        this.mitarbeiterguid = guid;
    }

    public void setAlter(int Alter)
    {
        this.mitarbeiteralter = Alter;
    }

    public static void main(String args[]){



        List<Mitarbeiter> storeMitarbeiter = new ArrayList<>();

        storeMitarbeiter.add(new Mitarbeiter("Max", "dsad213",48));
        storeMitarbeiter.add(new Mitarbeiter("Moritz", "dewr213",38));
        storeMitarbeiter.add(new Mitarbeiter("Max2", "dseww213",43));
        storeMitarbeiter.add(new Mitarbeiter("Max3", "dsewer213",68));
        storeMitarbeiter.add(new Mitarbeiter("Thomas", "ds213dsf",28));
        storeMitarbeiter.add(new Mitarbeiter("Meier", "dwe2sad213",68));

        for(int i = 0; i < storeMitarbeiter.size();i++)
            storeMitarbeiter.remove(storeMitarbeiter.remove(i));

        System.out.println(storeMitarbeiter.get(2).getMitarbeitername());
        System.out.println(storeMitarbeiter.get(3).getMitarbeitername());
    }

 

[LimDul]

Die Felder mitarbeitername und Co sollen mit Sicherheit nicht static sein - es sei denn, deine Mitarbeiter sollen alle Klone sein

Die MitarbeiterList verwendest du nie und sollte auch weg.

 

[doktordoom]

Abend @LimDul,
danke für deine schnelle Antwort und Support 👍

 

[KonradN]

Und die Schleife zum Löschen ist etwas dubios. Was genau ist Dein Ziel?

Zum einen rufst du remove doppelt auf. Wieso?

Zum anderen ist es schwer lesbar:
Das ist so ein typischer Missbrauch einer Zählschleife. Auf den ersten Blick sieht es so aus, als würdest du die Liste durchgehen und jeden Mitarbeiter löschen. Aber du löschst nur jeden Zweiten Mitarbeiter:
Du löschst den Mitarbeiter an Index 0. Dadurch verschieben sich die Indices: Index 1 wird zum neuen Index 0 und so.
Dann wird der Index der Schleife hochgezählt und du löschst dann Index 1.
u.s.w.

Das kann durchaus Deine Absicht gewesen sein, aber das ist so nicht direkt erkennbar. Das sollte man daher etwas umschreiben.

Ein einfacher Weg kann sein, hinten anzufangen und dann den Index immer um 2 zu reduzieren.

Das nur als kleiner Hinweis, weil es mir so ins Auge springt.