OOP Kleines Schülerverwaltungsprogramm

[jensWolf]

Hey bin neu in der Java-OOP-Programmierung,

um mich ein wenig zu verbessern, habe ich mir folgendes Szenario überlegt, um zu üben:

- Es gibt einen Schüler mit Namen und Alter
- Jeder Schüler gehört zu einer Schulklasse. Diese hat einen Namen und verwaltet die Schüler. Dabei muss jede Klasse einzigartig sein.
- Des Weiteren gibt es mehrere Jahrgangsstufe. Diese besteht aus einem Integer.
- Insgesamt gibt es dann noch die Schule, die unteranderem die Anzahl der Schüler beinhaltet

So als Software Design habe ich mir überlegt:

• Klasse Schule
  • HasListe mit den Jahrgängen
• Klasse Jahrgang
  • Integer für die Jahrgangsnummer
  • Hashliste mit den Klassen
• Klasse Schulklasse
  • ArrayListe mit Schülern
  • Namen für die Klasse
• Klasse Schüler
  • Name des Schülers
  • Alter des Schülers

Das wäre so mein grobes Design. Ist das ok so oder gibt es vielleicht Verbesserungsvorschläge? Die Hashliste habe ich genommen, damit die Elemente Klasse und Jahrgänge einzigartig sind oder gibt es hier eine bessere Möglichkeit?

Des Weiteren habe ich folgende Frage: Ich würde gerne die Anzahl der Schule ermitteln. Ist es hier geschickt diese über die Elemente Liste Jahrgang dann Liste der Klassen und dann über die Listengröße der Schüler zu ermitteln oder für jedes Objekt zusätzlich die größe der Gruppe zu speichern?

Wie gehe ich jetzt am besten vor wenn ich einen neuen Schüler erstellen möchte? Verwalte ich diese Erstellung am besten in der Klasse Schule?

 

[Jardcore]

Hey cool das du hier bist.

um?

 

[jensWolf]

Entschuldige. War etwas zu schnell

 

[Java20134]

Also ich finde den Aufbau gut. Ich weiß nicht ob du unter einer HashList eine HashMap verstehst, aber ich würde eine HashMap nehmen, da du dort die Einzigartigkeit und zudem auch noch der Zugriff auf die Elemente über den Jahrgang bspw gegeben ist.

Und das Hinzufügen eines Schülers würde ich allgemein in der Klasse Schule machen, aber zudem auch Methoden in den Jahrgangsstufen und Klasse hinzufügen! In der Schule müsste dann die Methode wie folgt aussehen:

public void addStudent(int year, int clazz){
     //TODO zum Jahrgang und Klasse hinzufügen u.a.
}

Überlegenswert wäre es vllt. noch, ob du in der Klasse Schulklasse statt der ArrayList eine HashMap nehmen könntest. Dabei könnte man als key den Namen und Vornamen des Schülers nehmen. Dadurch wäre der Zugriff auf die Daten etwas schneller, als wenn du dann nach jedem Schüler suchen müsstest.

Ob du dann die HashMap nimmst oder ein HashSet ist Dir überlassen, je nachdem wie es Dir besser gefällt. Und dann kann man sich auch noch überlegen, ob man alle Variablen braucht, wenn man eine HashMap nutzt.

 

[Jardcore]

Schule könnte noch ein Sekretariat haben und dafür sorgen das Schüler eingeschult werden
Schule könnte dann eine einschul()-Methode haben, welche das Sekretariat beauftragt einen Schüler in die Schule aufzunehmen. Das Sekretariat möchte dann nur ein Einschulformular, welches den Vor- und Nachnamen des Schülers, den Jahrgang und natürlich die Klasse enthält. Das Einschulformular wäre eine neue Klasse, die einfach Daten beinhaltet, damit du nicht Methoden mit n Parametern hast.

Wenn du die Anzahl der Schulen haben willst, könntest du eine statische Variable in Schule anlegen und diese im Konstruktor hochzählen. Und mit einer statischen Methode (getAnzahl()) kannst du die Anzahl abfragen.

 

[Tobse]

Wenn du die Anzahl der Schulen haben willst

Ich dachte, der TE will die Anzahl der Schüler bei der Schule?
@TE: Anmerkung hierzu: nimm blos keine Variable int anazahlSchüler in die Schule auf o.ä. Diese Variable auf dem richtigen Wert zu halten ist ein mega aufwand und total unnötig. Lieber einfach sowas:

public int getAnzahlSchueler() {
    int n = 0;
    for (Klasse klasse : klassen) {
        n += klasse.getSchueler().size();
    }
    return n;
}

 

[Jardcore]

Ich würde gerne die Anzahl der Schule ermitteln.

Ich mach nur das was dort steht

TE: Anmerkung hierzu: nimm blos keine Variable int anazahlSchüler in die Schule auf o.ä. Diese Variable auf dem richtigen Wert zu halten ist ein mega aufwand und total unnötig. Lieber einfach sowas:

Btw. man kann das aber auch hier durch eine statische Variable lösen und jedes Mal wenn ein Schüler zu einer Schule hinzugefügt wird diese erhöhen. Dann muss man nicht ständig iterieren.

Und mega Aufwand? Du hast eine Schüler hinzufüge Methode und eine Schüler entferne Methode... anzahlSchüler++... anzahlSchüler--. Das ist nicht gerade aufwendig finde ich^^... und O(1)

Meinetwegen kann man auch eine normale Variable mit einem Getter anlegen.

 

[Tobse]

Ich mach nur das was dort steht

Stimmt, hatte ich überlesen, sorry

Und mega Aufwand? Du hast eine Schüler hinzufüge Methode und eine Schüler entferne Methode... anzahlSchüler++... anzahlSchüler--. Das ist nicht gerade aufwendig finde ich^^... und O(1)

Ja, kann man, ist letztendlich Geschmackssache. Aber: jetzt kommt jemand und macht sowas: klasse.getSchueler().add(new Schueler("Hans", "Peter")); Um das zu verhindern muss man jetzt die nach außen gegebene Liste schützen. Selbiges gilt für schule.getKlassen().add(new Klasse("4b")).
Die Lösung hierfür, welche ich nutze, ist, Zugriff aut Iterator<Schueler> und Iterator<Klasse> zu geben und die Listenobjekte nicht nach außen zu geben. Für so ein Übungsprojekt ist dann IMO die berechnende Methode der deutlich kleinere Schmerz... (und sie ist auch unmöglich auszutricksen).

 

[Jardcore]

Sowas soll man nicht machen ^^ Listen sollten generell in den meisten Fällen nicht nach außen sichtbar sein. ("generell" und "in den meisten Fällen" in einem Satz klingt nicht sehr glaubwürdig... ich weiß )

 

[JAVAKEK]

Ich mach nur das was dort steht
Btw. man kann das aber auch hier durch eine statische Variable lösen und jedes Mal wenn ein Schüler zu einer Schule hinzugefügt wird diese erhöhen. Dann muss man nicht ständig iterieren.

Und mega Aufwand? Du hast eine Schüler hinzufüge Methode und eine Schüler entferne Methode... anzahlSchüler++... anzahlSchüler--. Das ist nicht gerade aufwendig finde ich^^... und O(1)

Meinetwegen kann man auch eine normale Variable mit einem Getter anlegen.

Falls eine HashMap oder ArrayList Implementierung genutzt wird, macht sie schon genau das was du hier beschreibst, nämlich eine interne Zähler-Variable verwalten. -> List.size() = O(1)

Sowas soll man nicht machen ^^ Listen sollten generell in den meisten Fällen nicht nach außen sichtbar sein. ("generell" und "in den meisten Fällen" in einem Satz klingt nicht sehr glaubwürdig... ich weiß )

Es gibt 3 Möglichkeiten:
- Zugriff auf die Liste von aussen und lass die Liste alles handlen
- Zugriff auf eine nicht modifizierbare Liste
- Zugriff auf Methoden, die die Liste intern modifizieren