C# Interface Methoden werden nicht angezeigt

[mosquito87]

Hallo zusammen,

hänge zurzeit an einem kleinen Problem mit Interfaces.
Evtl. ist es auch ein Verständnisproblem, da ich aus der Java-Ecke komme.

Ich definiere in einem Interface "ITest" eine "abstrakte" Methode:

int getId();

In einer anderen Klasse wird nun ein Objekt vom Typ "ITest" an die Methode übergeben (Übergabeparameter). Das Objekt heißt "test".
Warum funktioniert nun test.getId() nicht? Die Methode ist nicht bekannt. :-(

 

[kinglouy]

Mich würde der Rest vom Code noch interessieren.

 

[mosquito87]

Code vom Interface (das "Super"interface (IEntity) ist leer):

public interface ISqlEntity : IEntity
    {
        int Id
        {
            get;
            set;
        }

        int getId();
    }

Code der Methode, wo das Interface übergeben wird:

public override int updateEntity(DatabaseConnection db, ISqlEntity entity)
        {
            entity.getId() <- geht nicht ...
        }

 

[kinglouy]

Und die genaue Fehlermeldung lautet? Symbol nicht gefunden? Oder handelt es sich nur um einen IntelliSense-Fehler in Visual Studio ?

 

[mosquito87]

ISqlEntity does not contain a definition for 'getId' ...

 

[Tolotos]

Hi,

ich habe es mal kurz nachgestellt und bei mir (VS2010) funktioniert es (ich habe der Einfachheit halber mal IEntity weggelassen).
Sollte also kein grundsätzliches Problem sein.

Genaueres kann ich dir allerdings so nicht sagen...

Gibt der Compiler eine Fehlermeldung oder funktioniert die Code-Vervollständigung nicht?

Grüsse,

T.

 

[kinglouy]

Mir fällt da auch nichts Falsches dran auf.

 

[holy]

Du hast nicht zufällig 2 Interfaces, die sich einen Namen teilen, aber in unterschiedlichen Namespaces liegen?

 

[mosquito87]

Strange.

Intellisense funktioniert nicht (damit kann ich ja noch leben), aber der Compiler meckert ebenfalls:

"Error 1 'ISqlEntity' does not contain a definition for 'getId' and no extension method 'getId' accepting a first argument of type 'ISqlEntity' could be found (are you missing a using directive or an assembly reference?) C:\Users\Michael\Documents\Visual Studio 2010\Projects\MvcRepository\MvcRepository\Repositories\SqlRepository.cs 83 62 MvcRepository"

Ergänzung (16. November 2012)

@holy: Was meinst du mit Name teilen?

 

[holy]

Ergänzung (16. November 2012)

@holy: Was meinst du mit Name teilen?

Ob es in deinem Projekt zwei Interfaces mit dem Namen ISqlEntity gibt?
Es muss nichtmals sein, dass du so clever warst, evtl. exportiert eine deiner Referenzen dieses Interface. Einfach mal den Namespace von deinem ISqlEntity Argument prüfen.
Wenn es das auch nicht ist, dann keine Ahnung.

 

[mosquito87]

Scheint echt an der einen Klasse zu liegen.
In allen anderen Klassen funzt es.

 

[iks-deh]

Spring per Druck auf F12 mal zur Definition deiness ISQLEntity-Interfaces. Springt er in zur von dir erwartetetn Quellcode-Datei?

 

[mosquito87]

Ok.
Es liegt an meinen Generics.

Momentan habe ich:

public abstract class Repository<IEntity>

und

public abstract class SqlRepository<ISqlEntity> : Repository<ISqlEntity>

Ich will quasi erreichen, dass ein erzeugen eines Repository-Objekts nur mit einer Klasse möglich ist, die das IEntity Interface nutzt.
Das Erzeugen eines SqlRepositories soll nur möglich sein, wenn die Klasse das ISqlEntity Interface nutzt (unabhängig davon, dass man keine Objekte abstrakter Klassen erzeugen kann).

Wie würde das denn korrekt aussehen?

 

[lbm1305AtWork]

Das dürfte hefen

    public interface IRepository<T>
    {
    }

    public abstract class Repositor<T> : IRepository<T> where T : IEntity
    {
    }

    public abstract class SqlRepository<T> : IRepository<T> where T : ISqlEntity
    {
    }

EDIT: Wobei auch Objekte Typ ISqlEntity an das Repository<T> übergeben werden können.

Hier wäre es von Vorteil, wenn Du für jeden EntityType ein eigenes Interface oder eine Base Class (Abstrakte Klasse) erstellst und diese als Constrains in where - Klausel angibst.

public interface ISqlEntity : IEntity { }

public abstract class BaseEntity : IEntity { }

public abstract class BaseSqlEntity : ISqlEntity { }

// und dann 
where T : BaseEntity
where T : BaseSqlEntity

 

[mosquito87]

Danke.

So hat alles geklappt:

Die Interfaces:

public interface IEntity

public interface ISqlEntity : IEntity

Die 2 Klassen:

public abstract class Repository<T> where T : IEntity

public abstract class SqlRepository<T> : Repository<T> where T : ISqlEntity

Ihr seid spitze!

 

[lbm1305AtWork]

Hast Du mein EDIT gelesen?

 

[mosquito87]

Nein.
Eben erst.
Danke für den Hinweis.

Wenn ich dich richtig verstehe ist es so, dass an

public abstract class Repository<T> where T : IEntity

nicht nur IEntity sondern auch ISqlEntity übergeben werden kann, da ISqlEntity von IEntity erbt. Richtig?
Ich denke, dass ich damit leben könnte.
Gerade weil die Klassen ja abstrakt sind.
Mit weiteren abstrakten Klassen würde ich die Hierarchie verkomplizieren.
Oder hältst du das für sinnvoll?

Noch ein kleines Problem.
Ich hab eine weitere Unterklasse:

public class TaskSqlRepository<T> : SqlRepository<T> where T : Task

Die Klasse Task implementiert das Interface ISqlEntity.

protected override T getObject(SqlDataReader reader)
        {
            ...

            T task;
            task = new Task(id, number, description);

            return task;
        }

Die Methode funktioniert aber nicht:
"Cannot implicitly convert type 'MvcRepository.Models.Task' to 'T'. An explicit conversion exists (are you missing a cast?)".
Woran liegt das?
Task ist doch T!?

 

[lbm1305AtWork]

Nein.
Eben erst.
Danke für den Hinweis.

Wenn ich dich richtig verstehe ist es so, dass an

public abstract class Repository<T> where T : IEntity

nicht nur IEntity sondern auch ISqlEntity übergeben werden kann, da ISqlEntity von IEntity erbt. Richtig?

Korrekt.

public class TaskSqlRepository<T> : SqlRepository<T> where T : Task

Die Klasse Task implementiert das Interface ISqlEntity.

Die Methode funktioniert aber nicht:
"Cannot implicitly convert type 'MvcRepository.Models.Task' to 'T'. An explicit conversion exists (are you missing a cast?)".
Woran liegt das?
Task ist doch T!?[/QUOTE]

Task ist aber auch eine Klasse aus dem Namensraum System.Threading.Tasks (ab .NET 4).
Leitet dein Task auch von der ISqlEntity ab.

In persönlich würde in den Ableitungen nicht so tief gehen bzw. versuche ich, diese zu umgehen.

 

[roker002]

Wieso den so kompliziert machen.... man kann doch mehrere Interfaces an eine Klasse weitergeben. Wieso diese komische verkettung? Und ja du musst entity benutzen anstatt task. Sonst kann du einfach virual Methoden einbauen und diese dann in deine Hauptklasse überschreiben.