Objekt in Array abspeichern

[DuffterMatze]

Hallo, ich bin neu im Forum und auch noch totaler Anfänger in Programmierung, aber durch mein Studium bin ich jetzt zu Java gekommen und es macht mir auch echt riesig Spaß nur kommen natürlich so einige Fragen

Also mein Frage bezieht sich auf das abspeichern von Objekten in Arrays

Mit folgender Methode speicher ich (oder zumindest glaub ich das ^^) ein Objekt in ein Array ein (Wir sollen es mit Arrays und nich mit ArrayLists oder ähnlichem machen, deswegen muss ich das Array zuerst erweitern, da es beliebig viele abspeichern soll):

static int neuerArtikel(int artikelNr) {
		System.out.println("\nNeuen Artikel eingeben");
		System.out.println("======================");
		artikelListe = Arrays.copyOf(artikelListe, artikelListe.length+1);
		Artikel neuesAuto = new Artikel();
		artikelListe[artikelNr] = (Artikel)neuesAuto;
		while (artikelListe[artikelNr].eingeben() == false) {
			System.out.println("Fehlerhafte Eingabe!");
		}
		System.out.println("Neuer Artikel abgelegt");
		return (artikelNr + 1);
	}

und mit folgender Methode möchte ich dann eine Liste mit allen abgespeicherten Artikeln anzeigen lassen:

static void listeAnzeigen(int artikelNr) {
		System.out.println("\nArtikelliste");
		System.out.println("============");
		for (int i = 0; i < artikelNr; i++) {
			System.out.print((i + 1) + ") ");
			Artikel anzeige = (Artikel)artikelListe[i];
			System.out.println(anzeige.listEintrag());
		}
	}

Beim kompilieren gibt es kein problem, aber beim ausführen passiert es, dass wenn ich die Liste anzeigen lassen will er immer nur den zuletzt eingegebenen Artikel anzeigt, aber den sooft, so viele Eingaben ich gemacht habe (also wenn ich 5 Artikel eingegeben habe, zeigt er mir 5 mal den letzten)

Ich weiß jetzt nicht, was ihr noch an Code braucht, ich hoffe das passt so, wenn nicht, kann ich den Rest natürlich auch posten

Achja vielleicht noch die einegebn() und die listEintrag() Methoden ^^

public boolean eingeben() {
		Scanner eingabe = new Scanner(System.in);
		
		try {
			System.out.print("Artikelnummer: ");
			artikelnr = eingabe.nextInt();
			System.out.print("Bezeichnung: ");
			bez = eingabe.next();
			System.out.print("Gewichte in Gramm: ");
			gew = eingabe.nextInt();
			System.out.print("Preis: ");
			preis = eingabe.nextInt();
			System.out.print("Währung: ");
			währung = eingabe.next();
			return true;
		} catch (Exception e) {
			System.out.println(e);
			return false;
		} finally {
			eingabe = new Scanner(System.in);
		}
	}

public String listEintrag() {
		String text;
		text = bez + ", " + artikelnr + ", " + gew + "g, " + preis + " " + währung;
		return text;
	}

 

[Steff87]

Hi!
Wie ist denn deine artikelListe abgespeichert? Ist es vom Typ Atrikel?
Kannst du vielleicht noch die main Methode posten, um zu sehen, wie du das ganze aufrufst.

 

[Brainiac]

Aufgrund dieser Zeilen:

Artikel neuesAuto = new Artikel();
artikelListe[artikelNr] = (Artikel)neuesAuto;

gehe ich mal davon aus das artikelList sowas ist:

Artikel[] artikelListe = new Artikel[10];

Daher kannst Du dir das ganze Casten (Artikel) sparen.

Der obere Part geht einfach so:

artikelListe[artikelNr] = new Artikel();

 

[DuffterMatze]

Also ich poste jetzt einfach mal die ganze klasse

import java.util.Arrays;
import java.util.Scanner;
import einkauf.Artikel;

public class Blatt2_Aufgabe3 {
	static Artikel[] artikelListe = new Artikel[0];
	static int anzArtikel = 0;
	
	public static void main (String[] argumente) {
		boolean ende = false;
		
		do {
			System.out.println("\nShopper - zur Erstellung ihrer Einkaufsliste");
			System.out.println("============================================");
			System.out.println("\nHauptmenü");
			System.out.println("\n(1) Artikel anzeigen");
			System.out.println("(2) Neuen Artikel anlegen");
			System.out.println("\n(0) Programm beenden");

			Scanner wahl = new Scanner(System.in);
			try {
				System.out.print("\nAuswahl: ");
				short auswahl = wahl.nextShort();
				
				switch (auswahl) {
				case 1:
					listeAnzeigen(anzArtikel);
					break;
				case 2:
					anzArtikel = neuerArtikel(anzArtikel);
					break;
				case 0:
					System.out.println("Programm beenden");
					ende = true;
					break;
				default:
					System.out.println("Ungültige Eingabe");
				}
			} catch (Exception e) {
				System.out.println("ZAHLEN eingeben!");
				}
			wahl = new Scanner(System.in);
		} while (!ende);
	}
	
	static int neuerArtikel(int artikelNr) {
		System.out.println("\nNeuen Artikel eingeben");
		System.out.println("======================");
		artikelListe = Arrays.copyOf(artikelListe, artikelListe.length+1);
		Artikel neuesAuto = new Artikel();
		artikelListe[artikelNr] = (Artikel)neuesAuto;
		while (artikelListe[artikelNr].eingeben() == false) {
			System.out.println("Fehlerhafte Eingabe!");
		}
		System.out.println("Neuer Artikel abgelegt");
		return (artikelNr + 1);
	}
	
	static void listeAnzeigen(int artikelNr) {
		System.out.println("\nArtikelliste");
		System.out.println("============");
		for (int i = 0; i < artikelNr; i++) {
			System.out.print((i + 1) + ") ");
			Artikel anzeige = (Artikel)artikelListe[i];
			System.out.println(anzeige.listEintrag());
		}
	}
}

Aber ob ich das caste jetzt weglasse oder nicht öndert ja im prinzip nichts am programmablauf, oder? also würde es nichts an meinem problem ändern
natürlich is mir klar, dass mein programmierstil noch nicht der schönste ist ^^

Klasse Artikel:

package einkauf;

import java.util.Scanner;

public class Artikel {
	public int anzArtikel;
	static int artikelnr;
	static String bez;
	static int gew;
	static int preis;
	static String währung;
	
	public Artikel(int k_artikelnr, String k_bez, int k_gew, int k_preis, String k_währung) {
		artikelnr = k_artikelnr;
		bez = k_bez;
		gew = k_gew;
		preis = k_preis;
		währung = k_währung;
		anzArtikel++;
	}
	
	public boolean eingeben() {
		Scanner eingabe = new Scanner(System.in);
		
		try {
			System.out.print("Artikelnummer: ");
			artikelnr = eingabe.nextInt();
			System.out.print("Bezeichnung: ");
			bez = eingabe.next();
			System.out.print("Gewichte in Gramm: ");
			gew = eingabe.nextInt();
			System.out.print("Preis: ");
			preis = eingabe.nextInt();
			System.out.print("Währung: ");
			währung = eingabe.next();
			return true;
		} catch (Exception e) {
			System.out.println(e);
			return false;
		} finally {
			eingabe = new Scanner(System.in);
		}
	}
	
	public String listEintrag() {
		String text;
		text = bez + ", " + artikelnr + ", " + gew + "g, " + preis + " " + währung;
		return text;
	}
}

 

[SlaterB]

die Attribute in Artikel sind alle static, das darf nicht sein

anzArtikel dagegen sollte ironischerweise eher statisch sein,
ansonsten wird das pro Objekt einzeln immer nur von 0 auf 1 erhöht

 

[DuffterMatze]

Ah jetzt gehts, danke

Was war dann das Problem? Eclipse hat mir das nur während dem Programmiern als Lösung vorgeschlagen

also was genau machst static denn? weiß nur, dass es was damit zu tun hat welche methoden bzw. klassen sie aufrufen bzw. verändern können, oder?

 

[Brainiac]

gelöscht weil doppelt

 

[SlaterB]

Galileo Computing :: Java ist auch eine Insel – 5.3 Statische Methoden und statische Attribute