Collections ArrayList mit Comparator sortieren

[ve987]

Hallo zusammen,

ich möchte, dass die Methode VW.ausgabe(list); mir die Autos sortiert.
Leider bekomme ich überhaupt nichts zurück.

Liegt es daran, dass die List<Auto> list zweimal deklariert wird?
Wie sonst könnte man das lösen?

Vielen Dank im Voraus!

package Mobile2;

import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;

public class Main {


    public static void main(String[] args) {

        List<Auto> list = new ArrayList<Auto>();

        VW.addAuto(VW.erstelleAuto(true, "VW", 4.000, 14.000, "gelb", 2008));
        VW.addAuto(VW.erstelleAuto(false, "VW", 5.000, 4.000, "schwarz", 2008));
        VW.addAuto(VW.erstelleAuto(true, "VW", 9.000, 45.000, "weiss", 2007));
        VW.addAuto(VW.erstelleAuto(false, "VW", 18.000, 4.000, "lila", 2010));
        VW.addAuto(VW.erstelleAuto(false, "VW", 11.000, 6.000, "orange", 2012));
        VW.addAuto(VW.erstelleAuto(false, "VW", 34.000, 8.000, "blau", 2013));
        VW.addAuto(VW.erstelleAuto(true, "VW", 11.000, 14.000, "gelb", 2014));
        VW.addAuto(VW.erstelleAuto(true, "VW", 23.000, 14.000, "gelb", 2015));
        VW.addAuto(VW.erstelleAuto(true, "VW", 10.000, 14.000, "black", 2008));
        VW.addAuto(VW.erstelleAuto(true, "VW", 19.000, 14.000, "grün", 2017));
        VW.addAuto(VW.erstelleAuto(true, "VW", 7.000, 14.000, "rosa", 2019));

        Collections.sort(list, new sortErstZulassung());

        VW.ausgabe(list);

        VW.sucheAuto();

    }

    static class sortErstZulassung implements Comparator<Auto> {

        @Override
        public int compare(Auto o1, Auto o2) {

            return Integer.valueOf(o1.getErstzulassung()).compareTo(o2.getErstzulassung());

        }

    }
}

package Mobile2;

public class Auto {

    private boolean neuwagen;
    private String marke;
    private double preis;
    private double kilometerstand;
    private String farbe;
    private int erstzulassung;

    public Auto(boolean neuwagen, String marke, double preis, double kilometerstand, String farbe, int erstzulassung) {
        super();
        this.neuwagen = neuwagen;
        this.marke = marke;
        this.preis = preis;
        this.kilometerstand = kilometerstand;
        this.farbe = farbe;
        this.erstzulassung = erstzulassung;

    }

    public boolean isNeuwagen() {
        return neuwagen;
    }

    public void setNeuwagen(boolean neuwagen) {
        this.neuwagen = neuwagen;
    }

    public String getMarke() {
        return marke;
    }

    public void setMarke(String marke) {
        this.marke = marke;
    }

    public double getPreis() {
        return preis;
    }

    public void setPreis(double preis) {
        this.preis = preis;
    }

    public double getKilometerstand() {
        return kilometerstand;
    }

    public void setKilometerstand(double kilometerstand) {
        this.kilometerstand = kilometerstand;
    }

    public String getFarbe() {
        return farbe;
    }

    public void setFarbe(String farbe) {
        this.farbe = farbe;
    }

    public int getErstzulassung() {
        return erstzulassung;
    }

    public void setErstzulassung(int erstzulassung) {
        this.erstzulassung = erstzulassung;
    }

}

package Mobile2;

import java.text.DecimalFormat;
import java.util.*;

public class VW extends Auto {

    static List<Auto> list = new ArrayList<Auto>();

    public static int count;
    public static int countFarbe;
    public static int countPreis;
    public static int countErstzulassung;

    public VW(boolean neuwagen, String marke, double preis, double kilometerstand, String farbe, int erstzulassung) {
        super(neuwagen, marke, preis, kilometerstand, farbe, erstzulassung);
    }

    public static boolean addAuto(VW a) {
        list.add(a);
        return true;
    }

    public static VW erstelleAuto(boolean neuwagen, String marke, double preis, double kilometerstand, String farbe,
            int erstzulassung) {
        return new VW(neuwagen, marke, preis, kilometerstand, farbe, erstzulassung);
    }

    public static void ausgabe(List<Auto> list) {
        for (Auto auto : list) {
            System.out.println(auto);
        }
    }

    public static boolean sucheAuto() {

        Scanner scanner = new Scanner(System.in);
        System.out.println("Neuwagen oder Gebrauchswagen?");
        String wagenArt = scanner.nextLine();

        // ANFANG NEUWAGEN
        if (wagenArt.equalsIgnoreCase("Neuwagen")) {

            for (Auto auto : list) {
                if (auto.isNeuwagen()) {
                    count++;
                    System.out.println(auto);
                }
            }

            System.out.println("So viele Autos entsprechen deinen Kriterien: " + count);

            Scanner scanner2 = new Scanner(System.in);
            System.out.println("Wonach die Suche eingrenzen? Preis, Farbe, Erstzulassung?");
            String eingabe = scanner2.nextLine();

            if (eingabe.equalsIgnoreCase("Farbe")) {
                Scanner scanner4 = new Scanner(System.in);
                System.out.println("Welche Farbe?");
                String eingabeFarbe = scanner4.nextLine();

                for (Auto auto2 : list) {
                    if (eingabeFarbe.equalsIgnoreCase(auto2.getFarbe())) {
                        countFarbe++;
                        System.out.println(auto2);
                    }
                }

                System.out.println("So viele Autos entsprechen deinen Kriteien: " + countFarbe);

            } else if (eingabe.equalsIgnoreCase("Preis")) {
                Scanner scanner5 = new Scanner(System.in);
                System.out.println("Bis 10.000 oder bis 20.000?");
                double eingabePreis = scanner5.nextDouble();

                for (Auto auto3 : list) {
                    if (eingabePreis == 10 && auto3.getPreis() < 10.000 && auto3.isNeuwagen()) {
                        countPreis++;
                        System.out.println(auto3);
                    } else if (eingabePreis == 20 && auto3.getPreis() < 10.000 && auto3.isNeuwagen()) {
                        countPreis++;
                        System.out.println(auto3);
                    }
                }

                System.out.println("So viele Autos entsprechen deinen Kriteien: " + countPreis);

            } else if (eingabe.equalsIgnoreCase("Erstzulassung")) {
                Scanner scanner6 = new Scanner(System.in);
                System.out.println("bis 2010, 2015 oder 2020?");
                int eingabeErstzulassung = scanner6.nextInt();

                switch (eingabeErstzulassung) {

                case 2010:
                    for (Auto auto4 : list) {
                        if (auto4.getErstzulassung() < 2010 && auto4.isNeuwagen()) {
                            countErstzulassung++;
                            System.out.println(auto4);
                        }
                    }
                    break;
                case 2015:
                    for (Auto auto4 : list) {
                        if (auto4.getErstzulassung() < 2015 && auto4.isNeuwagen()) {
                            countErstzulassung++;
                            System.out.println(auto4);
                        }
                    }
                    break;
                case 2020:
                    for (Auto auto4 : list) {
                        if (auto4.getErstzulassung() < 2020 && auto4.isNeuwagen()) {
                            countErstzulassung++;
                            System.out.println(auto4);
                        }
                    }
                    break;

                default:
                    System.out.println("Falsches Jahr");
                }

                System.out.println("So viele Autos entsprechen deinen Kriteien: " + countErstzulassung);

            }
            // ENDE NEUWAGEN

            // ANFANG GEBRAUCHSWAGEN
        } else if (wagenArt.equalsIgnoreCase("Gebrauchswagen")) {

            for (Auto auto : list) {
                if ((!auto.isNeuwagen())) {
                    count++;
                    System.out.println(auto);
                }
            }
            System.out.println("So viele Autos entsprechen deinen Kriterien: " + count);
        } // ENDE WAGEN

        return true;
    }

    @Override
    public String toString() {

        DecimalFormat decimalFormat = new DecimalFormat("#.000");

        String string;

        if (isNeuwagen()) {
            string = "Neuwagen";
        } else {
            string = "Gebrauchswagen";
        }

        return "" + getFarbe() + " " + "Preis" + " " + decimalFormat.format(getPreis()) + " " + "Kilometerstand" + " "
                + getKilometerstand() + " " + string + " " + "Erstzulassung: " + " " + getErstzulassung();
    }


}

 

[MoxxiManagarm]

Bitte entschuldige wenn sich das sehr harrsch anhört, aber dein Code ist extrem schlimm. Besondern die Klasse VW musst du erstmal aufräumen.

Du lässt VW von Auto erben, daran ist erstmal nichts unbedingt verkehrt. Aber die Implementierung dazu passt nicht. Deine Implementierung ist eher eine VWGarage und hat mit einem Auto nichts gemein. Wenn du nur eine solche Garage haben willst, dann greif lieber zu einem Singleton anstatt diesen vielen static Methoden und Feldern. Insgesamt solltest du diese Models strikt trennen. Ein Auto ist halt keine Liste von Autos oder andersherum.

Die extra Klasse sortErstZulassung kannst du dir übrigens gekonnt sparen, schreibe einfach:

Collections.sort(list, Comparator.comparingInt(Auto::getErstzulassung));

Und ja, du fügst der Liste "list" ja nichts hinzu, sondern du fügst der "Garage" die Autos hinzu. Also ist "list" leer.

 

[ve987]

Hi MoxxiManagarm,

danke für deine Erklärung!

Könntest du bitte noch erklären, wie meinst du das "Ein Auto ist halt keine Liste von Autos oder andersherum."

Sorry für die blöde Frage, bin noch ganz am Anfang.

 

[MoxxiManagarm]

Damit meine ich, dass VW von Auto erbt, VW als solches aber nur den Zweck einer Listenführung bei dir darstellt. Das keyword "extends" kann quasi übersetzt werden als "VW ist ein Auto". Fachlich klingt das absolut korrekt, aber deine Implementierung dazu passt nicht. Die Implementierung bei dir ähnelt eher einer Garage, einer Ausstellungsfläche oder Ähnlichem.

 

[mihe7]

"VW ist ein Auto". Fachlich klingt das absolut korrekt

VW ist ein Automobilhersteller, aber kein Auto

 

[MoxxiManagarm]

VW ist ein Automobilhersteller, aber kein Auto

Ja auch! Aber die Leute sagen doch dauernd "Ich habe einen VW" wenn sie über ihr Auto reden ;-)

 

[mihe7]

Ich habe Leute auch schon hören sagen: "VW ist kein Auto, das ist ein Zustand."

 

[temi]

Du lässt VW von Auto erben, daran ist erstmal nichts unbedingt verkehrt. Aber die Implementierung dazu passt nicht. Deine Implementierung ist eher eine VWGarage und hat mit einem Auto nichts gemein.

Das hast du allerdings bereits hier gesagt bekommen und anscheinend gekonnt ignoriert

 

[temi]

Was ich gerade noch gesehen habe: Du hast sehr viele "static" in der Klasse "VW". Es sollte nur in begründeten Ausnahmefällen notwendig sein, "static" brauchen zu müssen. Einer davon ist "public static void main()". Versuche dein Programm so zu schreiben, dass dies das einzige "static" ist.

Dein Programm sollte dann ungefähr so aussehen:

public class Auto {
    // Code
}

public class Garage { // diese Klasse ist nur ein Sammler für Autos, sie erbt NICHT von Auto, weil sie kein Auto ist
    private List<Auto> autos = new ArrayList<>();
 
    // eine Beispielmethode zum Hinzufügen von Autos
    public void add(Auto auto) {
        autos.add(auto);
    }
 
    // mehr Code
}

public class Hauptprogramm {
    public static void main(String[] args) {
     
        Garage garage = new Garage(); // Instanz von Garage erstellen
     
        garage.add(new Auto(...)); // Hinzufügen eines Autos, Parameter von Auto() weggelassen
     
    }
}

ich möchte, dass die Methode VW.ausgabe(list); mir die Autos sortiert.

So etwas solltest du nicht wollen!

Der Name einer Methode sollte ausdrücken was die Methode tut. Von einer Methode ausgabe() erwarte ich nicht, dass sie etwas sortiert.

Nebenbei werden für Methodennamen i.d.R. Verben verwendet (eine Methode tut schließlich etwas), bei Methoden mit einem boolean-Rückgabewert sind auch Namen wie "isXXX()" oder "hasXXX()" gängige Bezeichner.

Besser wäre es demnach deine Klasse Garage hätte die Methoden sort() und print(). Da Englisch die bevorzugte Sprache beim Programmieren ist, ist es keine schlechte Idee, sich gleich daran zu gewöhnen. Irgendwie sehen sprachgemischten Programme immer seltsam aus.

 

[ve987]

Vielen Dank Euch alle für ausführliche Erklärung!

 

[ve987]

Das hast du allerdings bereits hier gesagt bekommen und anscheinend gekonnt ignoriert

Nicht absichtlich ignoriert, nur nicht ganz verstanden

 

[ve987]

Was ich gerade noch gesehen habe: Du hast sehr viele "static" in der Klasse "VW". Es sollte nur in begründeten Ausnahmefällen notwendig sein, "static" brauchen zu müssen. Einer davon ist "public static void main()". Versuche dein Programm so zu schreiben, dass dies das einzige "static" ist.

Dein Programm sollte dann ungefähr so aussehen:

public class Auto {
    // Code
}

public class Garage { // diese Klasse ist nur ein Sammler für Autos, sie erbt NICHT von Auto, weil sie kein Auto ist
    private List<Auto> autos = new ArrayList<>();

    // eine Beispielmethode zum Hinzufügen von Autos
    public void add(Auto auto) {
        autos.add(auto);
    }

    // mehr Code
}

public class Hauptprogramm {
    public static void main(String[] args) {
    
        Garage garage = new Garage(); // Instanz von Garage erstellen
    
        garage.add(new Auto(...)); // Hinzufügen eines Autos, Parameter von Auto() weggelassen
    
    }
}

So etwas solltest du nicht wollen!

Der Name einer Methode sollte ausdrücken was die Methode tut. Von einer Methode ausgabe() erwarte ich nicht, dass sie etwas sortiert.

Nebenbei werden für Methodennamen i.d.R. Verben verwendet (eine Methode tut schließlich etwas), bei Methoden mit einem boolean-Rückgabewert sind auch Namen wie "isXXX()" oder "hasXXX()" gängige Bezeichner.

Besser wäre es demnach deine Klasse Garage hätte die Methoden sort() und print(). Da Englisch die bevorzugte Sprache beim Programmieren ist, ist es keine schlechte Idee, sich gleich daran zu gewöhnen. Irgendwie sehen sprachgemischten Programme immer seltsam aus.

Das ist das, wo es gehängt hat: Eine Garage Klasse!
Die Benennung werde ich umsetzen, danke!

 

[temi]

Das ist das, wo es gehängt hat: Eine Garage Klasse!
Die Benennung werde ich umsetzen, danke!

Außer es handelt sich um einen "Parkplatz", ein "Autohaus" oder um etwas abstrakteres wie eine "AutoListe".

Benenne deine Klasse so, dass der Name ausdrückt, was die Klasse darstellen soll.

 

[ve987]

Außer es handelt sich um einen "Parkplatz", ein "Autohaus" oder um etwas abstrakteres wie eine "AutoListe".

Benenne deine Klasse so, dass der Name ausdrückt, was die Klasse darstellen soll.

Temi, worin würde sich "Parkplatz" / "Autohaus" von einer "Garage" Klasse, wo ich meine Array List rein mache, unterscheiden? Was könnte man dort reinpacken? Sorry für die blöde Frage, ich möchte das halt gerne richtig verstehen

 

[temi]

Temi, worin würde sich "Parkplatz" / "Autohaus" von einer "Garage" Klasse, wo ich meine Array List rein mache, unterscheiden? Was könnte man dort reinpacken? Sorry für die blöde Frage, ich möchte das halt gerne richtig verstehen

Im Prinzip gibt es vermutlich keinen Unterschied, aber nehmen wir an, du möchtest ein Autohaus modellieren, dann wäre es ja irgendwie nicht intuitiv, wenn die Klasse "Garage" heißt. Die Begrifflichkeiten sollten einfach fachlich zu dem passen, was man machen möchte. Würdest du eine Software zur Verwaltung von Gebrauchtwagen schreiben, dann wirst du ggf. ebenfalls eine "Auto" Klasse haben, aber die "Sammlerklasse" für die einzelnen Autos kaum "Garage" nennen wollen. Das würde in dem Zusammenhang einfach nicht richtig passen.

 

[MoxxiManagarm]

Sorry für die blöde Frage, ich möchte das halt gerne richtig verstehen

Inhaltlich würden sich diese nicht unterscheiden, es is eine fachliche Frage. Java ist eine Objektorientierte Sprache und du versuchst Objekte als solches zu modellieren. Es geht letztlich darum in welcher Domaine du dich in der Aufgabenstellung befindest. Wenn sich deine Aufgabe z.B. um eine Ausstellungsfläche handelt, dann nennst du die Klasse halt Ausstellungsfläche. Vielleicht ist es ja auch ein Gebrauchtwagenhandel, ein Laderaum einer Fähre, eine Maut-Station? Das wissen wir halt nicht, das weißt nur du.

 

[ve987]

Ihr habt natürlich Recht. Man muss am Anfang richtig planen. Das was ich gemacht hatte, war nur zu Übungszwecken gedacht. Ein erstes kleines Anfänger Projekt Es ist aber schon sehr wichtig, als Anfänger sich gleich an die Regeln halten. Danke euch alle!