избежать NullReferenceException в Request.QueryString

Этот код бросает NullReferenceException если mode не указана в строке запроса страницы:избежать NullReferenceException в Request.QueryString

bool isAdvancedMode = Request.QueryString["mode"].Equals("advanced");

Это, как я работаю вокруг этого:

bool isAdvancedMode = (Request.QueryString["mode"] + "").Equals("advanced");

Является ли этот стандарт практики или взлома?

источник

2013-07-18 Tom Gullen

Вы можете использовать null-coalescing operator:

bool isAdvancedMode = (Request.QueryString["mode"] ?? String.Empty).Equals("advanced");

Edit: Если вы хотите повторно использовать эту логику, попробуйте этот метод расширения:

public static bool EqualIfExists(this string source, string comparison) 
{ 
    return source != null && source.Equals(comparison); 
} 

Request.QueryString["mode"].EqualIfExists("advanced")

Добавить еще переопределение, чтобы соответствовать Equals подпись , Я не уверен, что это хорошее имя (я думаю, что это не так).

источник

2013-07-18 14:51:32

Ницца. Посмотрим, какой он выбирает! – AndrewK

Аха. В прошлый раз @newStackExchangeInstance и я (например, 30 минут назад) ответили на те же вопросы, что и на 30 секунд, я выиграл. Я чувствую, что нужно идти к нему. Так +1 к нему. ;) –

Спасибо за ответы, подумайте, что это идет к Саймону, как он первым ответил! –

Почему бы не использовать нулевой коалесцирующий оператор?

bool isAdvancedMode = (Request.QueryString["mode"] ?? String.Empty).Equals("advanced");

источник

2013-07-18 14:51:34 AndrewK

Ну, на самом деле, он компилируется в тот же код. Не верьте мне? http://ericlippert.com/2013/06/17/string-concatenation-behind-the-scenes-part-one/ –

Я не говорил, что это не так, просто заявив, что оператор с нулевым коалесцентом больше удобочитаемый. – AndrewK

Я знаю, просто интересный факт дня :) –

Ну, я бы рекомендовал вместо этого:

bool isAdvancedMode = (Request.QueryString["mode"] ?? "").Equals("advanced");

На самом деле, это то, что your code compiles to (ближе к нижней части, но это хорошо читать, так что я прочитал все). Ваша хорошая практика, но это немного более ясно.

источник

2013-07-18 14:52:10

Спасибо за ссылку! –

Другой подход, в то время как немного больше кода, я думаю, более ясное намерение.

bool isAdvancedMode = String.IsNullOrWhitespace(Request.QueryString["mode"]) ? 
         false : Request.QueryString["mode"].Equals("advanced")

источник

2013-07-18 14:53:21 EkoostikMartin

Хотя это хорошо, мне кажется, что мне грязно, чтобы получить доступ к значению словаря дважды (хотя в большинстве случаев это _O (1) _). Кроме того, это может быть сведено к '! String.IsNullOrWhitespace (Request.QueryString [" mode "]) && Request.QueryString [" mode "]. Equals (" advanced ")' –

@SimonBelanger хорошо понимает доступ к словарю, случай я бы использовал ваши ?? метод тоже :) – EkoostikMartin

, что об этом

bool isAdvancedMode=(Request.QueryString["mode"] ?? string.Empty).Equals("advanced");

источник

2013-07-18 14:54:35 Ehsan

Ищете один вкладыш, так как эта проблема - обычное явление –

избежать NullReferenceException в Request.QueryString

ответ

Смежные вопросы