Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5706)
2. C++ (2739)
3. Куча (2363)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (586)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. Java / Говнокод #5858

   +78

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   static List sort(List unsorted, Comparator comparator) { 
       List sorted = new ArrayList(unsorted); 
       Collections.sort(unsorted, comparator);
       return sorted;
   }

   feodal, 03 Марта 2011

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

2. JavaScript / Говнокод #5857

   +135

   1. 1

   <SCRIPT LANGUAGE=JavaScript src="../../../../../vstavka.txt"></script>

   http://www.helloworld.ru/texts/comp/lang/php/spravscript/stranica13.html что статьи говно что сайт.

   Vasiliy, 02 Марта 2011

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

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

   +146

   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
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66

   case enter:
   						{
   							TreeNodeBackColorChange();
   							if (Connection.Login == "" || Connection.Login == null)
   							{
   								new fmlogin().ShowDialog();
   								try
   								{
   									if (Connection.Login != "")
   										foreach (TreeNode item in tvMenuList.Nodes)
   										{
   											if (item.Name == lk)
   											{
   												item.NodeFont = new Font("arial", 10, FontStyle.Bold);
   												item.Text += " (" + Connection.Login + ")";
   											}
   										}
   								}
   								catch { }
   							}
   							else MessageBox.Show("Вы уже авторизованы!", "Вход в личный кабинет", MessageBoxButtons.OK, MessageBoxIcon.Warning);
   							break;
   						}
   					case leave:
   						{
   							TreeNodeBackColorChange();
   							if (Connection.Login != "" && Connection.Login != null)
   							{
   								if (MessageBox.Show("Вы уверены, что хотите выйти?", "Выход", MessageBoxButtons.OKCancel, MessageBoxIcon.Question) == DialogResult.OK)
   								{
   									Connection.Login = "";
   									Connection.Pass = "";
   									try
   									{
   										foreach (TreeNode item in tvMenuList.Nodes)
   										{
   											foreach (TreeNode item2 in item.Nodes)
   											{
   												foreach (TreeNode item3 in item2.Nodes)
   												{
   													if (item3.Name == lk)
   													{
   														item.NodeFont = tvMenuList.Font;
   														item3.Text = "Личный кабинет";
   													}
   												}
   												if (item2.Name == lk)
   												{
   													item.NodeFont = tvMenuList.Font;
   													item2.Text = "Личный кабинет";
   												}
   											}
   											if (item.Name == lk)
   											{
   												item.NodeFont = tvMenuList.Font;
   												item.Text = "Личный кабинет";
   											}
   										}
   									}
   									catch { }
   									MessageBox.Show("Выход произведен успешно!", "Выход", MessageBoxButtons.OK, MessageBoxIcon.Information);
   								}
   							}
   							else MessageBox.Show("Вы не авторизованы!", "Выход из личного кабинета", MessageBoxButtons.OK, MessageBoxIcon.Warning);
   							break;
   						}

   извиняюсь) форматирование сбивается когда из студии вставляю)
   PS жалко что нельзя вставить больше 100 строк. А тут такие красивые функции есть, которые теряют всю свою зрелишность при их урезании

   slavenin, 02 Марта 2011

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

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

   +116

   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
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60

   if (rbNewPackage.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id_user == ent.id
                                            select recb.date).Max() && r.id_status == 1
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }
                   else if (rbCancelled.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id_user == ent.id
                                            select recb.date).Max() && r.id_status == 5
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }
                   else if (rbConditional.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id == rp.id
                                            select recb.date).Max() && r.id_status == 3
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }
                   else if (rbOfficial.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id == rp.id
                                            select recb.date).Max() && r.id_status == 4
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }

   говногод моего бывшего коллеги по работе))

   slavenin, 02 Марта 2011

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

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

   +118

   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

   foreach (var aiett in _selectApplItemEntranceTrial)
               {
               lbl2:
                   ReceptionReason_EducationLayout rrel_temp = null;
                   EntranceTrialItem_RecResEducLayout eti_rrel = null;
                   bool flag = false;
                   foreach (var rrel in _recReasonEducLay)
                       foreach (var ap in aiett.ListAppItem)
                           foreach (var rr in rrel.ListRecReas_EducLayout)
                               if (ap.ApplItem.Id_ReceptionReason_EducationLayout == rr.Id)
                               {
                                   rrel_temp = rr;
                                   eti_rrel = rrel;
                                   flag = true;
                                   goto lbl;
                               }
               lbl: ;
                   if (flag)
                   {
                       _recReasonEducLay.FirstOrDefault(x => x == eti_rrel).ListRecReas_EducLayout.Remove(rrel_temp);
                       if (_recReasonEducLay.FirstOrDefault(x => x == eti_rrel).ListRecReas_EducLayout.Count == 0)
                           _recReasonEducLay.Remove(eti_rrel);
                       goto lbl2;
                   }
               }

   Мой личный говнокод! Работающий правильно))))

   slavenin, 02 Марта 2011

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

6. Си / Говнокод #5853

   +143

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7

   char str[256];
           char ** pstr =  (char **) &str;
           char *p = str;
   
           spprintf( &p, 0, "Ack errror; code=%d", res); 
           cnn->is_connected = 0;
           zend_throw_exception(amqp_queue_exception_class_entry, *pstr  ,0 TSRMLS_CC);

   Так похапешники пишут на C.

   symbix, 02 Марта 2011

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

7. C++ / Говнокод #5852

   +159

   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

   TCHAR aS[22];
            m_pCurrentLine->mainPortStatistics.TOs = 0;
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.TOs);
   	SetDlgItemText(IDC_EDIT_MAIN_TIMEOUTS, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_MODE = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_MODE);
   	SetDlgItemText(IDC_EDIT_MAIN_MODE, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_BREAK = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_BREAK);
   	SetDlgItemText(IDC_EDIT_MAIN_BREAK, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_FRAME = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_FRAME);
   	SetDlgItemText(IDC_EDIT_MAIN_FRAME, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_RXOVER = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_RXOVER);
   	SetDlgItemText(IDC_EDIT_MAIN_RXOVER, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_TXFULL = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_TXFULL);
   	SetDlgItemText(IDC_EDIT_MAIN_TXFULL, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_OVERRUN = 0;
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_OVERRUN);
   	SetDlgItemText(IDC_EDIT_MAIN_OVERRUN, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_RXPARITY = 0;
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_RXPARITY);
   	SetDlgItemText(IDC_EDIT_MAIN_RXPARITY, aS);

   каждому элементу структуры mainPortStatistics присваивается значение 0, затем каждый раз форматируется строка и засылается в функцию SetDlgItemText. Вменяемый человек пользуется для обнуления структуры функцией memset а затем ставит текст "0" во все controls. А говнокодер думает ,что чем больше дерьмокода развести - тем круче

   ShuraKotov, 02 Марта 2011

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

8. PHP / Говнокод #5851

   +160

   1. 1

   $VAR['server'] = trim(str_replace("www.","",strtolower($_SERVER['SERVER_NAME'])));

   Угадайте что этот код должен делать, и что он делает на самом деле. Да, это ужас. Это один из многих ужасов, доставшихся мне от предыдущего программиста. До этого самым большим был explode имени файла через точку и подставление второго элемента как расширение.

   BiSe_Trojanov, 02 Марта 2011

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

9. Си / Говнокод #5850

   +146

   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
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49

   // vba-c.h
   #include <stdio.h>
   #include <conio.h>
   
   #ifndef VBAC
   #   define VBAC
   #   define Sub void
   #   define EndSub }
   #   define main main() {
   #   define Dim __int32
   #   define As
   #   define Integer ;
   #   define Print( a ) printf( "%i\n", a );
   #   define Wait getch();
   #endif
   
   //main.c
   #include "vba-c.h"
   
   Sub main
       Dim a As Integer
       Dim b As Integer
       Dim c As Integer
       Dim d As Integer
       Dim mb, mc As Integer
   
       a = 10;
       b = 25;
       c = 34;
       d = 34;
       mb = 0x00000002;
       mc = 0x00000030;
   
       b = b - 1;
       c = c - 1;
       a = a + b + c;
       a = a / 3;
       b = b / 4;
       a = a - 1;
       b = b + 1;
       c = c - a + b;
       d = ( b & mb ) & ( c & mc );
   
       Print( a )
       Print( b )
       Print( c )
       Print( d )
       Wait
   EndSub

   После полугода изучения VBA, привыкание к хорошим, годным языкам, как Си очень проблематично. Что и порождает такие вот костыли...

   DarkWolf, 02 Марта 2011

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

10. SQL / Говнокод #5849

    −863

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5

    ...........
    UNION ALL
    (SELECT CONCAT("Помещене: ", rm_pomeshenie.nazvanie, ". Этаж: ", rm_etaj.nomer, ". Комната: ", rm_komnata.nazvanie, ". Раб. место: ", rm_rab_mesto.nazvanie, ". Сист. блок: ", sb_tipi_sist_blokov.nazvanie) AS raspolojenie, u_tip_oborudovaniya.id AS tip_id, u_tip_oborudovaniya.nazvanie AS tip_nazvanie, u_modeli_oborudovaniya.id AS model_id, CONCAT(u_modeli_oborudovaniya.nazvanie, " ", u_modeli_oborudovaniya.parametri) AS model_nazvanie, u_ustroystva.id AS ustr_id, u_ustroystva.seriyniy_nomer AS ustr_seriya, u_ustroystva.dop_svedeniya AS ustr_dop_svedeniya FROM rm_pomeshenie, rm_etaj, rm_komnata, rm_rab_mesto, sb_tipi_sist_blokov, sb_sistemnie_bloki, u_tip_oborudovaniya, u_modeli_oborudovaniya, u_ustroystva WHERE (rm_pomeshenie.id = rm_etaj.id_pomesheniya) AND (rm_etaj.id = rm_komnata.id_etaja) AND (rm_komnata.id = rm_rab_mesto.id_komnati) AND (sb_sistemnie_bloki.id_v_tablice_naznacheniya = rm_rab_mesto.id) AND (sb_sistemnie_bloki.id_tipa = sb_tipi_sist_blokov.id) AND (sb_sistemnie_bloki.imya_tablici_naznacheniya="rm_rab_mesto") AND (u_ustroystva.id_v_tablice_naznacheniya = sb_sistemnie_bloki.id) AND (u_tip_oborudovaniya.id = u_modeli_oborudovaniya.id_tipa_oborudovaniya) AND (u_modeli_oborudovaniya.id = u_ustroystva.id_modeli) AND (u_ustroystva.imya_tablici_naznacheniya = "sb_sistemnie_bloki") ORDER BY tip_nazvanie, model_nazvanie)
    UNION ALL
    ...........

    Выборка устройств. Таких блоков шесть штук и все "склеены" через UNION ALL.
    Наговнокодил от души ))

    nikitasnv, 02 Марта 2011

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

• ← влево
• вправо →

• 2105
• …
• 579
• 578
• 577
• 576
• 575
• 574
• 573
• 572
• 571
• …
• 1