Klassen Mehrere Java Klassen in einer .java Datei

[ImpCaligula]

Salvete!

Ich habe da circa 5 selbständige .Java Klassen, welche einen sehr großen redundanten Anteil an Code haben. Daher kam mir die Idee - wieso nicht zusammen fassen, was gleich ist und abstrahieren, was unterschiedlich ist. Also das typische - eine Oberklasse plus 5 Kindklassen... und in der Oberklasse zusammenfassen...

Die Idee ist folgende.

Eine .Java Klassendatei mit der Oberklasse und innerhalb dieser Klassendatei dann auch die Kindklassen. Denkste - darf ich nicht. Nur, wenn ich die Elternklasse auf public stelle und die Kindklassen den Modifizierer weg lasse...

public class Oberklasse
...
...

class KindklasseA extends Oberklasse
...
...

class KindklasseB extends Oberklasse
...
...

Wenn ich jetzt in einer anderen .Java Klassendatei (gleiches Package) mittels Object auf eine der Kindklassen instanziieren will - geht das dann denn? Die Kindklassen sind ja nicht public - oder?! Ich könnte jetzt natürlich so verfahren, dass ich die Oberklasse in eine .Java und die Kindklassen auch jeweils in eine .Java stecke.... aber dann bleibt es dennoch etwas unübersichtlich. Ich würde gerne die Eltern/Kindklassen in eine .Java stecken und dann in anderen .Java Klassendateien mittels Objekte darauf zugreifen.

Geht? Blödsinn? Mach es anders?
Ich frage vorab - da ich da einiges umbauen muss und die Arbeit nicht umsonst machen will wenn ich jetzt anfange. Danke für Eure sachdienliche Hilfe schon mal vorab...

 

[stg]

Mach dir das Leben nicht unnötig schwer und halte dich an gängige Konventionen, sprich in dem Fall: Je Klasse eine .java Datei. Sofern du mit einer anständigen IDE arbeitest wird das auch in keinster Weise unübersichtlich und du kannst bequem zwischen den Klassen hin- und hernavigieren.

 

[ImpCaligula]

Also doch Elternklasse und Kindklassen separat halten.... ?

 

[stg]

In der Regel ist das der beste Weg, ja.

 

[Joose]

Zuerst: Ich gebe stg Recht. Pro Klasse eine eigene Datei ist immer noch am übersichtlichsten

Geht? Blödsinn? Mach es anders?

Das geht theoretisch schon:

OuterClass.InnerClass innerClassObject = new OuterClass().new InnerClass();

Aber ich finde solche Zeilen schauen immer sehr unschön aus und sind kompliziert zu lesen.

 

[ImpCaligula]

Danke für Eure Ausführungen, Zeit und Mühe!

Jetzt habe ich noch die eine oder andere Frage.

Erstens...
Die Oberklasse ist nur eine Steuerklasse und stellt nur Attribute / Methoden für die Kindklassen bereit!
- Wie stelle ich sicher, dass die Kindklassen die Elemente verwenden können, Objekte, welche von den Kindklassen instanziiert werden aber nicht? Wenn ich in der Oberklasse public int a oder protected int a mache und dann ein Objekt von der Kindklasse mache, kann ich mittels objekt.a = 5 trotzdem darauf zugreifen - das sollte nicht sein.
- Wie stelle ich es am besten an, dass von der Oberklasse gar kein Objekt instanziiert werden kann? Abstract machen?

Zweitens...
Architektur in Ordnung? Zur Zeit ist es so gelöst, dass viel redundanter Code vorhanden ist. Es gibt 4 Klassen die etwas tun und dabei gleiche Methoden mit gleichem Code verwenden. Dazu gibt es eine 5 Klasse, welche von 2 der 4 anderen Klassen verwendet wird.

Also nochmals. 4 Klassen sind autark, verwenden aber ziemlich identische Attribute und Methoden. Zusätzlich wird von 2 (von den 4) Klassen eine 5te Klasse verwendet. Die Klassen benutzen dort in Klasse No. 5 eine public Methode, welche wiederum intern private Methoden verwenden und dann ein return senden...

Meine Idee ist also...
- die 4 Klassen werden alles 4 Kindklassen.
- die Oberklasse verwaltet die identischen Attribute und Methoden
- die Oberklasse übernimmt aber auch für 2 (nun Kindklassen) die Arbeit der nun nicht mehr benötigten 5ten Klasse

Achtung! In meiner Lösung aktuell instanziiert dazu die Kindklasse ein Objekt aus der Oberklasse in sich! Ich bin da noch nicht so glücklich darüber. Wäre das aber ok?

---
Danke nochmals.

 

[Joose]

Die Oberklasse ist nur eine Steuerklasse und stellt nur Attribute / Methoden für die Kindklassen bereit!
- Wie stelle ich sicher, dass die Kindklassen die Elemente verwenden können, Objekte, welche von den Kindklassen instanziiert werden aber nicht?
- Wie stelle ich es am besten an, dass von der Oberklasse gar kein Objekt instanziiert werden kann? Abstract machen?

Achtung! In meiner Lösung aktuell instanziiert dazu die Kindklasse ein Objekt aus der Oberklasse in sich! Ich bin da noch nicht so glücklich darüber. Wäre das aber ok?

Hier liegt ein grundlegender Denkfehler vor (soweit ich dich richtig verstanden habe):
Ein Sub-/Kindklasse leitet von einer Super-/Oberklasse ab. Eine Kindklasse erweitert das Verhalten einer Oberklasse.

Beispiel für eine Ableitung:

public abstract class Auto {
    protected String farbe;
    protected String marke;

    protected Auto(String marke, String farbe) {
....
}

public class Mercedes extends Auto {
    public Mercedes(String farbe) {
        super("Mercedes", farbe);
    }
}

Es sollte möglich sein zu sagen: Mercedes ist ein Auto ([Unterklasse] ist ein [Oberklasse])

Wenn du dann willst das von der Oberklasse keine Instanz erstellt werden kann dann diese als "abstract" deklarieren.
Und alle Attribute die in den Kindklassen verfügbar sein sollen als "protected".

Du schreibst davon das deine Kindklasse ein Objekt der Oberklasse instanziert. Das ist nicht notwendig wenn die Ableitung korrekt implementiert wurde da die Kindklasse ja auch ein Objekt der Oberklasse ist!

Architektur in Ordnung? Zur Zeit ist es so gelöst, dass viel redundanter Code vorhanden ist. Es gibt 4 Klassen die etwas tun und dabei gleiche Methoden mit gleichem Code verwenden. Dazu gibt es eine 5 Klasse, welche von 2 der 4 anderen Klassen verwendet wird.
- die 4 Klassen werden alles 4 Kindklassen.
- die Oberklasse verwaltet die identischen Attribute und Methoden
- die Oberklasse übernimmt aber auch für 2 (nun Kindklassen) die Arbeit der nun nicht mehr benötigten 5ten Klasse

Wie oben schon beschrieben. Eine Ableitung ist nur dann Sinnvoll das Verhalten einer gemeinsamen Basisklasse erweitert wird.
Nur weil der Code identisch ausschaut heißt es noch nicht das auch eine Ableitung sinnvoll sein muss.

Wenn etwas nur von 2 Kindklassen gebraucht wird könntest du dir auch überlegen eine weitere Abstraktionsebene einzuführen:

Superklasse A (eine Superklasse mit alle Methoden/Variablen die in allen Kindklassen gebraucht werden können.
+---Subklasse A1 (eine Superklasse mit dem Verhalten aus der 5.Klasse)
|  |
|  +-Subklasse A1a (deine beiden Klassen welche die 5.Klasse benötigt haben)
|  +-Subklasse A1b
|
+---Subklasse A2
|
+---Subklasse A3

 

[ImpCaligula]

Danke für Eure Hilfe!