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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2360)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (647)
9. Python (583)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   +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

   if(strpos($da,'Да')!==FALSE || strpos($da,'ДА')!==FALSE || strpos($da,'да')!==FALSE){$da='ДА';}
    else{$da = 'Нет';}
   
   .....
   
   if(strlen($document['Document_mark'])<=0){
         //манагеру разрешено ставить
      if($_SESSION['GL_firm'] == 1){$da = 'Нет';}
     
      if(strpos($da,'ДА')!==FALSE){$da='ДА';}
        else{
   
   .....

   Разбирая работающую (!) систему документооборота крупной компании.
   Фриланс.
   Цена 6000EUR.

   ZeiZ, 05 Апреля 2012

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

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

   +170

   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

   <?
   if (!$_SESSION['signed_id_user'] || $_SESSION['signed_id_group'] != 1) {
   	$pagetitle = 'Необходима авторизация';
   
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<div align=\"center\" style=\"width: 400px;\">\n";
   	$pagecontent .= "<form action=\"login.php\" method=\"post\">\n";
   	$pagecontent .= "<fieldset class=\"bw\">\n";
   	$pagecontent .= "	<legend class=\"bw\">авторизация пользователя</legend>\n";
   	$pagecontent .= "	<p>Поля, помеченные <span class=\"required\">*</span> являются обязательными для заполнения</p>\n";
   	$pagecontent .= "	<table border=\"0\" cellpadding=\"2\" cellspacing=\"0\" width=\"300\" align=\"center\">\n";
   	$pagecontent .= "	<tr>\n";
   	$pagecontent .= "		<td width=\"160\" align=\"right\">имя пользователя:</td>\n";
   	$pagecontent .= "		<td width=\"120\"><input type=\"text\" size=\"20\" class=\"baseinput\" name=\"username\" id=\"username\" /></td>\n";
   	$pagecontent .= "		<td><div id=\"msgUsername\" class=\"required\">*</div></td>\n";
   	$pagecontent .= "	</tr>\n";
   	$pagecontent .= "	<tr>\n";
   	$pagecontent .= "		<td width=\"160\" align=\"right\">пароль:</td>\n";
   	$pagecontent .= "		<td width=\"120\"><input type=\"password\" size=\"20\" class=\"baseinput\" name=\"password\" id=\"password\" /></td>\n";
   	$pagecontent .= "		<td><div id=\"msgPassword\" class=\"required\">*</div></td>\n";
   	$pagecontent .= "	</tr>\n";
   	$pagecontent .= "	<tr>\n";
   	$pagecontent .= "		<td width=\"160\">&nbsp;</td>\n";
   	$pagecontent .= "		<td width=\"120\"><input type=\"submit\" class=\"button\" name=\"posted\" value=\"Войти\" /></td>\n";
   	$pagecontent .= "		<td>&nbsp;</td>\n";
   	$pagecontent .= "	</tr>\n";
   	$pagecontent .= "	</table>\n";
   	$pagecontent .= "</fieldset>\n";
   	$pagecontent .= "</form>\n";
   	$pagecontent .= "</div>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   	$pagecontent .= "<p>&nbsp;<p>\n";
   
   
   	$tpl = new Template();
   	$tpl->set_file('tpl/e-con_bw.tpl');
   	$tpl->add_var('PAGETITLE', $pagetitle);
   	$tpl->add_var('PAGECONTENT', $pagecontent);
   	echo $tpl->output();
   	$tpl = NULL;
       exit();
   }
   ?>

   Подскажите, пожалуйста, нахер с таким стилем кодинга вообще шаблонизатор надо?..

   Anderson, 04 Апреля 2012

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

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

   +169

   1. 1
   2. 2
   3. 3

   if ( !defined("NOTIFICATION_ERROR_QUESTION_FIELD_REQUIRED") ) {
       define("NOTIFICATION_ERROR_QUESTION_FIELD_REQUIRED", "NOTIFICATION_ERROR_QUESTION_FIELD_REQUIRED");
   }

   Блеать, что это? Наверное, я чего-то определенно не знаю.

   zii, 04 Апреля 2012

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

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

   +149

   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

   $parseDocument = newDocumentHTML($parseFile);
           
           $data = $parseDocument->find('a.sAM');
           
           $keyArr = array();
           
           foreach($data as $d)
           {
               $pq = pq($d);
               
               $keyArr[] = strip_tags($pq->html());
           }
           
           unset($keyArr[0]);
           
           if(empty($keyArr))
           {
               throw new Exception('Fucking captcha');
               
               return null;
           }
           else return $keyArr;

   Archont12, 04 Апреля 2012

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

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

   +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

   if (isset ( $_POST ['atest'] ) && isset ( $_POST ['date_atest'] $_POST ['student'] ) && isset ( $_POST ['point'] ) ) {
   		( int ) $stud_id = $_POST ['student'];
   		( int ) $point = $_POST ['point'];
   		if (strlen ( $_POST ['atest'] ) > 3 && strlen ( $_POST ['date_atest'] ) == 10) {
   			$sql = "insert into `subjects` (`name`,`date`,`courses`,`group`)";
   			$sql .= "values (" . $db->qstr ( $_POST ['atest'] ) . "," . $db->qstr ( $_POST ['date_atest'] ) . "," . $course_input . "," . $group_input . ")";
   			$calc_attestation = "SELECT `users`.`id`, avg (`points`.`point`) attestation FROM `users`, `points`"; 
   			$calc_attestation .= "WHERE `users`.`id` = " . $stud_id . " AND `users`.`id` = `points`.`student`";
   			$result_calc_attestation = mysql_fetch_assoc(mysql_query($calc_attestation));
   			if ($result_calc_attestation['attestation'] >= 0 && $result_calc_attestation['attestation'] <= 0,5) {
   			$result_calc_attestation = 0;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   			if ($result_calc_attestation['attestation'] >= 0,5 && $result_calc_attestation['attestation'] <= 1) {
   			$result_calc_attestation = 1;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   			if ($result_calc_attestation['attestation'] >= 1 && $result_calc_attestation['attestation'] <= 1,5) {
   			$result_calc_attestation = 1;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   			if ($result_calc_attestation['attestation'] >= 1,5 && $result_calc_attestation['attestation'] <= 2) {
   			$result_calc_attestation = 2;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   			if ($result_calc_attestation['attestation'] >= 2 && $result_calc_attestation['attestation'] <= 2,5) {
   			$result_calc_attestation = 2;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   [Код сокращен]
   
   			if ($result_calc_attestation['attestation'] >= 11,5 && $result_calc_attestation['attestation'] <= 12) {
   			$result_calc_attestation = 12;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   			if ($result_calc_attestation['attestation'] == 12) {
   			$result_calc_attestation = 12;
   			$add_attestation = "insert into `points` (`point`, `course`, `student`)";
   			$add_attestation .= "values (" . $result_calc_attestation . ", " . $course_input . ", " . $group_input;
   			}
   			
   			if ($db->Execute ( $sql ) or $db->Execute ( $add_attestation ) === false) {
   				die ( 'Опаньки: ' . $db->ErrorMsg () );
   			}
   			$GLOBALS ['inform'] [] = "Атестація проведена";
   		} else {
   			$GLOBALS ['error'] [] = "Помилка додавання пари через те, що довжина назви менше 3-х символів або дата вказана не вірно.";
   		}
   	}

   Ну и как это использовать? Я уже представил что будет если будет болонская система)

   Alex_Bond, 04 Апреля 2012

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

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

   +167

   1. 1

   unlink(__FILE__);

   __proto__, 03 Апреля 2012

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

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

   +146

   1. 1
   2. 2
   3. 3

   <?php
   $userTimelog->add($userId, 2, strtotime(date('d.m.Y')));
   ?>

   Как можно знать о функции strtotime и не знать о $_SERVER['REQUEST_TIME'] или time()? :D

   miraage, 03 Апреля 2012

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

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

   +161

   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

   // Функция определения количества слов в запросе
   function kslov($s)
   {
   
   for($i=0;$i<=strlen($s)-1;$i++)
   {
    if(($s[$i]==' ')and(($s[$i+2]==' ')or($s[$i+3]==' '))){$s[$i]='';}
    if(($s[$i]==' ')and($s[$i+1]=='')){$s[$i]='';}
    if(($s[$i]==' ')and(($s[$i+2]=='')or($s[$i+3]==''))){$s[$i]='';}
    if(($s[$i]==' ')and(($s[$i-3]=='')or($s[$i-2]==''))){$s[$i]='';}
   }
   
    $mp=explode(' ',$s);
    $kp=count($mp);
    if($kp>3){$kp=3;}
    return $kp;
   }

   Реальный код...

   govnozmey, 03 Апреля 2012

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

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

   +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

   function createfile1($path,$info){
    if(file_exists($path))
    {
     $fp=fopen($path,'a+');
     fputs($fp,$info);
     fclose($fp);
    }
    else
    {
     $fp=fopen($path,'w+');
     fputs($fp,$info);
     fclose($fp);
    }
   }

   Функция с говорящим названием и пахнущим содержимым

   govnozmey, 03 Апреля 2012

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

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

    +159

    1. 1
    2. 2
    3. 3

    $time=date("d.m.Y.H.i.s");
    $t=explode('.',$time);
    $now=mktime($t[3],$t[4],$t[5],$t[1],$t[0],$t[2]);

    кручу-верчу, time() хочу

    govnozmey, 03 Апреля 2012

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

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

• 571
• …
• 337
• 336
• 335
• 334
• 333
• 332
• 331
• 330
• 329
• …
• 1