【问题标题】:ModelState is coming up as invalid?ModelState 出现无效?
【发布时间】:2015-02-13 17:31:41
【问题描述】:

我正在开发一个 MVC5 Code-First 应用程序。

在一个模型的Edit() 视图中,我包含了[Create] 按钮,用于从Edit() 视图中向其他模型添加新值,然后在DropDownFors() 上的DropDownFors() 中重新填充新值。

对于第一次尝试,我通过 AJAX 将 model_description 传递给我的控制器方法 createNewModel()

[HttpPost]
public JsonResult createNewModel(INV_Models model)
{
    // model.model_description is passed in via AJAX -- Ex. 411

    model.created_date = DateTime.Now;
    model.created_by = System.Environment.UserName;
    model.modified_date = DateTime.Now;
    model.modified_by = System.Environment.UserName;

    // Set ID
    int lastModelId = db.INV_Models.Max(mdl => mdl.Id);
    model.Id = lastModelId+1;

    //if (ModelState.IsValid == false && model.Id > 0)
    //{
    //    ModelState.Clear();
    //}

    // Attempt to RE-Validate [model], still comes back "Invalid"
    TryValidateModel(model);

    // Store all errors relating to the ModelState.
    var allErrors = ModelState.Values.SelectMany(x => x.Errors);

    // I set a watch on [allErrors] and by drilling down into
    // [allErrors]=>[Results View]=>[0]=>[ErrorMessage] I get
    // "The created_by filed is required", which I'm setting....?

    try
    {
        if (ModelState.IsValid)
        {
            db.INV_Models.Add(model);
            db.SaveChangesAsync();
        }

    }
    catch (Exception ex)
    {
        Elmah.ErrorSignal.FromCurrentContext().Raise(ex);
    }

    return Json(
        new { ID = model.Id, Text = model.model_description },
        JsonRequestBehavior.AllowGet);
}

我不明白为什么我的ModelState 会变成Invalid

ModelState 检查之前指定了所有属性;模型定义如下:

public class INV_Models
{
    public int Id { get; set; }

    [Required(ErrorMessage = "Please enter a Model Description.")]
    public string model_description { get; set; }

    [Required]
    [DisplayFormat(DataFormatString = "{0:MM/dd/yyyy}")]
    public DateTime created_date { get; set; }

    [Required]
    public string created_by { get; set; }

    [DisplayFormat(DataFormatString = "{0:MM/dd/yyyy}")]
    public DateTime modified_date { get; set; }

    public string modified_by { get; set; }
}

编辑

添加查看代码:

输入表格

        <span class="control-label col-md-2">Type:</span>
        <div class="col-md-4">
            @Html.DropDownListFor(model => model.Type_Id, (SelectList)ViewBag.Model_List, "<<< CREATE NEW >>>", htmlAttributes: new { @class = "form-control dropdown" })
            @Html.ValidationMessageFor(model => model.Type_Id, "", new { @class = "text-danger" })
        </div>
        <div class="col-md-1">
            <div class="btn-group">
                <button type="button" class="btn btn-success" aria-expanded="false">CREATE NEW</button>
            </div>
        </div>

脚本

        $('#submitNewModel').click(function () {

            var form = $(this).closest('form');
            var data = { model_description: document.getElementById('textNewModel').value };

            $.ajax({
                type: "POST",
                dataType: "JSON",
                url: '@Url.Action("createNewModel", "INV_Assets")',
                data: data,
                success: function (resp) {
                    alert("SUCCESS!");
                    $('#selectModel').append($('<option></option>').val(resp.ID).text(resp.Text));
                    alert("ID: " + resp.ID + " // New Model: " + resp.Text); // RETURNING 'undefined'...?
                    form[0].reset();
                    $('#createModelFormContainer').hide();
                },
                error: function () {
                    alert("ERROR!");
                }
            });
        });

【问题讨论】:

  • 提供您查看的代码以及您发送的数据。
  • 您没有发回 created_datecreated_by,因此您的 ModelState 在发回时无效。
  • @jumpingcode,你能详细说明一下吗?你的意思是当我从控制器发回视图时?

标签: c# asp.net-mvc asp.net-mvc-5 modelstate


【解决方案1】:

当您无法快速推断出 ModelState 验证失败的原因时,快速迭代错误通常很有帮助。

foreach (ModelState state in ModelState.Values.Where(x => x.Errors.Count > 0)) { }

您也可以直接提取错误。

var allErrors = ModelState.Values.SelectMany(x => x.Errors);

请记住,ModelState 是在执行 Action 主体之前构建的。因此,无论您在 Controller Action 中如何设置模型的属性,都已经设置了 IsValid。

如果您希望灵活地手动设置属性,然后重新评估对象的有效性,您可以在设置属性后在 Action 内手动重新运行验证。如 cmets 中所述,您应该在尝试重新验证之前清除您的 ModelState。

ModelState.Clear();
ValidateModel(model);

try
{
    if (ModelState.IsValid)
    {
        db.INV_Models.Add(model);
        db.SaveChangesAsync();
    }
}
...

顺便说一句,如果模型仍然无效,ValidateModel(model) 将抛出异常。如果您想阻止这种情况,请使用 TryValidateModel,它会返回 true/false:

protected internal bool TryValidateModel(Object model)

【讨论】:

  • 感谢大卫回复。我尝试了ValidateModel()TryValidateModel(),但仍然以Invalid 的形式返回。然后,我将您的方法与allErrors 一起使用,并关注allErrors。深入到[Results View]=&gt;[0]=&gt;[ErrorMessage] 我得到"The created_by field is required." 我已经确认我正在指定created_by 值...?
  • @AnalyticLunatic model.created_by = System.Environment.UserName;你确定这不是空的吗?请记住,如果引用类型的值为 null,[Required] 将返回无效。
  • 正确,model.created_by = System.Environment.UserName 不是 null。在我尝试TryValidateModel(model) 之前,我通过ModelState.Clear() 让它工作。一旦发生这种情况,一切都会按预期运行。我还必须更改为db.SaveChanges() vs db.SaveChangesAsync(),否则只有在我有断点的情况下才会将值添加到数据库中。否则,代码将在数据库添加发生之前快速处理并返回。
  • 有趣,我在您的代码中看不到任何需要 db.SaveChangesAsync(); 的内容。无论如何,我会更新我的答案以反映对 ModelState.Clear(); 的需求
【解决方案2】:

您不应该使用像 ModelState.Clear() 这样的 hack,也不需要 TryValidateModel(model);。您的问题源于您在 created_datecreated_by 属性上都有 [Required] 属性,但您没有回发值,因此它们是 null 并且验证失败。如果您要回发一个更复杂的模型,那么您将使用一个视图模型,它甚至没有 created_datecreated_by 的属性(它是一个 Create 方法,因此在您回发之前不应设置它们)。

在您的情况下,不需要视图模型,因为您只发回一个用于创建新 INV_Models 模型的值(对于 model-description)。

将脚本中的第二行改为

var data = { description: $('#textNewModel').val() };

将您的发布方式更改为

[HttpPost]
public JsonResult createNewModel(string description)
{
  // Initialize a new model and set its properties
  INV_Models model = new INV_Models()
  {
    model_description = description,
    created_date = DateTime.Now,
    created_by = System.Environment.UserName
  };
  // the model is valid (all 3 required properties have been set)
  try
  {
    db.INV_Models.Add(model);
    db.SaveChangesAsync();
  }
  catch (Exception ex)
  {
    Elmah.ErrorSignal.FromCurrentContext().Raise(ex);
  }
  return Json( new { ID = model.Id, Text = model.model_description }, JsonRequestBehavior.AllowGet);
}

旁注:

  1. 我建议modified_dateDateTime?(在数据库中可以为空 还)。您正在创建一个新对象,并且正在设置 created_datecreated_by 属性,但设置 modified_datemodified_by 属性似乎没有 合适(尚未修改)。
  2. 我怀疑你并不想将created_by 设置为 System.Environment.UserName (有没有意义 每条记录都设置为administrator 或任何UserName 服务器返回。相反,您需要从 Identity 获取用户名 或 Membership 您使用的任何授权系统。

【讨论】:

  • 这就是为什么我在没有使用ModelState.Clear()TryValidate() 的情况下如此迷失的原因。 [Required] 的所有 3 个属性都被赋予了一个值。 Description 通过 JSON 传递给控制器​​,然后 created_date/created_by 在控制器中被赋予一个值(在保存之前),但没有清除状态并重新验证,我仍然得到一个无效的模型错误。关于旁注,我已经实现了#1——在主类上做了DateTime?,但由于某种原因没有在其他类上实现。 #2 - 不确定将使用什么方法。只是一个正在等待审核的当前想法。
  • 我认为如果我将created_bycreated_date 的值以及model_description 的值传递给我的data 变量,则可以避免ModelState.Clear() 等,但是当我尝试@ 987654357@ 脚本永远不会执行到控制器,失败:“Unexpected Number” - var data = { model_description: document.getElementById('textNewModel').value, created_date: 2/17/2015 3:51:15 PM, created_by: JSmith };...?
  • 你遇到这个问题的原因是你的方法参数是INV_Models model。默认ModelBinder 首先初始化模型的新实例,然后根据发布的值设置其属性。因为您没有为 created_by 发布任何内容,所以它的值是默认值 (null),但是由于 Required 属性而添加了模型状态错误。只需像上面那样设置参数string description即可。
  • 您不应该尝试从视图中发送created_date。用户可能昨天开始做,今天分心并完成它。您的代码会将数据库中的创建日期设置为昨天的日期(即错误的日期!),因为@DateTime.Now 是页面首次加载的日期。
  • 对对对。我想我明白你现在在说什么了。我没有意识到您在上面的代码中更改了 Controller 参数。我将尝试您的建议,看看是否一切正常。我相信我现在明白您对 ModelBinder 查看我的模型的一个实例而我的参数正在初始化另一个实例的意思了。
【解决方案3】:

模型状态是在您的帖子数据与模型的绑定完成时计算的。 ModelState.IsValid 属性仅告诉您 ModelState.Errors 中是否存在错误。

当您设置创建日期时,您需要从 ModelState.Errors 中删除与其相关的错误

【讨论】:

  • 感谢迈克的回复。我没有意识到在绑定调用期间检查了ModelState(尽管这是有道理的)。我已经确认使用 David 的方法 ModelState 似乎认为它是无效的,因为 created_by 字段是必需的——但我指定了该值??
  • @AnalyticLunatic 正如我在回答中提到的,您需要从ModelState 中删除错误。查看这个问题的答案stackoverflow.com/questions/2588588/…
  • 抱歉,我以为TryValidateModel()ValidateModel() 会清除错误并进行新的验证?我究竟会为Remove 指定什么?
  • @AnalyticLunatic 您需要指定与ModelState.Errors 中相同的键,我相信它是"created_date"
  • 谢谢迈克!我不确定哪个会更好,但我只是在TryValidateModel(model) 之前调用ModelState.Clear() 让它工作。
猜你喜欢
  • 2018-04-10
  • 2017-05-30
  • 2012-03-09
  • 1970-01-01
  • 1970-01-01
  • 2019-08-01
  • 2020-02-10
  • 1970-01-01
  • 2011-02-09
相关资源
最近更新 更多