Arraylisten auslesen, kommt nur Quatsch raus

[kuzdu]

Gute Abend bzw. guten Morgen ,
habe ein kleines Problemchen woran ich iwie nicht weiterkomme und google hat mir auch (nocht) nicht (weiter)geholfen.
Habe eine neue Klasse zu meiner Main-Klasse. Die heißt Produktverwaltung.java.

Da steht folgendes (relevantes):

import java.util.*;
import java.io.*;


public class Produktverwaltung implements Serializable {
  
    
  
    private ArrayList Waren_und_Preise = new ArrayList();
    
      
    Produktverwaltung() {
    Waren_und_Preise.add(new Produkt("Südwein",24.99));
    Waren_und_Preise.add(new Produkt("Edelburgunder",129.99));  
    Waren_und_Preise.add(new Produkt("Speisekäse",3.49));  
    Waren_und_Preise.add(new Produkt("Erdammer",5.99));  
    Waren_und_Preise.add(new Produkt("Trüffel",1499.00));  
    }
    
    public void ProdukteAuslesen()
    {
       
    
       Iterator it = Waren_und_Preise.iterator();
       while(it.hasNext())
       System.out.println(it.next());
    }

Die Ausgabe ist:

onlineshop.Produkt@7d8a992f
onlineshop.Produkt@164f1d0d
onlineshop.Produkt@23fc4bec
onlineshop.Produkt@8dc8569
onlineshop.Produkt@45bab50a

Mit einer For-Schleife ist das Ergebnis genauso.
(Ich wette das ist ieine Lapalie auf die ich nicht komme -.-)

Falls überhaupt noch relevant meine Main-Klasse sieht so aus:

...
case 1: 
        {
           Produktverwaltung pv = new Produktverwaltung();
          
           //pv.inDateiSchreiben();
           pv.ProdukteAuslesen();
        
           break;
        }//Produkte anzeigen
...

Für mehr Code einfach fragen.

Gute Nacht

Gruß kuzdu

 

[ARadauer]

Was genau soll da stehen, wenn du eine Produkt mit System.out.println ausgibst?
Java weiß ja nicht, dass du zb den namen ausgeben möchtest..

Also ruft es die toString Methode auf: Object (Java Platform SE 6))

In other words, this method returns a string equal to the value of:

getClass().getName() + '@' + Integer.toHexString(hashCode())

Was kannst du machen?
Entweder die toString Methode in deiner Klasse überschreiben oder einfach eine Eigenschaft deines Objekts ausgeben..


bah... spät is heute schon...

 

[hdi]

Und bitte vermeide solche Namen wie "Waren_und_Preise". Das verstößt gegen die Namenskonvention für Variablen. Wenn schon dann sollte das "warenUndPreise" heißen - aber am sinnvollsten wäre eigentlich "produkte", denn genau das steckt da drin. Du hast deine Klasse ja auch "Produkt" genannt und nicht "Ware_und_Preis"

Und nochwas: Wenn nicht zwingend nötig dann vermeide es, als deklarierten Datentyp eine Collections-Implementierung zu nutzen. Nutze lieber den Typ des Interfaces. Außerdem kannst du die sog. Generics nutzen, um dem Compiler bekannt zu machen dass es sich bei den Elementen in der Liste um Produkte handelt. Ansonsten sieht er alles als "Object", und das kann dann u.U. etwas umständlich werden.

List<Produkt> produkte = new ArrayList<Produkt>();

 

[kuzdu]

Was genau soll da stehen, wenn du eine Produkt mit System.out.println ausgibst?

Als Ausgabe sollen eigentlich die Produkte auftauchen:
Südwein,24.99
Edelburgunder,129.99
...

Und bitte vermeide solche Namen wie "Waren_und_Preise". Das verstößt gegen die Namenskonvention für Variablen. Wenn schon dann sollte das "warenUndPreise" heißen - aber am sinnvollsten wäre eigentlich "produkte", denn genau das steckt da drin. Du hast deine Klasse ja auch "Produkt" genannt und nicht "Ware_und_Preis"

Okay wusste ich gar nicht.

Entweder die toString Methode in deiner Klasse überschreiben oder einfach eine Eigenschaft deines Objekts ausgeben..

Wie genau bewerkstellige ich sowas?

 

[jgh]

da ich deine Klasse [c]Produkt[/c] nicht kenne, habe ich mal schnell so eine erstellt...eine Möglichkeit wäre die folgende...du kannst natürlich auch die toString()-Methode überschreiben.

import java.io.Serializable;
import java.util.ArrayList;
import java.util.Iterator;

public class Produktverwaltung implements Serializable {

	private ArrayList Waren_und_Preise = new ArrayList();

	Produktverwaltung() {
		Waren_und_Preise.add(new Produkt("Südwein", 24.99));
		Waren_und_Preise.add(new Produkt("Edelburgunder", 129.99));
		Waren_und_Preise.add(new Produkt("Speisekäse", 3.49));
		Waren_und_Preise.add(new Produkt("Erdammer", 5.99));
		Waren_und_Preise.add(new Produkt("Trüffel", 1499.00));
	}

	public void produkteAuslesen() {

		Iterator it = Waren_und_Preise.iterator();
		while (it.hasNext()) {
			Produkt p = (Produkt) it.next();
			System.out.println(p.getName() + " " + p.getPreis());
		}
	}

	public static void main(String[] args) {
		Produktverwaltung p = new Produktverwaltung();
		p.produkteAuslesen();
	}
}

class Produkt {
	String name;
	double preis;

	public Produkt(String name, double preis) {
		this.preis = preis;
		this.name = name;
	}

	public String getName() {
		return name;
	}

	public double getPreis() {
		return preis;
	}
}

 

[Shulyn]

Als Ausgabe sollen eigentlich die Produkte auftauchen:
Südwein,24.99
Edelburgunder,129.99
...
Wie genau bewerkstellige ich sowas?

[Java]

/**
*
* @author jgh
*/
class Produkt {
String name;
double preis;

public Produkt(String name, double preis) {
this.preis = preis;
this.name = name;
}

public String getName() {
return name;
}

public double getPreis() {
return preis;
}

// So könntest du es machen...
@Override
public String toString() {
return name + " , " + Double.toString(preis);
}
}
// deine Ausgabe ist dann nur noch
public void produkteAuslesen() {

Iterator it = Waren_und_Preise.iterator();
while (it.hasNext()) {
Produkt p = (Produkt) it.next();
System.out.println(p);
// oder p.toString() , wobei das .toString() automatisch aufgerufen wird und somit unnötig ist...
}
}
[/Java]

 

[kuzdu]

Wollte das Thema doch nochmal aufgreifen. Also mir ist jetzt klar, wie ich die Arrays in einer Schleife auslese. Aber wie komme ich an einzelne Werte heran?

Konkreter:
Habe z.B.

###Klasse Kundenverwaltung
....

protected LinkedList Kundendaten = new LinkedList(); 

Kundenverwaltung(){
      Kundendaten.add(new Kunde("Herbert Maier","Kreutzweg 28a","57072 Siegen"));
      Kundendaten.add(new Kunde("Heidi Schiffer"," Wellnesgasse 4"," 13579 Schönhausen"));
      Kundendaten.add(new Kunde("Claudia Klum", "Wellnesgasse 4","13579 Schönhausen"));
      Kundendaten.add(new Kunde("Peter Held", "Katzenweg 5", "98123 Frömmersbach"));
      }
...

####Klasse Kunde
public class Kunde extends OnlineShop {
    
    protected String Name;
    protected String Adresse;
    protected String Ort;
    
    public Kunde(String Name_des_Kunden , String Anschrift_Kunde, String Ort_neu)
    {
    Name = Name_des_Kunden;
    Adresse = Anschrift_Kunde;      
    Ort = Ort_neu;
    }
    
     
    public String getName() {
        return Name;
    }
 
    public String getAdresse() {
        return Adresse;
    }
    
     public String getOrt() {
        return Adresse;
    }
}

###Hauptklasse
....

public void auslesenBestellung()
    {
    //Durch I/O wird ein Wert (integer) übergeben, den der User ausgesucht hat (Wert zwische 1-4)
    //       Kunde k = ???;
    //       System.out.println("KUNDE:" + k.getName());
   }

Also wie komme ich jetzt genau an den Namen?

Danke für Hilfe

 

[Michael...]

Kundendaten.get(2).getName()

Wobei in dem Fall eine LinkedList vermutlich nicht so gut geeignet scheint. Ich würde eine ArrayList nehmen.

 

[kuzdu]

Kundendaten.get(2).getName()

Sorry, aber daraus werde ich nicht ganz schlau.

Der Compiler kennt get.Name() aus der Klasse Kunde nicht.

Also:

System.out.println("KUNDE: " + kv.Kundendaten.get(2).getName());
//cannot find symbol,  symbol:   method getName()

Um das zu "beheben" habe ich es so geschrieben, aber das scheint hinten und Vorne nicht zu stimmen :/:

  Kundenverwaltung kv = new Kundenverwaltung();
            Kunde k = new Kunde(); //Fehler: constructor Kunde in class onlineshop.Kunde cannot 
                                                      //be applied to given types;
                                                      //required: java.lang.String,java.lang.String,java.lang.String
            
            System.out.println("KUNDE: " + kv.Kundendaten.get(2).getName());

Wenn ich nur kv.Kundendaten.get(2) schreibe, stehe ich wieder am Anfang und bekomme so etwas ausgegeben: onlineshop.Kunde@24a20892.

 

[Michael...]

Sorry, hatte übersehen, dass Du bei der Liste keine Generics nutzt, dann musst Du casten:

((Kunde)Kundendaten.get(2)).getName()

Besser die Liste wie folgt deklarieren:

LinkedList<Kunde> Kundendaten = new LinkedList<Kunde>();

Dann kann man sich das Casten sparen.

 

[njans]

Okay ich steige durch deinen, zum Teil doch recht unvollständigen, Code nicht durch.
Noch mal etwas zum Basisverständnis:

Wenn du System.out.println(MeinKundenObjekt); aufrufst, dann ruft java intern an dem Objekt "MeinKundenObjekt"die Methode toString() auf. Diese liefert dann eine Beschreibung des Objektes. Diese Beschreibung ist genau der Kauderwelsch, den du am Anfang ausgegeben bekommen hast. Was du eigentlich willst, sind die Daten, die in deinem Objekt gespeichert sind. Dazu musst du dann nur auf dem Objekt die Methoden aufrufen, die dir die entsprechenden Daten in einem passenden Format ausgeben.
So zum Beispiel die Methode getName(). Diese gibt dir den Namen als String zurück.
Wenn getName() bei deinem Beispiel nicht erkannt wird (//cannot find symbol, symbol: method getName()),d ann liegt das wahrscheinlich daran, dass du an dem Objekt, welches du mit kv.Kundendaten.get(2) zurückbekommst, diese Methode nicht aufrufen kannst.
Mögliche Ursache:
Du hast ein Array, welche aus "Kunden" besteht, sondern aus anderen Klassen (Evtl. die Klasse, von der Kunde erbt?).

 

[beastofchaos]

@njans: siehe über dir - da ist die Lösung (Generics)

Die Klasse Object hat schließlich nicht die Methode "getName()". Die hat nur "Kunden" und das muss eine LinkedList/ ArrayList wissen


@kuzdu:
Du meintest ja, dass es hier eine Fehlermeldung gibt:

 Kunde k = new Kunde();

So schau dir mal die Klasse Kunde an - da hast du einen Konstruktor erzeugt, der

public Kunde(String Name_des_Kunden , String Anschrift_Kunde, String Ort_neu)

lautet. Das heißt, dass du bei "new Kunde()" in Klammern auch die geforderten Parameter auch übergeben musst, sprich z.B.:

  Kunde k = new Kunde("Hans Dieter", "Dietersstr. 27b", "Berlin");

Und außerdem wie hdi um 7:55 schon bemerkt hat: Es ist eigentlich ein Standard, dass man bei Variablen und Methoden den ersten Buchstaben klein schreibt und jedes "neue" Wort in den Namen einfach groß dahinter schreibt. Also ich veranschauliche das mal an deinem Konstruktor:

public Kunde(String Name_des_Kunden , String Anschrift_Kunde, String Ort_neu)

zu:

public Kunde(String kundenName , String kundenAnschrift, String neuerOrt)

Ob nun "kundenName" oder "nameKunde" ist da egal, aber die Schreibweise wird besonders in Java groß geschrieben

Gruß, Thomas

 

[hdi]

@kuzdu Ich will dir ja nicht dumm kommen, aber du scheinst ein klassischer Fall von "kann nix und will viel" zu sein Wenn jemand noch nicht mal eine eigentlich unmissverständliche Fehlermeldung interpretieren kann, bzw keine Ahnung hat wie ein Konstruktor-Aufruf funktioniert, sollte er keine Klassen implements Serializable schreiben (weißt du überhaupt was das ist, oder von Eclipse einfügen lassen um eine Warning zu umgehen?), und auch nicht mit ArrayLists oder Generics arbeiten.

Mein (gut gemeinter) Ratschlag: Lass das erstmal stehen und liegen, und kümmere dich darum dass du dir die Basics aneignest. Wirf mal einen Blick in diesen Thread, da findest du genug Lernhilfen.

 

[kuzdu]

@Michael...
Funktioniert super. Danke!

@njans: siehe über dir - da ist die Lösung (Generics)

@kuzdu:
Du meintest ja, dass es hier eine Fehlermeldung gibt:

 Kunde k = new Kunde();

So schau dir mal die Klasse Kunde an - da hast du einen Konstruktor erzeugt, der

public Kunde(String Name_des_Kunden , String Anschrift_Kunde, String Ort_neu)

lautet. Das heißt, dass du bei "new Kunde()" in Klammern auch die geforderten Parameter auch übergeben musst, sprich z.B.:

  Kunde k = new Kunde("Hans Dieter", "Dietersstr. 27b", "Berlin");

Ja, war vielleicht ein bisschen blöd von mir diesen Codeschnipsel zu posten. Wurde mir von Netbeans angezeigt, dass drei Strings gefordert werden. Aber wenn ich die Daten so übergebe, dann ist der Kunde k ja statisch (wenn ich mich nicht irre?) Und der Benutzer sollte sich den Kunden ja aus der Linkedlist aussuchen, deshalb habe ich nach der Variante von "Michael..." gesucht.
Aber es funktioniert jetzt alles und deshalb besten Dank!

PS: Werde das mit den Standards in nächsten "Projekten" beachten

 

[ARadauer]

dann ist der Kunde k ja statisch (wenn ich mich nicht irre?)

du irrst dich, wenn du mit statisch static meinst

 

[kuzdu]

Ne, meine ich nicht.

Dann ist für mich aber alles klar