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

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

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

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

1. JavaScript / Говнокод #3807

   +162

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

   function repeattimer(functionname, looptime){
   	eval(functionname+"();");
   	setTimeout("repeattimer(\""+functionname+"\", "+looptime+");", looptime);
   }

   Не знаю насчёт того, ГК это или нет, но я бы сделал совсем не таким способом.

   XyHb, 28 Июля 2010

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

2. PHP / Говнокод #3806

   +158

   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

   <div class="item" style="padding-bottom:60px">
   	<a class="no-border" href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><img src="/bitrix/templates/producer/i/items/dis-generator.jpg" alt="Дизельные генераторы" title="Дизельные генераторы"/></a>
   	<a href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y">Дизельные генераторы</a>
   </div>
   <div class="item">
   	<a class="no-border" href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y"><img src="/_producer/i/items/oil-generator.jpg" alt="Бензиновые генераторы" title="Бензиновые генераторы"/></a>
   	<a href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y">Бензиновые генераторы</a>
   </div>
   <div class="item">
   	<a class="no-border" href="catalogue/?SECTION_ID=333"><img src="/bitrix/templates/producer/i/items/generator.jpg" alt="Газовые генераторы" title="Газовые генераторы"/></a>
   	<a href="catalogue/?SECTION_ID=333">Газовые генераторы</a>
   </div>
   <div class="item">
   	<a class="no-border" href="catalogue/?SECTION_ID=326"><img src="/bitrix/templates/producer/i/items/container.jpg" alt="Контейнеры" title="Контейнеры"/></a>
   	<a href="catalogue/?SECTION_ID=326">Контейнеры</a>
   </div>	
   <?
   // тут куча кода
   ?>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?><a class="no-border" href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/dis-generator.jpg" alt="Дизельные генераторы" title="Дизельные генераторы"/></div><? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?></a>
   	<a href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode("Producer(Турция)")?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><? } ?>Дизельные генераторы<? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?></a><? } ?>
   </div>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?><a class="no-border" href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/oil-generator.jpg" alt="Бензиновые генераторы" title="Бензиновые генераторы"/></div><? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?></a>
   	<a href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode("Producer(Турция)")?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Бензин А-92(95)")?>&set_filter=Y"><? } ?>Бензиновые генераторы<? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?></a><? } ?>
   </div>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($_GET['SECTION_ID'] != '333') { ?><a class="no-border" href="?SECTION_ID=333"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/generator.jpg" alt="Газовые генераторы" title="Газовые генераторы"/></div><? if ($_GET['SECTION_ID'] != '333') { ?></a>
   	<a href="?SECTION_ID=333"><? } ?>Газовые генераторы<? if ($_GET['SECTION_ID'] != '333') { ?></a><? } ?>
   </div>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($_GET['SECTION_ID'] != '326') { ?><a class="no-border" href="?SECTION_ID=326"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/container.jpg" alt="Контейнеры" title="Контейнеры"/></div><? if ($_GET['SECTION_ID'] != '326') { ?></a>
   	<a href="?SECTION_ID=326"><? } ?>Контейнеры<? if ($_GET['SECTION_ID'] != '326') { ?></a><? } ?>
   </div>

   Это часть шаблона сайта, сделанного на битриксе. Хотя битрикс тут и не нужен, автор и без него справился бы с задачей. Без него было бы даже проще :)))
   
   Название сайта я заменил в коде на "Producer", не хочу уже позорить. Жаль что весь код не влез, там в таком же духе дальше.

   alex322, 28 Июля 2010

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

3. PHP / Говнокод #3805

   +111

   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

   <?
   	class newPage
   	{
   		
   		public $bigMainImage;
   		
   		public $smallMainImage;
   	
   		protected $id;
   		
   		//Фамилия и имя
   		protected $surname;
   		protected $name;
   		
   		//город
   		protected $city;
   
   		//конструктор
   		public __construct($id,
   					$surname,
   					$name,
   					$city)
   		{
   			$this->id = htmlspecialchars($id);
   			$this->surname = htmlspecialchars($surname);
   			$this->name = htmlspecialchars($name);
   			$this->city = htmlspecialchars($city);
   
   			//копируем большое и маленькое изображение 
   			copy("photos/$id/1.jpg","photos/$id/Main.jpg");
   			copy("photos/$id/small/1.jpg","photos/$id/small/Main.jpg");
   			$this->bigMainImage = "photos/$id/Main.jpg";
   			$this->smallMainImage = "photos/$id/small/Main.jpg";
   		}
   		
   		//вывод страницы на экран
   		public get_html()
   		{
   			echo"<title>$this->name $this->surname</title>";
   			echo "
   					<table>
   						<tr>
   							<td>
   								<img src=\"$this->bigMainImage\"><br>Рейтинг:
   							</td>
   							<td>
   								<b>$this->name $this->surname</b>,<br>
   								$this->city								
   							</td>
   						</tr>
   					</table>";
   		}
   		public newFile($id)
   		{
   			$intPage = fopen("page/".$id.".php",w);
   		}
   					
   	}
   ?>

   Ржу не магу.

   Vasiliy, 28 Июля 2010

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

4. PHP / Говнокод #3804

   +121

   1. 001
   2. 002
   3. 003
   4. 004
   5. 005
   6. 006
   7. 007
   8. 008
   9. 009
   10. 010
   11. 011
   12. 012
   13. 013
   14. 014
   15. 015
   16. 016
   17. 017
   18. 018
   19. 019
   20. 020
   21. 021
   22. 022
   23. 023
   24. 024
   25. 025
   26. 026
   27. 027
   28. 028
   29. 029
   30. 030
   31. 031
   32. 032
   33. 033
   34. 034
   35. 035
   36. 036
   37. 037
   38. 038
   39. 039
   40. 040
   41. 041
   42. 042
   43. 043
   44. 044
   45. 045
   46. 046
   47. 047
   48. 048
   49. 049
   50. 050
   51. 051
   52. 052
   53. 053
   54. 054
   55. 055
   56. 056
   57. 057
   58. 058
   59. 059
   60. 060
   61. 061
   62. 062
   63. 063
   64. 064
   65. 065
   66. 066
   67. 067
   68. 068
   69. 069
   70. 070
   71. 071
   72. 072
   73. 073
   74. 074
   75. 075
   76. 076
   77. 077
   78. 078
   79. 079
   80. 080
   81. 081
   82. 082
   83. 083
   84. 084
   85. 085
   86. 086
   87. 087
   88. 088
   89. 089
   90. 090
   91. 091
   92. 092
   93. 093
   94. 094
   95. 095
   96. 096
   97. 097
   98. 098
   99. 099
   100. 100

   <?php
   
   .....
   $kredit=ereg_replace('[^0-9]*','',$_REQUEST[kredit]);
   $op=ereg_replace('[^1-2]*','',$_REQUEST[op]);
   
   @mysql_query("DELETE FROM `".prefix."clicks` WHERE `ts`<".strtotime(date("j F Y")));
   @mysql_query("DELETE FROM `ban` WHERE `date`<".time()."");
   @mysql_query("DELETE FROM `mrek` WHERE `date`<".time()."");
   
   If (!isset($_GET['mode']) || !ereg("^[a-z0-9_-]{1,15}$",$_GET['mode'])) $mode=false; else $mode=$_GET['mode'];
   
   include_once("header/oper.etc.php");
   if($my_type == 3)
   $ttype = "(`typeclick` = '3' or `typeclick` = '4' or `typeclick` = '5' or `typeclick` = '0')";
   elseif($my_type == 2)
   $ttype = "(`typeclick` = '2' or `typeclick` = '4' or `typeclick` = '6' or `typeclick` = '0')";
   elseif($my_type == 1)
   
   switch ($mode){
   
   case 'bank':
   
   @mysql_query("update `".prefix."users` set `mesto`='В банке' where `id` = '".$_USER['id']."'");
   echo '<div class="header_rek"><b>Банк кредитов</b></div><br/>
   Кредитов в банке '.round($_USER['bank'],1).'<br/>
   Кредитов в активе '.($_USER['cfrom']-$_USER['cto']).'<br/><br/>
   <b>Операции</b><br/>
   Количество кредитов:<br/>
   <form method="post" action="'.$_zapros.'mode=bank2">
   Операции проводятся только с целыми числами!';
   break;
   
   case 'bank2':
   
   $kredit = abs(intval($kredit));
   $op = abs(intval($op));
   if($op==1)
   {
   $err='';
   if($kredit<1 or $kredit>$_USER['bank'])
   print '<b>Неверно указано количество кредитов</b><br/>';
   else
   {
   mysql_query("UPDATE `".prefix."users` SET `bank`=`bank`-$kredit, `cfrom`=`cfrom`+$kredit WHERE login='".$_USER['login']."'") or die('err');
   print 'Операция успешно произведена!<br/><font color="blue"><b>'.$kredit.'</b></font> кредитов были сняты с банка<br/>';
   }
   }
   else
   {
   if($kredit<1 or $kredit>($_USER['cfrom']-$_USER['cto']))
   print 'Ошибка!Неверно указано количество кредитов<br/>'.'';
   else
   {
   mysql_query("UPDATE `".prefix."users` SET `bank`=`bank`+$kredit, `cfrom`=`cfrom`-$kredit WHERE login='".$_USER['login']."'") or die('err');
   print 'Операция успешно произведена!<br/><font color="blue"><b>'.$kredit.'</b></font> кредитов положены в банк<br/>';
   }
   }
   break;
   
   case 'goer':
   
   @mysql_query("update `".prefix."users` set `mesto`='Переводит кредиты' where `id` = '".$_USER['id']."'");
   echo '<div class="header_rek"><b>Перевод кредитов</b></div><br/>
   Кредитов в банке <b>'.$_USER['bank'].'</b><br/>
   Количество кредитов:<br/>
   <form method="post" action="'.$_zapros.'mode=send">
   break;
   
   case 'send':
   
   $kredit = abs(intval($kredit));
   $nick = mysql_escape_string($_POST[nick]);
   $nick = trim(htmlspecialchars($nick));
   
   if ($kredit<50 or $kredit>$_USER[bank])
   { print 'Ошибка! Возможно кредиты не лежат у вас в банке или лежат меньше 50кр!<br />'; $err=1;}
   $q = mysql_query("SELECT `bank` FROM `".prefix."users` WHERE login='$nick'");
   if (mysql_num_rows($q)!=1)
   { print 'юзер не найден<br />'; $err=1; }
   if ($err!=1)
   {
   @mysql_query("UPDATE `".prefix."users` SET `bank`=`bank`+$kredit WHERE login='".$nick."'") or die('err');
   @mysql_query("UPDATE `".prefix."users` SET `bank`=`bank`-$kredit WHERE login='".$_USER['login']."'") or die('err');
   @mysql_query("INSERT INTO `".prefix."perevod`
   (`otkogo`,`komy`,`skolko`)
   values
   ('".$_USER['login']."','".$nick."','".$kredit."')
   ") or die('err');
   print '<b>Кредиты успешно отправлены!</b>';
   }
   break;
   
   case 'stats':
   
   @mysql_query("update `".prefix."users` set `mesto`='Смотрит свою статистику' where `id` = '".$_USER['id']."'");
   $result = mysql_query("SELECT `to` FROM `".prefix."clicks` WHERE `who`='".$_USER['id']."'");
   $my = mysql_num_rows($result);
   $result = mysql_query("SELECT `who` FROM `".prefix."clicks` WHERE `id_user`='".$_USER['id']."'");
   $your = mysql_num_rows($result);

   Vasiliy, 28 Июля 2010

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

5. PHP / Говнокод #3803

   +163

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

   if (($state_client=='DEL')||(empty($descr)))
   	print "<!--";
   ?>
   тут код HTML всякий с использованием <?=$var?> и прочей красоты
   <?
   if (($state_client=='DEL')||(empty($descr)))
   	print "-->";

   В описании не нуждается

   Werdn, 28 Июля 2010

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

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

   +115

   1. 1

   s.replaceAll("(\\/\\.\\.\\/)+", "/").replaceFirst("^((\\.)+\\/)+", "");

   вот такие вот суровые паттерны
   как я это увидел - мне сразу вид двух друзей переханных трактором вспомнился
   
   String s="../../../a./../../files/file1.rar"; // надо убрать строки вида "../"

   3.14159265, 28 Июля 2010

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

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

   +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

   long GetMicroseconds();
   
   CTvoid cLog::GetTime (char * acLocal, time_t tTime)
   {
     struct tm   ltLocalTime;
     struct tm * ptLocalTime;
   
     tTime       = time (NULL);
     ptLocalTime = localtime_r (&tTime, &ltLocalTime);
   
     sprintf(acLocal,"%04d%02d%02d %02d%02d%02d-%06ld",
                      ptLocalTime->tm_year+1900,
                      ptLocalTime->tm_mon+1,
                      ptLocalTime->tm_mday,
                      ptLocalTime->tm_hour,
                      ptLocalTime->tm_min,
                      ptLocalTime->tm_sec,
                      GetMicroseconds());
   
   }
   
   long GetMicroseconds()
   {
       struct timeval timeVal;
   
       if (0 == gettimeofday( &timeVal, NULL ))
           return timeVal.tv_usec;
   
       return -1;
   }
   
   cLog::__Write(...)
   {
       /* ... */
       tTime          = time(NULL);
       GetTime (acDataTime, tTime);
       /* ... */
   }

   R&D дали задание добавить микросекунды ко всем таймстемпам в логах.
   сказано - сделано.
   ну ведь никто не говорил что таймстемпы должны быть еще и консистентными.
   
   ЗЫ ну и time() надо вызвать раза два-три - для надёжности.

   Dummy00001, 28 Июля 2010

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

8. bash / Говнокод #3800

   −126

   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

   start() {
     echo "starting Adaptec Storage Manager agent ..."
     kill `ps axwww | grep java | grep StorMan | grep ManagementAgent | cut -b 1-6` 1>/dev/null 2>&1
     sleep 2
     kill -9 `ps axwww | grep java | grep StorMan | grep ManagementAgent | cut -b 1-6` 1>/dev/null 2>&1
     sh /usr/StorMan/StorAgnt.sh >/dev/null 2>&1 &
     sleep 1
     echo
   }
   
                                                                   
   stop() {
     echo "stopping Adaptec Storage Manager agent ..."
     kill `ps axwww | grep java | grep StorMan | grep ManagementAgent | cut -b 1-6` 1>/dev/null 2>&1
     sleep 2
     kill -9 `ps axwww | grep java | grep StorMan | grep ManagementAgent | cut -b 1-6` 1>/dev/null 2>&1
     sleep 1
     echo
   }

   Кусок скрипта для RAID от фирмы Adaptec. Боюсь смотреть, что они там в java накодили.

   icCE, 28 Июля 2010

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

9. PHP / Говнокод #3799

   +144

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

   # Подключаем MySQL класс
   a_import('libraries/mysql');
   $db = new MySQL();
   $db->debugging = TRUE;
   $db->connect();
   $db->charset('utf8');

   Код взят из CMS http://mobilecms.ru/
   
   Нет чтобы объявить свойства приватными, а начальные параметры передать в конструктор

   Aligan, 28 Июля 2010

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

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

    +168

    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

    $action=$_GET["action"];
    ...
    if($action==1){ ... }
    if($action==2){ ... }
    ...
    if($action==9){ ... }
    if($action!=1 && 
       $action!=2 && 
       $action!=3 &&
       $action!=4 &&
       $action!=5 &&
       $action!=6 &&
       $action!=7 &&
       $action!=8 &&
       $action!=9) print("<html><body bgcolor=".'"#000000"'." color=".'"#FFFFFF"'."><div align=".'"center"'."><h1><b>Ошибка! Такой страницы нет!</b></h1></div><!--*copyright*--></body></html>");

    crazy code

    XyHb, 28 Июля 2010

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

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

• 2108
• …
• 375
• 374
• 373
• 372
• 371
• 370
• 369
• 368
• 367
• …
• 1