Vektor + auf Elemente zugreifen funktioniert nicht

[javanaggi]

Hallo Forum,

ich habe ein Problem mit meinem Programm.

Ich programmiere ein Adressbuch (bisher Konsolenprogramm).
Ich habe eine Klasse Address
mit den Attributen surname, firstname, zip, city
und den passenden getter und setter methoden.

außerdem habe ich eine datei, in der pro zeile ein wort steht...satzaufbau:
surname
firstname
zip
city
surname
firstname
zip
city
...
...
...

So, auslesen kann ich datei (funktion: read_database()).
read_database ruft in sich selbst eine funktion list_insert auf, die folgendes tut:

in read_database wird nach dem auslesen von 4 Zeilen (also einem Datensatz) list_insert aufgerufen.

public void list_insert(String helpsur, String helpfirst, String helpzip, String helpcity)
{
    ad.setSurname(helpsur);
    ad.setFirstname(helpfirst);
    ad.setZip(helpzip);
    ad.setCity(helpcity);
    vec.addElement(ad);
}

So vergrößert sich doch mein Vektor um ein Element je Aufruf von list_insert.

jetzt hab ich nur ein Problem mit der Ausgabe.

public void list_print()
{
    System.out.println("Vektor enthält Element ad: "+vec.contains(ad));
    System.out.println("Vektor ist "+vec.size()+" Elemente groß");

    for(int p=0; p<vec.size(); p++)
    {
        System.out.println(vec.elementAt(p);
        System.out.println(ad.getSurname());
        System.out.println(ad.getFirstname());
        System.out.println(ad.getZip());
        System.out.println(ad.getCity());
    }
}

Meine Ausgabe lautet wie folgt:

Vektor enthält Element ad: true
Vektor ist 2 Elemente groß
Address@19821f
nachname datensatz2
vorname datensatz2
zip datensatz2
city datensatz2
Address@19821f
nachname datensatz2
vorname datensatz2
zip datensatz2
city datensatz2

Ich habe bestimmt irgendwo einen logischen fehler. wo liegt der Fehler? Ich will Elemente in den vektor einfügen und am ende alle Elemente des vektors ausgeben. da ein Element des vektors ein objekt der klasse Address ist, nun meine frage, wie komme ich an die daten des objekts im vektor?

ich hoffe ich habe mich halbwegs klar ausgedrückt.

javanaggi

 

[Der Müde Joe]

ein neues Objekt pro Addresse!!!!

insert(...){
Addess address = new Address();
adrress.setStreet("JavaAvenue");
myVector.add(address);
}

is den ad ne GLOBALE Variable?

EDIT:
Adress@12345
ist ja 2x die geliche Referezaddresse..also 2x das gleiche Objekt an verschiedenen Stellen im vector

 

[NTB]

public void list_insert(String helpsur, String helpfirst, String helpzip, String helpcity)
{
    ad.setSurname(helpsur);
    ad.setFirstname(helpfirst);
    ad.setZip(helpzip);
    ad.setCity(helpcity);
    vec.addElement(ad);
}

So vergrößert sich doch mein Vektor um ein Element je Aufruf von list_insert.

Dein Vektor vergrößert sich immer um das gleiche Element.
Du musst in der obigen Methode jedes Mal ein neues Objekt erstellen. Zumindest sieht das von dem Codeschnipsel gesehen so aus, als wenn Du das Objekt "ad" nur einmal erzeugst.

public void list_insert(String helpsur, String helpfirst, String helpzip, String helpcity)
{
    Address ad = new Address();
    ad.setSurname(helpsur);
    ad.setFirstname(helpfirst);
    ad.setZip(helpzip);
    ad.setCity(helpcity);
    vec.addElement(ad);
}

 

[javanaggi]

danke für die antworten.
das war der fehler!
stimmt. ich habe vergessen in der list_insert Address ad = new Address(); zu schreiben.

habe es jetzt gemacht und bekomme jetzt:
Address@19821f
null
null
null
null
Address@addbf1
null
null
null
null

So, jetzt muss ich nur noch die Addressen selbst ausgeben.

also irgendwas in der richtung:

System.out.println(vec.elementAt(p).getSurname());

Mir ist klar, dass das so nicht geht. Aber wie kann ich sowas lösen?

Ich muss doch irgendwie auf ein Element meines Vektors zugreifen können und dort dann wieder auf die Methode meiner Address?

jemand ne idee?

javanaggi

 

[Der Müde Joe]

das gleiche umgekehrt

print(){
for(Address address : vector){
System.out.println(address.getName());
}
}

//oder

Iterator it = vector.iterator();
while(it.hasNext()){
Address address = it.next() //wenn nicht generischer Vector dann cast (Address)it.next();
System.out.println(address.getName());
}

dan sollt es gehen

 

[javanaggi]

mhh...das versteh ich nicht. also wie soll das funktionieren?

und warum gibt mir

System.out.println(vec.elementAt(p));

eine Adresse und nicht mein Element (ad) zurück?

javanaggi

 

[Der Müde Joe]

System.out.println(vec.elementAt(p));

eine Adresse und nicht mein Element (ad) zurück?

Es gibt dein Element ad zurück...Aber da du toString() von Object nicht überschreibst
und dann das Element Sysoutest wird das ausgeführt

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

geerbt von Object.

Wenn du was spezielles willst musst du toString() überschreiben.

public String toString(){
StringBuilder result = "";
result.append(this.name);
return result.toString();
}

oder in deiner Print methode das hatl machen wie du schon hast

public void list_print()
{
    System.out.println("Vektor enthält Element ad: "+vec.contains(ad));
    System.out.println("Vektor ist "+vec.size()+" Elemente groß");

    for(int p=0; p<vec.size(); p++)
    {
Address address = vec.elementAt(p);
        System.out.println(address .getSurname());
        System.out.println(address .getFirstname());
        System.out.println(address .getZip());
        System.out.println(address .getCity());
    }
}

EDIT:
Address address = (Address)vec.elementAt(p); wenn du keine generischen Vector hast (casten)

 

[javanaggi]

danke. jetzt funktionierts!!!!

ich kann die sachen ausgeben. ohne dein edit hätt ich es nicht geschafft. kannst du mir sagen warum das so umständlich geht (casten)?!?!

javanaggi

 

[Der Müde Joe]

kannst du mir sagen warum das so umständlich geht (casten)?!?!

Vor Java 1.5 gab es noch keine Generischen Typen. In einem vector konnte man
alles reintun. Also Address / Person ...

der Vector liefert dann auch ein Objekt vom Typ Object zurück, welches dann in das gewünschte Object
umgewandelt (gecastet) wird.

Das kann Probleme zur Laufzeit geben, da man nie sicher sein kann, ob das Objekt im Vector wirklich
von diesem Typ ist (ClassCastException)
Mit Java 1.5 wurden dann generics eingeführt, wo schon zur übersetzungszeit die Typsicherheit überprft wird,
wobei die cast überflüssig wurden.

Vector<Address> v = new Vector<Address>();
v.add(new Address());//ok
v.add(new Date()); //compiler error

Address a = v.elementAt(1); //ok, da sicher nur Addressen im vector

//ohne;
Vector v = new Vector();
v.add(new Address()); //ok
v.add(new Date()); //ok

Address a = (Address)v.elementAt(0); //ok
Address b = (Address)v.elementAt(1); // ClassCastException zur laufzeit

wird die so vielleicht ein wenig klarer

 

[javanaggi]

ok.

danke für die antworten.

ich habe ein weiteres (kleines ?) Problem.

Ich implementiere gerade eine Suchfunktion.

public void search (String name)
{
    for(int p=0; p<this.size(); p++)
    {
        Address ad = (Address)this.elementAt(p);
        if(ad.getSurname()==name)
        {
            System.out.println("gefunden");
        }
        else
        {
            System.out.println("nicht gefunden");
        }
    }
}

Es sei gesagt, dass ich diese Funktion in einer Klasse EnhancedVector stehen habe. Es wird auch jedes Element eines Vektors aufgerufen und auch die Elemente in meinem Objekt ad, aber er sagt er findet den Namen nicht. übersehe ich irgendwas? Meinem Verständnis nach müsste das doch stimmen?

jemand ne idee?

javanaggi

 

[Der Müde Joe]

String vergleicht man mit

equals !!!!

str1.equals("halloEcho");

 

[javanaggi]

danke.