Vererbung Kleine Hilfe bei einer aufgabe

[QuiXotiK]

hey liebes java-forum,
ich hab da ein kleines problemchen bei einer meiner aufgaben.

ich habe folgendes uml erhalten:

Spoiler

dazu sollen folgende kriterien erfüllt werden:

Spoiler

3.1. Angaben des Kunden
 Der Sportverein hat einen Namen und eine zugewiesene Vereinsnummer
 Der Sportverein hat eine Postadresse (Straße, Hausnummer, PLZ, Ort), an die alle Briefe und Pakete (auch die der Abteilungen) gesendet werden
 Der Sportverein hat mehrere Abteilungen
 Jede Abteilung hat zusätzlich einen Namen (der sich aus der Sportart ergibt) und einen Abteilungsleiter
 Der Sportverein hat maximal 100 Mitglieder
 Jedes Mitglied kann in zu mehreren Abteilungen gehören (z.B. Fußball und Tischtennis)
 Jedes Mitglied hat einen Namen, einen Vornamen und eine eindeutige, fortlaufende Mitgliedsnummer im Verein
3.2. Anforderungen an das
Programm
o Ein weiterer Sportverein kann nur durch Angabe des Namens und der Nummer erstellt werden
o Die aktuelle Anzahl an Mitgliedern des Vereins (fortlaufende Nummer) soll abrufbar sein
o Die Adresse des Sportvereins soll nur komplett (Straße, Hausnummer, PLZ, Ort) angezeigt werden
o Der Abteilungsleiter kann auch nach Erstellung der Abteilung eingetragen werden
o Die Mitglieder sollen innerhalb der Abteilungen gespeichert werden
o Innerhalb der Abteilungen sollen die Mitglieder ausgegeben werden können

an sich habe ich keine probleme mit solchen aufgaben jedoch ist hier iwie etwas faul.
hier mein derzeitiger stand:

Spoiler: Sportverein.java

package sportverein;

public class Sportverein {
	private String name;
	private int nummer;
	private static byte anzahlMitglieder;
	private String strasse_hnr;
	private short plz;
	private String ort;

	public Sportverein(String name, int nummer) {
		this.name = name;
		this.nummer = nummer;
	}

	public void setAdresse(String s, short p, String o) {
		strasse_hnr = s;
		plz = p;
		ort = o;
	}

	public void showAdresse() {
		System.out.println(strasse_hnr);
		System.out.println(plz + " " + ort);
	}

	public byte getAnzahlMitglieder() {
		return anzahlMitglieder;
	}
}

Spoiler: Abteilung.java

package sportverein;

public class Abteilung extends Sportverein {
	private String name;
	private String leiter;
	private Mitglied[] abteilungsliste = new Mitglied[100];

	public Abteilung() {
	}

	public Abteilung(String name, String leiter) {

	}

	public void setLeiter(String leiter) {
		this.leiter = leiter;
	}

	public String getLeiter() {
		return leiter;
	}

	public void setMitglied(Mitglied m) {
		int emptyField = 0;
		for (int i = 0; i < abteilungsliste.length
				&& abteilungsliste[i] != null; i++) {
			emptyField++;
		}
		if (emptyField < 100) {
			abteilungsliste[emptyField] = m;
		} else {
			System.out.println("Abteilung voll!");
		}

	}

	public void showMitglieder() {
		for (int i = 0; i < abteilungsliste.length; i++) {
			abteilungsliste[i].showName();
		}
	}
}

Spoiler: Mitglied.java

package sportverein;

public class Mitglied {
	private String name;
	private String vorname;
	private byte mitgliedsnummer;

	public Mitglied() {
	}

	public Mitglied(String name, String vorname) {
		this.name = name;
		this.vorname = vorname;
	}
	
	public void showName(){
		System.out.println(vorname + " " + name);
	}
	public byte getMitgliedsnummer(){
		return mitgliedsnummer;
	}
}

ich weiß nicht wie ich es realisieren soll dass beim erstellen eines Mitgliedsobjekt der anzahlMitglieder der Sportvereinklasse incrementiert wird.
Außerdem stehe ich grade aufm schlauch wie ich den konstruktur der Abteilungsklasse erstelle. immerhin muss innerhalb des konstruktors der konstruktor der oberklasse aufgerufen werden, da ich aber dafür name und nummer brauche ist hier iwie der wurm drinne.

wenn ich mich nicht strikt an das umldiagram halten müsste wüsste ich einige wege wie die lösen würde aber das ist halt eben NICHT gestattet.. hoffe ihr könnt ein wenig helfen

 

[knilch]

Hi,

Zum ersten Punkt: ich weiß nicht wie ich es realisieren soll dass beim erstellen eines Mitgliedsobjekt der anzahlMitglieder der Sportvereinklasse incrementiert wird.

Du kannst in der Klasse Abteilung einen Counter setzten. In der Methode

    public void setMitglied(Mitglied m) { ... }

inkrementierst du diesen Counter

in etwa so:

private int count = 0;
....

    public void setMitglied(Mitglied m) {
    	count ++;
        for (int i = 0; i < abteilungsliste.length
                && abteilungsliste[i] != null; i++) {
            emptyField++;
        }
        if (emptyField < 100) {
            abteilungsliste[emptyField] = m;
        } else {
            System.out.println("Abteilung voll!");
        }
 
    }

Um dann für jede Abteiling die Mitgliederanzahl korrekt zurück zu geben, überschreibst du in der Klasse Abteilung die Methode showMitglieder() , die du in der Klasse Sportverein() definiert hast.

	@Override 
        public byte getAnzahlMitglieder() {
		return ((byte) count );
	}

Wenn Du eine weitere Methode implementierst, die ein Mitglied aus einer Abteilung entfernst, kannst du dann den counter dekrementieren. So stimmt die Anzahl der Mitglieder beim Entferen und Hinzufügen...

Zu Punkt 2:
... immerhin muss innerhalb des konstruktors der konstruktor der oberklasse aufgerufen werden, da ich aber dafür name und nummer brauche ist hier iwie der wurm drinne.

In der Klasse Abteilung muss der Constructor geändert werden:

	public Abteilung(String name, int nummer) {
		super(name, nummer);
	}

Damit wird der Constructor der Klasse Sportverein() aufgerufen.

Gruss

Kilch

 

[MrClave]

Den Konstruktor der Super-Klasse aktivierst du über das super Keyword.

super( String, int );

Für den leeren Konstruktor der Abteilungsklasse, kannst du einfach

super( "", 0 );

verwenden.

Tja, ohne Mutator kannst du an der Mitgliederanzahl nichts rütteln. Da hilft auch Vererbung nicht.
Entweder protected/public machen, oder eine Methode erstellen.
Eine andere Lösung sehe ich gerade auch nicht.

 

[ZebaX]

Die Vergebung ist nicht wirklich sinnig.... Vererbung sollte man nach dem "is a" Prinzip gestalten und abteilung ist kein sportverein... das ist nen "has a" also nix mit Vergebung sondern als membervariable halten und in dem Fall einfach in ein hashSet packen. In die sportverein gehört die addAbteilung und in Abteilung die addMitgliedr und schon kann man überall drauf zugreifen....... wer gibt euch den solch ein uml

 

[MrClave]

Die Vergebung ist nicht wirklich sinnig.... Vererbung sollte man nach dem "is a" Prinzip gestalten und abteilung ist kein sportverein... das ist nen "has a" also nix mit Vergebung sondern als membervariable halten und in dem Fall einfach in ein hashSet packen. In die sportverein gehört die addAbteilung und in Abteilung die addMitgliedr und schon kann man überall drauf zugreifen....... wer gibt euch den solch ein uml

Offenbar die Schule.
Dass das UML Diagram unsinn ist, steht außer Frage.
Die Aufgabe ist aber klar definiert und sollte auch möglichst so gelöst werden.

Wenn man es ordentlich machen will, müsste man so einige Dinge ändern.
Das ist aber nicht der Ausgangspunkt der Aufgabe.

P.S. Ich glaube nicht, dass Klassen sind gegenseitig "Vergeben" müssen

 

[QuiXotiK]

in der tat, die schule hat mir diese aufgabe gegeben.
@knilch: 1. es ging mir eher hierum: "Die aktuelle Anzahl an Mitgliedern des Vereins (fortlaufende Nummer) soll abrufbar sein"
so wie ich es verstehe soll die gesamtzahl der mitglieder ausgegeben werden. in der klasse Sportverein gibt es eine klassenvariable anzahlMitglieder undb diese wollte ich beim erstellen eines neuen mitglieds um 1 erhöhen. auf deine weise wird mir aber die anzahl der mitglieder in der abteilung ausgegeben.

2. so ganz passt das doch nicht.. denn wenn ich über abteilung den konstruktor des sportvereins aufrufe ist es ja für mich möglich die die adresse unabhängig vom sportverein zu ändern. die aufgaben stellung sagt aber dass der sportverein DIE adresse für jede abteilung hat.

zebax hat hier gut aufn punkt gebracht

da werd ich wohl den lehrer aufsuchen müssen und fragen.

trotzdem danke für die hilfe