Fehler Calendar

[nick_10]

Hallo

Ich bekomme folgende Fehlermeldung beim Ausführen:

Exception in thread "main" java.lang.NullPointerException
at bk.Contact.setBirthDate(Contact.java:77)
at bk.TestDriver.main(TestDriver.java:9)


Hier der Code (Ich lass mal alles Überflüssige weg):

package bk;

import java.util.Calendar;
import java.lang.StringBuilder;
import java.text.SimpleDateFormat;

public class Contact {

	private String firstName;
	private String lastName;
	private Calendar birthDate;
	private String phoneNumber;
	private String eMail;
	private int idNumber;
	//address initialize all together
	private String street;
	private int streetNumber;
	private int postCode;
	private String residence;
	//
	private static int counter = 10000;
	private StringBuilder builder = new StringBuilder();
	
	//Constructor
	public Contact(String firstName, String lastName) {
		this.firstName = firstName;
		this.lastName = lastName;
		idNumber = counter;
		counter++;
	}

	public String toString() {
		
		//appends idNumber
		builder.append("ID Number: " + "\t" + idNumber + "\n");
		
		//appends name
		builder.append("First Name: " + "\t" + firstName + "\n");
		builder.append("Last Name:  " + "\t" + lastName + "\n");
		
		//appends birthDate
		if (birthDate == null) {}
		else builder.append("Birthday: " + "\t" + birthDateToString() + "\n");
		
		//appends phoneNumber
		if (phoneNumber == null) {}
		else builder.append("Phone Number: " + "\t" + phoneNumber + "\n");
		
		//appends eMail
		if (eMail == null) {}
		else builder.append("E-Mail: " + "\t" + eMail + "\n");
		
		//appends address
		if (street == null) {}
		else {
			builder.append("Street : " + "\t" + street + " " + streetNumber + "\n");
			builder.append("Residence: " + "\t" + postCode + " " + residence + "\n");
		}
	
		return builder.toString();
	}
	
	//helper class for toString, converts Date birthDate to String stringDate
	private String birthDateToString() {
		String stringDate;
		SimpleDateFormat sdf = new SimpleDateFormat();
		stringDate = sdf.format(birthDate);
		return stringDate;
	}
	
	
	/*	Accessor / Mutator
	*	address can only be initialized all together
	*	idNumber can not be changed
	*/
	public void setBirthDate(int initialDate, int initialMonth, int initialYear) {
		birthDate.set(initialYear, initialMonth, initialDate); //<-Fehler
	}
	
	public String getBirthDate() {
		return birthDateToString();
	}

}

package bk;


public class TestDriver {

	public static void main(String[] args) {
	
		Contact me = new Contact("Hans", "Muster");
		me.setBirthDate(1, 1, 1988); <- Fehler
		System.out.println(me.toString());
	}
}

Danke und Grüsse

 

[SlaterB]

birthdate ist null, welch unerwartete Ursache,
Variablen muss man doch erst mit Objekten befüllen?

Contact me = null;
me.setBirthDate(1, 1, 1988); 

statt

Contact me = new Contact("Hans", "Muster");
me.setBirthDate(1, 1, 1988); <- Fehler

bräche dir noch eher eine NullPointerException

Calendar.getInstance() kannst du zum Befüllen verwenden, oder sauberer new GregorianCalendar()

 

[nick_10]

Danke für deine antwort. Aber ich habs noch nicht ganz. Wenn ich den TestDriver so laufen lasse...

public class TestDriver {

	public static void main(String[] args) {
	
		Contact me = new Contact("Hans", "Muster");
		me.setPhoneNumber("123456789");
		System.out.println(me.toString());
	}
}

Console:
ID Number: 10000
First Name: Hans
Last Name: Muster
Phone Number: 123456789
--------------------------------
...gehts ja auch. Ich erstelle zuerst eine Objekt "me", dem ich firstName und lastName übergebe, dann führe ich setPhoneNumber() aus, womit ich phoneNumber einen Wert zuweise.
Das sollte doch analog mit birthDate auch gehen?

 

[bERt0r]

Du instanzierst dein birthdate im Konstruktor nicht:

public Contact(String firstName, String lastName) {
        this.firstName = firstName;
        this.lastName = lastName;
        idNumber = counter;
        counter++;
    }

Das heisst, deine Variable birthDate ist null, nun Versuchst du aber die Funktion set dieser Variable auszuführen und das geht nicht.

  public void setBirthDate(int initialDate, int initialMonth, int initialYear) {
        birthDate.set(initialYear, initialMonth, initialDate); //<-Fehler
    }

Bei deiner Phone Number sieht deine setPhoneNumber Funktion wahrscheinlich so aus:

public void setPhoneNumber(String number)
{
phoneNumber=number;
}

Merkst du den Unterschied? Hier machst du eine Zuweisung, es ist völlig egal welchen Wert phoneNumber vorher hatte. Beim BirthDate versuchst du aber in einem Objekt das bereits existieren muss Daten zu ändern.

 

[Xeonkryptos]

Beim BirthDate versuchst du aber in einem Objekt das bereits existieren muss Daten zu ändern.

Das ist wie mit der 0 in der Mathematik:
0 * irgendeine Zahl = 0!
0 / irgendeine Zahl = 0!

So ist es in der Programmierung genauso:
Du versuchst NICHTS (dein Objekt) mit irgendetwas zu ändern und das Ergebnis ist im Endeffekt nichts außer einer Exception, die dich darauf aufmerksam macht!

Wenn du es initialisieren würdest, dann hättest du nicht mehr nichts, dein Objekt eine Referenz (als hättest du ihm eine Zahl oder so zugewiesen) und könntest dieses auch verändern.

BTW: Mir ist bewusst, dass eine Referenz keine Zahl im eigentlichen Sinne ist, aber ich möchte gleich klarstellen, dass es nur als Verdeutlichung hier zu dienen hat.