Koray
Koray

Reputation: 1806

Use "Optional, DefaultParameterValue" attribute, or not?

Is there any difference between using Optional and DefaultParameterValue attributes and not using them?

public void Test1([Optional, DefaultParameterValue("param1")] string p1, [Optional, DefaultParameterValue("param2")] string p2)
{
}

public void Test2(string p1= "param1", string p2= "param2")
{
}

both work:

Test1(p2: "aaa");
Test2(p2: "aaa");

Upvotes: 26

Views: 7356

Answers (3)

user743382
user743382

Reputation:

The difference is that by using the attributes explicitly, the compiler doesn't enforce the same strictness on type requirements.

public class C {
  // accepted
  public void f([Optional, DefaultParameterValue(1)] object i) { }

  // error CS1763: 'i' is of type 'object'. A default parameter value of a reference type other than string can only be initialized with null
  //public void g(object i = 1) { }

  // works, calls f(1)
  public void h() { f(); }
}

Note that even with DefaultParameterValue, you don't throw out type-safety: if the types are incompatible, this will still be flagged.

public class C {
  // error CS1908: The type of the argument to the DefaultParameterValue attribute must match the parameter type
  //public void f([Optional, DefaultParameterValue("abc")] int i) { }
}

Upvotes: 9

mybirthname
mybirthname

Reputation: 18137

namespace System.Runtime.InteropServices {

    using System;

    //
    // The DefaultParameterValueAttribute is used in C# to set 
    // the default value for parameters when calling methods
    // from other languages. This is particularly useful for 
    // methods defined in COM interop interfaces.
    //
    [AttributeUsageAttribute(AttributeTargets.Parameter)]
    public sealed class DefaultParameterValueAttribute : System.Attribute
    {
         public DefaultParameterValueAttribute(object value)
         {
             this.value = value;
         }

         public object Value { get { return this.value; } }

         private object value;
    }
}

They are doing the same job. You can check things like this in Roslyn or in ReferenceSource

Upvotes: 3

Marc Gravell
Marc Gravell

Reputation: 1064184

They compile identically, and the compiler works fine with either. The only difference is the lack of using System.Runtime.InteropServices;, and easier to read code.

For reference, the IL is:

.method public hidebysig instance void TheName([opt] string p1,
    [opt] string p2) cil managed
{
    .param [1] = string('param1')
    .param [2] = string('param2')
    .maxstack 8
    L_0000: ret 
}

where TheName is the only thing that changes.

Upvotes: 19

Related Questions