Array wird falsch befüllt?

[cptlava]

Hey
Wir sollen Objektorientiert programmieren und das versuche ich gerade. Wir sollen ein Lager von einem Artikel mit mehreren Attributen anlegen.
Soweit auch kein Problem. Allerdings finde ich den Fehler nicht weshalb sich auf jedem Platz meines Arrays der selbe Artikel befindet.

Edit:
Nach ein paar weiteren Tests scheint es das sich das Array beim befüllen selbst überschreibt

Fachklasse Lager:

public class Lager
{
  public void anlegenArtikel(){
     lager[counter] = art1;
     counter++;
  }
 
  public void bucheZugang(int number,int value) { 
     for (int i = 0;i < counter; i++){
        art1 = lager[counter];
        if (number == art1.getNummer())  {
           art1.bucheZugang(value); 
        } 
     }
  }

  public void bucheAbgang(int number,int value) { 
     for (int i = 0;i < counter; i++){
        art1 = lager[counter];
        if (number == art1.getNummer())  {
           art1.bucheAbgang(value); 
        } 
     }
  }
 
  public void setart1(Artikel art1) {
      this.art1 = art1;
  }
 
  public Artikel art1() {
     return art1;
  }
  public Artikel[] lager() {
     return lager;
  }

  public String toString() { 
     String ausg = " hi", buffer = " "; 
     for (int i = 0;i < lager.length; i++){
        art2 = lager[i];
        buffer = "\tArtikelnummer: " + art1.getNummer() + "\n\tArtikelBezeichner: " + art1.getBezeichner()  + "\n\tArtikelBestand: " + art1.getBestand() + "\n\tArtikelpreis " + art1.getPreis() + "\n \n";
        ausg = ausg + buffer;
     }
     return ausg;
  }
 
  String[] str = new String[2];
  private int counter = 0;
  private Artikel art1,art2 = new Artikel();
  private Artikel[] lager = new Artikel[2];
}

Testklasse Lager

public class LagerTest
{
  public void start(){ 
     a1 = t1.start();
     l1.setart1(a1);
     l1.anlegenArtikel(); 
  }

  public void bestandAendern(){
     System.out.println("Nummer");
     iBuffer = input.nextInt();
     System.out.println("Menge");
     iBuffer1 = input.nextInt();
     l1.bucheZugang(iBuffer,iBuffer1);
  }
 
  public void bestandAendern1(){
     System.out.println("Nummer");
     iBuffer = input.nextInt();
     System.out.println("Menge");
     iBuffer1 = input.nextInt();
     l1.bucheAbgang(iBuffer,iBuffer1);
  }
 
  public void ausgabe(){
     String ausg = l1.toString();
     System.out.println(ausg); 
  }
 
  ArtTest t1 = new ArtTest();
     Lager l1 = new Lager();
     Artikel a1 = new Artikel();
     int iBuffer, iBuffer1;
     double dBuffer;
     String str;
     Scanner input = new Scanner(System.in);
}

Artikel Klassen

Spoiler

Fachklasse Artikel

public class Artikel
{
  //Buchen
  public void bucheZugang(int wert) {
     artBestand = artBestand + wert; 
  }
  public void bucheAbgang(int wert) {
     artBestand = artBestand - wert; 
  }
 
  //Set
  public void setartBestand(int artBestand) {
     this.artBestand = artBestand;
  }
  public void setartPreis(double artPreis) {
     this.artPreis = artPreis;
  }
  public void setartNummer(int artNummer) {
     this.artNummer = artNummer;
  }
  public void setartBezeichner(String artBezeichner) {
     this.artBezeichner = artBezeichner;
  }
 
  //Get
  public String getBezeichner() {
      return artBezeichner;
  }
  public int getNummer() {
      return artNummer;
  }
  public double getPreis() {
      return artPreis;
  }
  public int getBestand() {
      return artBestand;
  }
 
  //toString
  public String toString() {
     return "\tArtikelnummer: " + artNummer + "\n\tArtikelbezeichner " + artBezeichner +
     "\n\tArtikelbestand " + artBestand + "\n\tArtikelpreis " + artPreis;
  }
 
  //Variablen
  private double artPreis;
  private int artNummer, artBestand;
  private String artBezeichner;
}

Testklasse Artikel

public class ArtTest
{
  public Artikel start() { 
     System.out.println("Bestand");
     buffer = scanner.nextInt();
     a1.setartBestand(buffer);
 
     System.out.println("Nummer");
     buffer = scanner.nextInt();
     a1.setartNummer(buffer);
 
     System.out.println("Preis");
     dBuffer = scanner.nextDouble();
     a1.setartPreis(dBuffer);
 
     System.out.println("Bezeichner");
     buffer1 = scanner.next();
     a1.setartBezeichner(buffer1);
     return a1;
  }
 
  public void zugang(){
     System.out.println("Menge");
     buffer = scanner.nextInt();
     a1.bucheZugang(buffer);
  }
 
  public void abgang(){
     System.out.println("Menge");
     buffer = scanner.nextInt();
     a1.bucheAbgang(buffer);
  }
 
  public void ausgabe(){ 
     String ausg = a1.toString();
     System.out.println(ausg);
  }
 
  public static void main(String[] args){
     new ArtTest().start();
     new ArtTest().ausgabe(); 
  }

  Artikel a1 = new Artikel();
  Scanner scanner = new Scanner(System.in);
  int buffer;
  double dBuffer;
  String buffer1;
}

Im Anhang nochmal ein Bild mit der Ausgabe bei einem 2 stelligen Array.

Bin für jede Hilfe dankbar!

 

[Bitfehler]

In deiner toString von der Klasse Lager hast du eine Schleife und schreibst bei jedem Durchlauf etwas in art2. In der nächsten Zeile verwendest für die Ausgabe aber art1, der sich ja nicht ändert, sodass öfter art1 ausgeben wird.

EDIT:
Da sind noch mehr (potenzielle) Fehler in der Klasse Lager, vlt findet du die ja selber.

 

[cptlava]

Hatte ich schon gefunden und abgeändert, aber danke.

Problem ist auf jeden Fall nicht die Ausgabe sondern das befüllen des Arrays.
Sobald ich einen 2. artikel hinzufüge ändert sich der erste ebenfalls, ich kann mir nur nicht erklären weshalb -.-

 

[Bitfehler]

Guck dir mal die ersten drei Methoden deiner Klasse Lager an.

Prinzipiell wäre es schöner, wenn du drei Java-Klassen haben würdest. Eine Klasse Artikel, eine Klasse Lager und eine Klasse App/Programm/wieauchimmer. Die ersten beiden Klassen bilden dann die Objekte Lager und Artikel ab und werden von der dritten Klasse verwendet. Die dritte Klasse enthält also den Ablauf.

 

[cptlava]

Die Idee Artikel und Lagertest zusammen zu fassen hört sich gut an, werd ich gleich mal umsetzen.
Allerdings verstehe ich nicht ganz was die ersten 3 Methoden mit meinem Problem zu tun haben?
Gerade da ich die Zu und Abgangsmethode noch gar nicht benutzt habe da ich das Array ja nicht befüllt bekomme

 

[Bitfehler]

for (int i = 0;i < counter; i++){
  art1 = lager[counter];
...

Du müsstest hier den Index verwenden und nicht einen Teil deiner Abbruchbedingung. So liest du immer den selben Artikel aus dem Lager aus.

Ich dachte eher an sowas wie unten dargestellt.
Es ist absichtlich nicht alles von deinen Methoden implementiert und verwendet eine Liste anstatt eines Arrays. Ne bisschen sollst du ja auch selber machen.

Die Klassen Artikel und Lager stellen Klassen dar, die in der App verwendet werden.

Spoiler: Klasse Artikel

public class Artikel
{
  //Variablen
  private double artPreis;
  private int artBestand;
  private int artNummer;
  private String artBezeichner;
  public Artikel() {}
  public Artikel(double artPreis, int artBestand, int artNummer, String artBezeichner) {
      this.artPreis = artPreis;
      this.artBestand = artBestand;
      this.artNummer = artNummer;
      this.artBezeichner = artBezeichner;
  }
  //Set
  public void setartBestand(int artBestand) {
      this.artBestand = artBestand;
  }

  public void setartPreis(double artPreis) {
      this.artPreis = artPreis;
  }

  public void setartNummer(int artNummer) {
      this.artNummer = artNummer;
  }

  public void setartBezeichner(String artBezeichner) {
      this.artBezeichner = artBezeichner;
  }

  //Get
  public String getBezeichner() {
      return this.artBezeichner;
  }

  public int getNummer() {
      return this.artNummer;
  }

  public double getPreis() {
      return this.artPreis;
  }

  public int getBestand() {
      return this.artBestand;
  }

  //Achtung:Kann negativ werden
  public void updateBestand(int wert) {
      this.artBestand = this.artBestand - wert;
  }

  //toString
  public String toString() {
     return "\tArtikelnummer: " + this.artNummer + "\n\tArtikelbezeichner " + this.artBezeichner +
     "\n\tArtikelbestand " + this.artBestand + "\n\tArtikelpreis " + this.artPreis;
  }
}

Spoiler: Klasse Lager

public class Lager
{
    // instance variables - replace the example below with your own
    private ArrayList<Artikel> lager;

    /**
     * Konstruktor Class Lager
     */
    public Lager(){
        lager = new ArrayList<Artikel>();
    }

    /**
     *
     */
    public void anlegenArtikel(Artikel art) {
        lager.add(art);
    }

    /**
     * TODO: Siehe Anmerkung
     */
    public void updateArtBestand(int artNumber, int value) {
        //Durchlaufe Lager
        //Wenn ein Artikel die übergebene artNumber enthält,
        //rufe updateBestand aus der Klasse Artikel auf
        //Überlege, was passieren soll, wenn Artikel nicht gefunden
    }

    /**
     * public void bucheAbgang(int artNumber, int value) {}
     * wird nicht benötigt, gebe negative Zahl als Parameter in
     * updateArtBestand
     */

    /**
     * TODO; Bekommst du selber hin, hier fehlt wahrscheinlich noch was
     */
    public String toString() {
        String out = "";
        for(Artikel art : lager){
            out = out + " " + art.getBezeichner();
        }
        return out;
    }
}

Spoiler: Klasse App

public class App
{
    public static void main(String[] args) {
    
        //Lege ein Lager an
        Lager lager = new Lager();
    
        //Lege ein Artikel ins Lager
        lager.anlegenArtikel(new Artikel(10.0, 1, 1234, "Schokolade"));
    
        //Ausgabe
        System.out.println(lager.toString());
    
    }
}

 

[cptlava]

Erstmal Danke für die Arbeit die du dir gemacht hast. Hab jetzt auch verstanden was du meintest.
Noch als letzte Frage: Gibt es einen besonderen Grund warum du "this" außerhalb des Konstruktors verwendest?

 

[Bitfehler]

http://www.java-forum.org/thema/aufbau-einer-klasse-erzeugung-einer-instanz.2731/