OOP Optimales Zusammenspiel von 3 Klassen?

[LILA]

Hallo Forum,

ich habe 3 Klassen. Eine "Basisklasse", von der meine "EinlesenKlasse" erbt. Meine dritte Klasse heisst "ObjektBilden". In dieser Klasse möchte ich von der "EinlesenKlasse" eine Instanz bilden.

Der Sinn der Sache: In der Basisklasse wird allgemein der Zugriff auf Excel-Files mit der JExcel-API geregelt. In der "EinlesenKlasse" spezifiere ich den Zugriff genauer und in der "ObjektBilden"-Klasse bilde ich ein Objekt von der "EinlesenKlasse" und sage, wieviele Spalten eingelesen werden sollen und aus welcher Datei. Nach dem Start des Programms sollen dann die in der "ObjektBilden"-Klasse festgelegten Spalten in der Konsole dargestellt werden.

Bisher habe ich noch Probleme die Modifizierer optimal zu setzen, bzw. auf Variablen in anderen Klassen optimal zuzugreifen. Brauche ich da immer Getter und Setter, auch wenn ich von der Klasse erbe ("EinlesenKlasse" erbt ja von "Basisklasse"). Wie könnte ich dieses kleine Programm verbessern?
Welche Modifizierer nutze ich wann am Besten?

Besonders wichtig wäre mir, aus diesen Klassen ein Beispiel für das optimale Zusammenspiel verschiedener Klassen in der objektorientierten Programmierung (also Vererbung; Kapeselung...) zu erhalten.

Vielen Dank mal im Voraus für alle, die schon mehr Know-How im Zusammenhang der Objektorientierten Programmierung haben.

Hier mal meine Klassen:

Basisklasse:

//In dieser Klasse wird generell der Zugriff auf Excel-Dokumente geregelt.
//Eine bestimmte Anzahl an Excel-Zellen wird in einen String geschrieben

package beispiel;

import java.io.IOException;
import jxl.*;
import java.io.File;
import jxl.read.biff.BiffException;

public class Basisklasse 
{
    private Workbook exceldateiEingabe;
    
        public void readExceldatei() throws IOException, BiffException
        {
            //Workbook erstellen
            exceldateiEingabe = Workbook.getWorkbook(new File(datei));
      
        }

        public void closeExceldatei()
        {
            exceldateiEingabe.close();
        }
}

EinlesenKlasse:

package beispiel;

import java.io.IOException;
import jxl.*;
import java.io.File;
import jxl.read.biff.BiffException;

public class EinlesenKlasse extends Basisklasse
{
    //Anzahl der Spalten die eingelesen werden sollen
    private int var1;
    //Datei die eingelesen werden soll
    private String datei;

    //parameterloser Standardkonstruktor
    public EinlesenKlasse()
    {
    }

    //Konstruktor
    public EinlesenKlasse(int varA, String varB)
    {
        this.var1 = varA;
        this.datei = varB;
    }

    //Set-Methoden
    public void setVar1(int varA)
    {
        this.var1 = varA;
    }

    public void setVar2(String varB)
    {
        this.datei = varB;
    }

    //Get-Methoden
    public int getVar1()
    {
        return var1;
    }

    public String getDatei()
    {
        return datei;
    }

    //Hier wird die Funktion der Basisklasse überschrieben
    public void readExceldatei()
    {
            //Festlegen welches Tabellenblatt verwendet werden soll
            Sheet tabelle = exceldateiEingabe.getSheet(0);

            //Gesamte Spalte durchlaufen und Werte in String schreiben
            for (int y = 0; y < tabelle.getRows(); y++)
            {
                String a = tabelle.getCell(0, y).getContents();
                String b = tabelle.getCell(1, y).getContents();
                System.out.println(a + " | " + b);
            }
    }
}

ObjektBilden:

package beispiel;

import java.io.IOException;
import jxl.read.biff.BiffException;

public class ObjektBilden
{
    //Hauptmethode eines Java-Programms = Startpunkt für Programm
    public static void main(String[] args) throws IOException, BiffException
    {
        //paarameterloser Standardkonstruktor wird aufgerufen
        //EinlesenKlasse einlesen = new EinlesenKlasse();
        
        //Konstruktor mit Parametern wird aufgerufen
        EinlesenKlasse einlesen2 = new EinlesenKlasse(2, "C:/.../einfacheExcelDatei.xls");
        einlesen2.readExceldatei();
        einlesen2.closeExceldatei();
    }
}

 

[Ein Keks]

naja mach das ganze doch erstmal lauffähig...
[JAVA=53]Sheet tabelle = exceldateiEingabe.getSheet(0);[/code]
exceldateiEingabe ist null und nicht sichtbar in EinlesenKlasse also vorher noch

super.readExceldatei()

aufrufen und für exceldateiEingabe einen getter erstellen und benutzen

desweiteren wäre es ganz sinnvoll gescheite Namen zu benutzen und nicht var1, varA oder ähnliches

ach und wenn du ne methode überschreibst kann es ganz nützlich sein ein @Override drüber zu schreiben das verrät dir manchmal noch ein paar fehler

ob Basisklasse so wirklich einen Sinn hat ist allerdings etwas fragwürdig

 

[LILA]

Hallo Keks,

du hast mir wirklich sehr geholfen. Irgendwie stand ich am Schlauch.

Noch ein paar Fragen:

Dieses

super....();

brauche ich immer wenn ich eine Funktion überschreibe, oder um die Methode wie sie bisher war aufzurufen, oder?

Was genau sind deine Bedenken, dass du dir überlegst, ob "Basisklasse" einen Sinn macht? Ich wollte dieses ganze Beispiel nur machen, um mir selbst mehr Klar darüber zu werden, was es mir objektorientierter Programmierung auf sich hat.
Meinst du sie macht jetzt Sinn?

So habe ich meine Klassen jetzt abgeändert:
Meinst du sie entsprechen nun den Richtlinien objektorientierter Programmierung?
Könnte ich noch irgendwas verbessern. Ich denke die Kapselung passt.
Brauche ich vielleicht noch einen Konstruktor in der Basisklasse?

Basisklasse:

//In dieser Klasse wird generell der Zugriff auf Excel-Dokumente geregelt.
//Eine bestimmte Anzahl an Excel-Zellen wird in einen String geschrieben

package beispiel;

import java.io.IOException;
import jxl.*;
import java.io.File;
import jxl.read.biff.BiffException;

public class Basisklasse 
{
    private String datei;
    private Workbook exceldateiEingabe;

    //Getter für die Exceldatei
    public Workbook getExceldateiEingabe()
    {
        return exceldateiEingabe;
    }

    //Getter für Datei die eingelesen werden soll
    public String getDatei()
    {
        return datei;
    }

    //Setter für Datei die eingelesen werden soll
    public void setDatei(String datei)
    {
        this.datei = datei;
    }

    public void readExceldatei() throws IOException, BiffException
    {
        //Workbook erstellen
        exceldateiEingabe = Workbook.getWorkbook(new File(datei));
    }

    public void closeExceldatei()
    {
        exceldateiEingabe.close();
    }
}

EinlesenKlasse:

package beispiel;

import java.io.IOException;
import jxl.*;
import java.io.File;
import jxl.read.biff.BiffException;

public class EinlesenKlasse extends Basisklasse
{
    //Anzahl der Spalten die eingelesen werden sollen
    private int spaltenAnz;

    //parameterloser Standardkonstruktor
    public EinlesenKlasse()
    {
    }

    //Konstruktor
    public EinlesenKlasse(int spaltenAnz)
    {
        this.spaltenAnz = spaltenAnz;
    }

    //Setter
    public void setSpaltenAnz(int spaltenAnz)
    {
        this.spaltenAnz = spaltenAnz;
    }

    //Getter
    public int getSpaltenAnz()
    {
        return spaltenAnz;
    }

    //Hier wird die Funktion der Basisklasse überschrieben
    @Override
    public void readExceldatei() throws IOException, BiffException
    {
        //Macht die Methode sichtbar
        super.readExceldatei();

            //Festlegen welches Tabellenblatt verwendet werden soll
            Sheet tabelle = getExceldateiEingabe().getSheet(0);

            //Gesamte Spalte durchlaufen und Werte in String schreiben
            for (int y = 0; y < tabelle.getRows(); y++)
            {
                for (int x = 0; x < this.getSpaltenAnz(); x++)
                {
                    String a = tabelle.getCell(x, y).getContents();
                    System.out.print(a);
                    System.out.print(" | ");
                }

                System.out.println(" ");
            }
    }
}

ObjektBilden

package beispiel;

import java.io.IOException;
import jxl.read.biff.BiffException;

public class ObjektBilden
{
    //Hauptmethode eines Java-Programms = Startpunkt für Programm
    public static void main(String[] args) throws IOException, BiffException
    {
        //paarameterloser Standardkonstruktor wird aufgerufen
        //EinlesenKlasse einlesen = new EinlesenKlasse();
        
        //Konstruktor mit Parametern wird aufgerufen
        EinlesenKlasse einlesen2 = new EinlesenKlasse(2);
        einlesen2.setDatei("C:/.../einfacheExcelDatei.xls");
        einlesen2.readExceldatei();
        einlesen2.closeExceldatei();
    }
}

 

[Ein Keks]

Dieses

super....();

brauche ich immer wenn ich eine Funktion überschreibe, oder um die Methode wie sie bisher war aufzurufen, oder?

ja

super.methode

dient dazu die Methode wie sie bisher war aufzurufen wenn du sie überschreibst.

Was genau sind deine Bedenken, dass du dir überlegst, ob "Basisklasse" einen Sinn macht?

es hatte so ein bisschen den Anschein als wäre sie nur dazu da um von etwas Ableiten zu können, nur des Ableiten wegens ohne einen anderen Nutzen^^
Aber als Beispiel ist sie in Ordnung. Wobei ich noch für nen Basiskonstruktor wäre da ja auf jedenfall ne Datei benötigt wird.

Wichtig ist ja auch dass du statt

EinlesenKlasse einlesen2 = new EinlesenKlasse(2);

auch

BasisKlasse einlesen2 = new EinlesenKlasse(2);

schreiben könntest, da EinlesenKlasse eine BasisKlasse ist (da sie von BasisKlasse erbt).

Das Ganze könntest du jetzt noch weiterführen, indem du noch ein interface Reader machst, welches von BasisKlasse implementiert wird.

public interface Reader{
    public void closeExceldatei()
    public void readExceldatei() throws IOException, BiffException
}

Dann könnte man auch eine Implementierung mit POI machen und beide gleich benutzen (was ja der Sinn von Schnittstellen ist)
ob das dann sinnvoll ist, ist allerdings ne andere Frage^^

 

[LILA]

Hallo Keks,

klingt alles sehr gut.
Hat mir geholfen.

Grüsse und Danke