Array wird nicht an Methode übergeben

Bluebird209

Neues Mitglied
Hallo zusammen,

sorry falls die Frage für euch dämlich sein sollte, ich studiere Wirtschaftsinformatik im Fernstudium und habe daher keine Kommilitonen an die ich mich wenden könnte und aus den Büchern werde ich bei dem Problem nicht schlau. Für die aktuelle Hausarbeit soll ich die Klasse Warenkorb und die Klasse Artikel erstellen. Der Warenkorb soll gefüllt werden können, sowie die Preisberechnung durchführen etc.

Ich möchte die Bruttosumme des gesamten Warenkorbs berechnen. Die Netto-Summe ist in der Klasse Artikel gespeichert, in der Klasse Warenkorb gibt es die Methode getWarenkorbBruttoSumme() welche die Methode getBruttopreis() aufruft. In dem Array warenkorb[] vom Typ Artikel werden die einzukaufenden Artikel gespeichert.

Das Problem ist jetzt, dass beim Aufruf der Methode getBruttopreis() das Array warenkorb[] irgendwie verloren geht, d.h. dass innerhalb der Methode nur ein leeres Array vorhanden ist, obwohl in der Methode getWarenkorbBruttoSumme() das Array noch gefüllt ist. Ich scheine wohl beim Aufruf etwas falsch zu machen. Hier die beiden Methoden:
Java:
    public double getBruttopreis(int i)
    {
        double bruttopreis = 0;
        {
            bruttopreis += warenkorb[i].getNettopreis()*((this.warenkorb[i].getSteuersatz()+1));
        }
        return bruttopreis;
    }
   
    public double getWarenkorbBruttoSumme()
    {
        int i = 0;
        double summe = 0;
        while (warenkorb[i] != null)
        {
            summe += warenkorb[i].getBruttopreis(i);
            i++;
        }
        return summe;

Für den Gesamtzusammenhang stelle ich noch den kompletten Code hier rein.
Java:
public class Artikel extends Warenkorb
{

    private String Artikelnummer;
    private String Artikelbeschreibung;
    private double Nettopreis;
    private double Steuersatz;
   
    public Artikel (String n, String b, double p, double s)
    {
        Artikelnummer = n;
        Artikelbeschreibung = b;
        Nettopreis = p;
        Steuersatz = s; //Eingabe z.B. 0,19
    }
   
    public void Artikelausgeben()
    {
        System.out.println ("[ArtNr.] " + Artikelnummer + "; " + Artikelbeschreibung + "; Preis: " + Nettopreis + "; Steursatz: " + Steuersatz);
    }
   
    public void setArtikelnummer(String Artikelnummer)
    {
        this.Artikelnummer = Artikelnummer;
    }
   
    public void setArtikelbeschreibung(String Artikelbeschreibung)
    {
        this.Artikelbeschreibung = Artikelbeschreibung;
    }
   
    public void setNettopreis(double Nettopreis)
    {
        this.Nettopreis = Nettopreis;
    }
   
    public void setSteuersatz(double Steuersatz)
    {
        this.Steuersatz = Steuersatz;
    }
   
    public String getArtikelnummer()
    {
        return Artikelnummer;
    }
   
    public String getArtikelbeschreibung()
    {
        return Artikelbeschreibung;
    }
   
    public double getNettopreis()
    {
        return Nettopreis;
    }
   
    public double getSteuersatz()
    {
        return Steuersatz;
    }
}
Java:
public class Warenkorb
{
    private int AnzahlArtikelinWarenkorb = 0;
    public Artikel warenkorb[] = new Artikel[100];
   
    public static void main(String[] args)
    {
        Artikel a1 = new Artikel ("12345", "Musik-CD", 14.95, 0.19);
        Artikel a2 = new Artikel ("12346", "Video-DVD", 19.99, 0.19);
       
        a1.Artikelausgeben();
        a2.Artikelausgeben();
       
        Warenkorb w1 = new Warenkorb();
        Warenkorb w2 = new Warenkorb();
       
        w1.setArtikelinWarenkorb(a1, 0);
        w1.setArtikelinWarenkorb(a2, 1);
        System.out.println(w1.getWarenkorbBruttoSumme());
        System.out.println(w1.getWarenkorbNettoSumme());
        System.out.println(w1.getAnzahlArtikelinWarenkorb());
       
    }
   
   
    public void setArtikelinWarenkorb(Artikel a, int i)
    {
        warenkorb[i] = a;
        AnzahlArtikelinWarenkorb++;
    }
   
    private int findArtikel(String Artikelnummer)
    {
        int i = 0;
        while (warenkorb[i] != null)
        {
            if (Artikelnummer == warenkorb[i].getArtikelnummer())
                {
                return i;
                }
                else
                {
                    i++;
                }
        }
       
        return i=200; //Unerreichbaren Wert setzen, damit klar ist das kein Wert gefunden wurde
    }
   
    public int getAnzahlArtikelinWarenkorb()
    {
        return AnzahlArtikelinWarenkorb;
    }
   
    public double getBruttopreis(int i)
    {
        double bruttopreis = 0;
        {
            bruttopreis += warenkorb[i].getNettopreis()*((this.warenkorb[i].getSteuersatz()+1));
        }
        return bruttopreis;
    }
   
    public double getWarenkorbBruttoSumme()
    {
        int i = 0;
        double summe = 0;
        while (warenkorb[i] != null)
        {
            summe += warenkorb[i].getBruttopreis(i);
            i++;
        }
        return summe;
    }
   
    public double getWarenkorbNettoSumme()
    {
        int i = 0;
        double summe = 0;
        while (warenkorb[i] != null)
        {
            summe += warenkorb[i].getNettopreis();
            i++;
        }
        return summe;
    }
}

Ich hoffe jemand nimmt sich die Zeit mir weiter zu helfen. Vielen Dank schon mal dafür!
 

T_T

Bekanntes Mitglied
Ich würde prinzipiell eine ArrayList<Artikel>() verwenden anstatt eines vordefinierten Arrays. Dies lässt sich nämlich dynamisch erweitern.

Java:
private List<Artikel> warenkorb = new ArrayList<Artikel>();
...
...
...
warenkorb.add(a1);
warenkorb.add(a2);

Damit fällt schonmal deine eigene Methode "setArtikelinWarenkorb()" weg. Die beiden Warenkörbe w1 und w2 kannst du damit auch getrost weg schmeißen.
Den Rest musst du natürlich an die Liste anpassen, da du eigentlich auf ein Array zugreifst.
 

Flown

Administrator
Mitarbeiter
Ich hab für dich einmal ein Refactoring gestartet:
- Englische Bezeichner verwenden
- Klassen nach Verantwortlichkeit aufteilen
- Vererbung einsetzen, wenn man gemeinsamen State hat (also ist hier Komposition die richtige Entscheidung)
Java:
import java.util.ArrayList;
import java.util.List;
import java.util.NoSuchElementException;
import java.util.Objects;

public class Test {
  public static void main(String... args) {
    Article a1 = new Article("12345", "Musik-CD", 14.95, 0.19);
    Article a2 = new Article("12346", "Video-DVD", 19.99, 0.19);

    System.out.println(a1);
    System.out.println(a2);

    ShoppingCart w1 = new ShoppingCart();

    w1.addArticle(a1);
    w1.addArticle(a2);
    System.out.format("Brutto: %.2f%n", w1.getGrossSum());
    System.out.format("Netto: %.2f%n", w1.getNetSum());
    System.out.println(w1.size());
    System.out.println(w1.findArticle("12346"));
  }
}

class ShoppingCart {
  private List<Article> articles = new ArrayList<>();

  public void addArticle(Article a) {
    articles.add(a);
  }

  public int findArticle(String articleNumber) {
    for(int i = 0; i < articles.size(); i++) {
      if(articleNumber.equals(articles.get(i).getNumber())) {
        return i;
      }
    }
    throw new NoSuchElementException(String.format("No article with name(\"%s\") found.", articleNumber));
  }

  public int size() {
    return articles.size();
  }

  public double getGrossPrice(Article a) {
    double net = a.getNet();
    return net + net * a.getTaxRate();
  }

  public double getGrossSum() {
    double sum = 0;
    for (Article article : articles) {
      sum += getGrossPrice(article);
    }
    return sum;
  }

  public double getNetSum() {
    double sum = 0;
    for (Article article : articles) {
      sum += article.getNet();
    }
    return sum;
  }
}

class Article {

  private String number;
  private String description;
  private double net;
  private double taxRate;

  public Article(String articleNumber, String description, double net, double taxRate) {
    this.number = articleNumber;
    this.description = description;
    this.net = net;
    this.taxRate = taxRate;
  }

  public String getNumber() {
    return number;
  }

  public void setNumber(String number) {
    this.number = number;
  }

  public String getDescription() {
    return description;
  }

  public void setDescription(String description) {
    this.description = description;
  }

  public double getNet() {
    return net;
  }

  public void setNet(double net) {
    this.net = net;
  }

  public double getTaxRate() {
    return taxRate;
  }

  public void setTaxRate(double taxRate) {
    this.taxRate = taxRate;
  }

  @Override
  public boolean equals(Object o) {
    if (this == o) return true;
    if (!(o instanceof Article)) return false;
    Article article = (Article) o;
    return Double.compare(article.net, net) == 0 &&
      Double.compare(article.taxRate, taxRate) == 0 &&
      Objects.equals(number, article.number) &&
      Objects.equals(description, article.description);
  }

  @Override
  public int hashCode() {
    return Objects.hash(number, description, net, taxRate);
  }

  @Override
  public String toString() {
    return "Article{" +
      "number='" + number + '\'' +
      ", description='" + description + '\'' +
      ", net=" + net +
      ", taxRate=" + taxRate +
      '}';
  }
}

EDIT: findArticle sollte eigentlich so aussehen, da ja der Index geliefert wird und nicht ein konkreter Article:
Java:
  public int findArticle(String articleNumber) {
    for(int i = 0; i < articles.size(); i++) {
      if(articleNumber.equals(articles.get(i).getNumber())) {
        return i;
      }
    }
    return -1;
  }
 
Zuletzt bearbeitet:

Harry Kane

Top Contributor
Um mal etwas zur eigentlichen Fehlerursache zu sagen:
Die Klasse Artikel erweitert Warenkorb, ist damit auch ein Warenkorb, und hat alle seine Variablen. Diese Vererbungshierarchie ist nicht sinnvoll. Ein Warenkorb hat Artikel, und ein Artikel ist kein Warenkorb. Im vorliegenden Fall führt das zu folgendem Verhalten:
Beim Aufruf von Warenkorb.getWarenkorbBruttoSumme wird über die Artikel im Warenkorb iteriert und für jeden aktuellen Artikel (warenkorb[x]) die Methode getBruttoPreis(int i) aufgerufen. Diese methode ist nicht in der Artikel-Klasse deklariert, aber in der Warenkorb-Klasse, und kann, da Artikel unsinnigerweise Warenkorb erweitert, auch auf Artikel-Instanzen aufgerufen werden. Diese Methode versucht, einen Wert aus dem Warenkorb des aktuellen Artikels zu holen. Da der Warenkorb eines Artikels aber nie mit Waren gefüllt wird, und somit alle Einträge im Array Artikel warenkorb[] null sind, müsste an der Stelle eine NullPointerException fliegen.
Lösung:
Java:
summe += warenkorb[x].getBruttopreis(x);
ersetzen durch
Java:
summe += warenkorb[x].getNettopreis()*(warenkorb[x].getSteuersatz() + 1);
 

JStein52

Top Contributor
Hier:
Code:
  public int findArticle(String articleNumber) {
    for(int i = 0; i < articles.size(); i++) {
      if(articleNumber.equals(articles.get(i).getNumber())) {
        return i;
      }
    }
    throw new NoSuchElementException(String.format("No article with name(\"%s\") found.", articleNumber));
  }
würde ich allerdings keine Exception verwenden um anzuzeigen dass dieser Artikel nicht gefunden wurde. Exceptions sollten Ausnahmen behandeln, aber dass ein bestimmter Artikel nicht im Warenkorb ist ist ja keine Ausnahme sondern kann durchaus vorkommen. Ich würde also z.B. in diesem Fall -1 zurückliefern.
 

Bluebird209

Neues Mitglied
Vielen Dank euch allen, eure Antworten haben mir alle sehr weitergeholfen, vielen Dank dafür!

Eine Frage hätte ich noch @Flown: Ich verstehe leider nicht was die Overrides genau machen, bzw. wann diese ausgeführt werden. Könntest du mir das kurz noch erklären?
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
I Methoden Wieso wird mein Array "a" verändert und meine Variable "a" nicht? Java Basics - Anfänger-Themen 4
I Programm erkennt nicht an das Array zurückgegeben wird trotz Initialisierung *einfach* Java Basics - Anfänger-Themen 9
H Array wird als Rückgabewert nicht akzeptiert? Java Basics - Anfänger-Themen 2
NeoLexx Variable für Array wird nicht korrekt übergeben Java Basics - Anfänger-Themen 45
E Methoden 2D Array wird in der Methode nicht geändert/gespeichert Java Basics - Anfänger-Themen 5
S Array wird nicht übernommen NullPointerException Java Basics - Anfänger-Themen 6
C Array Aktualisierung wird nicht im Tooltip angezeigt Java Basics - Anfänger-Themen 2
A Array wird nicht durchlaufen Java Basics - Anfänger-Themen 10
M Array wird beim Neuaufruf nicht neu initialisiert Java Basics - Anfänger-Themen 9
A Ein Array aus einer Klasse wird nicht sauber abgefüllt Java Basics - Anfänger-Themen 4
I Array und variable gleicher wert, wird jedoch nicht erkannt Java Basics - Anfänger-Themen 4
B Array von Choice wird nicht von getSelectedItem erkannt Java Basics - Anfänger-Themen 3
sserio Zweidimensionales Array [][] wird untereinander ausgegeben Java Basics - Anfänger-Themen 14
A char array wird überschrieben Java Basics - Anfänger-Themen 6
L Iterieren durch eine ArrayList. Integer Array wird übergeben Java Basics - Anfänger-Themen 17
P Array wird automatisch überschrieben Java Basics - Anfänger-Themen 3
B Gefülltes Array wird mit nullwerten übergeben Java Basics - Anfänger-Themen 9
E Wie wird in Java ein Char-Array terminiert ? Java Basics - Anfänger-Themen 15
L [Verständnisproblem] Array wird trotz void rückgabe verändert. Java Basics - Anfänger-Themen 5
D Array wird immer mir 100 für jeden Wert initialisiert? Java Basics - Anfänger-Themen 2
Silvascus Array, der mit dem Namen, der per Methode übergeben wird, erstellt wird Java Basics - Anfänger-Themen 5
C Array wird falsch befüllt? Java Basics - Anfänger-Themen 7
T Array wird überschrieben Java Basics - Anfänger-Themen 4
J Warum wird Array durch Methode ohne Rückgabewert verändert? Java Basics - Anfänger-Themen 5
L Schleifen und Array, nur letzte Eingabe wird ausgegeben Java Basics - Anfänger-Themen 3
D Array Objekt wird überschrieben Java Basics - Anfänger-Themen 3
L Wie wird ein 2 Dimensionales Array befüllt? Java Basics - Anfänger-Themen 6
H Array wird verändert ohne Rückgabe? Java Basics - Anfänger-Themen 7
P Datentypen String auf JTextField wird zu Char-Array und anschließend zu int Java Basics - Anfänger-Themen 12
F Variablen String-Array wird ungewollt "überschreiben" Java Basics - Anfänger-Themen 7
D Erste Schritte Warum wird bei einem Array die Länge über Length ausgegeben? Java Basics - Anfänger-Themen 6
A OOP Array wird ohne grund sortiert?! Java Basics - Anfänger-Themen 4
B Array-objekte,. Es wird ein Fehler ausgegeben Java Basics - Anfänger-Themen 11
A Array wird überschrieben Java Basics - Anfänger-Themen 5
T Array verkleinern Java Basics - Anfänger-Themen 2
J Array aus Numberfield Eingaben Java Basics - Anfänger-Themen 7
D Array List mit Objekten sortieren Java Basics - Anfänger-Themen 2
onlyxlia Anzahl Random Zahlen mit Scanner abfragen und in Array speichern Java Basics - Anfänger-Themen 10
Ü Java Array - Buchstaben als Zahlen ausgeben Java Basics - Anfänger-Themen 22
Ü Zweidimensionales Array in der ersten Zeile deklarieren Java Basics - Anfänger-Themen 13
Thomas Uppe 2D Array Reihenfolge vermischen Java Basics - Anfänger-Themen 4
T array auslesen Java Basics - Anfänger-Themen 2
Nitrogames Variablen Variable aus JOptionPane Abfrage in Array einfügen Java Basics - Anfänger-Themen 4
moini Auf Array aus Superklasse zugreifen? Java Basics - Anfänger-Themen 2
J ArrayList in 2D-Array konvertieren. Java Basics - Anfänger-Themen 48
M NullPointerException: Cannot read the array length because "this.Kinder" is null Java Basics - Anfänger-Themen 1
P Wieso kann ich als Index für einen Array einen Char angeben? Java Basics - Anfänger-Themen 3
Finn_lol Fehlermeldung bei Schleife mit Array Java Basics - Anfänger-Themen 4
Proxy Chars vor array übergabe toLowerUpcase Java Basics - Anfänger-Themen 14
iAmFaiinez Primzahlen Tester ohne Array Java Basics - Anfänger-Themen 4
S array 2 dimensional treppe Java Basics - Anfänger-Themen 3
S Array 2x2 Blöcke mit 0 und 1 Java Basics - Anfänger-Themen 10
C Array von Klassen Java Basics - Anfänger-Themen 2
julian0507 2Dim-Array Spaltensummen Java Basics - Anfänger-Themen 1
XWing Doppelte Zahlen im Array Java Basics - Anfänger-Themen 8
melisax Java 2D-Array Tabelle Java Basics - Anfänger-Themen 4
melisax Java Array Wert an bestimmtem Index angeben Java Basics - Anfänger-Themen 14
W Items löschen aus String Array vom Custom Base Adapter Java Basics - Anfänger-Themen 2
Proxy Stack erweitern mit neuem Array falls der alte voll ist!? Java Basics - Anfänger-Themen 5
E Array, nächste Zahl zur 5 ausgeben, wie? Java Basics - Anfänger-Themen 42
J Array.list vergleichen Java Basics - Anfänger-Themen 1
W Java-Code mit Array Java Basics - Anfänger-Themen 14
D Reflections & Generisches Array Java Basics - Anfänger-Themen 4
T Array Java Basics - Anfänger-Themen 2
T Array Java Basics - Anfänger-Themen 15
T Wörteranzahl im Array zählen Java Basics - Anfänger-Themen 9
Ostkreuz Zweidimensionaler Array Index Java Basics - Anfänger-Themen 2
S String Array Buchstaben um einen gewissen Wert verschieben Java Basics - Anfänger-Themen 4
R Images aus einem Array ausgeben Java Basics - Anfänger-Themen 3
R 2d Array individuell machen Java Basics - Anfänger-Themen 4
D 2D Char Array into String Java Basics - Anfänger-Themen 2
J Array Median bestimmen Java Basics - Anfänger-Themen 6
S Array Maximum bestimmen mit for und foreach Java Basics - Anfänger-Themen 7
S Prüfen ob ein zweidimensionales Array rechteckig ist Java Basics - Anfänger-Themen 4
N Array Java Basics - Anfänger-Themen 1
J Array Mittleren Wert bestimmen Java Basics - Anfänger-Themen 2
D OOP Array einem Objekt zuweisen Java Basics - Anfänger-Themen 2
O Zahlen aus einem char-array per char + Zeichen addieren Java Basics - Anfänger-Themen 2
S leeres Array statt Null Pointer Exception ausgeben Java Basics - Anfänger-Themen 20
S Inhalte aus Array vergleichen und Max ausgeben Java Basics - Anfänger-Themen 3
M 2d array ohne längen anlegen Java Basics - Anfänger-Themen 4
S Bestimmte werte aus einem Array löschen Java Basics - Anfänger-Themen 2
S Ausgeben wie oft ein Wert in einem Array vorkommt Java Basics - Anfänger-Themen 7
E Reihenfolge der Werte umdrehen (mittels statischem int-Array Java Basics - Anfänger-Themen 3
O 2 Dimensionales Array Java Basics - Anfänger-Themen 6
M Bubble Sort - Int[] Array sortieren Java Basics - Anfänger-Themen 2
javaBoon86 Array mehrere Dimensionen Java Basics - Anfänger-Themen 10
B Array nach Elementwerten sortieren? Java Basics - Anfänger-Themen 1
B Explizit Array definieren geht nicht? Java Basics - Anfänger-Themen 14
D Kleinste Zahl in Array finden die vorher noch errechnet werden müssen. Java Basics - Anfänger-Themen 4
L Gegebenes Array sortieren, indem zufällige Zahlenpaare aus Array ausgewählt werden Java Basics - Anfänger-Themen 14
Say 2-DIM Array Code lesen und verstehen Java Basics - Anfänger-Themen 5
N Array beim erstellen mit Werten füllen Java Basics - Anfänger-Themen 6
C Java Array Struktur, welche ist wann besser? Java Basics - Anfänger-Themen 12
Temsky34 Array IndexOf nicht verfügbar Java Basics - Anfänger-Themen 18
belana wie am besten 2D Array von String to Integer Java Basics - Anfänger-Themen 18
S Array mit Methode löschen Java Basics - Anfänger-Themen 2
J Java To String Methode, Array mit For-Schleife Java Basics - Anfänger-Themen 2
E Durch Muster in Array iterieren Java Basics - Anfänger-Themen 3
L Frage zum Array Java Basics - Anfänger-Themen 1

Ähnliche Java Themen

Neue Themen


Oben