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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2379)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
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 (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   +169

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

   reset($err_list);
   //создаем все необходимые переменные
   while ($var = each($err_list)) {
       eval('if(!isset($_POST["'.$var['key'].'"])){$_POST["'.$var['key'].'"]="";}');
   }

   ...

   temka, 10 Сентября 2011

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

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

   +154

   1. 1
   2. 2

   $username=strtolower($this->username);
   $user=User::model()->find('LOWER(username)=?',array($username));

   чтоб наверняка
   http://yiiframework.ru/doc/blog/ru/prototype.auth

   DrFreez, 10 Сентября 2011

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

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

   +162

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

   switch($_GET['country']) {
               foreach ($getActiveChildren $k=>$v){   // тут цикл который создаёт case, echo, break - но он не работает
                   case "$v[id]":
                   echo 'Любой текст';
                   break;}
                   default:
                   print '[{value:"",text:"Выбрать автомобиль"}]';
                   break;
                   }

   Все поймём сразу что код не работает) Но удивляет пытливый ум новичка, который очень просил ему помочь, и он предлагал свои идеи, и говорил что не работает, это чудо одно из тех предложение) В конце концов бедолага разобрался сам... Но вот какое чудо он написал в итоге останется тайной =(

   nethak, 10 Сентября 2011

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

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

   +161

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

   $url = $_SERVER['REQUEST_URI'].'/';
   $counter = substr_count($url, 'page=');
   if($counter!=0){
   preg_match('/(page=)(.*?)(\/)/', $url, $pag);
   $sim_o = array('page=', '/');
   $sim_z= array('','');
   $pag = str_replace($sim_o, $sim_z, $pag);
   $page = $pag[0];
   echo $page;
   }

   Проверяем наличие GET и выводим его же.

   OJA, 10 Сентября 2011

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

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

   +159

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

   <? echo "$Welcome" ?> <? 	
   	$site = session_id();
   	 echo ", <b>" . $_SESSION["esb2b_username"] . "</b>" ;
   ?>

   Из одного китайско-арабского скрипта. Хорошо дан пробел после "Welcome".

   otnes, 09 Сентября 2011

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

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

   +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

   /*
   Содержимое news.xml
   <?xml version="1.0" encoding="utf-8"?>
    <news>
      <item>
        <title>Новые поступления</title>
        <text>
        This program is free software;
        </text>
        <date>13:30 - 12.08.2004</date>
      </item>
      <item>
        <title>ERERER поступления</title>
        <text>
          p pppppppppppppppp p p pp  p pppppppppppppppppp pppppppppppppppppppppp
        </text>
        <date>18:30 - 12.08.2004</date>
      </item>
    </news>
   */
   
   
    $file = fopen("news.xml","r");
    fseek($file, 47, SEEK_SET);
    $other = fread($file, 900000);
    fseek($file, 0, SEEK_SET);
    $prev = fread($file, 47);
    fclose($file);
    $file = fopen("news.xml","w");
   //...
   $text1 = "$text";
   $title = "$h1";
   $date = "$dates";
   echo $other;
   $all = $prev.'<item><title>'.$title.'</title><text>'.$text1.'</text><date>'.$date.'</date></item>'.$other;
   $buff = fputs ($file, $all);
   print $buff;

   Оригинальный способ дописывать новую новость в xml файл.
   Данный способ уже не кошерный:
   $doc = new DOMDocument();
   $doc->load($xml);
   $item = $doc->createElement('item');
   $news = $doc>getElementsByTagName('news')->item(0);
   $news->appendChild($item);
   $doc->save($xml);

   MoLe-X, 09 Сентября 2011

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

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

   +154

   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

   <?php
   function import_var($name,$source,$type,$maxlen=0) {
   	$retarr=array();
   	if(is_array($name)) { // если нужно обработать массив с именами
   		foreach($name as $v) {
   			$retarr[$v] = import_var($v,$source,$type,$maxlen);
   			// рекурсивно запускаем себя же
   		}
   	if(sizeof($retarr)>0) return $retarr; // возвращаем массив
   	}
   	// если нужно обработать одну переменную
   	switch($source) {
   		case 'G':
   			if (isset($_GET[$name])) {
   				$var=$_GET[$name];  // из GET
   			}
   		break;
   		case 'P':
   			if (isset($_POST[$name])) {
   				$var=$_POST[$name];  // из POST
   			}
   		break;
   		case 'C':
   			if (isset($_COOKIE[$name])) {
   				$var=$_COOKIE[$name];  // из Cookie
   			}
   		break;
   	}
   
   	if(!isset($var)) return false; // если переменная существует
   
   	if(is_array($var)) return false; // и если это не массив...
   
   	if(get_magic_quotes_gpc()) $var = stripslashes($var);
   	// убираем лишние бэкслэши
   
   	if(empty($var)) return false; // если переменная пуста
   
   	if ($maxlen>0) $var = substr($var,0,$maxlen); //обрубаем лишнее
   
   	switch($type) { // теперь обрабатываем в соответствии с типом
   		case 'INT' : // число integer
   			return is_integer($var) ? $var : intval($var);
   		break;
   		case 'NUM' : // число integer или float
   			return is_numeric($var) ? $var : false;
   		break;
   		case 'HTML' : // текст, в котором могут содержаться тэги
   			return trim(htmlspecialchars($var));
   		break;
   		case 'SQL' : // строка, которая попадет в SQL-запрос
   			return mysql_real_escape_string(htmlspecialchars($var));
   		break;
   		case 'MAIL'	: // email-адрес
   			return preg_match('/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{2,4}$/',$var) ? $var : false;
   		break;
   		case 'ALPHA' : // только буквенные символы
   			return preg_match("/^[а-яА-ЯёЁa-zA-Z]+$/",$var) ? $var : false;
   		break;
   		case 'ALPHANUM' : // числовые и буквенные символы
   			return preg_match("/^[а-яА-ЯёЁa-zA-Z0-9]+$/",$var) ? $var : false;
   		break;
   		case 'MD5' : // md5-хэш
   			return preg_match("/^[a-fA-F0-9]{32}+$/",$var) ? $var : false;
   		break;
   		case 'SHA1' : // sha1-хэш
   			return preg_match("/^[a-fA-F0-9]{40}+$/",$var) ? $var : false;
   		break;
   		case 'BOOL': // булева величина
   			return ($var=="1" || $var=="true" || $var=="on") ? true : false;
   		break;
   	}
   }
   ?>

   http://raz0r.name/releases/funkciya-dlya-obrabotki-vxodyashhix-dannyx/

   sectus, 09 Сентября 2011

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

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

   +161

   1. 1

   $period = in_array($bullRegionId,array_keys($periods)) ? $periods[$bullRegionId]['text'] : $periods['default']['text'];

   Интересна сама проверка на существование ключа, не правда ли? :-)
   Код мой. Коллега учуял, что от него несколько плохо пахнет и предложил запостить сюда))

   Nemoden, 09 Сентября 2011

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

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

   +153

   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

   function resetIsetSorting()
   	{
   		$db = 'i_set';
   		$co = -1;
   		$q = $this->db->get($db);
   		$r = $q->result();
   		foreach($r as $row)
   		{
   			$co++;
   			$upd['sort'] = $co;
   			$this->db->where('id_set', $row->id_set); 
   			$this->db->update($db, $upd);	
   		}
   	}
   	function isetSortExchange($sort, $canBeSort)
   	{
   		$db = 'i_set';
   		$bigNumber = 1470000;
   		$this->db->where('sort', $canBeSort);
   		$q = $this->db->get($db);
   		if ($q->num_rows() == 1)
   		{
   			$upd['sort'] = $bigNumber;
   			$this->db->where('sort', $canBeSort); 
   			$this->db->update($db, $upd);	
   			//
   			$upd['sort'] = $canBeSort;
   			$this->db->where('sort', $sort); 
   			$this->db->update($db, $upd);
   			//
   			$upd['sort'] = $sort;
   			$this->db->where('sort', $bigNumber); 
   			$this->db->update($db, $upd);
   		}
   	}

   Первая функция призвана обновить поле sort на уникальные последовательные значение. Сначала хотел провернуть такое дело на SQL в PHPmyADMIN, но мозгов моих оказалось мало, и я сделал это на CodeIgniter. Говнокод ли это?
   
   Вторая функция призвана для обмена значениями полей sort. Это нужно для сортировки чего либо в баз. Чутьё мне подсказывает что это можно проделать каким-то белее правильным путём, без использования $bigNumber = 1470000; (а вдруг количество строк достигнет этого числа??!!). Говнокод ли это?
   
   Всё рабочее, только что проверил. Свежий код, только что написан!

   increazon, 08 Сентября 2011

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

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

    +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

    function validemail($email){
    
     if (!ereg("^[^@]{1,64}@[^@]{1,255}$", $email))
    
     return false;
    
    
    
     $email_array = explode("@", $email);
    
     $local_array = explode(".", $email_array[0]);
    
     for ($i = 0; $i < sizeof($local_array); $i++) {
    
      if (!ereg("^(([A-Za-z0-9!#$%&'*+/=?^_`{|}~-][A-Za-z0-9!#$%&'*+/=?^_`{|}~\.-]{0,63})|(\"[^(\\|\")]{0,62}\"))$", $local_array[$i]))
    
      return false;
    
     }

    jQuery, 08 Сентября 2011

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

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

• 571
• …
• 277
• 276
• 275
• 274
• 273
• 272
• 271
• 270
• 269
• …
• 1