ASP.NET MVC Как преобразовать ошибки ModelState в json

127

Как получить список всех сообщений об ошибках ModelState? Я нашел этот код, чтобы получить все ключи: ( Возвращает список ключей с ошибками ModelState )

var errorKeys = (from item in ModelState
        where item.Value.Errors.Any() 
        select item.Key).ToList();

Но как мне получить сообщения об ошибках в виде IList или IQueryable?

Я мог бы пойти:

foreach (var key in errorKeys)
{
    string msg = ModelState[error].Errors[0].ErrorMessage;
    errorList.Add(msg);
}

Но это делается вручную - конечно, есть способ сделать это с помощью LINQ? Свойство .ErrorMessage находится так далеко по цепочке, что я не знаю, как писать LINQ ...

JK.
источник

Ответы:

192

Вы можете поместить в предложение все, что хотите select:

var errorList = (from item in ModelState
        where item.Value.Errors.Any() 
        select item.Value.Errors[0].ErrorMessage).ToList();

РЕДАКТИРОВАТЬ : вы можете выделить несколько ошибок в отдельные элементы списка, добавив fromпредложение, например:

var errorList = (from item in ModelState.Values
        from error in item.Errors
        select error.ErrorMessage).ToList();

Или:

var errorList = ModelState.Values.SelectMany(m => m.Errors)
                                 .Select(e => e.ErrorMessage)
                                 .ToList();

2- е РЕДАКТИРОВАНИЕ : вы ищете Dictionary<string, string[]>:

var errorList = ModelState.ToDictionary(
    kvp => kvp.Key,
    kvp => kvp.Value.Errors.Select(e => e.ErrorMessage).ToArray()
);
SLaks
источник
Это быстрый ответ :)! Выглядит неплохо, но что, если в ModelState [item.Key] больше одной ошибки? Ошибки [0] работает только для одного сообщения об ошибке
JK.
Как вы хотите их совместить?
SLaks
Спасибо, почти все, но он выбирает каждый ключ, даже если он не содержит ошибок - как мы можем отфильтровать ключи без ошибок?
JK.
4
Add.Where(kvp => kvp.Value.Errors.Count > 0)
SLaks
3
Чтобы получить тот же результат, что и у Request.CreateErrorResponse(HttpStatusCode.BadRequest, ModelState);вас, следует использовать var errorList = modelState.Where(elem => elem.Value.Errors.Any()) .ToDictionary( kvp => kvp.Key, kvp => kvp.Value.Errors.Select(e => string.IsNullOrEmpty(e.ErrorMessage) ? e.Exception.Message : e.ErrorMessage).ToArray());В противном случае у вас не будет ExceptionMessages
Сильвос
74

Вот полная реализация со всеми частями вместе:

Сначала создайте метод расширения:

public static class ModelStateHelper
{
    public static IEnumerable Errors(this ModelStateDictionary modelState)
    {
        if (!modelState.IsValid)
        {
            return modelState.ToDictionary(kvp => kvp.Key,
                kvp => kvp.Value.Errors
                                .Select(e => e.ErrorMessage).ToArray())
                                .Where(m => m.Value.Any());
        }
        return null;
    }
}

Затем вызовите этот метод расширения и верните ошибки из действия контроллера (если есть) как json:

if (!ModelState.IsValid)
{
    return Json(new { Errors = ModelState.Errors() }, JsonRequestBehavior.AllowGet);
}

И, наконец, покажите эти ошибки на стороне клиента (в стиле jquery.validation, но его можно легко изменить на любой другой стиль)

function DisplayErrors(errors) {
    for (var i = 0; i < errors.length; i++) {
        $("<label for='" + errors[i].Key + "' class='error'></label>")
        .html(errors[i].Value[0]).appendTo($("input#" + errors[i].Key).parent());
    }
}
JK.
источник
Это выглядит интересным методом, однако вспомогательный класс у меня не работает. Возможно, это связано с изменениями в MVC 2? Я получаю сообщение об ошибке, что метод ToDictionary не существует в modelState.
Cymen
@Cymen, вы забыли сослаться на System.Linq? ToDictionary () - это метод расширения LINQ.
Натан Тейлор,
8
В зависимости от ваших предпочтений .Where(m => m.Value.Count() > 0)можно также написать как .Where(m => m.Value.Any()).
Manfred
Его можно использовать аналогично ModelState.ToDataSourceResult () из Kendo.Mvc, чтобы возвращать ошибки в сетку и отображать сообщения об ошибках при редактировании.
malnosna
22

Мне нравится использовать Hashtableздесь, так что я получаю объект JSON со свойствами в качестве ключей и ошибками в качестве значения в форме массива строк.

var errors = new Hashtable();
foreach (var pair in ModelState)
{
    if (pair.Value.Errors.Count > 0)
    {
        errors[pair.Key] = pair.Value.Errors.Select(error => error.ErrorMessage).ToList();
    }
}
return Json(new { success = false, errors });

Таким образом вы получите следующий ответ:

{
   "success":false,
   "errors":{
      "Phone":[
         "The Phone field is required."
      ]
   }
}
Йовица Зарич
источник
8

Есть много разных способов сделать это, и все они работают. Вот теперь я это делаю ...

if (ModelState.IsValid)
{
    return Json("Success");
}
else
{
    return Json(ModelState.Values.SelectMany(x => x.Errors));
}
Дин Норт
источник
2
Вы также можете вернуться, BadRequest(ModelState)и он сериализует его в JSON для вас.
Фред
6

Самый простой способ сделать это - просто вернуть BadRequestс самим ModelState:

Например, на PUT:

[HttpPut]
public async Task<IHttpActionResult> UpdateAsync(Update update)
{
    if (!ModelState.IsValid)
    {
        return BadRequest(ModelState);
    }

    // perform the update

    return StatusCode(HttpStatusCode.NoContent);
}

Если мы используем аннотации данных, например, на номер мобильного телефона, например, в Updateклассе:

public class Update {
    [StringLength(22, MinimumLength = 8)]
    [RegularExpression(@"^\d{8}$|^00\d{6,20}$|^\+\d{6,20}$")]
    public string MobileNumber { get; set; }
}

Это вернет следующее при неверном запросе:

{
  "Message": "The request is invalid.",
  "ModelState": {
    "update.MobileNumber": [
      "The field MobileNumber must match the regular expression '^\\d{8}$|^00\\d{6,20}$|^\\+\\d{6,20}$'.",
      "The field MobileNumber must be a string with a minimum length of 8 and a maximum length of 22."
    ]
  }
}
Эрик А. Брандстадмоен
источник
1
BadRequest специфичен для WebAPI, и этот вопрос касается MVC.
rgripper 06
5

@JK мне это очень помогло, но почему бы и нет:

 public class ErrorDetail {

        public string fieldName = "";
        public string[] messageList = null;
 }

        if (!modelState.IsValid)
        {
            var errorListAux = (from m in modelState 
                     where m.Value.Errors.Count() > 0 
                     select
                        new ErrorDetail
                        { 
                                fieldName = m.Key, 
                                errorList = (from msg in m.Value.Errors 
                                             select msg.ErrorMessage).ToArray() 
                        })
                     .AsEnumerable()
                     .ToDictionary(v => v.fieldName, v => v);
            return errorListAux;
        }
h45d6f7d4f6f
источник
3

Взгляните на System.Web.Http.Results.OkNegotiatedContentResult.

Он преобразует все, что вы в него бросаете, в JSON.

Так я сделал это

var errorList = ModelState.ToDictionary(kvp => kvp.Key.Replace("model.", ""), kvp => kvp.Value.Errors[0].ErrorMessage);

return Ok(errorList);

Это привело к:

{
  "Email":"The Email field is not a valid e-mail address."
}

Мне еще предстоит проверить, что происходит, когда для каждого поля появляется более одной ошибки, но дело в том, что OkNegoriatedContentResult великолепен!

Получил идею linq / lambda от @SLaks

ozzy432836
источник
3

Простой способ добиться этого, используя встроенную функциональность

[HttpPost]
public IActionResult Post([FromBody]CreateDoctorInput createDoctorInput) {
    if (!ModelState.IsValid) {
        return BadRequest(ModelState);
    }

    //do something
}

Результат JSON будет

Nisfan
источник
2

ToDictionary - это расширение Enumerable, находящееся в System.Linq, упакованное в dll System.Web.Extensions http://msdn.microsoft.com/en-us/library/system.linq.enumerable.todictionary.aspx . Вот как для меня выглядит полный класс.

using System.Collections;
using System.Web.Mvc;
using System.Linq;

namespace MyNamespace
{
    public static class ModelStateExtensions
    {
        public static IEnumerable Errors(this ModelStateDictionary modelState)
        {
            if (!modelState.IsValid)
            {
                return modelState.ToDictionary(kvp => kvp.Key,
                    kvp => kvp.Value.Errors.Select(e => e.ErrorMessage).ToArray()).Where(m => m.Value.Count() > 0);
            }
            return null;
        }

    }

}
philrabin
источник
2

Почему бы не вернуть исходный ModelStateобъект клиенту, а затем использовать jQuery для чтения значений. Для меня это выглядит намного проще и использует общую структуру данных (.netModelState )

вернуть ModelState как Json, просто передайте его конструктору класса Json (работает с ЛЮБЫМ объектом)

C #:

return Json(ModelState);

ЯШ:

        var message = "";
        if (e.response.length > 0) {
            $.each(e.response, function(i, fieldItem) {
                $.each(fieldItem.Value.Errors, function(j, errItem) {
                    message += errItem.ErrorMessage;
                });
                message += "\n";
            });
            alert(message);
        }
d.popov
источник
1

Вариант с возвращаемым типом вместо возврата IEnumerable

public static class ModelStateHelper
{
    public static IEnumerable<KeyValuePair<string, string[]>> Errors(this ModelStateDictionary modelState)
    {
        if (!modelState.IsValid)
        {
            return modelState
                .ToDictionary(kvp => kvp.Key, kvp => kvp.Value.Errors.Select(e => e.ErrorMessage).ToArray())
                .Where(m => m.Value.Any());
        }

        return null;
    }
}
Джефф Чирчео
источник
0

Я сделал и расширение, которое возвращает строку с разделителем "" (можно использовать свое):

   public static string GetFullErrorMessage(this ModelStateDictionary modelState) {
        var messages = new List<string>();

        foreach (var entry in modelState) {
            foreach (var error in entry.Value.Errors)
                messages.Add(error.ErrorMessage);
        }

        return String.Join(" ", messages);
    }
Нияз Мухамедья
источник
-1
  List<ErrorList> Errors = new List<ErrorList>(); 


        //test errors.
        var modelStateErrors = this.ModelState.Keys.SelectMany(key => this.ModelState[key].Errors);

        foreach (var x in modelStateErrors)
        {
            var errorInfo = new ErrorList()
            {
                ErrorMessage = x.ErrorMessage
            };
            Errors.Add(errorInfo);

        }

если вы используете jsonresult, верните

return Json(Errors);

или вы можете просто вернуть modelStateErrors, я не пробовал. Я назначил коллекцию Errors моей модели ViewModel, а затем зациклил ее. В этом случае я могу вернуть свои ошибки через json. У меня есть класс / модель, я хотел получить источник / ключ, но все еще пытаюсь в этом разобраться.

    public class ErrorList
{
    public string ErrorMessage;
}
CyberNinja
источник