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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1927)
6. Java (1501)
7. Си (1099)
8. Pascal (647)
9. Python (582)
10. 1C (534)
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 / Говнокод #8566

   +162

   1. 1
   2. 2

   $id = $this->db->fetchCol('SELECT id FROM data;');
   $names = $this->db->fetchCol('SELECT name FROM points INNER JOIN data ON points.id = data.point_id_from;');

   Zend Db usage

   jf4ever, 18 Ноября 2011

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

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

   +160

   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

   function &GetObject($cls, $p=null, $flags=0)
   {	//HIDE($objects,$i,$ext,$n,$v)
   	static $objects;
   	if (is_array($cls)) {
   		$i = null;
   		foreach($cls as $n=>$v) $i->$n = $v;
   		$result = &$i;
   		return $result;
   	}
   	if (!($flags&O_NEW) && isset($objects[$cls])) return $objects[$cls];
   
   	$ext = new WExtern($cls, 'c');
   	if (!($flags&O_NEW) && isset($objects[$i=$ext->Id()])) return $objects[$i];
       if ($flags&O_INSTANCE) {$x = null; $result = &$x; return $result;}
   
   	if (!$ext->Exists()) $ext->Prepare();
   	if (($flags&O_NOERROR) && !class_exists($ext->c)) {
   		$co = $ext->c;
   		$cc = $co{0}.'B'.substr($co,1);
   		if (($flags&O_NOERROR) && !class_exists($cc)) {
   			$i = false;
   			return $i;
   		} else {
   			$ext->c = $cc;
   		}
   	}
   	
   	if (!class_exists($ext->c)) {
   		$co = $ext->c;
   		$cc = $co{0}.'B'.substr($co,1);
   		if (!class_exists($cc)) {
   			$i = false;
   			return $i;
   		} else {
   			$ext->c = $cc;
   		}
   	}
   	$c = $ext->c;
   	$i = new $c;
   
   	if (isset($ext->I) && $ext->I) $i->_I = reset($ext->I);	// Instance ID
   	if (isset($ext->A) && $ext->A) $i->_A = $ext->A;		// Alias ID
   
   	if (!($flags&O_NEW)) {
   		$objects[$ext->Id()] = &$i;
   		if ($ext->Id()!=$cls) $objects[$cls] = &$i;
   	}
   	$options = &WConfig::Get('OPTIONS');
   	if (/*is_a($i, 'wobject')*/ $i instanceof wobject && $options) {
   		$o = $options;
   		$c = get_class($i);
   		do{
   			$c = strtolower($c);
   			if (isset($o[$c])) {
       			$i->o = array_merge($o[$c], isset($i->o) ?$i->o :array());
   			}
   		}while($c=get_parent_class($c));
   		if (isset($ext->A) && isset($o[$ext->A])) 	$i->o = array_merge(isset($i->o) ?$i->o :array(), $o[$ext->A]);
   		if ($ext->I && isset($o[$ext->Id()])) 		$i->o = array_merge(isset($i->o) ?$i->o :array(), $o[$o[$ext->Id()]]);
   	}
   
   	if (is_object($p)) {
   		if (isset($p->o)) $i->o = array_merge(isset($i->o) ?$i->o :array(), $p->o);
   		$p = isset($p->c) ? $p->c : null;
   	}
   	if (method_exists($i,'Create')  && $p!==false) {
   		call_user_func_array(array(&$i,'Create'), is_array($p) ?$p : (isset($p) ? array($p) : array()));
   	}
   
   	return $i;
   }

   Одна из базовых функций фремворка.

   Orion189, 18 Ноября 2011

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

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

   +166

   1. 1

   foreach ($_GET as $xxx => $rvar) { $$xxx = $rvar; }

   может боян конешн

   pusinsky, 18 Ноября 2011

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

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

   +163

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

   if (isset($_GET['last_user']) && (int)$_GET['last_user']){
       $last_user = 0;
   }else{
       $last_user = (int)$_GET['last_user'];
   }

   no comments.. )

   Igogo, 17 Ноября 2011

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

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

   +162

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

   $tmp_arr=array();
   $id1_str='';
   $root_str='';
   foreach($id1 as $id){
   	$id1_str.=$id['id'].' , ';
   	if(!isset($tmp_arr[$id['root']])){
   		$root_str.=$id['root'].' , ';
   		$tmp_arr[$id['root']]=true;
   	}
   }

   это типа такой способ собрать distinct root ids в строку!
   не забыть потом откусить хвост ' , '
   нормальные герои не ищут легких путей

   shitcoder, 17 Ноября 2011

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

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

   +157

   1. 1

   $newUrl = $newUrl = implode("?", array_merge($arUrl, array(implode("&", $pgAddUrl))));

   Вот так некоторые люди формируют URL.

   alexxxnf, 17 Ноября 2011

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

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

   +146

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

   <?php
   $myKey = array('а','б','в','г','д','е','ё','ж','з','и','й','к','л','м','н','п','р','с','т','о','у','ф','х','ц','ч','ш','щ','ь','ъ','э','ю','я','1','2','3','4','5','6','7','8','9','0');
   $myRand = rand(0, count($myKey));
   $key = $myKey[$myRand];
   echo $key;
   ?>

   Sulik78, 16 Ноября 2011

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

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

   +145

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

   <?php
   //O_o
   echo dirname(__DIR__);
   ?>

   Diwms, 16 Ноября 2011

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

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

   +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

   //create image array
            $images = array();
            //fill it
            for ($i = 1; $i <= 180; $i++) 
               if (($i != $passimage1)&&($i != $passimage2)&&($i != $passimage3)&&($i != $passimage4)) 
               $images[] = $i;
            //randomize it
            for ($i = 0; $i < 176; $i++) {
               $r = rand(0,175);
               $swap = $images[$r];
               $images[$r] = $images[$i];
               $images[$i] = $swap;
            }

   denis90, 16 Ноября 2011

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

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

    +151

    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

    (1)Сначала:
    <!DOCTYPE html>
    <html>
    <head>
    <script type='text/javascript' src='./Plugins/jQuery/jquery.min.js'></script>
    
        <script>
    	function rezlt(a,b,c) {
                $.ajax({  
                    url: "test.php",
    				type:	'POST',
    				data: 'a='+a+'&b='+b+'&c='+c+'',				
                    cache: false,				
                    success: function(html){  
                        $(".content").html(html)  ;
                    }  
                });
         }
    
        </script>  
    </head>
    <body>
    //тут уже результат работы, строк свыше 1000
    <span class="content" onclick="rezlt(1,2,3)" >answer1</span><br/>
    <span class="content" onclick="rezlt(4,5,6)" >answer2</span><br/>
    <span class="content" onclick="rezlt(7,8,9)" >answer3</span><br/>
    </body>  
    </html>
    
    
    (2)Решение:
    <!DOCTYPE html>
    <html>
    <head>
    <script type='text/javascript' src='./Plugins/jQuery/jquery.min.js'></script>
    
        <script>
    	function rezlt(a,b,c,nid) {
                $.ajax({  
                    url: "test.php",
    				type:	'POST',
    				data: 'a='+a+'&b='+b+'&c='+c+'',				
                    cache: false,				
                    success: function(html){  
                        $(nid).html(html)  ;
                    }  
                });
         }
    
        </script>  
    </head>
    <body>
    //тут уже результат работы, строк свыше 1000
    <span id="content" onclick="rezlt(1,2,3,seq1)" ><div id="seq1">answer1</div></span><br/>
    <span id="content" onclick="rezlt(4,5,6,seq2)" ><div id="seq2">answer2</div></span><br/>
    <span id="content" onclick="rezlt(7,8,9,seq3)" ><div id="seq3">answer3</div></span><br/>
    </body>  
    </html>

    Уже второй день бъюсь над проблемой.
    Задача: Перегружать всю страницу не надо. По клику по объекту content отправляются данные на test.php
    и некий результат присваивается объекту content, на который был совершен клик. Сначала было вот так - (1) но тут, присвоение производится сразу всем объектам content
    
    Чую, что решение(2) говно, учитывая что объектов content будет свыше 1000.

    truedizzy, 16 Ноября 2011

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

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

• 570
• …
• 299
• 298
• 297
• 296
• 295
• 294
• 293
• 292
• 291
• …
• 1