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

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

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

    +109

    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
    public static bool IsLocalAddress(IPAddress address)
{
   if (address.ToString() == IPAddress.Loopback.ToString())
   {
      return true;
   }
   foreach (IPAddress address2 in LocalAddresses)
   {
      if (address2.ToString() == address.ToString())
      {
         return true;
      }
   }
   return false;
}

    Выдрано решарпером из одного коммерческой библиотеки.

    absolut, 02 Марта 2012

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

  3. C# / Говнокод #9555

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    ...
XElement body = new XElement(obj.Body);

obj.HasText = body.Element("text") != null;
obj.HasPhoto = files!=null && body.Element("files");

obj.Body = new XElement(body);
...

    Lourian, 27 Февраля 2012

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

  4. Си / Говнокод #7393

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    int rnd() {
  srand( rand()*rand() );
  int r = 0;
  for (int i=0;i<=10;i++)
    r=rand();
  srand( ++r - time(NULL) );
  return rand()/r;
}

    NEED MOAR RANDOM NUMBERS!!!

    Fai, 29 Июля 2011

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

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

    +109

    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
    [Test]
public void TryToParseFromIntToString()
{
            string str="12";
            int resultat;
            bool accept=int.TryParse(str, out resultat);
            Assert.AreEqual(resultat,12);
            Assert.IsTrue(accept==true);
}

[Test]
public void TestEqualsWhenThe2NumbersAreReallyEquals()
{
            Assert.IsTrue(AmountComparer.Equals(123.3456m,123.3456m));
}

    Вот такие юнит-тесты мы сегодня вычищаем! Все знаки препинания и символы сохранены ;)
    Зато всегда зеленые! ;)

    _Ru55_, 14 Июля 2011

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

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

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if (File.Exists(local)) {
                    if (Connect()) {
                        if (ftpConnection.Exists(remote)) {
//code
                        }
                    }
                }

    Irdis, 27 Июня 2011

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

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

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    try
 {
//тут другой гавнокод связаный с регистрацией
}
catch(ApplicationException)
{
    this.ModelState.AddModelError("DuplicateUser", Strings.DuplicateUser);
    return View(model);
}

    Вот так, если приложение падает значит юзверь уже существует....

    biff, 07 Апреля 2011

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

  8. Си / Говнокод #5923

    +109

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    while(1) {
    :cycle
    goto cycleCode;
}

// ...

:cycleCode
//код
goto cycle;

    danilissimus, 08 Марта 2011

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

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

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    try
            {
                splash.Close(new TimeSpan(0, 0, 0, 3, 0));
            }
            catch (Exception ex) { System.Diagnostics.Debug.WriteIf(false, ex.ToString()); }

    Вот так вот эксепшн обработал. Помню, недавно сказал такую вещь "В нашем игровом движке содержится лучший код, который я когда-либо писал" про свой игровой движок. Казалось бы, удобная имплементация, структура и принципы ООП правильно используются. Но потом я увидел ЭТО (см. выше). Надо больше спать, видимо... %)

    RaZeR, 16 Февраля 2011

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

  10. Pascal / Говнокод #5172

    +109

    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
    procedure TfmLab3.mmVesClick(Sender: TObject);
var
  sngVes, sngFlag: single;
  intFlag: integer;
begin
  try
    sngVes:=StrToFloat(InputBox('Ввод исходных данных','Введите вес от 40 до 170',''));
    sngFlag:=sqrt(sngVes-40);
    if sngVes>170 then
      begin
        sngFlag:=1/intFlag;
        //ShowMessage(FloatToStr(sngFlag));
      end;
    edWeight.Text:=FloatToStr(sngVes);
  except
    on EConvertError do ShowMessage('Вводить можно только действительные числа!');
    on EInvalidOp do ShowMessage('Минимальный вес 40кг');
    on EZeroDivide do ShowMessage('Максимальный вес 170кг');
  end;
 
end;

    Брутальненькая лаба с венгеркой и исключениями.

    bugmenot, 07 Января 2011

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

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

    +109

    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
    31. 31
    32. 32
    private void btnSearch_Click(object sender, EventArgs e)
        {
            Thread thrd = new Thread(ShowProgress);
            if ((cbCategory.Text != "")
                && (txtBoxCriteria.Text != ""))
            {
                if (dgvFound.RowCount != 0)
                    dgvFound.Rows.Clear();// очистка результатов предыдущего поиска
                thrd.Start(); // запуск прогресса в дополнительном потоке
                frmSiteManager.tvDoc.BeginUpdate();
                frmSiteManager.tvDoc.Nodes[cbCategory.SelectedIndex].Expand();
                FindDocuments(frmSiteManager.tvDoc.Nodes[cbCategory.SelectedIndex].Nodes);
                if (thrd.IsAlive)
                {
                    Thread.Sleep(1000);
                    thrd.Abort();// завершить поток прогресса
                    thrd.Join();
                }
                fProgress.Close();
                frmSiteManager.tvDoc.EndUpdate();
                if (dgvFound.RowCount == 0)
                    MessageBox.Show("По Вашему запросу ничего не найдено.",
                                    "Результаты поиска",
                                    MessageBoxButtons.OK,
                                    MessageBoxIcon.Exclamation);
            }
            else
                MessageBox.Show("Задайте пожалуйста критерии поиска.",
                                    "Поиск документа",
                                    MessageBoxButtons.OK,
                                    MessageBoxIcon.Error);
        }

    обработчик нажатия btnSearch_Click - находится в другой форме. в ДокСеарч )))
    А теперь объясните что здесь не правильно ?

    Maxim546, 21 Декабря 2010

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