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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2397)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (44)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. Си / Говнокод #5990

   +127

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

   [code=Си]
   switch(n)
   case 1:
   {
     /* code1 */
     /* fallthrough */
     case 2:
     /* code 2 */
   }
   [/code]

   Все имена и явки изменены!
   Сцуко, работает. Щас в стандарт полезу, интересно же! Обвиняют меня, я киваю на издержки мержа. Но смешно.

   nil, 15 Марта 2011

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

3. C++ / Говнокод #5974

   +167

   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

   float ArenaTeam::GetChanceAgainst(uint32 own_rating, uint32 enemy_rating)
   {
       // returns the chance to win against a team with the given rating, used in the rating adjustment calculation
       // ELO system
      // return 1.0f/(1.0f+exp(log(10.0f)*(float)((float)enemy_rating - (float)own_rating)/400.0f));
   	float a,b;
   
   	a = own_rating;
   	b = enemy_rating;
   
   		 if(a<1800 && b<1800) return 16;
   		 if(a>=1800 && a<2000 && b<2000 || b>=1800 && b<2000 && a<2000) return 15;
   		 if(a>=2200 && b>=2200) return 15;
   	if(a>b){
   		 if(a>=2000 && a<2100 && b>=2100 && b<2200) return 17;
   		 if(a>=2000 && a<=2100 && b>=2200) return 18;
   		 if(b>=2000 && b<2100 && a>=2200) return 9;
   		 if(a>=2000 && a<2100 && b>=2000 && b<2100) return 15;
   		 if(a>=2100 && a<2200 && b>=2100 && b<2200) return 15;
   		 if(a>=2100 && a<2200 && b>=2200) return 16;
   		 if(b>=2100 && b<2200 && a>=2200) return 11;
   		 if (a>=2000 && a<2100 && b >=1850) return 10;
   		 if (a>=2000 && a<2100 && b <=1850) return 5;
   		 if (a>=2100 && a<=2200 && b >=1950 && b <2000) return 9;
   		 if (a>=2100 && a<=2200 && b >=1900 && b <2000) return 7;
   		 if (a>=2100 && a<=2200 && b >=2000 && b <2100) return 11;
   		 if (a>=2100 && a<2200 && b <=1900 && b >=1750) return 4;
   		 if (a>=2100 && a<2200 && b <1750) return 3;
   		 if (a>=2200 && b >=1950 && b <=2000) return 4;
            if (a>=2200 && b <=1950 && b>=1800) return 2;
   		 if (a>=2200 && b <=1850) return 1;
   		 if(a>=1500 && a<1600 && b >=2200) return 31;
   		 if(a>=1500 && a<1600 && b>=2000 && b <2100) return 28;
   		 if(a>=1500 && a<1600 && b>=2100 && b<2200) return 29;
   		 if(a>=1500 && a<1600 && b>=2100 && b>2200) return 31;
   		 if (a>=1900 && a<2000 && b>=2000 && b<= 2075) return 18;
   		 if (a>=1900 && a<2000 && b>=2075 && b<= 2150) return 24;
   		 if (a>=1900 && a<2000 && b>=2150 && b<= 2200) return 25;
   		 if (a>=1900 && a<2000 && b>=2150 && b> 2200) return 27;
   		 if (a>=1800 && a<1900 && b>=2000 && b<= 2075) return 22;
   		 if (a>=1800 && a<1900 && b>=2075 && b<= 2150) return 27;
   		 if (a>=1800 && a<1900 && b>=2150 && b<= 2200) return 29;
   		 if (a>=1800 && a<1900 && b>=2150 && b> 2200) return 30;
   		 if (a>=1600 && a<1800 && b>=2000 && b<= 2075) return 22;
   		 if (a>=1600 && a<1800 && b>=2075 && b<= 2150) return 28;
   		 if (a>=1600 && a<1800 && b>=2150 && b<= 2200) return 29;
   		 if (a>=1600 && a<1800 && b>=2150 && b> 2200) return 30;
   		 if(a<1500 & b>=2000) return 31;
   	}else{
                    // same just b instead of a and a instead of b
   	}
       
   }

   New developer for wow server ;) Best part is the cast to floats at start imo

   kerhong, 14 Марта 2011

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

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

   +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

   <?php
   class security extends engine{
       var $temp=array();
   
       function getUserId(){
           $hash=explode('::',$_COOKIE['site_hash']);
           $id=$hash[0];
           return $id;
       }
   
       function getUserSecurityAccess($id){
           if(is_numeric($id)){
               $id=rawurlencode($id);
               $conn_id=mysql_connect('host','user','passwd');
               mysql_select_db('database');
               $q=mysql_query("SELECT groupid FROM `users`
                               WHERE id='".$id."'",$conn_id);
               if($q){
                   if(mysql_numrows($q)!=0){
                       $result=@mysql_fetch_assoc($q);
                       return $result['group_id'];
                   }else{
                       return -1;
                   }
               }else{
                   return -1;
               }
               mysql_close($conn_id);
           }else{
               return -1;
           }
       }
   
       function checkUserPermission($module,$act){
           #return true;
           $this->temp=array();
           $this->temp['_result']=0;
           $this->temp['_uid']=explode('::',$_COOKIE['site_hash']);
           $this->temp['_uid']=$this->temp['_uid'][0];
           $this->temp['_gid']=$this->getUserSecurityAccess($this->temp['_uid']);
           $this->temp['_conn_id']=mysql_connect('host','user','passwd');
           mysql_select_db('database');
           $this->temp['_q1']=mysql_query('SELECT perms'
                           .'FROM `secure_groups`' 
                           .'WHERE id='.$this->temp['_gid']);    
           $this->temp['_access_stamp']=mysql_fetch_assoc($this->temp['_q1']);
           $this->temp['_access_stamp']=$this->temp['_access_stamp']['perms'];
           $this->temp['_access_stamp']=explode(';',$this->temp['_access_stamp']);
           $this->temp['_access_stamp']=array_slice($this->temp['_access_stamp'],0,-1);
           foreach($this->temp['_access_stamp'] as $this->temp['v']){
               $this->temp['_mod_access']=explode(':',$this->temp['v']);
               $this->temp['_mod_indefier']=$this->temp['_mod_access'][0];
               if($this->temp['_mod_indefier']==$module){
                   $this->temp['_perms']=explode(',',$this->temp['_mod_access'][1]);
                   switch($act){
                       case 'r':
                           $this->temp['_result']=($this->temp['_perms'][0]==1)? 1:0;
                           break;
                       case 'w':
                           $this->temp['_result']=($this->temp['_perms'][1]==1)? 1:0;
                           break;
                   }
                   break;
               }
           }
           mysql_close($conn_id);
           return $this->temp['_result'];
       }
   }
   ?>

   Понравилось окончание статьи
   "Надеюсь, что мы скоро встретимся...
   Кто хочет почитать остальные мои статьи - прошу на http://e-code.tnt43.com.
   С уважением Карпенко Кирилл, глава IT-отдела ИНПП. "

   MoLe-X, 12 Марта 2011

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

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

   +122

   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

   <?php require_once "db_config.php";
    
    class db extends db_config {
    
     private $connection;
     
     function __construct(){
      $this->open_connection();
   //   echo "Соединение установлено ";
     }
     
     private function open_connection(){
      $this->connection=mysql_connect($this->DB_HOST,$this->DB_USER,$this->DB_PASS);
      if (!$this-connection){
       die("Соедитение с базой данных не установлено: ".mysql_error());
      } else {
       $db_select=mysql_select_db($this->DB_NAME);
   	if (!$db_select){
   	 die("База данных не определена: ".mysql_error());
   	}
      }
      mysql_query("set names utf8")or die("set name utf8 failed");
      mysql_query("set lc_time_names=ru_RU");
     }
     
     public function sql($query){
      $result=mysql_query($query,$this->connection);
      if (!result){
       die("Запрос не выполнен: ".mysql_error());
      }
      return $result;
     }
    }
    $db = new db();?>

   Типа сингелтон

   Vasiliy, 10 Марта 2011

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

6. C++ / Говнокод #5836

   +154

   1. 1

   function(&(*(--Iter)++);

   Вот как надо использовать итератор...

   Tirect, 01 Марта 2011

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

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

   +159

   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

   <?
   			if(($socle == 1302 || $socle == 2723 || $socle == 2723 || $socle == 211) && ($power == 1816 || $power = 1296 || $power = 2741 || $power = 2741 || $power = 216  || $power = 1854))//G9 40W
   				{echo '<br/><a href="/catalog/?id=1&section=2163&dep=12">Купить лампочку: цоколь G9, мощность 40W</a>';}
   			else if(($socle == 1324 || $socle == 2964 || $socle == 212) && ($power == 1826 || $power == 2942 || $power == 1325 || $power == 1861 || $power == 217  || $power == 1734))//GU10 50W
   				{echo '<br/><a href="/catalog/?id=2&section=2163&dep=12">Купить лампочку: цоколь GU10, мощность 50W</a>';}
   			else if(($socle == 2855 || $socle == 1294 || $socle == 2733 || $socle == 1883 || $socle == 1721) && ($power == 1816 || $power = 1296 || $power = 2741 || $power = 2741 || $power = 216  || $power = 1854))//E14 40W
   				{
   				echo '<br/><a href="/catalog/?id=3&section=2163&dep=12">Купить лампочку: цоколь E14, мощность 40W</a><br/>';
   				echo '<br/><a href="/catalog/?id=4&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E14, мощность 7W</a>';
   				}
   			else if(($socle == 2748 || $socle == 1305 || $socle == 3051 || $socle == 203 || $socle == 1724) && ($power == 1816 || $power = 1296 || $power = 2741 || $power = 2741 || $power = 216  || $power = 1854))//E27 40W
   				{
   				echo '<br/><a href="/catalog/?id=5&section=2162&dep=12">Купить лампочку: цоколь E27, мощность 40W</a><br/>';
   				echo '<br/><a href="/catalog/?id=6&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E27, мощность 11W</a>';
   				}
   			else if(($socle == 2748 || $socle == 1305 || $socle == 3051 || $socle == 203 || $socle == 1724) && ($power == 1919 || $power = 1817 || $power = 5163 || $power = 1306 || $power = 2727 || $power = 205 || $power = 3052 || $power = 1850))//E27 60W
   				{
   				echo '<br/><a href="/catalog/?id=7&section=2162&dep=12">Купить лампочку: цоколь E27, мощность 60W</a><br/>';
   				echo '<br/><a href="/catalog/?id=8&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E27, мощность 11W</a>';
   				}
   			else if(($socle == 2748 || $socle == 1305 || $socle == 3051 || $socle == 203 || $socle == 1724) && ($power == 1919 || $power = 1817 || $power = 5163 || $power = 1306 || $power = 2727 || $power = 205 || $power = 3052 || $power = 1850))//E27 60W
   				{
   				echo '<br/><a href="/catalog/?id=9&section=2162&dep=12">Купить лампочку: цоколь E27, мощность 60W</a><br/>';
   				echo '<br/><a href="/catalog/?id=10&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E27, мощность 11W</a>';
   				}		
   		?>

   Говнокодеры не спят, они какают и какают :))))
   Учитесь, как надо давать покупателю возможность выбрать подходящую лампочку)

   greevex, 25 Февраля 2011

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

8. C# / Говнокод #5794

   +139

   1. 1

   Response.Write("<" + '/'.ToString() + "script>");

   Даже не знаю, кому и зачем это понадобилось...

   Ordos, 24 Февраля 2011

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

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

   +145

   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

   function ruslat ($string) # Задаём функцию перекодировки кириллицы в транслит.
   {
   $string = ereg_replace("ж","zh",$string);
   $string = ereg_replace("ё","yo",$string);
   $string = ereg_replace("й","i",$string);
   $string = ereg_replace("ю","yu",$string);
   $string = ereg_replace("ь","'",$string);
   $string = ereg_replace("ч","ch",$string);
   $string = ereg_replace("щ","sh",$string);
   $string = ereg_replace("ц","c",$string);
   $string = ereg_replace("у","u",$string);
   $string = ereg_replace("к","k",$string);
   $string = ereg_replace("е","e",$string);
   $string = ereg_replace("н","n",$string);
   $string = ereg_replace("г","g",$string);
   $string = ereg_replace("ш","sh",$string);
   $string = ereg_replace("з","z",$string);
   $string = ereg_replace("х","h",$string);
   $string = ereg_replace("ъ","''",$string);
   $string = ereg_replace("ф","f",$string);
   $string = ereg_replace("ы","y",$string);
   $string = ereg_replace("в","v",$string);
   $string = ereg_replace("а","a",$string);
   $string = ereg_replace("п","p",$string);
   $string = ereg_replace("р","r",$string);
   $string = ereg_replace("о","o",$string);
   $string = ereg_replace("л","l",$string);
   $string = ereg_replace("д","d",$string);
   $string = ereg_replace("э","yе",$string);
   $string = ereg_replace("я","jа",$string);
   $string = ereg_replace("с","s",$string);
   $string = ereg_replace("м","m",$string);
   $string = ereg_replace("и","i",$string);
   $string = ereg_replace("т","t",$string);
   $string = ereg_replace("б","b",$string);
   $string = ereg_replace("Ё","yo",$string);
   $string = ereg_replace("Й","I",$string);
   $string = ereg_replace("Ю","YU",$string);
   $string = ereg_replace("Ч","CH",$string);
   $string = ereg_replace("Ь","'",$string);
   $string = ereg_replace("Щ","SH'",$string);
   $string = ereg_replace("Ц","C",$string);
   $string = ereg_replace("У","U",$string);
   $string = ereg_replace("К","K",$string);
   $string = ereg_replace("Е","E",$string);
   $string = ereg_replace("Н","N",$string);
   $string = ereg_replace("Г","G",$string);
   $string = ereg_replace("Ш","SH",$string);
   $string = ereg_replace("З","Z",$string);
   $string = ereg_replace("Х","H",$string);
   $string = ereg_replace("Ъ","''",$string);
   $string = ereg_replace("Ф","F",$string);
   $string = ereg_replace("Ы","Y",$string);
   $string = ereg_replace("В","V",$string);
   $string = ereg_replace("А","A",$string);
   $string = ereg_replace("П","P",$string);
   $string = ereg_replace("Р","R",$string);
   $string = ereg_replace("О","O",$string);
   $string = ereg_replace("Л","L",$string);
   $string = ereg_replace("Д","D",$string);
   $string = ereg_replace("Ж","Zh",$string);
   $string = ereg_replace("Э","Ye",$string);
   $string = ereg_replace("Я","Ja",$string);
   $string = ereg_replace("С","S",$string);
   $string = ereg_replace("М","M",$string);
   $string = ereg_replace("И","I",$string);
   $string = ereg_replace("Т","T",$string);
   $string = ereg_replace("Б","B",$string);
   return $string;
   }

   Sulik78, 23 Февраля 2011

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

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

    +160

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

    <?php
    $countH =1;
    $T = 0;
    foreach ($aSelectedUnits as $v) {
      if ($T != $v['UnitType']) {
        $countH = 1;
      }
      $countHired[$v['UnitNameId']] = $countH ++; // calculate hired units by type
      $T = $v['UnitNameId'];
    }
    ?>

    всего навсего нужно было подсчитать количество юнитов каждого типа.
    Как ни странно этот кусок кода считает юнытов, но не всегда верно.

    Sergunik, 22 Февраля 2011

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

11. Java / Говнокод #5755

    +84

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

    transDate = ( ExtBaApiStaging.class.equals( extExcep.getClass() ) )?((ExtBaApiStaging)extExcep).getBeginDate():
    		( ExtRsApiCancel.class.equals( extExcep.getClass() ) )?((ExtRsApiCancel)extExcep).getTransDate():
    		( ExtRsApiWriteCustContact.class.equals( extExcep.getClass() ) )?((ExtRsApiWriteCustContact)extExcep).getTransDate():
    		( ExtRsApiPayAdjust.class.equals( extExcep.getClass() ) )?((ExtRsApiPayAdjust)extExcep).getTransDate():
    		( ExtRsApiChgCustomer.class.equals( extExcep.getClass() ) )?((ExtRsApiChgCustomer)extExcep).getTransDate():
    		( ExtRsApiChgService.class.equals( extExcep.getClass() ) )?((ExtRsApiChgService)extExcep).getTransDate():
    		( ExtRsApiChgBoxData.class.equals( extExcep.getClass() ) )?((ExtRsApiChgBoxData)extExcep).getTransDate():
    		( ExtCLApiIsfMnp.class.equals( extExcep.getClass() ) )?((ExtCLApiIsfMnp)extExcep).getLoadDate():
    		( ExtCLApiNoTruckSro.class.equals( extExcep.getClass() ) )?((ExtCLApiNoTruckSro)extExcep).getLoadDate():
    		( ExtCLApiNsfHoldEcOsa.class.equals( extExcep.getClass() ) )?((ExtCLApiNsfHoldEcOsa)extExcep).getLoadDate():
    		( ExtRsApiSendAHit.class.equals( extExcep.getClass() ) )?((ExtRsApiSendAHit)extExcep).getTransDate() : null;

    Это писала одна тимлид

    tr00_gr1m_doomster, 21 Февраля 2011

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

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

• 1976
• …
• 1262
• 1261
• 1260
• 1259
• 1258
• 1257
• 1256
• 1255
• 1254
• …
• 1