ModelState est à venir comme invalide?

Je suis en train de travailler sur un MVC5 Code-Première application.

Sur un Modèle de Edit() vue, j'ai compris [Create] boutons pour ajouter de nouvelles valeurs à d'autres modèles dans le Edit() la vue et de repeupler la nouvelle valeur dans DropDownFors() sur le Edit().

Pour cette première tentative, je suis de passage d'un model_description via AJAX à ma méthode de contrôleur 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);
}

Ce que je ne peut pas comprendre est pourquoi mon ModelState est à venir comme Invalid?

Toutes les propriétés sont spécifiées avant le ModelState vérifier; le Modèle est défini comme suit:

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; }
}

MODIFIER:

Ajouté Afficher le code:

Formulaire De Saisie:

        <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>

SCRIPT:

        $('#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!");
                }
            });
        });

Fournir le code de vous voir avec les données que vous envoyez.
Vous n'êtes pas publier un created_date ou created_by de sorte que votre ModelState est pas valide lorsque vous postez en arrière.
Pouvez-vous développer? Voulez-vous dire quand je suis à l'affichage de la Vue à partir de la manette?

InformationsquelleAutor Analytic Lunatic | 2015-02-13

asp.net-mvc asp.net-mvc-5 c#modelstate

3

Vous ne devriez pas utiliser un hack comme ModelState.Clear() ni TryValidateModel(model); nécessaire. Votre problème provient du fait que vous avez un [Required] attribut à la fois sur votre created_date et created_by propriétés, mais vous ne publiez pas de retour à une valeur, de sorte qu'ils sont null et la validation échoue. Si vous étiez à publier un modèle plus complexe, alors vous devez utiliser un modèle de vue qui n'ont même pas de propriétés pour created_date et created_by (ses une méthode de création, de sorte qu'ils ne devraient pas être ensemble jusqu'à ce que vous publiez en arrière).

Dans votre cas, un modèle de vue n'est pas nécessaire puisque votre seul affichage d'une seule valeur ( pour model-description) qui permet de créer un nouveau INV_Models modèle.

Changer la 2ème ligne dans le script pour
```
var data = { description: $('#textNewModel').val() };
```
Changer votre méthode post pour
```
[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);
}
```
Notes:
1. Je suggère modified_date être DateTime? (prendre la valeur null dans la base de données
  également). Vous êtes à la création d'un nouvel objet, et de la définition de la
  created_date et created_by propriétés, mais la définition d'
  modified_date et modified_by propriétés ne semble pas
  approprié (il n'a pas été modifié encore).
2. Je soupçonne que vous ne voulez pas vraiment à définir created_by à
  System.Environment.UserName (il serait inutile d'avoir
  chaque ensemble d'enregistrements à administrator ou quoi que UserName de l'
  serveur renvoie. Au lieu de cela vous avez besoin pour obtenir des noms d'utilisateurs, de Identity
  ou Membership quel que soit le système d'autorisation que vous utilisez.
- C'est pourquoi j'ai été tellement perdu sans l'aide de la ModelState.Clear() & TryValidate(). Tous les 3 propriétés avec [Required] ont une valeur. Le Description transmis au contrôleur via JSON, puis created_date/created_by étant donné une valeur dans le contrôleur (à droite avant l'enregistrement), mais sans compensation de l'état et de la revalidation, j'ai été encore une défaillance de l'erreur du modèle. Concernant les notes, j'ai mis en œuvre n ° 1 -- Ne DateTime? sur la classe principale, mais pas pour les autres pour une raison quelconque. #2 - Pas sûr de ce que la méthode va être encore utilisé. Juste une idée courante en attente d'examen.
- Je pensais que le ModelState.Clear() etc. peut être éviter si j'ai passé une valeur pour created_by et created_date avec mon model_description pour mon data variable, mais lorsque je tente var data = { model_description: document.getElementById('textNewModel').value, created_date: @DateTime.Now, created_by: @System.Environment.UserName }; le script ne s'exécute jamais au contrôleur, à défaut avec: "Nombre Inattendu" - var data = { model_description: document.getElementById('textNewModel').value, created_date: 2/17/2015 3:51:15 PM, created_by: JSmith };...?
- La raison, vous avez eu le problème est que vous avez des paramètres de méthode a été INV_Models model. La valeur par défaut ModelBinder première initialise une nouvelle instance du modèle, puis définit ses propriétés basées sur les valeurs envoyées. Parce que vous ne publiez pas n'importe quoi pour created_by sa valeur est la valeur par défaut (null) mais ensuite, un modèle d'état d'erreur est ajoutée en raison de la Required attribut. Il suffit de faire le paramètre string description comme ci-dessus.
- Vous ne devriez pas être en train d'essayer d'envoyer un created_date de la vue. Un utilisateur pourrait avoir commencé à le faire hier, se distraire et de le finir aujourd'hui. Votre code devrait définir la date de création de la base de données comme hiers date (j'.e la mauvaise date!) parce que @DateTime.Now est la date à laquelle la page a été chargée pour la première fois.
- À droite, à droite. Je crois que je vois ce que vous dites maintenant. Je ne savais pas que tu avais changé le paramètre du Contrôleur dans le code ci-dessus. Je vais essayer votre suggestion et de voir si tout fonctionne.Je crois que je comprends maintenant ce que tu veux dire à propos de la ModelBinder à la recherche d'une instance de mon modèle alors que mon paramètre est l'initialisation de l'autre.
- Merci Stephen, j'ai mis en place votre exemple ci-dessus, et tout semble fonctionner comme prévu -- si je ne me demande quelque chose. Quand/Où exactement (je suis en supposant que ModelBinding ou pendant Save()) le ID obtenir une valeur de définir les coulisses d'un nouveau Modèle d'enregistrement?
- Si votre ID propriété est une auto-incrémenté clé dans la base de données, EF mise à jour de la valeur lorsque vous ajoutez un nouvel enregistrement lorsque vous appelez db.SaveChangesAsync(); - mais pourquoi faites-vous ce que asynchrone?
- Honnêtement, pour aucune autre raison que c'est la façon dont la méthode a été généré en raison de m'vérification [X] create async methods lors de la création de la Controller.
InformationsquelleAutor
6

Lorsque vous ne pouvez pas rapidement en déduire pourquoi votre ModelState la validation échoue, il est souvent utile de rapidement effectuer une itération sur les erreurs.
```
foreach (ModelState state in ModelState.Values.Where(x => x.Errors.Count > 0)) { }
```
Sinon, vous pouvez sortir directement les erreurs.
```
var allErrors = ModelState.Values.SelectMany(x => x.Errors);
```
Gardez à l'esprit que la ModelState construit AVANT le corps de votre Action est exécutée. En conséquence, IsValid sera déjà ensemble, indépendamment de la façon dont vous définissez vos propriétés du modèle une fois que vous êtes à l'intérieur de l'Action du Contrôleur.

Si vous voulez la flexibilité de définir manuellement les propriétés et puis re-evalute la validité de l'objet, vous pouvez le relancer manuellement la validation à l'intérieur de votre Action après la mise à l'propriétés. Comme indiqué dans les commentaires, vous devez effacer vos ModelState avant de tenter de le revalider.
```
ModelState.Clear();
ValidateModel(model);

try
{
    if (ModelState.IsValid)
    {
        db.INV_Models.Add(model);
        db.SaveChangesAsync();
    }
}
...
```
En aparté, si le modèle n'est toujours pas valide ValidateModel(modèle) lèvera une exception. Si vous voulez éviter cela, utilisez de l' TryValidateModel, qui renvoie la valeur true/false à la place:
```
protected internal bool TryValidateModel(Object model)
```
- Merci pour la réponse de David. J'ai essayé ValidateModel() et TryValidateModel(), mais les deux sont de retour en tant que Invalid. Ensuite, j'ai utilisé votre approche avec allErrors et mettre une montre sur allErrors. Le forage vers le bas dans le [Results View]=>[0]=>[ErrorMessage] - je obtenir "The created_by field is required." j'ai confirmé que je SUIS en précisant le created_by valeur...?
- de modèle.created_by = Système.De l'environnement.Nom d'utilisateur; Êtes-vous bien certain que ce n'est pas null? Rappelez-vous, [Obligatoire] sera de retour invalide si votre valeur est nulle sur un type de référence.
- Correct, model.created_by = System.Environment.UserName n'est pas null. Je l'ai eu à travailler en faisant un ModelState.Clear() AVANT d'essayer TryValidateModel(model). Une fois que cela se produit, tout fonctionne comme prévu. J'ai aussi dû changer db.SaveChanges() vs db.SaveChangesAsync() sinon la valeur n'aurez ajouté DB si j'avais un point d'arrêt. Sans, le code de traiter rapidement et de rentrer avant la DB ajouter a eu lieu.
- Intéressant, je ne vois rien dans votre code qui nécessiterait db.SaveChangesAsync();. Peu importe, je vais mettre à jour ma réponse à tenir compte de la nécessité pour ModelState.Clear();
InformationsquelleAutor David L
1

Le modèle de l'état est calculée lors de la liaison à partir de votre poste de données de modèle est fait.
Le ModelState.La propriété IsValid ne vous raconte pas si il ya des erreurs dans ModelState.Erreurs.

Lorsque vous définissez votre date de création, vous devrez supprimer l'erreur liées à de ModelState.Les erreurs
- Merci pour la réponse de Mike. Je ne savais pas que le ModelState a été vérifié lors de la liaison d'appel (si cela fait sens). J'ai confirmé à l'aide de David est l'approche que le ModelState semble le penser il est invalide en raison de la created_by champ est requis, mais je suis en précisant que la valeur??
- Comme je l'ai mentionné dans ma réponse, vous devrez supprimer l'erreur de la ModelState. Voir la réponse à cette question stackoverflow.com/questions/2588588/...
- Désolé, je pensais que TryValidateModel() et ValidateModel() serait effacer l'erreur et effectuer une nouvelle Validation? Qu'est-ce exactement aurais-je préciser à Remove?
- vous devez spécifier la même clé que dans ModelState.Errors qui, je crois, serait "created_date"
- Merci Mike! Je ne suis pas sûr de ce qui serait mieux, mais j'ai juste réussi à le faire fonctionner en appelant ModelState.Clear() avant TryValidateModel(model).
- Correction, il a travaillé une fois... et maintenant plus rien. Il semble que si je marche dans le code où il ajoute la nouvelle description de DB, il fonctionne. Mais si je le laisse tourner sans point de rupture, la nouvelle description est ajoutée à la liste déroulante, mais PAS de la DB. Je suppose que peut-être db.SaveChangesAsync() n'est pas totalement terminer avant le retour se fait?
InformationsquelleAutor Mike Norgate

Vous devez vous connecter pour publier un commentaire.