Question

I’m having trouble getting the C# compiler to call an extension method I created, since its preferring an instance method with a params argument instead.

For example, say I have the following class and its method:

public class C
{
    public void Trace(string format, params object[] args)
    {
         Console.WriteLine("Called instance method.");
    }
}

And and extension:

public static class CExtensions
{
    public void Trace(this C @this, string category, string message, params Tuple<string, decimal>[] indicators)
    {
        Console.WriteLine("Called extension method.");
    }
}

In my sample program:

public void Main()
{
    var c = new C();

    c.Trace("Message");
    c.Trace("Message: {0}", "foo");
    c.Trace("Category", "Message", new KeyValuePair<string, decimal>("key", 123));
}

All calls print Called instance method..

I do not have access to class C, obviously, or I wouldn’t bother creating extension methods, and my extension is important because it would allow my users to continue using a class that they already know with some added value.

From what I’ve understood, the compiler will favor instance methods over extension methods, but is this the only rule? That would mean that any class with a method that looks like Method(string format, params object[] args) cannot have extension methods with a first parameter of type string.

Any explanation on the reasons for this behavior or a way to work around it (that is not “simply call CExtensions.Trace(c, "Category", ...“) would be greatly appreciated.

Answer 1

You can’t use extensions to “take over” existing class methods.

If the call works without the extension, the behaviour should not change when you add the extension. The reason for this is that existing code should not break by introducing an extension later on.

You have to use a different name for it, or use parameter types different from the class method.