Using interfaces on abstract classes in C#

Question

I'm learning C# coming from C++ and have run into a wall.

I have an abstract class AbstractWidget, an interface IDoesCoolThings, and a class which derives from AbstractWidget called RealWidget:

public interface IDoesCoolThings
{
    void DoCool();
}

public abstract class AbstractWidget : IDoesCoolThings
{
    void IDoesCoolThings.DoCool()
    {
        Console.Write("I did something cool.");
    }
}

public class RealWidget : AbstractWidget
{

}

When I instantiate a RealWidget object and call DoCool() on it, the compiler gives me an error saying

'RealWidget' does not contain a definition for 'DoCool'

I can cast RealWidget object to an IDoesCoolThings and then the call will work, but that seems unnecessary and I also lose polymorphism (AbstractWidget.DoCool() will always be called even if i define RealWidget.DoCool()).

I imagine the solution is simple, but I've tried a variety of things and for the life of me can't figure this one out.

To have polymorphism in widgets' DoCool(), you'd need to declare the AbstractWidget's one virtual. — Anton Tykhyy
– Anton Tykhyy, Commented Mar 26, 2009 at 2:06
Erg. I'd read somewhere that virtual was implied for interface methods. — JubJub
– JubJub, Commented Mar 26, 2009 at 2:23

John Feminella · Accepted Answer · 2009-03-26 02:45:33Z

42

You're running into the issue because you used explicit interface implementation (EII). When a member is explicitly implemented, it can't be accessed through a class instance -- only through an instance of the interface. In your example, that's why you can't call DoCool() unless you cast your instance to IDoesCoolThings.

The solution is to make DoCool() public and remove the explicit interface implementation:

public abstract class AbstractWidget : IDoesCoolThings
{
    public void DoCool()      // DoCool() is part of the abstract class implementation.
    {
        Console.Write("I did something cool.");
    }
}

// ...

var rw = new RealWidget();
rw.DoCool();                  // Works!

In general, you use EII in two cases:

You have a class that must implement two interfaces, each of which contains a member that has an identical name/signature to another member in the other interface.
You want to force clients not to depend on the implementation details of your class, but rather on the interface that's being implemented by your class. (This is considered a good practice by some.)

edited Mar 26, 2009 at 2:45

answered Mar 26, 2009 at 2:03

John Feminella

313k49 gold badges348 silver badges362 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

JubJub Over a year ago

God, I thought I'd tried that. Thanks! I seemed to have missed this key line in my reference book: "a method deﬁned by using explicit interface implementation cannot be declared as virtual, whereas omitting the interface name allows this behavior."

JubJub Over a year ago

I was confused that simply making the method public in the abstract class didn't fix the problem, plus the examples I read didn't define methods as public or private, so I came to believe interface methods are implicitly public and virtual. Overall, an embarrassing question :). Thanks though!

supercat Over a year ago

Explicit interface implementation can also be good if it's possible that a derived class' override of a member may wish to return a type which is derived from the base class' implementation. For example, both CarFactory and ICarFactory may have MakeCar() method which returns Car, but FordCarFactory may want to have a MakeCar() method return a FordCar. That could often be best handled by having a protected virtual DoMakeCar method that returns Car, and having both ICarFactory.MakeCar and a non-virtual CarFactory.MakeCar method chain to that.

Community · Accepted Answer · 2017-05-23 12:09:21Z

8

The way you implement the interface is explicit implement void IDoesCoolThings.DoCool(), if you choose implicit implement interface.

public abstract class AbstractWidget : IDoesCoolThings
{
    public void DoCool()
    {
        Console.Write("I did something cool.");
    }
}

Then it will work.

Read this :

C# Interfaces. Implicit implementation versus Explicit implementation

edited May 23, 2017 at 12:09

CommunityBot

11 silver badge

answered Mar 26, 2009 at 2:04

J.W.

18.2k7 gold badges47 silver badges76 bronze badges

1 Comment

hitec Over a year ago

Thanks for adding the link to the SO question on implicit and explicit implementation !

Matthew Murdoch · Accepted Answer · 2012-08-25 18:37:02Z

8

Change your declaration to:

public abstract class AbstractWidget : IDoesCoolThings 
{
    public void DoCool()
    { 
        Console.Write("I did something cool."); 
    }
}

edited Aug 25, 2012 at 18:37

Matthew Murdoch

31.6k32 gold badges99 silver badges127 bronze badges

answered Mar 26, 2009 at 2:04

Mitch Wheat

302k44 gold badges482 silver badges552 bronze badges

Comments

venkat · Accepted Answer · 2013-04-15 17:35:32Z

3

You should do it this way:

public interface IDoesCoolThings 
{
   void DoCool();
}

public abstract class AbstractWidget 
{
   public void DoCool()
   {
      Console.WriteLine("I did something cool.");
   }
}

public class Widget : AbstractWidget, IDoesCoolThings 
{
}

Usage:

var widget = new Widget();
widget.DoCool();

edited Apr 15, 2013 at 17:35

venkat

5,77816 gold badges62 silver badges83 bronze badges

answered Jan 30, 2013 at 7:38

Toffee

4,6362 gold badges18 silver badges6 bronze badges

Collectives™ on Stack Overflow

Using interfaces on abstract classes in C#

4 Answers 4

3 Comments

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

3 Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related