Personalnummer automatisch generieren

[SteffiBee]

Hallo,

ich suche nach einer Möglichkeit, eine Personalnummer als int-Wert automatisch beim
erzeugen eines neuen Objekts (hier Mitarbeiter) zu generieren.

Wie es rein theoretisch funktioniert ist mir schon bewusst, umgesetzt habe ich sowas
bisher wie im folgenden Code:

[/B]
public abstract class Mitarbeiter implements IGehalt {

    private static int counter = 1000;
    private int personalNr;
    private String nachname;
    private String vorname;
    private Adresse adresse;
    private String telefon;
    private LocalDate geburtsdatum;
    private Firmenwagen firmenwagen;
    private Einsatzort einsatzort;
    private double stundenanzahl;
    private double stundenlohn;

    public Mitarbeiter(String nachname, String vorname, String telefon, LocalDate geburtsdatum, double stundenlohn) {

        this.personalNr = counter++;
        this.nachname = nachname;
        this.vorname = vorname;
        this.telefon = telefon;
        this.geburtsdatum = geburtsdatum;
        this.stundenlohn = stundenlohn;
        this.stundenanzahl = 0;
    }
[B]

Die Personalnummer soll beim ersten Mitarbeiter bei 1000 beginnen und für jeden weiteren eben um eins erhöht werden.
Jeder Mitarbeiter wird in einer Liste gespeichert. Diese Liste lässt sich speichern und bei erneutem Ausführen
des Programms auch laden, so dass die erstellten Mitarbeiter nach Programmende eben nicht verloren gehen.
Das ist auch soweit machbar.
Aber natürlich ist bei einem erneuten Programmstart die Variable Counter wieder bei 1000.
Somit kann ich zwar meine Mitarbeiter wieder laden, aber wenn ich jetzt neue hinzufüge, fängt
der Counter ja wieder bei 1000 an. Und genau das soll verhindert werden.

Es ist zwingend notwendig, dass jeder Mitarbeiter eine eindeutige nur einmal vergebene Personalnummer hat.

Mir ist mit meinem derzeitigen Anfängerstand bisher nur eingefallen, eventuell eine Liste mit Personalnummern
zu führen, der bei jedem neu erzeugten Mitarbeiter nur die Personalnummer hinzugefügt bekommt. Diese Liste
ebenfalls zu speichern und bei Programmstart das letzte Element (also die letzte vergebene Personalnummer)
der Variablen Counter zuzuweisen.

Das kommt mir sehr aufwendig vor, allerdings reichen meine Kenntnisse noch nicht sehr weit.
Gibt es eine effizientere Möglichkeit?

Für jegliche Hilfestellung wäre ich sehr dankbar.

Kleine Anmerkung: Dies ist mein erster Post in diesem Forum. Ich bemühe mich, mich an alle Regeln
hier zu halten. Sollte ich irgendetwas nicht richtig umgesetzt haben, bin ich für Hinweise offen

Liebe Grüße
Steffi

 

[Kirby.exe]

Du könntest beim Laden die letzte Personalnummer rufen (also die größte) und setzt diese als deine static variable

 

[SteffiBee]

Jaaa, hab auch schon an sowas gedacht. Bin nur irgendwie nicht fähig an die größte Personalnummer zu kommen

 

[Kirby.exe]

Naja da die Liste vermutlich von vorne bis hinte gefüllt wird, steht die Person mit der größten Personalnummer vermutlich am Ende Also am Index List.size()-1

 

[SteffiBee]

Das hört sich gut an. Vielen Dank!!!!!!!
Versuche ich mal.

 

[kneitzel]

Zwei Möglichkeiten:
a) Du speicherst auch den statischen Wert mit. Wenn Du also einen Stand lädst, dann lädst du auch diese Variable. Hat den Vorteil, dass beim Löschen von Mitarbeitern die Nummer nicht noch einmal vergeben wird.

b) Wenn Du die Mitarbeiter lädst, dann erzeugst Du ja auch Instanzen. Du hast also einen Konstruktor, der aufgerufen wird. In diesem Konstruktor kannst Du prüfen, ob die personalNr + 1 > counter ist. Ist dies der Fall, dann setzt du counter auf personalNr + 1.

Aber bei b) sehe ich eine mögliche Wiederverwendung. Wenn das ok ist, ist das natürlich kein Thema.
Also Szenario noch einmal zum Verständnis:
- Neuer Mitarbeiter bekommt persNr. x
- neuer Mitarbeiter wird gekündigt, daher fliegt er aus der Datenbank (evtl. gibt es aber noch Dokumente mit personalNr x!)
- speichern der Mitararbeiter
- laden der Mitarbeiter
- erneut neuer Mitarbeiter, persNr x wird wieder vergeben.

Generell ist daher a) zu empfehlen. Der Wert gehört also auch gespeichert.

 

[SteffiBee]

Richtig, die Personalnummer soll nicht noch einmal vergeben werden können.
Auch nicht, wenn der Mitarbeiter gelöscht wird.

Die Lösung a) finde ich super. Bin allerdings noch nicht ganz sicher, wie ich das Speichern der Variablen umsetzen soll.
Evtl. ganz einfach über eine txt-Datei, in der der Wert gespeichert wird?
Bin da noch nicht so ganz fit drin, habe das mit dem Laden und Speichern erst einmal in einem Praktikum umgesetzt.

 

[kneitzel]

Wie speicherst Du denn die Mitarbeiter derzeit? Dann könnte man dir anhand dessen aufzeigen, wie Du diese statische Variable mit speichern könntest.

 

[SteffiBee]

In etwa soll das so aussehen. Das waren die Methoden aus dem letzten Praktikum, die haben super
funktioniert und würde die gerne wieder verwenden

[/B]
public Kontakt[] laden() {

        File file = new File("kontakte.ser");

        try (FileInputStream fis = new FileInputStream(file); ObjectInputStream ois = new ObjectInputStream(fis)) {
            
            HashSet<Kontakt> kontakte = (HashSet<Kontakt>) ois.readObject();
            kontaktliste = kontakte;
            
            return kontakte.toArray(new Kontakt[kontakte.size()]);
            
        } catch (IOException | ClassNotFoundException e) {
            e.printStackTrace();
        }
        return new Kontakt[0];
    }

    public void speichern() {

        File file = new File("kontakte.ser");

        try (FileOutputStream fos = new FileOutputStream(file); ObjectOutputStream oos = new ObjectOutputStream(fos)) {

            oos.writeObject(kontaktliste);

        } catch (IOException e) {
            e.printStackTrace();
        }
    }
[B]

Nicht irritieren lassen von den anderen Bezeichnungen. Das waren jetzt die Methoden aus dem vorherigen Projekt.
Die Vorgehensweise ist aber identisch.

 

[kneitzel]

Ja danke, das reicht so schon. Du nutzt also ObjectInputStream / ObjectOutputStream.

Damit kannst Du auch einzelne int Werte speichern. Mit writeInt schreibst du ein int und mit readInt liest du diesen.

Wichtig ist die Reihenfolge: Wenn Du erst den counter in die Datei schreibst und dann die Mitarbeiter, dann muss es beim Lesen genau so sein: Also erst den counter lesen und dann die Mitarbeiter.

Was noch etwas unschön ist: Du musst natürlich counter verändern. Wenn Du in einer anderen Klasse das Lesen / Schreiben machst, dann benötigt diese Klasse Zugriff auf counter.

Generell gefällt mir so ein Ansatz nicht ganz so gut. Aber das liegt an der Nutzung der statischen Variable. Da dies aber so erst einmal gegeben ist, würde ich das so in der beschriebenen Art vorschlagen.

 

[SteffiBee]

Super, vielen Dank für die Infos und Hinweise.

Werde berichten, ob es funktioniert hat

 

[SteffiBee]

Hallo.

Der Hinweis, den int-Wert zu speichern war Gold wert. Danke dafür.
Hab es so jetzt umgesetzt und es funktioniert 1A.

Also nochmal: DANKEEEEEEEEEEEEEEE!