Ambiguity with Action and Func parameter

Question

How is it possible that this code

TaskManager.RunSynchronously<MyObject>(fileMananager.BackupItems, package);

causes a compile error

The call is ambiguous between the following methods or properties:
'TaskManager.RunSynchronously<MyObject>(System.Action<MyObject>, MyObject)' and
'TaskManager.RunSynchronously<MyObject>(System.Func<MyObject, bool>, MyObject)'

when signature of the action is

public void BackupItems(MyObject package)

and "ambiguous" methods are

static class TaskManager
{
    public static void RunSynchronously<TInput>(Action<TInput> task, TInput param)
    {
        Task.Factory.StartNew(() => task(param));
    }

    public static bool RunSynchronously<TInput>(Func<TInput, bool> task, TInput param)
    {
        return Task.Factory.StartNew(() => task(param)).Result;
    }
}

It seems to me that there is an ample difference between these methods. What am I missing here?

EDIT:

Besides the accepted answer I just came across a solution in a similar question. Here is the link.

When you say “throws an exception”, do you mean “causes a compile error”? — svick, Sep 10 '13 at 10:53
@svick Yes, you are right. Sorry for misleading. Thank you, I edited the post. — Ondrej Janacek, Sep 10 '13 at 10:57
Possible duplicate of [Compiler Ambiguous invocation error - anonymous method and method group with Func<> or Action](https://stackoverflow.com/questions/2057146/compiler-ambiguous-invocation-error-anonymous-method-and-method-group-with-fun) — user247702, Jul 19 '17 at 08:24

Daniel Hilgarth · Accepted Answer · 2013-09-10T10:43:13.420

26

The reason is that the return type of a method is not part of its signature. Thus, while resolving the correct overload, the compiler only looks at the parameter of the method.

The easiest solution is to simply not use the implicit method group conversion. All of the following compile:

TaskManager.RunSynchronously<MyObject>(
    x => fileMananager.BackupItems(x), package);

TaskManager.RunSynchronously<MyObject>(
    (Action<MyObject>)fileMananager.BackupItems, package);

TaskManager.RunSynchronously<MyObject>(
    new Action<MyObject>(fileMananager.BackupItems), package);

The first one is the most elegant of them, but it also is the only one with a - slight - runtime performance impact, because of an additional redirection. However, this impact is so small that you actually shouldn't care.

edited Sep 10 '13 at 10:43

answered Sep 10 '13 at 10:18

Daniel Hilgarth

171,043
40
335
443

Is this valid for all methods or only lambdas? – Alessandro D'Andria Sep 10 '13 at 10:21
@AlessandroD'Andria: What exactly do you mean? – Daniel Hilgarth Sep 10 '13 at 10:22
I didn't realize that. Is there a way how to design my `TaskManager.RunSynchronously` method so that I would not need explicit conversion? – Ondrej Janacek Sep 10 '13 at 10:32
@OndrejJanacek: Right now, I don't see how, sorry. – Daniel Hilgarth Sep 10 '13 at 10:41
@DanielHilgarth All right, it is not the subject of this question. But if you ever come across a solution, please update your answer and let me know. By the way, what the magic variable `x` in the first solution stands for, when the `package` should be the parameter? – Ondrej Janacek Sep 10 '13 at 10:46
@OndrejJanacek: There is no magic variable there. This is a simple lambda expression. When the lambda is called, `x` will have the value of whatever you pass as the parameter to the lambda. If that's not clear, be sure to read up on lambdas. – Daniel Hilgarth Sep 10 '13 at 10:48

score 2 · Answer 2 · answered Sep 25 '19 at 08:34

Another possible explanation for this nowadays is this:

The code was written for C# version 7.3 (used by default by MSBuild 16.x, corresponding to VS2019), but the build is attempted with an earlier version of C# (which is the default for MSBuild 15.x, corresponding to VS2017).

Earlier versions of C# throw this error, but the overload is resolved correctly in C# 7.3.

score 0 · Answer 3 · answered Sep 10 '13 at 10:13

0

I came across the same problem, the solution was:

var r = RunSynchronously<bool>(x =>
{
    return true;
}, true);

RunSynchronously<bool>(x =>
{
}, true);

Now, why the compiler cannot do that???

answered Sep 10 '13 at 10:13

Alessandro D'Andria

8,663
2
36
32

1

You can use `x => true` instead of `x => { return true; }`. – Decade Moon Jan 06 '15 at 20:52

Ambiguity with Action and Func parameter

3 Answers3

Linked