- 1
- 2
- 3
long id;
try { id = long.Parse(e.Item.Cells[0].Text); }
catch { id = 0; }
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 3
+132
long id;
try { id = long.Parse(e.Item.Cells[0].Text); }
catch { id = 0; }
Эта жемчужина прекрасно описывает весь говнокод производимый различными группами людей в моей компании.
+123
private bool IsInt(object ValueToCheck)
{
int Dummy = new int();
string InputValue = Convert.ToString(ValueToCheck);
//If user enters 45.00 This should not be allowed
//User must enter numbers without .00
if(InputValue.Contains("."))
return false;
bool Int = int.TryParse(InputValue, System.Globalization.NumberStyles.Any, null, out Dummy);
return Int;
}
Уже другой индусский автор наговнокодил. Орфография сохранена. Причем он сам себе в ногу выстрелил используя NumberStyles.Any...
Вот как надо:
private static bool IsInt(string valueToCheck)
{
int dummy;
return int.TryParse(valueToCheck, System.Globalization.NumberStyles.None, null, out dummy);
}
+115
char[] splitter = { ',' };
string types = hashtable[FlagsEnumValue].ToString();
string[] typesStringArray = types.Split(splitter, StringSplitOptions.RemoveEmptyEntries);
ArrayList typesArray = new ArrayList();
foreach (string str in typesStringArray)
{
foreach (string type in Enum.GetNames(typeof(FlagsEnum)))
{
if (type == str.Trim())
{
typesArray.Add((FlagsEnum)Enum.Parse(typeof(FlagsEnum), str, true));
break;
}
}
}
foreach (FlagsEnum type in typesArray)
{
if ((someObject.field & type) > 0)
{
typeFound = true;
}
else
{
typeFound = false;
break;
}
}
Автор хотел чтобы его любили. Точнее он хотел сконвертировать строковое представление битового енама в инам и сравнить по маске с проперти обьекта. Если бы автор прочел документацию то написал бы так:
string types = hashtable[FlagsEnumValue].ToString();
if (types != "")
{
FlagsEnum enum = (FlagsEnum)Enum.Parse(typeof(FlagsEnum), types, ignoreCase: true);
if ((enum & someObject.field) == enum)
typeFound = true;
}