Проверка, если ViewBag имеет свойство или нет, условно инъекционные JavaScript

Question

Рассмотрим простой контроллер:

Porduct product = new Product(){ 
    // Creating a product object; 
}; 
try 
{ 
    productManager.SaveProduct(product); 
    return RedirectToAction("List"); 
} 
catch (Exception ex) 
{ 
    ViewBag.ErrorMessage = ex.Message; 
    return View("Create", product); 
} 

Сейчас, на мой Create зрения, я хочу, чтобы проверить ViewBag объект, чтобы увидеть, если он имеет Error собственности или нет. Если у него есть свойство error, мне нужно добавить некоторый JavaScript на страницу, чтобы показать сообщение об ошибке моему пользователю.

Я создал метод расширения, чтобы проверить это:

public static bool Has (this object obj, string propertyName) 
{ 
    Type type = obj.GetType(); 
    return type.GetProperty(propertyName) != null; 
} 

Затем, в Create зрения, я написал эту строку кода:

@if (ViewBag.Has("Error")) 
{ 
    // Injecting JavaScript here 
} 

Однако, я получаю эту ошибку:

Cannot perform runtime binding on a null reference

Есть идеи?

Answer 1

Ваш код не работает, потому что ViewBag является dynamic object не 'реальный' тип.

следующий код должен работать:

public static bool Has (this object obj, string propertyName) 
{ 
    var dynamic = obj as DynamicObject; 
    if(dynamic == null) return false; 
    return dynamic.GetDynamicMemberNames().Contains(propertyName); 
} 

Answer 2

Я бы полностью избежал ViewBag здесь. См. Мои мысли здесь: http://completedevelopment.blogspot.com/2011/12/stop-using-viewbag-in-most-places.html

Альтернативой было бы выбросить пользовательскую ошибку и поймать ее. откуда вы знаете, если база данных не работает, или если ошибка бизнес-логики сохраняет ошибку? в приведенном выше примере вы просто поймаете единственное исключение, как правило, есть лучший способ поймать каждый тип исключения, а затем общий обработчик исключений для действительно необработанных исключений, таких как встроенные страницы пользовательских ошибок или использование ELMAH.

Так выше, я бы вместо ModelState.AddModelError() Вы можете посмотреть на этих ошибках (если вы Арент этого нужно просто собираетесь использовать встроенные проверки) через How do I access the ModelState from within my View (aspx page)?

Поэтому, пожалуйста, внимательно рассмотреть отображающим когда вы поймаете «любое» исключение.

Answer 3

Вместо использования ViewBag используйте ViewData, чтобы вы могли проверить предмет, который вы храните. Объект ViewData используется в качестве словаря объектов, к которым вы можете ссылаться по ключу, он не является динамическим, как ViewBag.

// Set the [ViewData][1] in the controller 
ViewData["hideSearchForm"] = true;  

// Use the condition in the view 
if(Convert.ToBoolean(ViewData["hideSearchForm"]) 
    hideSearchForm(); 

Answer 4

@if (ViewBag.Error!=null) 
{ 
    // Injecting JavaScript here 
} 

Answer 5

Вы можете использовать ViewData.ContainsKey("yourkey").

В контроллере:

ViewBag.IsExist = true; 

В свете:

if(ViewData.ContainsKey("IsExist")) {...}