如何将参数传递给控制器​​构造函数?这是正确的做法吗?

时间:2013-11-23 20:20:33

标签: c# asp.net-mvc asp.net-mvc-4 asp.net-mvc-routing

我可能不会这样做,因为我是这个框架的新手,我希望你能帮忙^^

正如您在下面的代码中看到的,在每个动作中我都找到了要更新的表,然后我调用表中的方法。

public class TableController : Controller
{
    private Lobby L;

    public TableController()
    {
        L = Lobby.Instance;

    }

    public ActionResult Index(uint id)
    {
        Table T = L.Tables[id];
        return View(T);
    }

    public ActionResult AddPlayer(byte pos, uint id)
    {
         Table T = L.Tables[id];

        ...
        T.AddPlayer(p, pos);
        ...
    }
 ...

}

但是我注意到我在每个方法中都做同样的事情,所以我可以把表变成一个属性,所以我不需要为每个动作找到它。

我想有这样的事情:

public class TableController : Controller
{
    private Lobby L;
    private  Table T;


    public TableController(uint tableId)
    {
        L = Lobby.Instance;
        T = L.Tables[tableId];
    }

    public ActionResult Index()
    {
        return View(T);
    }

    public ActionResult AddPlayer(byte pos)
    {
        ...
        T.AddPlayer(p, pos);
        ...
    }

这种方法有什么不对吗?

如果这在概念上是正确的,我如何将表ID传递给我的构造函数? 这不起作用:(

        routes.MapRoute(
                "Table",
                "Table_{tableId}/{action}/",
                new { controller = "Table", action = "Index"}
            );

2 个答案:

答案 0 :(得分:4)

通常,Controller构造函数用于注入依赖项,而不是数据。此外,在此阶段,this.Request|Response|Session以及其他基本属性仍为null

请改为尝试:

protected override void Initialize(RequestContext requestContext)
{
    var tableId = Convert.ToUInt32(requestContext.RouteData.GetRequiredString("tableId"));

    L = Lobby.Instance;
    T = L.Tables[tableId];

    base.Initialize(requestContext);
}

答案 1 :(得分:0)

将参数传递给不属于控制器的操作。 每次请求发生时都会创建控制器。 如果您需要它们,可以使用静态属性。 可以从任何控制器或操作访问静态属性。