Klassen Auto Klasse

[edebro]

Hallo, ich übe zurzeit etwas für mein Informatik Modul und bin mir sehr unsicher ob meine Lösung wirklich so akzeptiert werden kann.

Die Aufgabe ist es eine einfache Auto Klasse zu erstellen die :
1. Erzeugung einer Auto-Instanz unter Angabe des Typs (Typ unveränderbar nach Erzeugung)
2. Erzeugung einer Auto-Instanz unter Angabe des Typs und Preises (Typ unveränderbar)
3. Setzen den Preises einer Instanz (nur von Auto-Methoden aus)
4. Abfragen des Preises einer Instanz
5. Anzahl der Objekte und die Summe der Preise

Meine Probleme waren :
1. Ich kann nicht nur durch einen Typ eine Instanz erstellen, da er einen Preis erwartet.
2. Ich setze zwar den Preis aber jedoch kommt mir meine Lösung extrem falsch vor

public class Auto{
    public final String typ;
    private double preis;
    public static int anzahl = 0;
    public static double gesamt = 0;
   
    public Auto(String t, double p){
        typ = t;
        preis = p;
        Auto.anzahl++;
        Auto.gesamt += preis;
    }
   
    public double getPreis(){
        return preis;
    }
       
    public double setPreis(){
        preis = StdIn.readDouble();
        return preis;
    }
   
    public static void main(String[]args){
        Auto a = new Auto ("Mercedes",50000);  // Erzeugung eines Auto Objekts
        Auto b = new Auto ("Lupo",5000);
        System.out.println(a.getPreis());   // Abfragen des Preises
        a.preis = a.setPreis();             // Setzen des Preises einer Instanz
        System.out.println(anzahl);
        System.out.println(gesamt);
    }
}

Wäre sehr dankbar für jede Hilfe!

 

[httpdigest]

public class Auto {
  private final static java.util.WeakHashMap<Auto, Void> autos = new java.util.WeakHashMap<>();
  private final String typ;
  private double preis;
  public Auto(String typ) {
    this(typ, 0.0);
  }
  public Auto(String typ, double preis) {
    this.typ = typ;
    this.preis = preis;
    autos.put(this, null);
  }
  public double getPreis() {
    return preis;
  }
  public void setPreis(double preis) {
    this.preis = preis;
  }
  public String getTyp() {
    return typ;
  }
  public static int anzahlAllerAutos() {
    return autos.size();
  }
  public static double preisAllerAutos() {
    return autos.keySet().stream().mapToDouble(Auto::getPreis).sum();
  }
  public static void main(String[] args) {
    Auto a1 = new Auto("a1", 3.0);
    Auto a2 = new Auto("a2", 1.0);
    Auto a3 = new Auto("a3");
    a3.setPreis(5.0);
    System.out.println("Preis aller Autos: " + preisAllerAutos());
    System.out.println("Anzahl aller Autos: " + anzahlAllerAutos());
  }
}

 

[Robat]

@httpdigest Glaubst du nicht, dass der Code für Anfänger etwas zu hoch ist?

 

[httpdigest]

Naja, offensichtlich ist die Lösung mit "ich speichere den Gesamtpreis in einer static double Variable und aktualisiere diese nur beim Anlegen eines Autos mit gegebenem Preis" falsch, da eine Aktualisierung des Preises nicht mehr berücksichtigt wird.
Um das aber abzubilden, benötigt man ja wenigstens ein Array oder eine Liste.
Das heißt, ich musste davon ausgehen, dass er/sie Listen/Arrays bereits kennt, und dann ist eine WeakHashMap ja auch nicht mehr so weit von entfernt.
Jaaa, okay, das mit dem Garbage Collector, dass keine Instanzen mitgezählt werden, die bereits collected sind, kommt da noch dazu.

 

[httpdigest]

Okay, es geht ja doch ohne eine Liste/Array.

public class Auto {
  private static double gesamtPreis;
  private static int gesamtAnzahl; 
  private final String typ;
  private double preis;
  public Auto(String typ) {
    this(typ, 0.0);
  }
  public Auto(String typ, double preis) {
    this.typ = typ;
    this.preis = preis;
    gesamtPreis += preis;
    gesamtAnzahl++;
  }
  public double getPreis() {
    return preis;
  }
  public void setPreis(double preis) {
    gesamtPreis = gesamtPreis - this.preis + preis;
    this.preis = preis;
  }
  public String getTyp() {
    return typ;
  }
  public static int anzahlAllerAutos() {
    return gesamtAnzahl;
  }
  public static double preisAllerAutos() {
    return gesamtPreis;
  }
  public static void main(String[] args) {
    Auto a1 = new Auto("a1", 3.0);
    Auto a2 = new Auto("a2", 1.0);
    Auto a3 = new Auto("a3");
    a3.setPreis(5.0);
    System.out.println("Preis aller Autos: " + preisAllerAutos());
    System.out.println("Anzahl aller Autos: " + anzahlAllerAutos());
  }
}

 

[edebro]

Vielen Dank für die Hilfe! Das Thema ist noch komplett neu und mir haben die Ansätze gefehlt.

 

[Robat]

@httpdigest das wäre auch meine Idee gewesen