ArrayList

[krackmoe]

Hab folgenden Code, in meiner main will ich dann mit System.out.println(p.toString());
Mir die Telefonnumer ausgeben lassen, die ich vorher eingegeben habe.
Das tut er mir aber nicht, er gibt mir einfach nichts aus!
Weiß jmd woran das liegt?

import java.util.*;

public class Person{
	private String name;
	private String nr;
	ArrayList<Person> arr = new ArrayList<Person>();
	
	public Person(String name, String nr){
		this.name = name;
		this.nr = nr;
	}
	public void add(String name, String nr){
		arr.add(new Person(name, nr));
	}
	public String toString(){
		String daten="";		
		for(int i=0; i < arr.size(); i++){
			daten = arr.get(i);
		}
		return daten;
	}
}

 

[diggaa1984]

arr.get(i);

das sollte dir eigentlich was vom typ "Person" zurückgeben, und kein String, aus der Person kannst du dann eleganterweise das entsprechende Attribut holen und ausgeben.

 

[ARadauer]

eine Person hat eine Liste von Personen... naja.. ist nicht sehr sinnvoll, ich würd die Arraylist in einer seperaten klasse halten

zeile 19, eine person ist kein String..., und du weißt daten immer wieder einen anderen wert zu, hier am besten stringbuffer oder stringbuilder verwenden, += würde auch gehen, performt aber schlecht

public class Person{ 
   private String name; 
   private String nr; 
   ArrayList<Person> arr = new ArrayList<Person>(); 
    
   public Person(String name, String nr){ 
      this.name = name; 
      this.nr = nr; 
   } 
   public void add(String name, String nr){ 
      arr.add(new Person(name, nr)); 
   } 
   public String toString(){ 
      StringBuffer daten= new StringBuffer();       
      for(int i=0; i < arr.size(); i++){ 
        daten.append(arr.get(i).name);
        daten.append(" - ");
        daten.append(arr.get(i).nr);
        daten.append("\n"); 
      } 
      return daten.toString(); 
   } 
   
   public static void main(String[] args) {
    Person p = new Person("test", "nummer");
    p.add("test2", "nummer2");
    p.add("test3", "nummer3");
    
    System.out.println(p.toString());
  }
}

 

[krackmoe]

Mh....

Ja dann bräucht ich ja eigene getter Methoden für name und number oder?
Dann bringt mir ja die ArrayList die ich jetzt habe nichts mehr!?
Dachte man kann das auch so ausgeben?

Was bringt mir das getrennt? Weil dann sind die 2 ja nicht miteinander verknüpft oder?

 

[Saxony]

Hiho,

Naja so wie es aber hier da steht, würde er eh nur den letzten Namen der Person bekommen, welche als letztes in der ArrayList steht.
Wieso hat eigentlich eine Person eine ArrayList von anderen Personen? Sind das die Kinder, Freunde oder Ehefrauen dieser Person?

bye Saxony

 

[diggaa1984]

Naja, betrachte die Klassen die du hast derart, das sie nur das tun wofür sie da sind, und bei deiner Benennung wäre eine Klasse Person nur etwas, was Namen und Nummer hinterlegt.
Einer weitere Klasse würde nun die ArrayList mit vielen einzelnen Personen beinhalten, quasi ein Datencontainer.

und weil ich das immer so gerne mache, am besten für die Main immer ne extra Klasse ^^ ... hat kein tieferen Sinn ausser das sie da völlig für sich alleine rumdümpelt und es in meinen Augen aufgeräumter ist.

 

[Saxony]

und weil ich das immer so gerne mache, am besten für die Main immer ne extra Klasse ^^ ... hat kein tieferen Sinn ausser das sie da völlig für sich alleine rumdümpelt und es in meinen Augen aufgeräumter ist.

Na wir hoffen doch mal, dass ARadauer es auch immer so macht und nur hier in diesem ganz speziellen Fall, um das Beispiel kurz zu halten, es so gemacht hat.

bye Saxony

 

[ARadauer]

aja member variablen macht man private und greift über public und getter und setter methoden darauf zu ;-.)

 

[krackmoe]

Okay hab das jetzt alles mal umgeschrieben so wie ihr gesagt habt.

book.java:

import java.util.*;

public class book{
	private String name;
	private String nr;
	Person p = new Person(name, nr);
	
	public book(String name, String nr){
		this.name = name;
		this.nr = nr;
	}
	public void add(String name, String nr){
		p = new Person(name, nr);
	}
	public String toString(){
		return p.toString();
	}
}

person.java:

import java.util.*;

public class Person{
	ArrayList<Person> arr = new ArrayList<Person>();
	int i=0;
	private String name="";
	private String nr="";
	
	public Person(String name, String nr){
		arr.add(new Person(name, nr));
	}
	public String toString(){
		StringBuffer daten = new StringBuffer();
		for(int i=0; i < arr.size(); i++){
			daten.append(arr.get(i).name);
			daten.append(" - ");
			daten.append(arr.get(i).nr);
			daten.append("\n");
		}
		return daten.toString();
	}
}

Und in der main die ich jetzt nicht gepostet habe, ruf ich wieder die toString auf.

Ist der Konstruktor in meiner Person.java richtig? Übergeb ich da die Arraylist richtig? Weiß ned wie ichs sonst tun sollte..
Gibts da ned irgendwo ne Seite wo das gut erklärt wird..

 

[Saxony]

Na ich versteh den Sinn der ArrayList von Personen innerhalb einer Person noch nicht ganz!
Verwenden tust du eh nur eine Person!
In der Klasse Book gibt es zwar eine Methode add, aber die macht was ganz anderes als der Name vermuten lässt.
Sie setzt die EINE Person von Book, klingt aber so als ob sie jedesmal eine neue Person hinzufügt.

bye Saxony

 

[diggaa1984]

machen wir mal so:

import java.util.ArrayList; 

public class Adressbuch{ 
   ArrayList<Person> bekannte = new ArrayList<Person>(); 
    
   public Adressbuch(){ 
   } 

   public void add(String name, String nr){ 
      p = new Person(name, nr); 
      bekannte.add(p)
   } 

   public String toString(){ 
      StringBuffer daten= new StringBuffer(); 
      int size = bekannte.size(); 

      for(int i=0; i < size; i++){ 
        daten.append(bekannte.get(i).getName()); 
        daten.append(" - "); 
        daten.append(bekannte.get(i).getNr()); 
        daten.append("\n"); 
      } 

      return daten.toString();
   } 
}

public class Person {
   
   private String name;
   private String nr;

   public Person(String name, String nr) {
      this.name = name;
      this.nr = nr;
   }

   public String getName() {
      return name;
   }

   public String getNr() {
      return nr;
   }

   public void setNewNr(String nr) {
      this.nr = nr;
   }

   //finde ich aus realistischer Sicht nicht zwingend notwendig, wenn dann äusserst selten
   public void setNewName(String name) {
      this.name = name;
   }

   //alternativ ein toString, dann müsstest aber das in dem Adressbuch umbauen
}

 

[krackmoe]

Danke!

Wozu brauch ich jetzt eigentlich noch die Person.java? Die hat ja gar keine Funktion mehr, da sowieso alles in der ArrayList drin steht, oder?

 

[Final_Striker]

ohne Personen gibt es keine ArrayList mit Personen!

 

[diggaa1984]

Wozu brauch ich jetzt eigentlich noch die Person.java? Die hat ja gar keine Funktion mehr, da sowieso alles in der ArrayList drin steht, oder?

So nicht ganz korrekt, die ArrayList speichert nicht die Personendaten, sondern gibt dir lediglich den Verweis auf die eigentlichen Personenobjekte mit den Infos. Die ArrayList kennt also nur nen Ort, wo die Personeninfos stehen. Den gibt sie dir zurück, und dann kannst dort quasi schaun was die Person für Daten inne hat.

du willst Daten für Personen speichern, da liegt es schonmal nahe, das Person also eine Klasse wird, welcher du nur die Attribute verpasst die du von ihr speichern willst ... name, adresse, diverse nummern, augenfarbe was nich alles. Mehr macht die Personenklasse nicht. Quasi wie n Ausweis.

So nun willst du mehrere Personen organisieren, bzw im Programm festhalten, brauchen wir also wie du schon erkannt hast zB eine Liste. Nur diese Liste hat nix mit einer einzelnen Person zu tun, wie und wo diese einzelne Person genutzt und referenziert wird, ist dieser egal (Ob du bei deinen Bekannten oder Verwandten im Adressbuch stehst weisst ja auch nicht direkt).

Also legen wir diese Liste ausserhalb der Person.java an, und schon nenn ich das mal Datencontainer. Dieser beinhaltet die Liste und Methoden welche du brauchst um was mit der Liste anzustellen (wie zB die "toString", welche dir alle Personen der Reihe nach ausgibt).

Wenn wir dabei bleiben das deine Personen-klasse Ausweis-infos beinhaltet dann wäre die Containerklasse vielleicht dein Portemonee (schreibt man das so?? ???:L ) ... ist nich ganz perfekt das Beispiel, aber andere Personeninfos würden dann andere Ausweise/Karten in deinem Portm. darstellen.

 

[Saxony]

Portemonee (schreibt man das so?? ???:L )

Ja! Alternativ auch Portemonnaie.

bye Saxony