sprawdzając, czy ViewBag ma właściwość lub nie, warunkowo wstrzyknąć JavaScript

Question

Rozważmy prosty kontroler:

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

Teraz, moim Create widzenia, chcę sprawdzić ViewBag obiektu, aby zobaczyć, czy to ma Error własność lub nie. Jeśli ma właściwość błędu, muszę wstrzyknąć trochę kodu JavaScript na stronę, aby wyświetlić komunikat o błędzie dla mojego użytkownika.

I stworzył metodę rozszerzenia aby to sprawdzić:

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

Następnie w widoku Create pisałem ten wiersz kodu:

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

Jednak uzyskać ten błąd:

Cannot perform runtime binding on a null reference

Każdy pomysł?

Answer 1

Twój kod nie robi praca bo ViewBag to nie dynamic object a 'prawdziwy' typ.

następujący kod powinien działać:

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

Unikałbym tutaj całkowicie ViewBag. Zapoznaj się z moimi przemyśleniami na ten temat: http://completedevelopment.blogspot.com/2011/12/stop-using-viewbag-in-most-places.html

Alternatywą byłoby rzucenie błędu niestandardowego i uchwycenie go. skąd wiesz, czy baza danych nie działa, czy też logika biznesowa zapisuje błąd? w powyższym przykładzie po prostu wychwycisz jeden wyjątek, zazwyczaj istnieje lepszy sposób na wychwycenie każdego typu wyjątku, a następnie ogólny wyjątek dla naprawdę nieobsługiwanych wyjątków, takich jak wbudowane niestandardowe strony błędów lub użycie ELMAH.

Więc powyżej, chciałbym zamiast ModelState.AddModelError() Następnie można spojrzeć na tych błędów (zakładając, że nie jest z ciebie jsut zamiar użyć wbudowanego w walidacji) poprzez How do I access the ModelState from within my View (aspx page)?

Więc proszę dokładnie rozważyć, wyświetlając wiadomość po złapaniu "jakiegokolwiek" wyjątku.

Answer 3

Zamiast korzystać z ViewBag, należy użyć ViewData, aby można było sprawdzić, czy przedmiot jest przechowywany. Obiekt ViewData jest używany jako słownik obiektów, do których można odwoływać się za pomocą klucza, nie jest dynamiczny jak 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

Można użyć ViewData.ContainsKey("yourkey").

W sterowniku:

ViewBag.IsExist = true; 

Biorąc:

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