- 1
- 2
bool isManualExecute = (Request.QueryString["Manual"] != null && Request.QueryString["Manual"].ToLower() == "true")
|| (Request.QueryString["manual"] != null && Request.QueryString["manual"].ToLower() == "true");
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+121
bool isManualExecute = (Request.QueryString["Manual"] != null && Request.QueryString["Manual"].ToLower() == "true")
|| (Request.QueryString["manual"] != null && Request.QueryString["manual"].ToLower() == "true");
Передаю Вовчику привет!
Eugene 20.09.2010 16:11 # −2
bool isManualExecute = string.Equals(manual, bool.TrueString, StringComparison.InvariantCultureIgnoreC ase);
Semargl 20.09.2010 16:45 # +1
if (!bool.TryParse(Request.QueryString["Manual"], out isManual)) {
// set default value;
}
Eugene 20.09.2010 17:02 # +1
public static bool TryParse(string value, out bool result)
{
result = false;
if (value != null)
{
if ("True".Equals(value, StringComparison.OrdinalIgnoreCase))
{
result = true;
return true;
}
if ("False".Equals(value, StringComparison.OrdinalIgnoreCase))
{
result = false;
return true;
}
value = TrimWhiteSpaceAndNull(value);
if ("True".Equals(value, StringComparison.OrdinalIgnoreCase))
{
result = true;
return true;
}
if ("False".Equals(value, StringComparison.OrdinalIgnoreCase))
{
result = false;
return true;
}
}
return false;
}
Так чем твой вариант хуже моего?
Semargl 20.09.2010 18:04 # +1
1. Я использую то, что уже написано в .net
2. Ясно, пришло в качестве аргумента все же false или некорректные данные.
iqorek 20.09.2010 17:15 # +6
Pifagor 24.08.2021 02:09 # 0