1. Лучший говнокод

    В номинации:
    За время:

  2. C# / Говнокод #6628

    +115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    ----Something.cs
public partial class Something
{
  // some implementation.
}
----Something.Bla.cs
partial class Something
{
  private class Bla
  {
  }
}
----Something.Foo.cs
partial class Something
{
  private class Foo
  {
  }
}
----Something.Bar.cs
partial class Something
{
  private class Bar
  {
  }
}

    Нормально ли использовать partial классы исключительно для хранения private nested классов? К примеру если количество таковых доходит до 5-15?

    walash, 11 Мая 2011

    Комментарии (14)

  3. Pascal / Говнокод #6422

    +115

    1. 1
    2. 2
    If Val = true then Proc(True)
 Else Proc(False);

    Just simple coprocode.

    Govnocoder#0xFF, 20 Апреля 2011

    Комментарии (12)

  4. C# / Говнокод #6125

    +115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    columnDomain.Visible =
 (grid.MainView.RowCount >
0
&&
 !String.IsNullOrEmpty(
     ((ListItem)
  grid.MainView.
     GetRow(0)).Domain)
&&
  ((ListItem)
 grid.MainView.GetRow(0))
      .Domain !=
 ((ListItem)
 grid.MainView.GetRow(0))
     .DisplayName);

    Это реальное форматирование кода, очевидно сделанное для удобства чтения на узком и высокои мониторе :) И такого многие и многие экраны

    eval_2009, 29 Марта 2011

    Комментарии (17)

  5. C# / Говнокод #5842

    +115

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    RepaymentEvent e = DatabaseHelper.GetString("event_type", pReader) == "RBLE"
	                ? new BadLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                : (DatabaseHelper.GetString("event_type", pReader) == "RRLE"
	                       ? new RescheduledLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                       :(DatabaseHelper.GetString("event_type", pReader).StartsWith("P") 
                                ? new PendingRepaymentEvent (DatabaseHelper.GetString("event_type", pReader)) {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
                                : new RepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}));

    Вот такую "элегантную" строчку нашел сегодня коллега в коде нашего проекта :)
    Создаем событие пойди разбери какое :)

    _Ru55_, 02 Марта 2011

    Комментарии (9)

  6. C# / Говнокод #5837

    +115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    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;
    }

    piocsic, 01 Марта 2011

    Комментарии (5)

  7. C# / Говнокод #5765

    +115

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    var model = Service.GetPlanGatheringIndicators(periodId).ToList();
ViewData["PeriodName"] = Service.GetPeriods().Where(p => p.Id == periodId).Select(p => p.TextAlias).FirstOrDefault();
var leafEls = model.Where(x => x.HasChildren == false);
var leafEl = (leafEls.Count() > 0) ? leafEls.First() : null;
ViewData["PlanActivated"] = ((leafEl != null) && (leafEl.PlanActivate == true)
		                             && (leafEl.PredictActivate == true)
		                             && (leafEl.FactActivate == true));

    Вот в таком чудесном виде и лежит, ожидая рефактора и форматирования.

    fr0mrus, 22 Февраля 2011

    Комментарии (18)

  8. C# / Говнокод #5696

    +115

    1. 1
    2. 2
    3. 3
    4. 4
    public bool SameTerritoryAsEditor(string terrName)
{
   return _personService.GetTerritoryByUserIdentity(Thread.CurrentPrincipal.Identity.Name).Name.Equals(terrName);
}

    Определяем принадлежность пользователя к определенной территории по её названию.

    St0rm, 16 Февраля 2011

    Комментарии (0)

  9. C# / Говнокод #5678

    +115

    1. 1
    2. 2
    3. 3
    4. 4
    ViewData["PlanActivated"] = ((model.Count > 0) && (model.First().PlanActivate == true) 
                 && (model.First().PredictActivate == true) 
                 && (model.First().FactActivate == true)) 
                 ? true : false;

    Даже не стал разбираться, стер и переписал....

    fr0mrus, 15 Февраля 2011

    Комментарии (2)

  10. C# / Говнокод #4917

    +115

    1. 1
    pi.UserName = pi.UserName.Trim().TrimStart(',').Trim();

    удаление запятой между именем и фамилией если фамилии нет.

    eval_2009, 14 Декабря 2010

    Комментарии (12)

  11. C# / Говнокод #4819

    +115

    1. 1
    2. 2
    if(!string.IsNullOrEmpty(((AccessLevelObject)cbAccessLevel.SelectedItem).Description))
        txtAccessLevelDescription.Text = ((AccessLevelObject)cbAccessLevel.SelectedItem).Description;

    догадайтесь что здесь лишнее )))

    Maxim546, 03 Декабря 2010

    Комментарии (14)