带有可选参数的 MVC 操作——哪个更好？

Question

在您的动作签名中使用以下两种替代方案是否有任何优点/缺点：

public ActionResult Action(int? x) // get MVC to bind null when no parameter is provided
{
    if(x.HasValue)
    {
        // do something
    }
}

或

public ActionResult Action(int? x = null) // C# optional parameter (virtual overload)
{
    if(x.HasValue)
    {
        // do something
    }
}

Answer 1

我在实践中从未见过第二个动作签名，也看不出它有什么用处。

第一个通常涵盖所有场景：

• 如果不发送任何参数 (GET /somecontroller/action)，x 参数的值将在操作内为空
• 如果发送了 x 参数，但它不是有效的整数 (GET /somecontroller/action?x=abc)，则 x 参数的值将在操作中为 null，并且模型状态将无效
• 如果发送了 x 参数并且值表示有效整数 (GET /somecontroller/action?x=123)，则将为其分配 x。

在我的示例中，我使用了带有查询字符串参数的 GET 请求，但显然这同样适用于其他 HTTP 动词，如果 x 是一个路由参数。

Answer 2

如果它是null以外的任何其他值，您只需要指定可选参数值。

如果在重载或对 Action 的调用中未指定任何内容，MVC3 将自动将 null 设置为您的参数值。

但值得注意的是，如果签名中此参数后有任何非可选参数，则必须在调用中指定null。

因此最好将所有可选参数放在签名的末尾。

Answer 3

最佳 Asp.net MVC 解决方案 - 使用操作方法选择器

为什么不通过删除不必要的代码分支来简化控制器操作方法，并拥有如下所示的这种代码：

public ActionResult Index()
{
    // do something when there's no id
}

[RequiresRouteValues("id")]
public ActionResult Index(int id)
{
    // do something when id is present
}

这当然是可能的，只要您提供RequiresRouteValuesAttribute 操作方法选择器的非常简单的代码即可。您可以在 this blog post 中找到执行此操作的代码。

在我看来，这是解决这个问题的最佳方法，因为：

1. 它通过删除不必要的分支来简化代码
2. 使代码更易于维护（由于复杂性较低）
3. 尽可能扩展 Asp.net MVC 框架
4. 保持参数类型应有的状态，而不需要使它们可以为空
5. 等

无论如何。有关此技术的所有细节都在链接的帖子中进行了详细说明。