小知识:C#可选参数的一个陷阱

时间:2022-12-26 15:50:36
一、背景:
互联网行业,为了降低程序维护、升级的部署风险,往往会将程序拆分成很多项目,编译成多个dll部署,这样发布的时候,只需要部署修改过的dll即可。
 
二、问题:
有一个函数,在很多个地方被使用:
public fun1(A a ,B b)
{
//代码主体
}
突然有一天,有的地方调用的时候需要加入一个参数C c,但是又不想其他客户程序有任何变动,可以充分利用.net4.0新增的可选参数特性,这样改:
方法一:使用可选参数
public fun1(A a ,B b,C c = )
{
//代码主体
}
程序修改完后,在本地程序完美运行,将dll发测试。在测试环境,莫名其妙的报错。由于系统分层处理,web站点和Web API分离,错误返回到最上层,已经看不到错误原因,查了老半天日志,终于发现是另外一个dll(未重新编译部署)调用了fun1函数,报找不到方法 fun1的错误。
 
 
将程序修改为:
方法二:重载
public fun1(A a ,B b)
{
fun1(A a ,B b,);
}
public fun1(A a ,B b,C c)
{
//代码主体
}
重新发布dll,程序正常运行。
 
三、初步结论:
 
将代码修改成可选参数,客户程序需要重新编译,不然客户程序调用的时候会报找不到方法的错误。如果,客户程序太多,为了避免部署所有客户程序的dll,可以用方法二,重载的方法。
 
四、验证:
方法无默认参数:
 
小知识:C#可选参数的一个陷阱
 
小知识:C#可选参数的一个陷阱
 
方法有可选参数:
小知识:C#可选参数的一个陷阱
 
客户程序Program编译后的IL:
 
小知识:C#可选参数的一个陷阱
 
五、最终结论:
     由图可知,客户程序的c#源码虽然没有变,但是编译后的IL中间代码确改变了,调用的函数也传了两个参数,并把函数默认参数888在客户程序声明,使用。
可以得出的结论是,C#可选参数函数其实是在编译的时候,在函数调用的客户程序做了处理,把默认参数传了进去。
 
六、现在问题来了,大家请思考
微软为什么要这样做呢?为什么不在编译的时候,IL像以下代码一样,做类似重载的处理呢? 这样的话就不用重新编译客户程序了
 
public fun1(A a ,B b)
{
fun1(A a ,B b,);
}
public fun1(A a ,B b,C c)
{
//代码主体
}

七、网友解答:感谢34楼CYJB和其他网友的热心解答,非常详尽

楼主为方法添加了一个默认参数,那么这个新的方法和之前的方法的签名是完全不同的——对应的参数个数是不同的,所以才会出现找不到方法的错误。.Net 里的可选参数就像 @冲杀 说的,就是一个语法糖,仅仅是编译器在方法调用的时候自动加上了默认参数值而已,编译器完全可能不支持这一功能。

实际上,根据 VS “代码分析”功能,可以发现公共方法使用默认参数,是具有一定风险的,因为:在公共语言规范 (CLS) 中允许方法使用默认参数;但是 CLS 允许编译器忽略为这些参数分配的值。 为忽略默认参数值的编译器编写的代码必须为每个默认参数显式提供变量。 为了跨编程语言维护所需的行为,必须使用提供默认参数的方法重载来替换使用默认参数的方法。完整的文档可以参考 MSDN CA1026:不应使用默认参数

最后,公共方法的确是应该只扩展不修改的,以保持良好的兼容性,楼主应该是没有想到 C# 的默认参数只是一个语法糖,而不慎中招了。而微软为何没有将默认方法实现为多个重载,不是不愿,而是不能。例如下面的方法:

public void TestMethod(int a = , int b = , int c = )
{
// ...
}

如果自动实现为多个重载,显然是不可能的,因为参数的类型都是 int,无法根据参数类型来区分出参数 a、b 和 c,因此微软会完全放弃这个方案。而代码膨胀之类的,可能也在微软的考虑范围内。