Asp.net MVC2 IModelBinder试图让我疯狂(并取得成功)

时间:2010-07-30 16:53:00

标签: asp.net-mvc-2 c#-4.0 imodelbinder

假设我有

class FooClass { }

class BarClass
{
    public FooClass Foo;
}

这个BarClass是我传递给ViewPage的模型。

我也传递(通过ViewData)一个IEnumerable<SelectListItem>,其中包含所有Foo,并且选择了与bar.Foo匹配的那个(在运行时检查)。

然后我打电话给Html.DropDownList("Foo", foos);

下拉列表呈现得很好,但它没有选择正确的项目,因为html控件具有属性的名称,并且它与内部运行的ViewData.Eval()混淆。这似乎是一种公认​​的行为(在SO上看到很多关于这个问题的答案),所以我不是在争论这个并且将对扩展的调用改为:

Html.DropDownList("DDL_Foo", foos);

选择了正确的值,我很高兴。所以我发回表格。

可悲的是,在我的控制器的相应Action中,Foo成员为null。所以我添加一个FooModelBinder来实现IModelBinder来拦截表单的DDL_Foo并正确初始化FooClass。

FooModelBinder.BindModel永远不会被解雇,bar.Foo为空。如果我再次更改我的视图并将下拉列表重命名为Foo,则FooModelBinder将按预期触发并且bar.Foo将按原样初始化。

那么,我错过了什么?更重要的是,我应该怎样做正确的方式。我想到了大量的黑客攻击和解决方法,但这不是我想要的。我想知道如何正确行事。

谢谢!

[编辑] 感谢您的反馈,但我不认为前缀是问题。

关于Binder,我添加了它,因为它无法正确初始化。请注意,我正在处理的实际案例比这里提供的内容更复杂。这个解决方案只是我可以做的最小的模型来重现这个问题。

这里是引用的代码(或download the full solution):

CONTROLLER

    [HttpGet]
    public ActionResult Index()
    {
        var dp = new DummyProvider();
        var bar = dp.GetBar();
        var foos = new List<SelectListItem>();

        dp.GetAllFoos().ForEach(
            f => foos.Add(new SelectListItem {Text = f.Name, Value = f.Id.ToString(), Selected = f.Id == bar.Foo.Id }));

        ViewData["foos"] = foos;

        return View(bar);
    }

    [HttpPost]
    public ActionResult Index(BarClass bar)
    {
        var dp = new DummyProvider();
        var foos = new List<SelectListItem>();

        dp.GetAllFoos().ForEach(
            f => foos.Add(new SelectListItem { Text = f.Name, Value = f.Id.ToString(), Selected = f.Id == bar.Foo.Id }));

        ViewData["foos"] = foos;
        ViewData["selectedItem"] = bar.Foo.Name;

        return View(bar);
    }

查看

<%
    var foos = ViewData["foos"] as List<SelectListItem>;

    using(Html.BeginForm())
    {
        %>
        <p>
            <h3>Enter Another Value</h3>
            <%= Html.TextBox("AnotherValue", Model.AnotherValue) %>
        </p>
        <p>
            <h3>Enter Yet Another Value</h3>
            <%= Html.TextBox("YetAnotherValue", Model.YetAnotherValue) %>
        </p>

        <p>
            <h3>Choose a foo</h3>
            <%= Html.DropDownList("DDL_Foo", foos)%>
        </p>
        <button type="submit">Send back !</button>
        <%
    } 
%>

MODEL

public class BarClass
{
    public FooClass Foo { get; set; }
    public string AnotherValue { get; set; }
    public string YetAnotherValue { get; set; }
}

public class FooClass
{
    public Guid Id { get; set; }
    public string Name { get; set; }

}

public class FooClassCollection : List<FooClass> { }

public class FooModelBinder : IModelBinder
{
    public object BindModel(ControllerContext controllerContext, ModelBindingContext bindingContext)
    {
        var foo = new FooClass();

        var guid = Guid.Empty;
        if (Guid.TryParse(controllerContext.HttpContext.Request.Form["DDL_Foo"], out guid))
        {
            foo.Id = guid;    
        }


        return foo;
    }
}

public class DummyProvider
{
    public FooClassCollection GetAllFoos()
    {
        return new FooClassCollection
                       {
                           new FooClass {Name = "Item 1", Id = new Guid("4a402abd-ab85-4065-94d6-d9fcc0f9b69e")},
                           new FooClass {Name = "Item 2", Id = new Guid("cf20bfd6-0918-4ffc-a6ec-c4cc4ed30e7f")},
                           new FooClass {Name = "Item 3", Id = new Guid("ad81b882-b93e-42b9-a42c-78376dd8f59d")},
                           new FooClass {Name = "Item 4", Id = new Guid("1511c15d-9ae4-4b18-9e10-e02588c21b27")},
                           new FooClass {Name = "Item 5", Id = new Guid("855e4a2f-fc5b-4117-a888-1dc3ebb990fc")},
                       };
    }

    public BarClass GetBar()
    {
        return new BarClass
                   {
                       AnotherValue = "Nice value",
                       YetAnotherValue = "This one is awesome",
                       Foo = new FooClass {Name = "Item 3", Id = new Guid("ad81b882-b93e-42b9-a42c-78376dd8f59d")}
                   };
    }
}

Global.asax中

    protected void Application_Start()
    {
        AreaRegistration.RegisterAllAreas();
        RegisterRoutes(RouteTable.Routes);

        ModelBinders.Binders.Add(typeof(FooClass), new FooModelBinder());
    }

[编辑] codeplex上有一个未解决的问题,如果你想解决它,请投票支持它(即使它已经开放近一年了)

2 个答案:

答案 0 :(得分:1)

我设法通过制作完成所有工作的BarClassModelBinder来完成所有工作。这是代码:

public class BarModelBinder : IModelBinder
{
    public object BindModel(ControllerContext controllerContext, ModelBindingContext bindingContext)
    {
        var bar = new BarClass();

        // In real code, check for nulls, etc.

        bar.AnotherValue = controllerContext.HttpContext.Request.Form["AnotherValue"];
        bar.YetAnotherValue = controllerContext.HttpContext.Request.Form["YetAnotherValue"];

        var guid = Guid.Empty;
        if (Guid.TryParse(controllerContext.HttpContext.Request.Form["DDL_Foo"], out guid))
        {
            bar.Foo = new FooClass {Id = guid};
        }


        return bar;
    }
}

因此,我在Controller中使用FormCollection再次看到的唯一好处是代码的清晰度。我唯一不满意的是字段名称在ModelBinder中“隐藏”,所以如果有人更改了视图,他必须非常小心字段名称。也许有一些方法可以绕过这个问题,也许有一个属性。但即使没有这个,这是较小的邪恶,所以我会解决它。

整个问题看起来仍然是DropDownListFor实现的不良副作用。

答案 1 :(得分:0)

花了半个小时玩这个。我不会为编写自定义模型绑定器而烦恼。我只使用视图模型而不是整个FooClass,而是使用Guid FooId。无论如何,你不会从下拉列表中获得更多。然后这将工作:

<%: Html.DropDownListFor(m => m.FooId, foos) %>

回发后,它会正确绑定FooId属性。

如果BarClass是域模型类,则视图模型可能如下所示(obv):

public class BarViewModel
{
    public Guid FooId { get; set; }
    public string AnotherValue { get; set; }
    public string YetAnotherValue { get; set; }
}