Frage zu Konstruktoren

[Guest]

Hallo, habe folgendes Problem:

Ich muss eine Klasse implementieren, die fast nur aus Konstruktoren besteht.

Folgendes Problem:
Bei folgendem Code macht er einfach die Abfrage nicht richtig.
D.h. es kommt immer false raus, obwohl ich weiß, dass das Bbjekt mit der Eigenschaft spouse = null erschaffen wurde.
=> NullPointerException.

if(this.getChild(i).spouse == null){

Muss ich die Abfrage anders machen, oder ist dies gar nicht möglich?

Vielen Dank im Vorraus.

 

[SlaterB]

möglich ist sie, probiere folgendes:

System.out.println("child: "+this.getChild(i));
if (this.getChild(i) == null) {
   System.out.println("aus die Maus");
} else {
   System.out.println("spouse : "+this.getChild(i).spouse );
   if (this.getChild(i).spouse == null) {
       System.out.println("aus die Maus");
    } else {
       System.out.println("supi, ist nicht null");
    }
}

 

[Guest]

Vlt. sollte ich noch erwähnen, dass spouse ein Objekt vom Typ Person ist.

Private Person spouse;

Person(String n){
nachname = n;
father = null;
mother = null;
spouse = null;
}

Liegt es daran? Kann ich keiner Person eine null zuweisen?

 

[SlaterB]

> Kann ich keiner Person eine null zuweisen?

-> Kann ich keiner Person-Variable den Wert null zuweisen?

doch, geht, probiere bitte meinen Code aus und alles wird klar
(oder poste die Ausgaben)

 

[lukian]

Hi,

das Problem ist, dass Du und nur einen Mini-Ausschnitt deines Codes zeigst und wir herumraten müssen. Das kleine i in deiner Codezeile lässt mich vermuten, dass du eine for-Schleife drumherumgebastelt hast und irgendeine Instanz von spouse wird einfach != null sein.

Gruß,
lukian

 

[Guest]

Okay, dann poste ich hier mal den ganzen Code:
Habe das Problem anders gelöst, indem ich eine weitere Variable verwendet habe (boolean spoused)!
Allerdings hätte ich doch gerne gewusst, obs auch ohne geht. {Person spouse = null; if(spous == null)}

Weiteres Problem an der Sache: Ich habe das Person[], allerdings muss ich Person[] = Person zuweisen. Deshalb das rumgewurschtel mit der Hilfsvariable und dem vorinitialisiertem Feld.
Habe noch keine Vektoren behandelt!

Und falls jemand Ideen hat, die "Sippe" einfacher auszugeben, bitte nicht zurückhalten!

/**
 * Private Objektvariablen, öffentliche Konstruktoren u. Methoden
 * @author 
 *
 */
public class Person {
	
	private int childs = 0;
	private boolean spoused = false;
	private String[] vornamen;
	private String vorname;
	private String nachname;
	private Person[] children = new Person[10];
	private Person father;
	private Person mother;
	private Person spouse;

	Person(String n){
		nachname = n;
		father = null;
		mother = null;
	}
	
	Person(String[] gs, String n){
		this (n);
		vornamen = gs;
	}
	
	Person(String g, String n){
		this(n);
		vorname = g;
	}
	
	Person(Person f, Person m){
		nachname = m.nachname;
		father = f;
		mother = m;
		f.children[++f.childs] = this;
		m.children[++m.childs] = this;
	}
	
	Person(String[] gs, Person f, Person m){
		this(f, m);
		vornamen = gs;
	}
	
	Person(String g, Person f, Person m){
		this(f, m);
		vorname = g;
	}
	
	// Alle Vornamen
	String[] getGivenNames(){
		return this.vornamen;
	}
	
	// Ersten Vornamen
	String getGivenName(){
		return this.vorname;
	}
	
	// Nachname
	String getName(){
		return this.nachname;
	}
	
	// Vater / Mutter / Ehepartner abfragen (Resultatwert ggf. null).
	Person getFather(){
		return this.father;
	}
	
	Person getMother(){
		return this.mother;
	}
	
	Person getSpouse(){
		return this.spouse;
	}
	
	// Anzahl Kinder
	int getNumChildren(){
		return this.childs;		
	}
	
	// i-tes Kind
	Person getChild (int i){
		return this.children[i];
	}
	
	// Aktuelle Person heiratet Person s; s nimmt den Nachnamen von this an.
	void marry (Person s){
		this.spouse = s;
		s.spouse = this;
		s.nachname = this.nachname;
		this.spoused = true;
		s.spoused = true;
	}
	
	// Vornamen und Nachname in einer Zeile ausgeben.
	void print (){
		System.out.print("\t" + this.vorname + " " + this.nachname + "   ");
	}

Hier bastel ich grad rum:

	void printFam(){
		if(this.spoused == true){
			System.out.print("\n\t\t\t" + this.spouse.vorname + " " + this.spouse.nachname + "\n\t\t");
		}
				
		if(this.getNumChildren() > 0){
			for(int i=1;i<=this.getNumChildren();i++){
				System.out.print(this.getChild(i).vorname + " " + this.getChild(i).nachname+ "  ");
				}
			for(int i=1;i<=this.getNumChildren();i++){
				this.getChild(i).printFam();
			}
		}
	}
	
	// Gesamte "Sippe" ausgeben: eigene Namen, ggf. Namen des Ehepartners,
	// ggf. (sukzessive eingerückt) Namen aller Kinder (ggf. mit Ehepartnern),
	// Kindeskinder etc.
	void printFamily (){
		System.out.print(this.vorname + " " + this.nachname);
		System.out.print("\n\t" + this.spouse.vorname + " " + this.spouse.nachname + "\n\t\t");
		
		if(this.getNumChildren() > 0){
			for(int i=1;i<=this.getNumChildren();i++){
				System.out.print(this.getChild(i).vorname + " " + this.getChild(i).nachname+ "  ");
				}
			for(int i=1;i<=this.getNumChildren();i++){
				this.getChild(i).printFam();
			}
		}
	}

	public static void main(String[] args){
		
		// Personenerzeugung
		Person p1 = new Person("Peter", "Holz");
		Person p2 = new Person("Anna", "Wein");
		Person p3 = new Person("Torsten", p1, p2);
		Person p4 = new Person("Flo", p1, p2);
		Person p5 = new Person("Hans", p1, p2);
		Person p6 = new Person("Sabine", "Sabienchen");
		Person p7 = new Person("Klaudia", "Ludolf");
				
		// Heirat, etc.
		p1.marry(p2);
		p5.marry(p6);
		p3.marry(p7);
		
		
		// Ausgabe
		p1.printFamily();
		//p1.print();
		//p2.print();
		//p3.print();
		//p4.print();
		//p5.print();
		
		//p1.getSpouse().print();
		//p2.getSpouse().print();
		//p3.getSpouse().print();
		//p4.getSpouse().print();
		
		//System.out.println(p1.getNumChildren());
		//System.out.println(p2.getNumChildren());
		//System.out.println(p3.getNumChildren());
		//System.out.println(p4.getNumChildren());
		
		//p1.getChild(p1.getNumChildren()).print();
		//p1.getChild(p1.getNumChildren()-1).print();
		
		//p2.getChild(1).print();
		//p2.getChild(2).print();
		//p2.getChild(3).print();
	}
}

[/code]

 

[SlaterB]

> obs auch ohne geht. {Person spouse = null; if(spous == null)}

geht,
wenn du schon so viele Code postest, dann doch noch mal die alte Version
und benenne die Stelle, an der es nicht geht (nicht zu gehen scheint)

 

[Guest]

Ist eigentlich nicht so viel verändert, ich greife nur auf die andere print Funktion zu und bei der Erstellung der Person wird spouse = null gesetzt.

public class Person {
	
	private int childs = 0;
	private String[] vornamen;
	private String vorname;
	private String nachname;
	private Person[] children = new Person[10];
	private Person father;
	private Person mother;
	private Person spouse;

	Person(String n){
		nachname = n;
		father = null;
		mother = null;
spouse = null;
	}

Person(String[] gs, String n){
		this (n);
		vornamen = gs;
	}
	
	Person(String g, String n){
		this(n);
		vorname = g;
	}
	
	Person(Person f, Person m){
		nachname = m.nachname;
		father = f;
		mother = m;
		f.children[++f.childs] = this;
		m.children[++m.childs] = this;
	}
	
	Person(String[] gs, Person f, Person m){
		this(f, m);
		vornamen = gs;
	}
	
	Person(String g, Person f, Person m){
		this(f, m);
		vorname = g;
	}
	
	// Alle Vornamen
	String[] getGivenNames(){
		return this.vornamen;
	}
	
	// Ersten Vornamen
	String getGivenName(){
		return this.vorname;
	}
	
	// Nachname
	String getName(){
		return this.nachname;
	}
	
	// Vater / Mutter / Ehepartner abfragen (Resultatwert ggf. null).
	Person getFather(){
		return this.father;
	}
	
	Person getMother(){
		return this.mother;
	}
	
	Person getSpouse(){
		return this.spouse;
	}
	
	// Anzahl Kinder
	int getNumChildren(){
		return this.childs;		
	}
	
	// i-tes Kind
	Person getChild (int i){
		return this.children[i];
	}
	
	// Aktuelle Person heiratet Person s; s nimmt den Nachnamen von this an.
	void marry (Person s){
		this.spouse = s;
		s.spouse = this;
		s.nachname = this.nachname;
	}

Ich will hier den Ehepartner ausgeben, falls vorhanden.

	void print (){
		if(this.spouse != null){
			System.out.print(this.spouse.vorname + " " + this.spouse.nachname);
		}
	}

public static void main(String[] args){
		
		// Personenerzeugung
		Person p1 = new Person("Peter", "Holz");
		Person p2 = new Person("Anna", "Wein");
		Person p3 = new Person("Torsten", p1, p2);
		Person p4 = new Person("Flo", p1, p2);
		Person p5 = new Person("Hans", p1, p2);
		Person p6 = new Person("Sabine", "Sabienchen");
		Person p7 = new Person("Klaudia", "Ludolf");
				
		// Heirat, etc.
		p1.marry(p2);
		p5.marry(p6);
		p3.marry(p7);
		
		
		// Ausgabe
		p1.printFamily();
		//p1.print();
		//p2.print();
		//p3.print();
		//p4.print();
		//p5.print();
		
		//p1.getSpouse().print();
		//p2.getSpouse().print();
		//p3.getSpouse().print();
		//p4.getSpouse().print();

 

[lukian]

Tja,

es kann so einfach sein. Leider besitzt die Klasse Person viele Konstruktoren (vielleicht etwas zu viele?) und Du setzt spouse dummerweise genau in dem Konstruktor auf null, den du gar nicht benutzt, um Instanzen zu erstellen :shock:

Gruß,
lukian

 

[Guest]

Ok, im Code fehlt bei

 Person(Person f, Person m){
      nachname = m.nachname;
      father = f;
      mother = m;
      f.children[++f.childs] = this;
      m.children[++m.childs] = this;
   }

das spouse = null;

Aber wenn ich einfach in meinem Code p1=New Person("test", "testchen");

und p1.spouse().print(); aufrufe, wird ja

   Person(String g, String n){
      this(n);
      vorname = g;
   }

aufgerufen. Und this(n) ruft ja

 Person(String n){
      nachname = n;
      father = null;
      mother = null;
spouse = null;
   }

auf. Der fehler kommt trotzdem!

Überseh ich grad was?!?

 

[SlaterB]

zunächst mal:
spouse = null;
im Konstruktor kannst dir sparen, wenn kein Objekt gesetzt wird, dann ist die Variable automatisch null,

-------

und nun bitte endlich mal ein Beispiel..
deine Ausgaben haben bei mir alle geklappt oder es gab ne NullPointerException z.B. bei
p4.getSpouse().print();
weil p4 ja nicht verheiratet ist,

dein neuestes Beispiel
p1=New Person("test", "testchen");
p1.spouse().print();
ist wieder so wirr,
eine Operation spouse() gibts gar nicht und wenn p1 nicht verheiratet ist, dann wird
p1.getSpouse().print();
selbstverständlich wieder eine NullPointerException geben

ich sehe nirgendwo die Ursprungsfrage:
> es kommt immer false raus, obwohl ich weiß, dass das Bbjekt mit der Eigenschaft spouse = null erschaffen wurde
> if(this.getChild(i).spouse == null){

 

[lukian]

Ja,
denn bei den vielen Konstruktoren würde ich auch nicht mehr durchsteigen. :wink: Thorsten, Flo und Hans machen Dir das Leben schwer, da die in einem Konstruktor erzeugt werden, der wiederum einen Konstruktor aufruft, der leider spouse NICHT auf null setzt. Puh... was für ein durcheinander.

[edit] SlaterB hat völlig recht und ich lag falsch.

Gruß,
luk

 

[Guest]

Okay, mein Code ist wirr.

Aber das hat damit zu tun, dass mein Prof. Codedopplung hasst (Punktabzug). Normalerweise würde ich auch lieber n bisschen mehr Code schreiben, um die Übersicht zu behalten. :wink:
Desweiteren habe ich schon weiterprogrammiert und meinen alten Code teilweise überschrieben (=> Wirr²!).

Aber da ihr ja meint, dass dies generell funktioniert, danke ich euch für die Hilfe und werde es später nochmal neu schreiben und ausprobieren.

So, nochmals vielen Dank und noch einen schönen Abend.