25

First time question for me :)

I need some way to define a default predicate using a generic on the format

Func<T, bool>

and then use this as a default argument. Something like this:

public bool Broadcast(byte command, MemoryStream data, bool async, Func<T, bool> predicate = (T t) => true)

When i do this i get the compile error:

Default parameter value for 'predicate' must be a compile-time constant

Is there a smooth way of doing this that I am missing or should a make the predicate function nullable and change my function logic accordingly?

Thanks,

Øyvind Bråthen
  • 59,338
  • 27
  • 124
  • 151
Bakery
  • 406
  • 1
  • 5
  • 10

3 Answers3

35

Default values for method parameters have to be compile-time constants, as the default values are actually copied to all the call sites of the method by the compiler.

You have to use an overload to do this:

public bool Broadcast(byte command, MemoryStream data, bool async) {
    return Broadcast(command, data, async, t => true);
}

public bool Broadcast(byte command, MemoryStream data, bool async, Func<T, bool> predicate) {
    // ...
}

Also, there is a specific Predicate<T> delegate in mscorlib which you can use instead. It's the same signature as Func<T, bool>, but it explicitly marks it as a delegate which decides whether an action is performed on instances of T

thecoop
  • 45,220
  • 19
  • 132
  • 189
3

Make an overload for Broadcast which does not take the last argument.

tenfour
  • 36,141
  • 15
  • 83
  • 142
3

Try this:

public bool Broadcast(byte command, MemoryStream data, bool async, Func<T, bool> predicate = default(Func<T, bool>))

But I think you have to check for predicate!=null.

Amir Rezaei
  • 4,948
  • 6
  • 33
  • 49
  • 3
    In this case i might as well just do public bool Broadcast(byte command, MemoryStream data, bool async, Func predicate = null) – Bakery Feb 19 '11 at 12:11