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

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

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

1. PHP (5709)
2. C++ (2740)
3. Куча (2390)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (38)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. Куча / Говнокод #5864

   +121

   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

   <taskdef resource="flexTasks.tasks" classpath="${FLEX_TASK_JAR}"/>
   
   		<target name="tetris_activity">
   			<mxmlc file="${FREELANCERS_DIR}\orientsys\skyblockwars\src\tetriswars.as"
   						output="${basedir}\${DEPLOY_DIR}\tetris\tetriswars.swf"
   						static-link-runtime-shared-libraries="true"
   						debug="false"
   						use-network="true">
   			           	<load-config filename="${FLEX_HOME}\frameworks\flex-config.xml"/>
   						<source-path path-element="${basedir}/LIB"/>
   						<source-path path-element="C:\freelancers\orientsys\skyblockwars\src"/>
   						<source-path path-element="${basedir}\RoundsDeveloperAPI\src"/>
   						<library-path dir="${basedir}\LIB\swc">
   							<include name="fl.swc"/>
   							<include name="as3corelib.swc"/>
   							<include name="greensock.swc"/>
   							<include name="icodecoder.swc"/>
   							<include name="maccherone-json.swc"/>
   							<include name="smartfox.swc"/>
   						</library-path>
   						<library-path dir="${FLEX_HOME}\frameworks\libs">
   							<include name="framework.swc"/>
   							<include name="textLayout.swc"/>
   						</library-path>
   			</mxmlc>
   			<echo>flickr</echo>			
   		</target>

   Просто нет слов... мудак, с которым я работаю удалил мои билд файлы и заменил вот этим. Я работаю в Убунту, и у меня даже эмулятора Виндовса нет.

   wvxvw, 03 Марта 2011

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

3. 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)

4. Си / Говнокод #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)

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

   +123

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12

   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);
   
   }

   piocsic, 01 Марта 2011

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

6. Java / Говнокод #5797

   +81

   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

   public void actionPerformed(java.awt.event.ActionEvent e)
   {
   	/* code	here */					
   	thr = new Potok(Spisok,f,papk);
   	thr.setWork(true);
   	try { thr.join(); } 
   	catch (InterruptedException e1) 
   	{
   		// TODO Auto-generated catch block
   		e1.printStackTrace();
   	}
   	/* another code here */
   }
   
   /* in the galaxy far-far away */
   class Potok extends Thread
   {
   	private boolean isWork = true;
   
   	public Potok() {}
   
   	public void run()
   	{
   		while ( isWork )
   		{
   			/* code here */
   		}
   	}
   	public void setWork(boolean work)
   	{
   		isWork = work;
   	}
   }

   Внутри обработчика нажатия на кнопку раскрывается вся суть многопоточных приложений.

   Elvenfighter, 24 Февраля 2011

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

7. PHP / Говнокод #5773

   +163

   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
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72

   <?$APPLICATION->IncludeFile('/inc/mail.php')?> <?$err=-1;?> <?if($_REQUEST['act']=="add" && $_REQUEST['text'] && $_REQUEST['phone'] && $_REQUEST['captcha'] && !isset($_REQUEST['from'])):?> <?$err=0;?> <?if($_SESSION['pass']!=$_REQUEST['captcha']):?>
   <div class="red"><b>Неверно введен код подтверждения!</b></div>
   <?$err=1;?> <?else:?> <?
   $el = new CIBlockElement;
   
   if($new_id = $el->Add($arLoadProductArray))
   {
   $sect2 = GetIBlockSection($_REQUEST['podr_'.$_REQUEST['rubrika']]);
   $sect1 = GetIBlockSection($_REQUEST['rubrika']);
   echo "Ваше объявление успешно принято и будет рассмотрено модератором";
   $domen = "http://www.auto-city.ru/";
   $mail_body = 'Добавлено новое объявление "'.$_REQUEST['text'].'" в рубрике "'.$sect1['NAME'].'"->"'.$sect2['NAME'].'" и ожидает модерации: <a href="'.$domen.'bitrix/admin/iblock_element_edit.php?ID='.$new_id.'&type=obyav&IBLOCK_ID=47">'.$domen.'bitrix/admin/iblock_element_edit.php?ID='.$new_id.'&type=obyav&IBLOCK_ID=47</a>';
   email('[email protected]', '[email protected]', 'Робот АвтоCity', 'Новое объявление на сайте', $mail_body);
   }
   else
   echo "Возникла ошибка при добавлении, обратитесь к слежбе поддержки";
   ?> <?endif?> <?elseif($_REQUEST['act']=="add"):?> <?if(!$_REQUEST['from']):?>
   <div class="red"><b>Заполните все поля!</b></div>
   <?else:?> <b>Пожалуйста, укажите подрубрику и введите код подтверждения</b> <?endif?> <?$err=1;?> <?endif?> <?if($err || $err==-1):?>
   <script language="JavaScript">
   lastKey = 0;
   function ch_r(obj)
   {
   document.getElementById('r_'+obj.value).style.display = 'block';
   document.getElementById('r_'+lastrid).style.display = 'none';
   lastrid = obj.value;
   }
   
   function checkForm(obj)
   {
   var str = obj.phone.value;
   var re = /[^\d ()-+]+/;
   for(i=0;i<1000;i++)
   obj.phone.value = obj.phone.value.replace(re, "");
   if(obj.phone.value!=str){
   alert('В поле "Номер телефона" необходимо вводить только цифры!');
   return false;
   
   }
   if(!obj.text.value || !obj.captcha.value || !obj.phone.value)
   {
   alert('Заполните все поля!');
   return false;
   }
   }
   /*
   function checkChar2(){
   var re = /[^\d ()-+]+/;
   for(i=0;i<1000;i++)
   obj.phone.value = obj.phone.value.replace(re, "");
   }
   function checkChar()
   {
   key = event.keyCode;
   if(key<48 || key>57)
   
   // || key==109 || key==46 || key==16 || key==8 || key==32 || key==37 || key==39 || key==36 || key==35 || (key==57 || key==48 || key==107 || key==36 || key==35 || key==37 || key==39 || key==51 || key==56))
   {
   alert(key);
   lastKey = key;
   return false;
   }
   lastKey = key;
   return true;
   }*/
   </script>
   <form name="iblock_add" method="post" onsubmit="return checkForm(this)">
   <table class="tabs" style="width: 100%; border-collapse: collapse;" id="give_obyav">
   <tbody>
   <tr> <td> Текст объявления: </td> <td style="width: 250px;"> <textarea maxlength="200" size="20" rows="5" cols="25" name="text" style="width: 100%;">&lt;img src=&quot;/bitrix/images/fileman/htmledit2/php.gif&quot; __bxsrc=&quot;/bitrix/images/fileman/htmledit2/php.gif&quot; border=&quot;0&quot; __bxtagname=&quot;php&quot; __bxcontainer=&quot;{'code': '&lt;?=$_REQUEST[\'text\']?&gt;'}&quot; /&gt;</textarea> </td> </tr>
   
   <tr> <td> Рубрика: </td> <td> <?$l1 = GetIBlockSectionList(47,0,Array("SORT"=>"ASC"))?> <select name="rubrika" onchange="ch_r(this);" style="width: 100%;"> </select> </td> </tr>

   qbasic, 22 Февраля 2011

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

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

   +173

   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

   $pattern = '/[^A-Za-zА-Яа-я0-9]/u';
   $phrase = $this->data['ProjectsPhrase']['phrase'];
   
   $phrase = preg_replace($pattern, ' ', $phrase);
   
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = str_replace('  ', ' ', $phrase);
   $phrase = trim($phrase);

   Вот такое говнокодище встретил в проекте. Паранойя такая паранойя…

   govnouzver, 17 Февраля 2011

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

9. JavaScript / Говнокод #5690

   +175

   1. 1

   $("input[id='yearB']")

   striker, 16 Февраля 2011

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

10. PHP / Говнокод #5662

    +162

    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

    <?php
    function pagination($page, $z, $url) 
       // номер стр, число страниц в табл, УРЛ)
       {
       if (!preg_match('/\\?/is', $url)) $url = $url.'?'; else $url = $url.'&';
       // если в адресе встречается ?, то в конце добавляем &, если нет - то ?
       
        echo "<div style='text-align:center;'> <a href='".$url."z=".$z."&&page=1'>Первая</a>&nbsp; &nbsp;";
        if ($page>1) print("<a href='".$url."z=".$z."&&page=".($page-1)."'>Предыдущая</a> &nbsp; &nbsp;");
            print ("Страница $page из $z &nbsp; &nbsp;");
            if ($page<$z) print("<a href='".$url."z=".$z."&&page=".($page+1)."'>Следующая</a>&nbsp; &nbsp;");
            echo "<a href='".$url."z=".$z."&&page=$z'>Последняя</a>";
            echo "<p style='margin-top:10px; margin-bottom:10px'>";
            
            for ($i = 1; $i <= $z; $i++) 
        if ($i == $page) echo "<span class='greenbig'> $i </span> &nbsp;";
        else echo "<a href='".$url."z=".$z."&&page=$i'>$i</a> &nbsp;"; 
            echo "</p></div>";
        }
     
     
    include('db.php');
    if(isset($_GET['page']))  $page=$_GET['page']; else $page="1";
    $plim=($page-1)*20;
    $query=mysql_query("SELECT * FROM zomplog_news ORDER BY `id` DESC LIMIT $plim, 20");
    while($row = mysql_fetch_array($query)) {
    $title = $row['title'];
    $text = $row['text'];
    $date = $row['date'];
     
    ?>
     
     
       <br><table class='panel'><tr class='phead'>
       <td><? echo "$title";?><td align='right'> <? echo "$date";?><tr><td colspan=2 class='pcontend'>
      <? echo "$text";?>
       <tr><td class='pend'>
       <td class='pend' align='right'></table>
     
     <?}
      $table = 'zomplog_news';
       if (isset($_GET['z'])) $z=$_GET['z']; else  
       {
       $r = mysql_query("SELECT COUNT(*) FROM $table");
       $z1 = mysql_result($r,0);
       $z = ceil($z1/20); ## z - общее число страниц
       }
    pagination ($page, $z, 'index.php');
     
    ?>

    qbasic, 13 Февраля 2011

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

11. PHP / Говнокод #5651

    +164

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

    class Dialog_view extends Controller {
    
        var $LutskZakarpatyeId = array('1225', '1233');
        var $respublicaKrim = '1229';
    
    ...
    
    }

    красота.

    shmaltorhbooks, 12 Февраля 2011

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

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

• 1975
• …
• 1139
• 1138
• 1137
• 1136
• 1135
• 1134
• 1133
• 1132
• 1131
• …
• 1