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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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 (28)
22. Swift (27)
23. Kotlin (14)

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

   +156

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

   $c=count($arr1);
   for($i=0;$i<$c;$i++){
   	$key=key($arr1);
   	ksort($arr1[$key]);
   	next($arr1);
   }

   foreach() чем-то не угодил

   Khvorostin, 16 Сентября 2014

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

2. Perl / Говнокод #16701

   −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

   eval {
           my $from = $class->get_env_sender($message);
   
           $smtp->mail($from)
               || croak("Email::Send::Gmail: error sending 'from' $from");
   
           my @to = $class->get_env_recipients($message);
   
           my @ok = $smtp->to( @to, { SkipBad => 1 } )
               || croak("Email::Send::Gmail: error sending 'to' @to");
   
           if ( @to != @ok ) {
               my %to;
               @to{@to} = (1) x @to;
               delete @to{@ok};
               @bad = keys %to;
           }
   
           croak("Email::Send::Gmail: no valid recipients") if @bad == @to;
       };

   Думаю понятно, какой это модуль с CPAN. Убило незнание приоритетов, в результате чего "|| croak" всегда возвращало в @ok число вместо результата выполнения $smtp->to

   NeoliteBSD, 16 Сентября 2014

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

3. Python / Говнокод #16700

   −99

   1. 1
   2. 2

   def str_to_float(i):
       return int (i) + 0.0

   Так надо!

   zadrot, 16 Сентября 2014

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

4. C++ / Говнокод #16699

   +52

   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

   double cCompositeBlock::determinant4x4(double *d){ // WARNING It's not logically connected with class.
       return  d[3]*d[6]*d[9]*d[12]  -d[2]*d[7]*d[9]*d[12]-
               d[3]*d[5]*d[10]*d[12] +d[1]*d[7]*d[10]*d[12]+
               d[2]*d[5]*d[11]*d[12] -d[1]*d[6]*d[11]*d[12]-
               d[3]*d[6]*d[8]*d[13]  +d[2]*d[7]*d[8]*d[13]+
               d[3]*d[4]*d[10]*d[13] -d[0]*d[7]*d[10]*d[13]-
               d[2]*d[4]*d[11]*d[13] +d[0]*d[6]*d[11]*d[13]+
               d[3]*d[5]*d[8]*d[14]  -d[1]*d[7]*d[8]*d[14]-
               d[3]*d[4]*d[9]*d[14]  +d[0]*d[7]*d[9]*d[14]+
               d[1]*d[4]*d[11]*d[14] -d[0]*d[5]*d[11]*d[14]-
               d[2]*d[5]*d[8]*d[15]  +d[1]*d[6]*d[8]*d[15]+
               d[2]*d[4]*d[9]*d[15]  -d[0]*d[6]*d[9]*d[15]-
               d[1]*d[4]*d[10]*d[15] +d[0]*d[5]*d[10]*d[15];
   }

   Abbath, 16 Сентября 2014

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

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

   +73

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

   public static <T extends Comparable<T>> boolean isLessThan(T a, T b, double numericTolerance) {
       if (a == null) {
           return b != null;
       }
       boolean isLessThan = a.compareTo(b) < 0;
       if (!isLessThan && a instanceof Number && b instanceof Number) {
           isLessThan = ((Comparable) (((Number) a).doubleValue() - numericTolerance)).compareTo(((Number) b).doubleValue()) < 0;
       }
       return isLessThan;
   }

   Один из методов сравнения значений в пределах допустимой погрешности (последняя только для чисел).
   Вроде бы и проще никак, но чувство говна не покидает. Советы по упрощению приветствуются.

   Actine, 16 Сентября 2014

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

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

   +155

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

   <?php
   
   $data = file_get_contents("/path/to/photo.jpg"); // Read the file's contents
   $name = 'myphoto.jpg';
   
   force_download($name, $data);

   Я знаю конечно что это не говнокод, но последствия будут ужасными если не передать сразу же в функцию данные (т.к. PHP будет копировать содержимое дважды), представьте себе файл в 2МБ и много запросов, сервер капут в два раза быстрее.
   - https://ellislab.com/codeigniter/user-guide/helpers/download_helper.html

   volter9, 15 Сентября 2014

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

7. 1C / Говнокод #16695

   −126

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

   &НаСервере
   Процедура ПриСозданииНаСервере(Отказ, СтандартнаяОбработка)
   
           НовыйДокумент = Документы.ДокументПредЗаявка.СоздатьДокумент();
            НовыйДокумент.Дата = ТекущаяДата();
            НовыйДокумент.Записать(РежимЗаписиДокумента.Проведение);
            СПД = НовыйДокумент.Ссылка;
            
            //Открываем документ
            ОткрытьЗначение(СПД);
         
   КонецПроцедуры

   Я не вкурил, но полагаю, что ЭТО новый способ создания документа на основании...

   Danik, 15 Сентября 2014

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

8. JavaScript / Говнокод #16694

   +151

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

   //...
   <script>splashRadar('leftcol', 1);</script>
   <script>splashRadar('baseStyles');</script>
   //...

   На мейл.сру.

   gost, 14 Сентября 2014

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

9. JavaScript / Говнокод #16693

   +157

   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

   $('#id_accept_eula').click(function () {
       // код аффтара
       try {
           var v = 1 - this.getAttribute('value');
           this.setAttribute('value', v);
           if (v == 1) {
               $('#submit_button').removeAttr('disabled');
           } else {
               $('#submit_button').attr('disabled', 'disabled');
           }
       } catch (e) {
           alert('exc: ' + e);
       }
       // заменил на
       $("#submit_button").prop('disabled', !this.checked);
   });

   Когда платят за строки кода...

   govnozmey, 14 Сентября 2014

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

10. Си / Говнокод #16692

    +131

    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
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84

    /*
     * Copyright (C) 2007-2012 Allwinner Technology Co., Ltd.
     *
     * This program is free software; you can redistribute it and/or
     * modify it under the terms of the GNU General Public License as
     * published by the Free Software Foundation; either version 2 of
     * the License, or (at your option) any later version.
     *
     * This program is distributed in the hope that it will be useful,
     * but WITHOUT ANY WARRANTY; without even the implied warranty of
     * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the
     * GNU General Public License for more details.
     *
     * You should have received a copy of the GNU General Public License
     * along with this program; if not, write to the Free Software
     * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
     * MA 02111-1307 USA
     */
    
    #ifndef __DISP_DISPLAY_I_H__
    #define __DISP_DISPLAY_I_H__
    
    #include "ebios_de.h"
    #include "ebios_lcdc_tve.h"
    #include "drv_disp_i.h" /* For DISP_XXXXXX status codes */
    
    #define DE_INF __inf
    #define DE_WRN __wrn
    
    #define OSAL_IRQ_RETURN IRQ_HANDLED
    
    #define HANDTOID(handle)  ((handle) - 100)
    #define IDTOHAND(ID)  ((ID) + 100)
    
    #define INTC_IRQNO_SCALER0  SW_INT_IRQNO_DEFEBE0
    #define INTC_IRQNO_SCALER1  SW_INT_IRQNO_DEFEBE1
    #define INTC_IRQNO_LCDC0    SW_INT_IRQNO_LCDCTRL0
    #define INTC_IRQNO_LCDC1    SW_INT_IRQNO_LCDCTRL1
    
    #define MAX_SPRITE_BLOCKS   32
    
    /*basic data information definition*/
    enum {
    	FALSE = 0,
    	TRUE
    };
    
    #define DIS_NULL 0
    
    #define BIT0    0x00000001
    #define BIT1	0x00000002
    #define BIT2	0x00000004
    #define BIT3	0x00000008
    #define BIT4	0x00000010
    #define BIT5	0x00000020
    #define BIT6	0x00000040
    #define BIT7	0x00000080
    #define BIT8	0x00000100
    #define BIT9	0x00000200
    #define BIT10	0x00000400
    #define BIT11	0x00000800
    #define BIT12	0x00001000
    #define BIT13	0x00002000
    #define BIT14	0x00004000
    #define BIT15	0x00008000
    #define BIT16	0x00010000
    #define BIT17	0x00020000
    #define BIT18	0x00040000
    #define BIT19	0x00080000
    #define BIT20	0x00100000
    #define BIT21	0x00200000
    #define BIT22	0x00400000
    #define BIT23	0x00800000
    #define BIT24	0x01000000
    #define BIT25	0x02000000
    #define BIT26	0x04000000
    #define BIT27	0x08000000
    #define BIT28	0x10000000
    #define BIT29	0x20000000
    #define BIT30	0x40000000
    #define BIT31	0x80000000
    
    
    #endif

    Нашёл в драйвере дисплея. Вместо 1<<N.

    mittorn, 13 Сентября 2014

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

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

• 2107
• …
• 1415
• 1414
• 1413
• 1412
• 1411
• 1410
• 1409
• 1408
• 1407
• …
• 1