incompatibel return type bei vererbung

[guni]

Hallo, ich habe eine Klasse Database, die mir eine Verbindung zu einer db herstellt:
(der Einfachkeit halber bild ich sie hier nicht vollständig ab)

public class Database {
	private static Database INSTANCE;
	public Connection sql;
	public Statement stmnt;

	public static Database getInstance() throws Exception {
		if (INSTANCE == null) {
			INSTANCE = new Database();
		}
		return INSTANCE;
	}
	protected Database() throws ClassNotFoundException, SQLException {
		Class.forName("oracle.jdbc.driver.OracleDriver");
		sql = DriverManager.getConnection(dbconnect, user, password);
		stmnt = sql.createStatement();
	}
}

also, Schema ist bekannt, oder?!
hier habe ich ein Klasse, die von Database erbt:

public class SagDatabase extends Database {
	private static SagDatabase INSTANCE;

	public static SagDatabase getInstance() throws Exception {
		if (INSTANCE == null) {
			INSTANCE = new SagDatabase();
		}
		return INSTANCE;
	}
	
	private SagDatabase() throws Exception {
		super();
	}
	
}

... soweit auch klar, oder?!
jetzt kommt noch eine Klasse, die in meinen Augen gleich aufgebaut ist:

public class CdbDatabase extends Database {
	private static CdbDatabase INSTANCE;

	public static CdbDatabase getInstance() throws Exception {
		if (INSTANCE == null) {
			INSTANCE = new CdbDatabase();
		}
		return INSTANCE;
	}

	private CdbDatabase() throws Exception {
		super();
	}
}

der kleine Unterschied: die Klasse SAG-Database funktioniert; die Klasse CdbDatabase gibt einen Fehler:
Exception in thread "main" java.lang.Error: Unresolved compilation problem:
The return type is incompatible with Database.getInstance()

wie kann das sein?!

mfg, guni

 

[SlaterB]

statische Methoden werden nicht vererbt,
bei mir ist auch kein Fehler zu sehen,
nimmt man das static raus, so klappt die Vererbung bei mir ohne Fehler

ganz schön mysteriös

Screenshots, alles in eine Datei schreiben?

import java.sql.Connection;
import java.sql.SQLException;
import java.sql.Statement;


public class Test
{


    public static void main(String[] args)
        throws Exception
    {
    }
}


class Database
{
    private static Database INSTANCE;
    public Connection sql;
    public Statement stmnt;

    public Database getInstance()
        throws Exception
    {
        if (INSTANCE == null)
        {
            INSTANCE = new Database();
        }
        return INSTANCE;
    }

    protected Database()
        throws ClassNotFoundException, SQLException
    {
        Class.forName("oracle.jdbc.driver.OracleDriver");
        // sql = DriverManager.getConnection(dbconnect, user, password);
        stmnt = sql.createStatement();
    }
}


class SagDatabase
    extends Database
{
    private static SagDatabase INSTANCE;

    public SagDatabase getInstance()
        throws Exception
    {
        if (INSTANCE == null)
        {
            INSTANCE = new SagDatabase();
        }
        return INSTANCE;
    }

    private SagDatabase()
        throws Exception
    {
        super();
    }

}


class CdbDatabase
    extends Database
{
    private static CdbDatabase INSTANCE;

    public CdbDatabase getInstance()
        throws Exception
    {
        if (INSTANCE == null)
        {
            INSTANCE = new CdbDatabase();
        }
        return INSTANCE;
    }

    private CdbDatabase()
        throws Exception
    {
        super();
    }
}

 

[guni]

Was meinst du mit Screenshot?
Hier ist einer:


Und was soll es bringen, wenn ich alles in eine Datei schreib?!
mfg, guni

 

[maki]

*quatsch*

 

[guni]

hmm ...
was heißt das jetzt für mich?
Warum funktioniert 'SagDatabase' und 'CdbDatabase' nicht?!
Sie sind doch genau gleich aufgebaut?!
Instance wird ja ausserdem von einer statischen Methode aufgerufen ... insofern dürfte innerhalb von der Methode ja auch keine Vererbungsproblematik auftreten! ...

lg, guni

 

[SlaterB]

da ist noch ein zweites rotes X weiter unten, steht da was interessantes?
funktioniert es, wenn in der Oberklasse keine getInstance()-Methode vorhanden ist?

einen Sinn sehe ich aber immer noch nicht in einer derartigen Fehlermeldung,
vielleicht hilft Project -> Clean

> Und was soll es bringen, wenn ich alles in eine Datei schreib?!

sämtlichen anderen Code, Imports, gleichnamige Klassen, usw. ignorieren

 

[guni]

nein ... das 2. 'x' ist nicht relevant; hab nur eine Klammer falsch gesetzt; mein Fehler taucht aber unabhängig davon auf.

die getInstance-Methode möchte ich aus der Parent-Class eigentlich nicht entfernen; schließlich soll man auch von dieser Klasse ein Objekt erzeugen können ...

ein 'clean' werde ich gleich mal ausprobieren ...

 

[guni]

hmm ... das 'clean' hat nichts gebracht :-(
irgendeine Ahnung woran es sonst noch liegen könnte?!

 

[maki]

hmm ... das 'clean' hat nichts gebracht :-(
irgendeine Ahnung woran es sonst noch liegen könnte?!

Der Code ist falsch, anscheinend werden statische Methoden doch vererbt, aber beim überladen darf ebem der return type nicht geändert werden.

Singletons & Vererbung funktionieren nicht in Java, hat mich noch nie gestört, ist imho eh daneben wenn man von Singletons erben will.

 

[guni]

hmm ... wenn du oben schaust hab ich aber eine Klasse SagDatabase.
In dieser Klasse überschreib ich ganz genauso meine getInstance aus dem Database-Singleton. Trotzdem gehts. Es muss also an irgend was anderem liegen -> ???:L

lg, guni

 

[SlaterB]

der Code läuft bei mir und damit anscheinend bei der Allgemeinheit korrekt,
insofern braucht man über ihn nicht weiter nachzudenken

die Java-Version, die Entwicklungsumgebung, das Betriebssystem usw. könnten interessanter sein,
aber mir brächte das auch nix, einen solchen Fehler habe ich noch nicht gesehen

 

[maki]

hmm ... wenn du oben schaust hab ich aber eine Klasse SagDatabase.
In dieser Klasse überschreib ich ganz genauso meine getInstance aus dem Database-Singleton. Trotzdem gehts. Es muss also an irgend was anderem liegen -> ???:L

lg, guni

Das liegt eher am inkrementellen Kompilieren unter Eclipse, SlaterBs Code mit static bringt den Fehler bei jedem abgeleiteten SIngleton.

der Code läuft bei mir und damit anscheinend bei der Allgemeinheit korrekt,
insofern braucht man über ihn nicht weiter nachzudenken

Wenn du das static wieder reinnimmst, nicht

 

[SlaterB]

bei dir kommt es also auch zum Compilerfehler?
nun gut, dann erkläre du das, bei mir gehts

ein Fehler wäre natürlich, wenn es in der Oberklasse nicht statisch ist,
aber in den Subklassen statisch, dann allerdings ein anderer als angegeben
(This static method cannot hide the instance method from Database)

 

[maki]

bei dir kommt es also auch zum Compilerfehler?
nun gut, dann erkläre du das, bei mir gehts

Hmmm... gestern Abend bei mir zuhause hat es die gleichen Fehler wie beim TS angezeigt, aber bei jeder Subklasse... jetzt nicht mehr... muss mal zuhause nachsehen was ich da anders hatte.

 

[guni]

ok ... vielleicht muss ich einfach mein konzept überdenken ... mich ärgert nur, dass es bei der einen Klasse geht und bei der anderen nicht ...
aber gut: überdenken wir:
* ich habe eine klasse Database, die in verschiedenen Objekten benötigt wird. da ich sie zur Laufzeit allerdings nur ein einziges Mal initialisieren möchte und beim Prgrammieren nicht dauernd überlegen will, ob ich sie woanders schon initialisiert hab, hab ich mich hier für einen Singleton unterschieden.
* ich hab ein paar konkretere Klassen, die im Prinzip das gleiche wie Database machen (Vererbung!). sie unterscheiden sich eigentlich eher dadurch, dass in konkreteren Fällen schon mehr Verbindungseinstellungen bekannt sind -> man braucht also weniger Parameter übergeben. Allerdings sollen auch diese Klassen wieder nur einmalig initialisiert werden -> Kombination aus Vererbung und Singleton.
-- ich will hier keinen Thread darüber starten, ob Singleton philosophisch gesehen schön ist (gibt es glaub ich schon *gg*) ... habt ihr irgendeine andere Idee, das Ganze zu konzeptionieren?!

lg, guni

 

[maki]

Mit einer abstrakten Factory wie hier zB: Core J2EE Patterns - Data Access Object

Was mich an deinem Code etwas wundert ist, dass du "angeblich" 3 (!) Datenbanken hast, die du lesen möchtest, alle Oracle, mit demselben User & Password... alle dasselbe Schema??? Ja -> 1 DB, nicht 3..

 

[SlaterB]

nenne die Methoden doch unterschiedlich und finde eine zentrale Klasse mit Methoden wie
getInstanceDatabase()
getInstanceSagDatabase()
usw

Vererbung gibts nicht, also muss du eh jede Methode direkt eintippen,
vielleicht auch ne Factory mit
getInstance(Klasse/-nname)
die anhand der Klasse das richtige intern verwaltete Objekt zurückgibt

 

[guni]

@maki.
der code den du oben siehst ist stark vereinfacht: natürlich verbinden nicht alle 3 klassen mit selber dbconnect, user, pw zur datenbank.
hab die 3 attribute dbconnect, user, pw protected angelegt.
in der Database hab ich dann genaugenommen im Konstruktor noch einen Aufruf auf eine OverrideSettings, die in den Kindklassen zu redefinition der Verbindungseinstellungen überschrieben werden kann.
Die einzelnen Klassen unterscheiden sich dann einfach durch die Art und Weise wie die Verbindungseinstellungen ermittelt werden.

@SlaterB: hmm ... die statischen getInstance-Methoden unterschiedlich nennen: Einfach aber wirksam. Tolle Idee ... manchmal sieht man vor lauter Bäumen den Wald nicht mehr ;-)

 

[maki]

Hier wie versprochen der Code der bei mir zuhause die Fehler (incompatible return type) bei allen Unterklassen zeigt:

package de.test;

import java.sql.Connection;
import java.sql.SQLException;
import java.sql.Statement;
 
 
public class Database
{
    private static Database INSTANCE;
    public Connection sql;
    public Statement stmnt;
 
    public static Database getInstance()
        throws Exception
    {
        if (INSTANCE == null)
        {
            INSTANCE = new Database();
        }
        return INSTANCE;
    }
 
    protected Database()
        throws ClassNotFoundException, SQLException
    {
        Class.forName("oracle.jdbc.driver.OracleDriver");
        // sql = DriverManager.getConnection(dbconnect, user, password);
        stmnt = sql.createStatement();
    }
}
 
 
class SagDatabase
    extends Database
{
    private static SagDatabase INSTANCE;
 
    public static SagDatabase getInstance()
        throws Exception
    {
        if (INSTANCE == null)
        {
            INSTANCE = new SagDatabase();
        }
        return INSTANCE;
    }
 
    private SagDatabase()
        throws Exception
    {
        super();
    }
 
}
 
 
class CdbDatabase
    extends Database
{
    private static CdbDatabase INSTANCE;
 
    public static CdbDatabase getInstance()
        throws Exception
    {
        if (INSTANCE == null)
        {
            INSTANCE = new CdbDatabase();
        }
        return INSTANCE;
    }
 
    private CdbDatabase()
        throws Exception
    {
        super();
    }
}

k.A. warum das so ist...

Nachtrag: Derselbe Code liefert diese Fehler wenn es sich um ein Maven2 Project handelt, bei einem normalen Eclipse Java Projekt aber nciht.
Könnte es an den unterschiedlichen Compilern liegen???
:bahnhof:???:L