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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1098)
8. Pascal (647)
9. Python (582)
10. 1C (533)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. PHP / Говнокод #3828

   +164

   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

   function isGood($race,$class) {
            switch ($race) {
                   case 1:
                        if ($class == 1 || $class == 2 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
                        break;
                   case 2:
                        if ($class == 1 || $class == 3 || $class == 4 || $class == 6 || $class == 7 || $class == 8 || $class == 9) { return true; }
                        break;
                   case 3:
                        if ($class == 1 || $class == 2 || $class == 3 || $class == 4 || $class == 5 || $class == 6) { return true; }
                        break;
                   case 4:
                        if ($class == 1 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 11) { return true; }
                        break;
                   case 5:
                        if ($class == 1 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
                        break;
                   case 6:
                        if ($class == 1 || $class == 3 || $class == 6 || $class == 7 || $class == 11) { return true; }
                        break;
                   case 7:
                        if ($class == 1 || $class == 4 || $class == 6 || $class == 8 || $class == 9) { return true; }
                        break;
                   case 8:
                        if ($class == 1 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 7 || $class == 8) { return true; }
                        break;
                   case 10:
                        if ($class == 2 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
                        break;
                   case 11:
                        if ($class == 1 || $class == 2 || $class == 3 || $class == 5 || $class == 6 || $class == 7 || $class == 8) { return true; }
                        break;
            }
            return false;
   
   }

   Oops, 30 Июля 2010

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

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

   +156

   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

   // в данном случае count($xls_data) не подойдет, потому что
   // он меньше изза того что индексы не всегда по +1 идут
   end($xls_data);
   list($xls_dataCount,$unused) = each($xls_data);
   
   // тут некоторый код, тоже пахнет
   
   for($i = 3; $i <= $xls_dataCount; $i++){
   	$row = $xls_data[$i];
   	if(count($row) == 1 && $estnames[$row[1]])$ename = $row[1];
   	else{
   		foreach($years as $yindex => $year)
   			$data[$ename][$row[1]][$year] = $row[$yindex];
   	}
   }

   Сижу и ковыряюсь в говнице, по уши увяз((((

   asics, 30 Июля 2010

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

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

   +164

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

   try {
       ...
   } catch ( Exception $O_o ) {
       error_log( $O_o->getMessage() );
       ...
   }

   Совсем неожиданный эксепшин O_o

   eee_qqq, 30 Июля 2010

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

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

   +147

   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
   73. 73
   74. 74
   75. 75

   <?php
   
   require_once 'MDB2.php';
   
   $dsn = "mysql://user:pass@localhost/db";
   
   $mdb2 = & MDB2::singleton($dsn);
   if (PEAR::isError($mdb2)) {
       die($mdb2->getMessage());
   }
   
   class DB {
   
       static private $instance = NULL;
       static private $mdb2 = NULL;
   
       private function  __construct() {
           self::$mdb2 = & MDB2::singleton();
           self::$mdb2->exec("SET NAMES utf8");
           self::$mdb2->setFetchMode(MDB2_FETCHMODE_ASSOC);
       }
   
       static function getInstance() {
           if(self::$instance == NULL) {
               self::$instance =  & new DB();
           }
           return self::$instance;
       }
   
       public function query($sql = false) {
           $res = self::$mdb2->query($sql);
           if (PEAR::isError($res)) {
               die($res->getMessage());
           }
           if(!$res->numRows()) {
               return FALSE;
           }
           return $res;
       }
   
       private function __clone() {
   
       }
   }
   
   
   class Page{
   
       public   $limit = 10;
       private $conn = FALSE;  
   
       function  __construct() {
           $this->conn = & DB::getInstance();
       }
   
       public function getPageList() {
   
           $result = FALSE;
   
           $sql = "SELECT * FROM table LIMIT ".$this->limit;
           $res = $this->conn->query($sql);
           if($res) {
               $result = $res->fetchAll();
           }
           
           return $result;
       }
   
   }
   
   $p = & new Page();
   $nodes = $p->getPageList(25);
   print '<pre>'.print_r($nodes, 1).'</pre>';
   
   ?>

   Дайте, пожалуйста, оценку такой конструкции. Не говнокод ли?

   cartman, 29 Июля 2010

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

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

   +165

   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

   $config["maxcount"]=50;
   ...
   $qwe = mysql_query("SELECT * FROM ...");
   $i=0;
   while($i<$config["maxcount"]){
   	$asd=mysql_result($qwe, $i, 0);
   	$asdasd=mysql_result($qwe, $i, 1);
   	$asdasdasd=mysql_result($qwe, $i, 2);
   	if($asd&&$asdasd&&$asdasdasd){
   		...
   		$i=$i+1;	
   	}
   }

   Красивый код + красивые переменные.
   И что будет если $asd=0 или в результате запроса не 50 строк? xD

   XyHb, 29 Июля 2010

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

6. PHP / Говнокод #3813

   +125

   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

   $string="привет мир";
   
   $string=encodeURI($string);
   
   echo($string);
   
   function encodeURI($string) {
      $result = "";
      for ($i = 0; $i < strlen($string); $i++) {
         $result .= encodeURIbycharacter(urlencode(iconv ( "windows-1251","utf-8" , $string[$i])));
      }
      return $result;
   }
   
   function encodeURIbycharacter($char) {
      if ($char == "+") { return "%20"; }
      if ($char == "%21") { return "!"; }
      if ($char == "%23") { return "#"; }
      if ($char == "%24") { return "$"; }
      if ($char == "%26") { return "&"; }
      if ($char == "%27") { return "\""; }
      if ($char == "%28") { return "("; }
      if ($char == "%29") { return ")"; }
      if ($char == "%2A") { return "*"; }
      if ($char == "%2B") { return "+"; }
      if ($char == "%2C") { return ","; }
      if ($char == "%2F") { return "/"; }
      if ($char == "%3A") { return ":"; }
     .... Очень много строк
      if ($char == "%9A") { return "%C5%A1"; }
      if ($char == "%9B") { return "%E2%80%BA"; }
      if ($char == "%9C") { return "%C5%93"; }
      if ($char == "%B1") { return "%C2%B1"; }
      if ($char == "%B2") { return "%C2%B2"; }
      if ($char == "%B3") { return "%C2%B3"; }
      if ($char == "%B4") { return "%C2%B4"; }
      if ($char == "%B5") { return "%C2%B5"; }
      if ($char == "%B6") { return "%C2%B6"; }
      if ($char == "%B7") { return "%C2%B7"; }
      if ($char == "%B8") { return "%C2%B8"; }
      if ($char == "%B9") { return "%C2%B9"; }
      if ($char == "%BA") { return "%C2%BA"; }
      if ($char == "%BB") { return "%C2%BB"; }
      if ($char == "%BC") { return "%C2%BC"; }
      if ($char == "%BD") { return "%C2%BD"; }
      if ($char == "%BE") { return "%C2%BE"; }
      if ($char == "%BF") { return "%C2%BF"; }
      if ($char == "%C0") { return "%C3%80"; }
      if ($char == "%C1") { return "%C3%81"; }
      if ($char == "%C2") { return "%C3%82"; }

   По моему школьникам на каникулах скучно они прожат. Вот что я нашел

   Vasiliy, 29 Июля 2010

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

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

   +124

   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

   <?php
   //вот это наверное самый тупой вывод случаных ссылок из бд
   $zap="select * from stat";$zap=mysql_query($zap);$zap=mysql_num_rows($zap);$id=rand($zap-5,-1);
   $query="select * from stat where (id=$id)";$result=mysql_query($query);$r=mysql_fetch_array($result);$dop=$r[name];$inc=$r[inc];$inc = explode("<|>", $inc);$name=$inc[0];//первая ссылка
   $query="select * from stat where (id=$id+1)";$result=mysql_query($query);$r=mysql_fetch_array($result);$dop2=$r[name];$inc=$r[inc];$inc = explode("<|>", $inc);$name2=$inc[0];//Вторая ссылка
   $query="select * from stat where (id=$id+2)";$result=mysql_query($query);$r=mysql_fetch_array($result);$dop3=$r[name];$inc=$r[inc];$inc = explode("<|>", $inc);$name3=$inc[0];//третья ссылка
   $query="select * from stat where (id=$id+3)";$result=mysql_query($query);$r=mysql_fetch_array($result);$dop4=$r[name];$inc=$r[inc];$inc = explode("<|>", $inc);$name4=$inc[0];//четвертая  ссылка
   $query="select * from stat where (id=$id+4)";$result=mysql_query($query);$r=mysql_fetch_array($result);$dop5=$r[name];$inc=$r[inc];$inc = explode("<|>", $inc);$name5=$inc[0];//пятая ссылка
   ?>
   <a href="<?php echo ("$nhost/stat/$dop/");?>"><?php echo ("$name");?></a>
   <a href="<?php echo ("$nhost/stat/$dop2/");?>"><?php echo ("$name2");?></a>
   <a href="<?php echo ("$nhost/stat/$dop3/");?>"><?php echo ("$name3");?></a>
   <a href="<?php echo ("$nhost/stat/$dop4/");?>"><?php echo ("$name4");?></a> 
   <a href="<?php echo ("$nhost/stat/$dop5/");?>"><?php echo ("$name5");?></a>
   </b>
   <div  class="artif" ></div>

   Vasiliy, 29 Июля 2010

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

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

   +163

   1. 1

   if(in_array($rating,array_keys($ratings))) ...

   ужос средь бела дня)

   xakki, 29 Июля 2010

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

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

   +168

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

   if (array_keys($value) !== array_values(array_keys($value))) {
   
   } else {
   
   }

   Встретилось в ядре по работе с БД :)

   firefred, 29 Июля 2010

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

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

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

• 570
• …
• 137
• 136
• 135
• 134
• 133
• 132
• 131
• 130
• 129
• …
• 1