Konstruktor mit Datum

[Robotron]

Hallo,


ich bin Java-Rookie und möchte Objekte anhand einer Klasse erzeugen, die jeweils ein objektspezifisches Datum enthalten: und zwar eine Klasse "Mitarbeiter" (im Projekt Unternehmen) soll für jeden Mitarbeiter nicht nur Name, Abteilung und Beruf liefern, sondern auch das Einstellungsdatum.


Für die Objekterzeugung habe ich in der Klasse "Mitarbeiter" die folgenden Variablen festgelegt:


public int Tag;


public int Monat;


public int Jahr;


protected int Personalnummer = Mitarbeiter_Anzahl + 1;


protected String Name;


protected String Abteilung;


protected String Beruf;


protected LocalDate Einstellungsdatum = LocalDate.of(Tag, Monat, Jahr);


(Die Mitarbeiter_Anzahl für die Generierung der Personalnummer wird aus einem Interface implementiert.)


Für die Objekterzeugung habe ich dann einen Konstruktor in der Klasse "Mitarbeiter" erzeugt:


public Mitarbeiter() {


this.Personalnummer = Mitarbeiter_Anzahl ++;


this.Name = Name;


this.Abteilung = Abteilung;


this.Beruf = Beruf;


this.Einstellungsdatum = Einstellungsdatum;}






Getter und Setter habe ich erzeugt, zur besseren Übersicht lasse ich diese hier weg.


Die folgenden Importe habe ich in der Klasse "Mitarbeiter" gesetzt:


import java.time.LocalDate;


import java.time.Month;


import java.time.Year;


import java.time.format.DateTimeFormatter;






In der Hauptklasse möchte ich in der Main-Methode wie folgt Objekte der Klasse Mitarbeiter erzeugen:


Mitarbeiter mitarbeiter1 = new Mitarbeiter();


geselle1.setName("Alfred Altmann");


mitarbeiter1.setAbteilung("Fertigung");


mitarbeiter1.setBeruf("Industriemechaniker");


mitarbeiter1.setEinstellungsdatum(01,07,2012);






Leider klappt das so nicht.


Wahrscheinlich muss ich beim Konstruktor auch die Parameter (Tag, Monat, Jahr) irgendwie berücksichtigen.


Hinzu kommt ja dann noch die Formatierung des Einstellunsdatums, so dass in meinem Beispiel der 01.07.2012 ausgegeben werden soll.


Kann mir bitte jemand dabei weiterhelfen, wie ich Objekte der Klasse "Mitarbeiter" mit einem Einstellungsdatum erzeugen kann bzw.wie man die Parameter Tag, Monat, Jahr im Konstruktor berücksichtigt?


Für Hilfe wäre ich sehr dankbar.

 

[MJannek]

zuerst wäre es mal gut zu wissen, welche Fehlermeldungen du bekommst. Was mir beim ersten daraufschauen aufgefallen ist, dass du beim Aufruf von

LocalDate.of(Tag, Monat, Jahr);

Tag und Jahr vertauschen musst. und ich würde es wie folgt machen

protected LocalDate Einstellungsdatum;

public void setEinstellungsdatum(int tag, int monat, int jahr){
    this.Einsteluungsdatum = LocalDate.of(jahr, monat, tag);
}

Wenn du das gleich beim initialisieren so definierst, kommt eine Fehlermeldung, da Tag, Monat und Jahr noch nicht mit Werten gewüllt wurden. Wenn du deinen Konstuktor in Mitarbeiter so verwenden möchtest musst du diesem noch in der Definition die Werte für Name, Abteilung, Beruf und Einstellungsdatum übergeben. Außerdem würde ich die Variablen im Camelcase (einstellungsDatum) oder Snakecase (einstellungs_Datum) benennen

 

[KonradN]

Also wenn ich da deinen Codeausschnitte ansehe, dann wäre meine erste Empfehlung: Such Dir einen Grundlegenden Ansatz an um Java von Grund auf zu lernen. Per Buch wäre hier z.B. "Java von Kopf bis Fuß" eine gute Wahl. Aber evtl. sind da auch einfach viele Copy & Paste Fehler, so dass mein Eindruck komplett falsch war.

Aber schauen wir einmal die Punkte an, die mir so ins Auge gefallen sind:

1. Code bitte in Code-Tags posten. Das geht am einfachsten über den Knopf </> links oberhalb des Eingabefeldes für Deinen Post.

2. Keine Parameter beim Konstruktor und vermutlich auch unklar, was die Parameter vom Setter beim Einstellungsdatum sind.

public Mitarbeiter() {


this.Personalnummer = Mitarbeiter_Anzahl ++;


this.Name = Name;


this.Abteilung = Abteilung;


this.Beruf = Beruf;


this.Einstellungsdatum = Einstellungsdatum;}

Der Konstruktor Mitarbeiter hat keine Parameter. Daher dann später der Aufruf ohne Argumente. Und die Zuweisungen bringen dann natürlich auch (fast) alle nichts.
Da z.B. die Instanzvariable Name nicht durch einen Parameter Name verdeckt wird, sind this.Name und Name gleich. Der Inhalt der Instanzvariable Name wird der Instanzvariable Name zugewiesen ....

Der Setter vom Einstellungsdatum wird vermutlich nur ein Parameter nehmen vom Typ LocalDate. Wenn Du das nicht zu drei Parametern ändern willst wie von MJannek gezeigt, kannst Du beim Aufruf auch direkt das LocalDate erzeugen. Also etwas wie setEinstellungsDatum(LocalDateOf(jahr, monat, tag));

3. Was machst Du da mit einem Interface?

(Die Mitarbeiter_Anzahl für die Generierung der Personalnummer wird aus einem Interface implementiert.)

Das braucht evtl. noch eine gewisse zusätzliche Erläuterung. Es klingt erst einmal dubios. Typisch wäre eigentlich etwas wie eine statische Variable Mitarbeiter_Anzahl

4. Reihenfolge der Initialisierung!

public int Tag;


public int Monat;


public int Jahr;


// ...


protected LocalDate Einstellungsdatum = LocalDate.of(Tag, Monat, Jahr);

Tag, Monat und Jahr werden nicht direkt initialisiert. Dadurch erfolgt bei Einstellungsdatum immer eine Initialisierung bon LocaleDate.of(0, 0, 0).
(Das mit der Reihenfolge der Parameter wurde ja schon genannt - Jahr Monat Tag ist die Reihenfolge!)

Wichtig ist: Das läuft einmal durch. Wenn dann später Tag, Monat oder Jahr verändert werden, dann verändert sich das Einstellungsdatum nicht.

Dann paar generelle Hinweise - die aber keine wirklichen Fehler sind

5. Die übliche Namensgebung bei Instanzvariablen und Parametern ist das Lower Camel Case. Also tag, monat, einstellungsDatum, ....
Also erster Buchstabe klein und bei zusammengesetzten Wörtern die jeweils folgenden "Anfangsbuchstaben" groß.

6. Keine Werte mehrfach speichern. Wenn Du das Einstellungsdatum in einem LocalDate speicherst, dann solltest Du da nicht zusätzlich Tag, Monat und Jahr vorhalten. Doppelt gespeicherte Daten können leicht mal auseinander gehen und dann hat schwer zu findende Fehler.