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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. PHP / Говнокод #3717

   +161

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

   $razn = mktime(0,0,0,$m,$d,$y); 
   
   $vozrast = (time() - $razn) / 31536000; 
   
   return explode(".",$vozrast);

   из кода одного крупного сайта, сделанного на заказ

   Мартин, 16 Июля 2010

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

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

   +157

   1. 1
   2. 2

   $wpdb->query($wpdb->prepare("INSERT INTO `".WPSC_TABLE_CART_CONTENTS."` (`prodid`, `name`, `purchaseid`, `price`, `pnp`,`tax_charged`, `gst`, `quantity`, `donation`, `no_shipping`, `custom_message`, `files`, `meta`) VALUES ('%d', '%s', '%d', '%s', '%s', '%s', '%s', '%s', '%d', '0', '%s', '%s', NULL)", $this->product_id, $this->product_name, $purchase_log_id, $this->unit_price, (float)$shipping, (float)$tax, (float)$tax_rate, $this->quantity, $this->is_donation, $this->custom_message, serialize($this->custom_file)));
   		$cart_id = $wpdb->get_var("SELECT LAST_INSERT_ID() AS `id` FROM `".WPSC_TABLE_CART_CONTENTS."` LIMIT 1");

   Это код WP-Ecommerce. Ахтунг!

   karevn, 16 Июля 2010

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

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

   +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

   <?php  
   // Создание массива из строки 
   if($_POST['add_array']=="Получит массив") 
   { 
      // новый массив для заполнения значениями 
      $new_arr = array(); 
      $text = explode(",", htmlspecialchars($_POST['text'])); 
      for($i=0; $i<count($text); $i++) 
      { 
            $vol = "<".$text[$i].">"; 
            array_push($new_arr,$vol); 
      } 
      print_r($new_arr); 
   } 
   ?>
   <form  method="post" action="">
     Введите  текст:<br>
     <textarea name="text" cols="70" rows="5"></textarea>
     <br>
     <br>
     <input name="add_array" type="submit" value="Получит массив">
   </form>

   Нашел на одном сайте с уроками по PHP.

   Мартин, 16 Июля 2010

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

4. Java / Говнокод #3714

   +75

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

   float lan = getLan();
          float lon = getLan();
          if(lan == lan || lon == lon)
          {
              throw new RuntimeException("Bad value");
          }

   Из моего реалного проекта. Этот код реально делает полезную вещь

   xaoc, 16 Июля 2010

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

5. C++ / Говнокод #3713

   +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
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47

   switch(filter)
      {
         case FILTER_SPECIAL:
         {
   
            if ( ( u8Direction >= 0xF0 ) || ( u8Direction <= 0x0F ) )
            {
               return 0x0;
            }
            else if ( ( u8Direction >= 0x10 ) && ( u8Direction <= 0x2F ) )
            {
               return 0x2;
            }
            else if ( ( u8Direction >= 0x30 ) && ( u8Direction <= 0x4F ) )
            {
               return 0x4;
            }
            else if ( ( u8Direction >= 0x50 ) && ( u8Direction <= 0x7F ) )
            {
               return 0x6;
            }
            else if ( u8Direction == 0x80 )
            {
               return 0x8;
            }
            else if ( ( u8Direction >= 0x81 ) && ( u8Direction <= 0xAF ) )
            {
               return 0xA;
            }
            else if ( ( u8Direction >= 0xB0 ) && ( u8Direction <= 0xCF ) )
            {
               return 0xC;
            }
            else if ( ( u8Direction >= 0xD0 ) && ( u8Direction <= 0xEF ) )
            {
               return 0xE;
            }
            break;
         }
   
         case FILTER_NORMAL:
         --//--
         break;
         case FILTER_BACKWARD_DIRECTION:
         --//--
        break;
     }

   Сферический индокод в вакууме

   Zielscheibe, 16 Июля 2010

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

6. C# / Говнокод #3712

   +117

   1. 1

   if (str_45_ik.ToString() == "".ToString())

   Вот такое неоптимальное говно осталось от предков. Нужно было использовать string.IsNullOrEmpty.

   WebKill_HochetVBan, 16 Июля 2010

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

7. C# / Говнокод #3711

   +109

   1. 1

   if (embededDatasourceItem.Text.Localized(language) == string.Empty)

   Это вместо string.IsNullOrEmpty

   imshaman, 16 Июля 2010

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

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

   +114

   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

   <?php
   function initiate()
   {
   
   $logged_in = false;
   if(isset($_SESSION['user_name']))
   {
   $logged_in = true;
   }
   
   // Check that cookie is set
   if(isset($_COOKIE['auth_key']))
   {
   $auth_key = safe_var($_COOKIE['auth_key']);
   
   if($logged_in === false)
   {
   // Select user from database where auth key matches (auth keys are unique)
   $auth_key_query = mysql_query("SELECT username, password FROM users WHERE auth_key = '" . $auth_key . "' LIMIT 1");
   if($auth_key_query === false)
   {
   // If auth key does not belong to a user delete the cookie
   setcookie("auth_key", "", time() - 3600);
   }
   else
   {
   while($u = mysql_fetch_array($auth_key_query))
   {
   // Go ahead and log in
   login($u['username'], $u['password'], true);
   }
   }
   }
   else
   {
   setcookie("auth_key", "", time() - 3600);
   }
   }
   
   }
   
   ?>

   Там целый сайт такого кода полный http://www.ruseller.com/lessons.php?rub=28&id=427

   Vasiliy, 16 Июля 2010

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

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

   +176

   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

   function NDS($poisk)
   {
   	preg_match('#БЕЗ НДС#', $poisk, $matches);
   	if(empty($matches[0]))
   	{
   		preg_match('#без НДС#', $poisk, $matches);
   	   	if(empty($matches[0]))
   		{
   			preg_match('#НДС НЕТ#', $poisk, $matches);
   		 	if(empty($matches[0]))
   			{
   			  	 preg_match('#НДС нет#', $poisk, $matches);
   		 		 if(empty($matches[0]))
   				 {
   			  		preg_match('#НДС не облагается#', $poisk, $matches);
   				 	if(empty($matches[0]))
   					{
   					   	preg_match('#НДС НЕ ОБЛАГАЕТСЯ#', $poisk, $matches);
   					 	if(empty($matches[0]))
   					 	{
   					 		preg_match('#НДС НЕ ПРЕДУСМОТРЕН#', $poisk, $matches);
   					 		if(empty($matches[0]))
   					 		{
   							   preg_match('#Без налога (НДС)#', $poisk, $matches);
   					 		   if(empty($matches[0]))
   					 		   {
   									preg_match('#НДС: БЕЗ НАЛОГА#', $poisk, $matches);
   					 				if(empty($matches[0]))
   					 				{
   					 				   preg_match('#Без НДС#', $poisk, $matches);
   					 				   if(empty($matches[0]))
   					 				   {
   					 				   	  preg_match('#без налога (НДС)#', $poisk, $matches);
   					 					  if(empty($matches[0]))
   					 					  {}
   					 					  else  return true;
   					 				   }
   					 				   else  return true;
   					 				}
   					 				else  return true;
   					 		   }
   					 		   else  return true;
   					 		}
   					 		else  return true;
   					 	}
   						else  return true;
   					}
   					else return true;
   				  }
   				  else return true;
   			}
   			else return true;
   		}
   		else return true;
   	}
   	else return true;
   }

   Обнаружено в старом проекте заказчика

   UncleRus, 16 Июля 2010

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

10. Pascal / Говнокод #3708

    +97

    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

    {
    ----------------------8<----------------------
    Цей юніт призначений лише для використання
    разом із пакетом програм ███████
    будь-якої версії.
    Він не може розповсюджуватися окремо, так
    як являється частиною пакету, який може
    розповсюджуватися на платній основі.
    
    (С) Copyright ███████████████
    ----------------------8<----------------------
    }
    
    (* ~15 строк поскипано за унылостью *)
    
    procedure TranslateForm(var form:TObject; const LNGFile:String);
    var
    ini:TIniW;
    c1,c2,p1,p2,i:Integer;
    Sct,par,val:string;
    keys:TStringList;
    cobg:TObject;
    comp:TComponent;
    begin
      initini(ini,LNGFile);
      keys:=TStringList.Create;
      sct:=Trim((form as TForm).Name);
        c1:=ini.KeyCount(Sct);
        ini.ReadSection(Sct,keys);
        cobg:=(form as TObject);
        for I := 0 to c1-1 do
        begin
          par:=keys[I];
          p1:=Pos('=',par);
          par:=Trim(Copy(par,0,p1-1));
          val:=ini.ReadString(Sct,par,'read error');
          typinfo.SetStrProp(cobg,par,val);
        end;
        c2:=(form as TForm).ComponentCount;
        for p2 := 0 to c2 - 1 do
          begin
          comp:=(form as TForm).Components[p2];
            cobg:=(comp as TObject);
            sct:=typinfo.GetStrProp(cobg,'Name');
    
              c1:=ini.KeyCount(Sct);
              if c1>0 then
              ini.ReadSection(Sct,keys);
                for I := 0 to c1-1 do
                begin
                  par:=keys[I];
                  p1:=Pos('=',par);
                  par:=Trim(Copy(par,0,p1-1));
                  val:=ini.ReadString(Sct,par,'read error');
                  try
                  typinfo.SetStrProp(cobg,par,val);
                  except
    
                  end;
                end;
    
          end;
    end;
    
    
    end.

    Сверху - жадность, снизу - отсутствие скиллов.

    bugmenot, 16 Июля 2010

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

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

• 2108
• …
• 366
• 365
• 364
• 363
• 362
• 361
• 360
• 359
• 358
• …
• 1