Aus einer Methode eine eigene Klasse machen - modular coden

[LogicCube]

Hallo,

ich muss für eine Programmieraufgabe (die ich schon fertiggestellt habe) die Architektur umbauen, und zwar habe ich momentan folgenden Aufbau:

class MeineKlasse {

<hier stehen die Variablen>

main-Methode{}

methode1{}

methode2{}

methode3{}

....

}

nun sollen aber die Methoden als eigene Klassen definiert werden, und dann ein eigenes Hauptprogg erstellt werden welches wohl Objekte dieser Klassen erzeugen soll welchedann bearbeitet werden (im Hauptprogramm).

Mein Problem ist, das ich ein Beispiel bräuchte, wie man eine Methode quasi "umschreibt" in eine eigene Klasse.

Ob das sinnvoll ist oder nicht sei jetzt mal außen vor gelassen, mein Prof verlangt es halt als "modulares Programm" und daher würde ich ihm das auch gerne so umbauen !

Hier ist mal eine der Methoden, dabei geht es um die Verbindung zu einer Datenbank:

 public static Connection connect() throws SQLException
 {
    String treiber;
    OracleDataSource ods    = new OracleDataSource();
    treiber = "oracle.jdbc.driver.OracleDriver";
    Connection dbConnection = null;

    try{
        Class.forName(treiber).newInstance();}
    catch (Exception e){
        System.out.println("Fehler beim laden des Treibers: "+ e.getMessage());}
    try{
        /*  Anmeldedaten für Home-Server            */
        ods.setURL("jdbc:oracle:thin:system/Haube99@127.0.0.1:1521/XE");
  
        /*  Anmeldedaten für DB-Labor              */
        /*  ods.setURL("jdbc:oracle:thin:dbprak03/dbprak03@fichte.nt.fh-koeln.de:1521/orcl"); */
        dbConnection = ods.getConnection();}

    catch (SQLException e){
    System.out.println("Fehler beim Verbindungsaufbau zur Datenbank!");
    System.out.println(e.getMessage());}
    return dbConnection;
 }

Kann mir da bitte bitte jemand kurz bei helfen? Ich denke wenn man das einmal gemacht hat, dann weiß man wie das geht und der Rest sollte sich dann zumindest größtenteils von selber ergeben....

[/code]

 

[SlaterB]

erstelle eine neue Klasse mit beliebigen Namen,

kopiere die Methode 1 zu 1 rüber,

entferne 'static'

fertig

 

[LogicCube]

Hey,

das klingt einfach, scheitert aber leider zunächst (aber das kriegen wir hin...!):

1. kopieren der Methode inklusive Kopf?

2. Ich habe da ein incompatible_types Problem was die Connection angeht, das ist noch etwas kniffelig. Ich ziehe daher zunächst erstmal eine andere Methode heran, und zwar

static void simpleSelect()
 {
    try
    {
        Connection con = connect();
        Statement Stmt;
        ResultSet RS;
        String SQL;
        
        Stmt = con.createStatement();
        SQL  = "SELECT * FROM KUNDE03";
        RS   = Stmt.executeQuery(SQL);
       
        while(RS.next())
        {
            kdenr   = RS.getInt("kdenr");
            kdenam  = RS.getString("kdenam");
            plz     = RS.getInt("plz");
            ort     = RS.getString("ort");
            gjahr     = RS.getInt("gjahr");
      
            System.out.println("\nKundennummer: "+kdenr);
            System.out.println("Kundenname:   "+kdenam);
            System.out.println("PLZ:          "+plz);
            System.out.println("Ort:          "+ort);
            System.out.println("Geburtsjahr:  "+gjahr);
            System.out.println();
        }    
    }
    catch (SQLException e){
        System.out.println(e.getMessage());
        System.out.println("SQL Exception wurde geworfen!");}
 }

wo müssen jetzt die Variablen wie kdenr, kdenam usw hin?
VOR die Methode wenn denn der Kopf mitbenutzt wird, oder innerhalb der Methode?

Ich weiß klingt etwas konfus aber wie gesagt, wenn ich das System einmal geschnallt habe denke ich kann ich das umsetzen...!

 

[SlaterB]

zu einer Methode gehört auch der Kopf, und auch bei den Variablen ändert sich nix, alles komplett rüber ohne irgendwelche Änderungen,

natürlich kann man generell überlegen, irgendwas anders zu machen,
z.B. Variablen vor die Methode zu schreiben,
aber nur wenn man weiß WAS man WARUM macht,

wenn z.B. zwei Methoden die gleichen Variablen benötigen oder so,
pauschal gibts dafür aber kein Rezept

 

[LogicCube]

Hey,

ok das habe ich verstanden, und das funktioniert denke ich auch, allerdings kriege ich bei der connect-Klasse einen Fehler der sich dummerweise auch auf die anderen Methoden auswirkt, da die alle Objekte vom Typ "Connection" erstellen und wenn die Connection-Klasse nicht ordentlich ist, gibts da natürlich Fehler.

Problem sieht man hier hoffentlich:

import java.io.*;
import java.sql.*;
import oracle.jdbc.pool.*;

class Connection
{

 Connection connect() throws SQLException
 {
    String treiber;
    OracleDataSource ods    = new OracleDataSource();
    treiber = "oracle.jdbc.driver.OracleDriver";
    Connection dbConnection = null;

    try{
        Class.forName(treiber).newInstance();}
    catch (Exception e){
        System.out.println("Fehler beim laden des Treibers: "+ e.getMessage());}
    try{
        /*  Anmeldedaten für Home-Server            */
        ods.setURL("jdbc:oracle:thin:XXXX/XXXX@127.0.0.1:1521/XE");

        /*  Anmeldedaten für DB-Labor              */
        /*  ods.setURL("jdbc:oracle:thin:XXXX/XXXX@server:1521/orcl"); */
        dbConnection = ods.getConnection();}

    catch (SQLException e){
    System.out.println("Fehler beim Verbindungsaufbau zur Datenbank!");
    System.out.println(e.getMessage());}
    return dbConnection;
 }

}

Fehler:

Connection.java:25:41: incompatible types
found   : java.sql.Connection
required: Connection
        dbConnection = ods.getConnection();}

schreibe ich allerdings

   dbConnection = (Connection)ods.getConnection();

wird die Klasse allerdings compiliert, aber ich glaube nicht das das richtig ist oder?!

 

[SlaterB]

zuächst mal solltest du keine eigenen Klassen wie vorgegebene Klassen nennen,
dann klappts auch mit dem Compiler

 

[LogicCube]

Hey,

habe jetzt die Klasse "Verbindung" genannt, der Fehler mit den incompatible-types besteht aber innerhalb der Methode immernoch.

Gib mir einen Tip !

 

[SlaterB]

was ist denn die Fehlermeldung?

bisher hattest du eine Variable

> Connection dbConnection = null;

welche deine eigene Klasse Connection meint,
in welche du aber ein Objekt vom Typ java.sql.Connection
speichern wolltest,

durch Umbennen der Klassen kann es das Problem in dieser Form nicht mehr geben

 

[LogicCube]

Die Fehlermeldung ist immernoch:

Verbindung.java:25:41: incompatible types
found   : java.sql.Connection
required: Connection
        dbConnection = ods.getConnection();}

die Klasse sieht insgesamt folgendermaßen aus:
(Verbindung.java)

import java.io.*;
import java.sql.*;
import oracle.jdbc.pool.*;

class Verbindung
{
   public Connection connect() throws SQLException
   {
    String treiber;
    OracleDataSource ods    = new OracleDataSource();
    treiber = "oracle.jdbc.driver.OracleDriver";
    Connection dbConnection = null;

    try{
        Class.forName(treiber).newInstance();}
    catch (Exception e){
        System.out.println("Fehler beim laden des Treibers: "+ e.getMessage());}
    try{
        /*  Anmeldedaten für Home-Server            */
        ods.setURL("jdbc:oracle:thin:XXXX/XXXX@127.0.0.1:1521/XE");

        /*  Anmeldedaten für DB-Labor              */
        /*  ods.setURL("jdbc:oracle:thin:XXXX/XXXX@server:1521/orcl"); */
        dbConnection = ods.getConnection();}

    catch (SQLException e){
    System.out.println("Fehler beim Verbindungsaufbau zur Datenbank!");
    System.out.println(e.getMessage());}
    return dbConnection;
 }
}

Leider braucht man zum compilieren eine classes12.jar in der die JDBC klassen enthalten sind; ist also schwer nachzutesten für andere.

Das kann doch nicht so schwer sein

 

[maki]

Sehr empfehlenswerte Lektüre zu diesem Thema:"Refactoring: Improving the Design of Existing Code" von Martin Fowler

 

[SlaterB]

was soll das
> import oracle.jdbc.pool.*;
?

hat sich vielleicht automatisch eingeschlichen und dazu gehört dann vielleicht auch
oracle.jdbc.pool.Connection

.* zu importieren ist generell schlecht
-----

wenn es weiter noch nicht hinhaut:
als erstes musst du herausfinden, was das für eine 'Connection' ist,

kommentiere allen fraglichen Code aus und schreibe

System.out.println(Connection.class.getName());
und lasse das auch ausführen, welche Ausgabe kommt?

 

[LogicCube]

Hey,


also ich habe das import mal mit import oracle.jdbc.pool.Connection.*; probiert, dann kennt der zum Beispiel die

OracleDataSource ods    = new OracleDataSource();

Zeile nicht mehr da er nicht mehr weiß was Objekte vom Typ OracleDataSource sind (ist ja auch logisch).

Aber mal ein allgemeinerer Ansatz: Wie kann denn die Methode in einem großen Hauptprogramm funktionieren, indem aber weder eine andere Methode noch andere Variablen irgendwas mit dieser Methode zu tun haben?!

Die imports sind gleich, die Methode ist exakt kopiert, und als einzelne Klasse läuft das nicht. Da muss es doch irgendwo nen Trick geben wie das aussehen muss...

 

[SlaterB]

> import oracle.jdbc.pool.Connection.*;

dies hat ja auch nichts mit korrekten Java zu tun,

du sollst
> import oracle.jdbc.pool.*;
streichen und für deine OracleDataSource von mir aus

import oracle.jdbc.pool.OracleDataSource;

wieder hinschreiben

--------

> und als einzelne Klasse läuft das nicht.

warum das so ist, interessiert mich auch,
bei deinen versuchen a la
> import oracle.jdbc.pool.Connection.*;

muss man natürlich von allen möglichen Fehlern und auch Fehlinterpretationen der Fehlersituationen ausgehen,
also kann man das schlecht von außer erraten,
oder ich zumindest kann das nicht

 

[LogicCube]

Hey,

habs jetzt.

Ich Idiot hatte vergessen dass ich mal ne Connection.class compiliert hatte (war ne alte Version wo ich viel rausgenommen hatte, die wurde dann fehlerfrei compiliert) und anscheinend hat sich der Compiler gedacht, er soll davon Objekte erstellen.

Datei gelöscht -> jetzt compiliert es ohnne Probleme!

Da trau ich mich schon kaum zu fragen, wie man jetzt per Punktnotation in der SimpleSelect die Klasse Verbindung benutzt...

 

[SlaterB]

umso besser, ich wollt eh nicht mehr antworten