是否确定改变可选参数

Question

值

修改参数值被认为是一种反模式，但我觉得它在C＃中的可选参数有时是有用的：

public void Foo(int p1, MyClass fooObj = null) 
{ 
    if (fooObj == null) 
    { 
     fooObj = LoadFooObj(.... 
    } 

    . . . 
} 

是这里的东西可能有害的我可能会丢失？

谢谢。

Answer 1

这绝对没问题。事实上，这是一个让参数可选的好方法，不需要将数值作为常量进行烘焙。

您可以使用空合并运算符，使其略微更具可读性，但：

public void Log(string message, DateTime? timestamp = null) 
{ 
    DateTime actualTimestamp = timestamp ?? DateTime.UtcNow; 
    ... 
} 

一个缺点：

fooObj = fooObj ?? LoadFooObj(); 

使用值类型相同的方法，你甚至可以考虑这是因为它可以防止null被用作“正常”有意义的值 - 虽然 - 考虑您是否会在特定情况下需要这样的值。

Answer 2

如果你觉得它有味道，那么简单的超载怎么样。

public void Foo(int p1, MyClass fooObj) 
{ 
    . . . 
} 

public void Foo(int p1) 
{ 
    var fooObj = LoadFooObj(....); 
    Foo(p1, fooObj); 
} 

这样就明确了每种方法的作用，并且不会改变调用中的参数。

Answer 3

我要去争辩乔恩斯基特完全相反，并说这不是罚款，原因有二：

1. 尽可能，所有变量（包括参数）应被视为是不可变的。如果你真的必须改变它们的价值。这导致更清晰，更易于理解的代码。
2. 避免可选参数。一个带有可选参数的方法，然后立即测试该参数是一个清晰的代码味道：你有两条路径通过代码，所以使它有两种方法。

你可能只是重载Foo，但想想这些方法的作用：他们可能应该被赋予不同的名称来描述他们做不同事情的事实。不要依赖评论来解释这一点;用代码本身清楚说明。