ArrayList

[Dimbel]

Hi, ich hab ne Frage bzgl. ArrayList.

Ich soll für ne Übung ein simples Warenkorb-System erstellen (mit BlueJ).
Es soll 2 Klassen geben, Warenkorb und Artikel.

Artikel:
3 Variablen(Artikelbezeichnung, Artikelnummer und Preis)
2 Methoden(Preis des Artikels ausgeben, alle 3 Variablen in einem toString ausgeben)

Warenkorb:
2 Variablen(Auftragsname und Auftragsnummer)
1 ArrayList(Positionen)
3 Methoden(Artikel zu Positionen hinzufügen, Gesamtsumme des Warenkorbs berechnen und eine Warenkorbausgabe durch die toString-Methode der Artikel-Klasse)

Die Klasse Artikel habe ich fertig, die ist nicht das Problem.
Die einzelnen Artikel sind jeweils eine Instanz der Artikel-Klasse und diese soll ich dann in der ArrayList Positionen speichern.

Das Hinzufügen von Artikeln zum Warenkorb habe ich so gelöst:

public void addArtikel(Artikel x)
    {
        positionen.add(x);
    }

Meine Frage ist, wie kann ich auf die Elemente im ArrayList zugreifen, da es sich ja um eine Instanz von der Artikel-Klasse handelt und wie kann ich auf die einzelnen Variablen zugreifen?

 

[njans]

public Artikel getArtikel(index i)
    {
        return positionen.get(i);
    }

So holst du die Artikel.

 

[Dimbel]

public Artikel getArtikel(index i)
    {
        return positionen.get(i);
    }

So holst du die Artikel.

Dann habe ich den kompletten Artikel oder? Wie könnte ich nur auf dein Preis des Artikels im Warenkorb zugreifen?

 

[Marco13]

Und falls das nicht geht, weil die Liste als
private ArrayList positionen = new ArrayList();
definiert ist: Ändern in
private List<Artikel> positionen = new ArrayList<Artikel>();

Um auf einzelne Elemente zuzugreifen sowas wie

Artikel artikel = position.get(i); 
// Oder von außen: 
// Artikel artikel = warenkorb.getArtikel(i); 
int preis = artikel.getPreis(); // oder so..

 

[pro2]

private List<Artikel> positionen = new ArrayList<Artikel>();

Die vollständige Schreibweise ist doch gar nicht mehr nötig, oder?

private List<Artikel> positionen = new ArrayList();

Dürfte reichen.

 

[nillehammer]

Die vollständige Schreibweise ist doch gar nicht mehr nötig, oder?

private List<Artikel> positionen = new ArrayList();

Dürfte reichen.

Nope, es fehlt der Diamond-Operator:

private List<Artikel> positionen = new ArrayList<>();

Den gibt's aber erst seit 1.7 Marco's Schreibweise ist also sicherer, für den Fall, dass der TO evtl. mit 1.6 oder gar 1.5 arbeitet.

 

[C_A]

Und wenn Du schon mit BlueJ lernst (was ich sehr sinnvoll finde), kannst Du ein Objekt auch auf der Objekt-Bar ablegen und mit rechtem Mausklick "inspizieren". Das kann Dir, wenn Du Probleme mit OO hast, den Einstieg erleichtern.

 

[Dimbel]

Hey, tut mir Leid das ich erst jetzt antworte.

Mit dieser Methode habe ich einen Artikel in den Warenkorb (in die ArrayList positionen) gelegt:

public void addArtikel(Artikel x)
    {
        positionen.add(x);
    }

Ist das soweit richtig?

Mit dieser Methode möchte ich die Gesamtsumme aller Artikel berechnen:

public double summe()
    {
        double gesamtsumme = 0;
        int i = 0;
        
        while(i < positionen.size()){
            Artikel artikel = positionen.get(i);
            gesamtsumme = gesamtsumme + artikel.getpreis(i);
        }
        
        return gesamtsumme;
    }

Irgendwas ist an dieser Methode falsch, ich hoffe ihr könnt mir helfen

 

[njans]

public double summe()
    {
        double gesamtsumme = 0;
        int i = 0; // i bleibt immer 0 
        
        while(i < positionen.size()){
            Artikel artikel = positionen.get(i);
            gesamtsumme = gesamtsumme + artikel.getpreis(i); // getPreis(i) ? Wieso den i-ten preis? Was ist der i-te Preis eines Artikels? 
        }
        
        return gesamtsumme;
    }

 

[Dimbel]

Ich dachte so kommt man an die einzelnen Variablen der Artikel-Klasse.
Wenn ich einen ganzen Artikel (mit Name, Nummer und Preis) in den Warenkorb bzw das Array lege, bekomme ich durch artikel.get(i) doch auch wieder den kompletten Artikel als Ausgabe oder?

 

[njans]

Du hast bei dir die Artikel Klasse liegen.
Hat die Methode getpreis überhaupt einen Parameter int ?
Mich würde es wundern, da ein Artikel ja nicht verschiedene Preise hat, sondern nur einen.
Generell hat ein Shop mehrere Artikel (deswegen auch while-Schleife) und jeder Artikel hat genau einen Preis.

Es wäre vielleicht auch mal gut die Klasse Artikel hier mal zu posten.

 

[Dimbel]

Ich glaube ich hab da was komplett falsch verstanden.
Ich dachte wenn ich eine Instanz der Klasse Artikel in das ArrayList in der Klasse Warenkorb lege, kann ich einfach so auf die verschiedenen Variablen zugreifen.
Das geht wohl nicht? Ich muss also in der Klasse Artikel für jede Variable ne Ausgabe-Methode schreiben?

In der Artikel-Klasse sind einfach nur die 3 Variablen definiert:

public class Artikel
{
    // instance variables - replace the example below with your own
    private double preis;
    private String bezeichnung;
    private int artikelnr;

    // konstruktor
    public Artikel(double kosten, String artikelbez, int nr)
    {
       preis = kosten;
       bezeichnung = artikelbez;
       artikelnr = nr;
    }
}

 

[njans]

    private double preis;

Der modifier private bedeutet, dass diese Variable nur innerhalb dieses Objektes gesehen werden kann.
Du kannst also innerhalb deines Objektes mit dieser Variable arbeiten:

public class Artikel
{
    // instance variables - replace the example below with your own
    private double preis;
    private String bezeichnung;
    private int artikelnr;
 
    //...

    public void machIrgendwas()
    {
       int irgendwas = preis * 10 + Math.pow(2,10);
       //....
    }
}

Aber von außen kannst du nicht darauf zugreifen:

Artikel artikel = new Artikel(2.1,"foo",1);
artikel.preis = 10.0; //Fehler!

Was du anstelle davon hast (was ebenfalls Gang und Gebe ist) sind getter/setter Methoden.
Diese dienen der Abkapselung und der Kontrolle des Zugriffs:

public class Artikel
{
    // instance variables - replace the example below with your own
    private double preis;
    private String bezeichnung;
    private int artikelnr;
 
    // konstruktor
    public Artikel(double kosten, String artikelbez, int nr)
    {
       preis = kosten;
       bezeichnung = artikelbez;
       artikelnr = nr;
    }

    public double getPreis()
    {
      return preis;
    }

    public void setPreis(double neuerPreis)
    {
      preis = neuerPreis;
    }

     //... usw für die anderen Variablen
}

Damit kannst du dann an dem Objekt artikel (aus dem Beispiel zuvor) folgendes machen:

Artikel artikel = new Artikel(2.1,"foo",1);
artikel.setPreis(10.0); //Korrekt!

Hier bestimmst du aber, was passiert, wenn du setPreis und getPreis aufrufst. So kannst du intern jedes mal, wenn sich der Preis verändert, den alten Preis speichern (nur als Beispiel gedacht).
Soviel zu den Basics über Zugriffe von variablen von Objekten.

Du willst in deinem Beispiel schon auf deine Variablen zugreifen, aber bei getPreis macht der index i wenig Sinn. Da, wie gesagt, ein Artikel ja nur einen Preis hat.

public double summe()
    {
        double gesamtsumme = 0;
        int i = 0; // i bleibt immer 0 
        
        while(i < positionen.size()){
            Artikel artikel = positionen.get(i); // Du holst dir den Artikel
            gesamtsumme = gesamtsumme + artikel.getPreis(); //  Und addierst dann einfach den Preis des Artikels auf die Summe
            // hier musst du noch dafür sorgen, dass i nicht immer nur 0 bleibt
        }
        return gesamtsumme;
    }

 

[Dimbel]

Jetzt hab ichs verstanden, vielen dank

 

[godi]

Hallo

Wenn du die GesamtSumme deines Warenkorbes berechnen willst und du deine Artikel in der ArrayList gespeichert hast dann verwendest du keine while schleife sondern eine foreach.
Mit dieser kannst du über alle Artikel in deiner ArrayList iterieren.

int sum = 0;
for(Artikel artikel : positionen) { //Positionen? deine ArrayList mit den Artikeln drinn
     sum += artikel.getPreis();
}
// Jetzt hast du in sum den Gesamtpreis deines Warenkorbes.

godi