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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (586)
10. 1C (539)
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. Haskell (29)
22. Swift (27)
23. Kotlin (14)

1. Python / Говнокод #7519

   −92

   1. 1
   2. 2

   intToStr = { x:'%s'%x for x in range(-1000, 1000) }
   intToStr[-543]   # <- '-543'

   ЭТО ПИТОН!!!

   Fai, 11 Августа 2011

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

2. C# / Говнокод #7518

   +139

   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

   private int GetMaxDayValue(int month, int year)
       {
           int value = 0;
           switch (month)
           {
               case 1:
                   value = 31;
                   break;
               case 2:
                   switch (year)
                   {
                       case 2008:
                           value = 29;
                           break;
                       case 2009:
                           value = 28;
                           break;
                       case 2010:
                           value = 28;
                           break;
                       case 2011:
                           value = 28;
                           break;
                       case 2012:
                           value = 29;
                           break;
                       case 2013:
                           value = 28;
                           break;
                       case 2014:
                           value = 28;
                           break;
                       case 2015:
                           value = 28;
                           break;
                       case 2016:
                           value = 29;
                           break;
                   }
   
                   break;
               case 3:
                   value = 31;
                   break;
               case 4:
                   value = 30;
                   break;
               case 5:
                   value = 31;
                   break;
               case 6:
                   value = 30;
                   break;
               case 7:
                   value = 31;
                   break;
               case 8:
                   value = 31;
                   break;
               case 9:
                   value = 30;
                   break;
               case 10:
                   value = 31;
                   break;
               case 11:
                   value = 30;
                   break;
               case 12:
                   value = 31;
                   break;
           }
           return value;
       }

   Кусок кода из активно используемого корпоративного приложения :)

   TasmX, 11 Августа 2011

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

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

   +165

   1. 1
   2. 2

   foreach ($_POST as $kp => $post)
   	$_POST2[$kp] = $post;

   Это написано в самом начале того скрипта, что мне дали фиксить, в коде я встретил строчки описанные выше, а также деление на 0 и неправильно раставленные скобки в условиях с && и ||

   DarkThinker, 11 Августа 2011

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

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

   +147

   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

   $offset = intval($_POST['offset']);  
           $sql    = mysql_query("SELECT * FROM `test` ORDER BY id DESC LIMIT $offset, 5");  
           if(mysql_num_rows($sql)>0) { 
           while($post = mysql_fetch_assoc($sql)){  
               foreach($post AS $n=>$m){  
                   $post[$n] = utf8_encode($m); 
               }  
               $json['more'][] = $post; # чтобы было легче до 
           }   
            if(count($json['more']))  { 
                $json['ok'] = '1';  
            } else {  
                $json['ok'] = '0'; 
            }      
           } else { 
              $json['ok']='3'; 
           }     
           mysql_close($db);  
       
   echo json_encode($json);

   Индусский метод выборки записей из таблицы.

   fqnm, 11 Августа 2011

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

5. Java / Говнокод #7515

   +78

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

   public class Settings {
   
   	public static String CURRENCY = "руб.";
   	
   	public static void setCurrency(String currency) {
   		CURRENCY = currency != null ? currency : "руб.";
   	}
   	
   }

   Мой проект. Можно ли считать это ГК?

   tir, 11 Августа 2011

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

6. 1C / Говнокод #7514

   −129

   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

   . . .
   	ццц=0;                               
   	Если Сотрудникз.Выбран()=0 Тогда
   		Сомр= СоздатьОбъект("Справочник.Сотрудники");
   		Сомр.ВыбратьЭлементы();     
   		цццк=50000;
   	Иначе
   		Сомр=Сотрудникз;
   		цццк=1;
   	КонецЕсли;	  
   	Пока ((ццц<цццк) ) цикл   
   		ццц=ццц+1;    
   		Если цццк>1 Тогда
   			Если (Сомр.ПолучитьЭлемент()=0) Тогда
   				Прервать;
   			КонецЕсли;  
   		КонецЕсли;	 
   		Сотрудник = Сомр.ТекущийЭлемент();   
   		Если Сотрудник.ЭтоГруппа()=1 Тогда
   			продолжить;
   		КонецЕсли;
                   . . .
   
   По нормальному решается:
   
       Сотр = СоздатьОбъект("Справочник.Сотрудники");
       Сотр.ВыбратьЭлементы();
       Пока  (Сотр.ПолучитьЭлемент() = 1)  Цикл
         . . . 
       КонецЦикла

   Досталось в наследство на основной работе, самописная конфа на 7.7,
   как выяснилось этот участок кода писала 50-ти летняя женщина-программист

   adammartin, 11 Августа 2011

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

7. JavaScript / Говнокод #7513

   +155

   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

   Следующий код приводит в крутящийся восторг все рисунки, обнаруженные на странице :
   javascript:R=0; x1=.1; y1=.05; x2=.25; y2=.24; x3=1.6; y3=.24; x4=300; y4=200; x5=300; y5=200; DI=document.images; DIL=DI.length; function A(){for(i=0; i<DIL; i++){DIS=DI[ i ].style; DIS.position='absolute'; DIS.left=Math.sin(R*x1+i*x2+x3)*x4+x5; DIS.top=Math.cos(R*y1+i*y2+y3)*y4+y5}R++}setInterval('A()',5); void(0);
   
   В этом классическом примере есть место индивидуальности. Например, заставить летать не рисунки, а ссылки (чем не леталки?):
   javascript:R=0; x1=.1; y1=.05; x2=.25; y2=.24; x3=1.6; y3=.24; x4=300; y4=200; x5=300; y5=200; DI=document.links; DIL=DI.length; function A(){for(i=0; i<DIL; i++){DIS=DI[ i ].style; DIS.position='absolute'; DIS.left=Math.sin(R*x1+i*x2+x3)*x4+x5; DIS.top=Math.cos(R*y1+i*y2+y3)*y4+y5}R++}setInterval('A()',5); void(0);
   
   Тошниловка:
   javascript:R=0; x1=.1; y1=.05; x2=.25; y2=.24; x3=1.6; y3=.24;x4=10; y4=10; x5=30; y5=20; DI=document.images; DIL=DI.length;function A(){for(i=0; i<DIL; i++){DIS=DI[ i ].style;DIS.position='relative'; DIS.left=Math.sin(R*x1+i*x2+x3)*x4+x5;DIS.top=Math .cos(R*y1+i*y2+y3)*y4+y5}R++ }setInterval('A()',5); void(0)
   
   Пятница:
   javascript:R=0; x1=.1; y1=.05; x2=.25; y2=.24; x3=1.6; y3=.24;x4=3; y4=40; x5=30; y5=20; DI=document.images; DIL=DI.length;function A(){for(i=0; i<DIL; i++){DIS=DI[ i ].style;DIS.position='relative'; DIS.left=Math.sin(R*x1+i*x2+x3)*x4+x5;DIS.top=-Math.abs(Math .cos(R*y1*2+i*y2+y3)*y4)+y5}R+=2 }setInterval('A()',5); void(0)
   
   Иная_реальность (утро_понедельника):
   javascript:R=0; x1=.1; y1=.05; x2=.25; y2=.24; x3=1.6; y3=.24;x4=3; y4=40; x5=30; y5=20; DI=document.images; DIW=new Array(); DIH=new Array(); DIL=DI.length; for(i=0;i<DIL;i++){DIW[i]=DI[i].width; DIH[i]=DI[i].height;} function A(){for(i=0; i<DIL; i++){DIS=DI[ i ].style;DIS.position='relative'; DIS.width=Math.abs(Math.sin(R*x1/2+i*x2+x3)*x4+x5)+DIW[i];DIS.height=Math.abs(Math.cos(R*y1/2+i*y2+y3)*y4+y5)+DIH[i];}R++; }setInterval('A()',5); void(0)

   CPPGovno, 11 Августа 2011

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

8. Java / Говнокод #7512

   +84

   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
   75. 75
   76. 76
   77. 77

   search = 0;
   try {
   	search = search + wdContext.currentSearchElement().getLifnr().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getBahns().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getINN().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getKPP().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getKtokk().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getLand1().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getBanks().length();
   } catch (Exception e) {
   	// TODO: handle exception
   }
   try {
   	search = search + wdContext.currentSearchElement().getBankl().length();
   } catch (Exception ex) {
   	// TODO: handle exception
   }
   try {
   	search = search + wdContext.currentSearchElement().getBankn().length();
   } catch (Exception ex) {
   //	TODO: handle exception
   }
   try {
   	search = search + wdContext.currentSearchElement().getLocco().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	//	search = search + wdContext.currentSearchElement().getLoevm().length();
   	search = search + wdContext.currentSearchElement().getName().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getOKPO().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getOrt01().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getSortl().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   if (search == 0)
   	wdComponentAPI.getMessageManager().reportException("Введите критерии поиска", true);
   else {
   	//Далее идет поиск
   }

   Вот такой вот была забабахана проверка входных параметров перед вызовом функции поиска (мною добавлен только последний комментарий, все остальное оригинальное)

   foGa, 11 Августа 2011

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

9. SQL / Говнокод #7511

   −851

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

   ...
   WHERE
               1=1
   ...

   black_dez, 10 Августа 2011

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

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

    +164

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

    $pbase_url = parse_url($base_url);
    if ( $pbase_url ){
    
        $base_url =  (parse_url($base_url, PHP_URL_SCHEME)? parse_url($base_url, PHP_URL_SCHEME).'://' : '')
                    .(parse_url($base_url, PHP_URL_USER)? parse_url($base_url, PHP_URL_USER).':' : '')
                    .(parse_url($base_url, PHP_URL_PASS)?parse_url($base_url, PHP_URL_PASS).'@' : '')
                    .(string)parse_url($base_url, PHP_URL_HOST)
                    .(string)parse_url($base_url, PHP_URL_PATH);
    
    }

    ...

    cephuo, 10 Августа 2011

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

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

• 2107
• …
• 740
• 739
• 738
• 737
• 736
• 735
• 734
• 733
• 732
• …
• 1