- 1
- 2
- 3
- 4
decimal d = DomainObject.AssignedPercent;
if (d.Equals(0) == false) {
...
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+109.6
decimal d = DomainObject.AssignedPercent;
if (d.Equals(0) == false) {
...
}
+118
void govnofunction(int m)
{
bool rigth = false, middle = false, left = false;
switch(m)
{
case 1: rigth = true; break
case 2: middle = true; break;
case 3: rigth = true; middle = true; break;
case 4: left = true; break;
case 5: left = true; rigth = true; break
case 6: left = true; middle = true; break;
case 7: left = true; rigth = true; middle = true; break;
}
...
}
писал по памяти, но общая картина сохранилась
+118.3
private string StrColor(decimal x)
{
if (x.ToString().Contains("-"))
{
string adj = x.ToString();
adj = string.Concat("(", adj.Substring(1, adj.Length - 1), ")");
//adj = System.Drawing.Color.Red.ToString();
return adj;
}
else
{
return x.ToString();
}
}
Определение знака числа в строке 3
+123.7
7. MyColorStops[0].Color = System.Drawing.Color.FromArgb(((int)(((byte)(161)))), ((int)(((byte)(255)))), ((int)(((byte)(67)))));
Лишние касты
+145
....
var parent = from o in objects where o.Id == oi.ParentId select o;
ObjectInfo grpInfo = null;
foreach (var grp in parent)
{
grpInfo = grp;
break;
}
.....
Чувак(Зимбабвийский программист) выбирает первый элемент(или не выбирает).
+152
try {
if (something) {
throw new MyGotoException();
}
DoSomethingElse();
}
catch (MyGotoException) {
}
DoGotoBlock();
Очень хитрый вариант реализации схемы goto.
+147
if (chk.Checked)
{
foreach (BV.Star.BLL.PersonGroups.PersonGroup trainee in trainees)
{
if (trainee.TraineeID.ToString() == gvwTrainees.DataKeys[gvwTrainees.PageIndex == 0 ? gvr.DataItemIndex : (gvr.DataItemIndex % gvwTrainees.PageSize)].Value.ToString())
goto Next;
}
InsertTraineeIntoGroup(Int32.Parse(ddlGroups.SelectedValue),
Int32.Parse(gvwTrainees.DataKeys[gvwTrainees.PageIndex == 0 ? gvr.DataItemIndex : (gvr.DataItemIndex % gvwTrainees.PageSize)].Value.ToString()),
false, DateTime.MaxValue,
"");
Next: ;
}
И снова на сцене - goto!
+143.1
if (((HttpContext.Current.Profile as ProfileCommon).Role == "3") || ((HttpContext.Current.Profile as ProfileCommon).Role == "4"))
{
if (e.Item.Value == " Add New Trainee ")
e.Item.Enabled = false;
if (e.Item.Value == " Add New Group ")
e.Item.Enabled = false;
if (e.Item.Value == " Audit Log ")
e.Item.Enabled = false;
if (e.Item.Value == " Manage Holidays ")
e.Item.Enabled = false;
}
if ( (( HttpContext.Current.Profile as ProfileCommon).Role != "0") && (( HttpContext.Current.Profile as ProfileCommon).Role != "") )
{
if (e.Item.Value == " Manage Users ")
e.Item.Enabled = false;
if (e.Item.Value == " Manage Courses ")
e.Item.Enabled = false;
if (e.Item.Value == " Audit Log ")
e.Item.Enabled = false;
}
if (((HttpContext.Current.Profile as ProfileCommon).Role == "") ||
((HttpContext.Current.Profile as ProfileCommon).Role == "0") ||
((HttpContext.Current.Profile as ProfileCommon).Role == "1") ||
((HttpContext.Current.Profile as ProfileCommon).Role == "2"))
{
if (e.Item.Value == " Holidays ")
e.Item.Enabled = false;
}
Новое место работы. Новые порции говнокода.
Это классика - no comments...
+145
/// <summary>
/// Child border.
/// </summary>
private Border childBorder;
Я бы никогда без комментария не догадался, зачем же это поле
+958.8
idString.Length == Guid.NewGuid().ToString.Length()
Самый имхо индусский (см. лукмор) код проверить что строка это Guid