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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2325)
4. JavaScript (2027)
5. C# (1925)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. SQL / Говнокод #24273

   0

   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
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93

   "SELECT " +
                       "2 AS recType," +
                       "1 AS cnt," +
                       "''  AS vndName," +
                       "'' AS catName," +
                       "off.orderBy1S AS orderBy1S," +
                       "off._id AS _id," +
                       "off.vendorCode AS vendorCode," +
                       "off.price AS price," +
                       "off.currencyId AS currencyId," +
                       "off.thumbnail AS thumbnail," +
                       "off.name AS name," +
                       "IFNULL(crt.quantity,0) AS quantityItem," +
                       "off.queryRest AS queryRest," +
                       "off.valueRest AS valueRest," +
                       "off.lastChangedRest AS lastChangedRest," +
                       "off.price_uah_ir AS price_uah_ir," +
                       "IFNULL(vlt.[rate],0) AS rate " + //валютный курс для currencyId товара
                       "FROM tOffer AS off " +
                       "LEFT JOIN tCart AS crt ON off.vendorCode=crt.offerId " +
                       "LEFT JOIN tCurrency AS vlt ON off.currencyId=vlt.valute " +
   
                       "WHERE  off.vendorName='" + BRAND_NAME + "' AND off.categoryId=" + CATEGORY_ID + " AND  (off.name LIKE " + nameFilter + ") " +
                       "UNION " +
                       "SELECT " +
                       "1," +
                       "COUNT(*)," +
                       "MAX(vnd.name)," +
                       "IFNULL(cat.name,'')," +
                       "0," +
                       "0," +
                       "0," +
                       "0," +
                       "''," +
                       "''," +
                       "vnd.name," +
                       "0," +
                       "0," +
                       "''," +
                       "''," +
                       "0," +
                       "0 " +
                       "FROM tOffer AS off " +
                       "LEFT JOIN tVendor AS vnd ON off.vendorName=vnd.name " +
                       "INNER JOIN tCategory AS cat ON off.categoryId=cat._id " +
                       "WHERE  off.vendorName='" + BRAND_NAME + "' AND off.categoryId=" + CATEGORY_ID + " AND  (off.name LIKE " + nameFilter + ") " +
                       "GROUP BY vnd.name HAVING COUNT(*)>0 " +
                       "UNION " +
                       "SELECT " +
                       "3," +
                       "COUNT(*)," +
                       "MAX('Другой товар')," +
                       "''," +
                       "0," +
                       "0," +
                       "0," +
                       "0," +
                       "''," +
                       "''," +
                       "'Другой товар'," +
                       "0," +
                       "0," +
                       "''," +
                       "''," +
                       "0," +
                       "0 " +
                       "FROM tOffer AS off " +
                       "WHERE  off.vendorName<>'" + BRAND_NAME + "' AND  (off.name LIKE " + nameFilter + ") " +
                       "GROUP BY 'Другой товар' HAVING COUNT(*)>0 " +
                       "UNION " +
                       "SELECT " +
                       "4," +
                       "1," +
                       "''," +
                       "''," +
                       "off.orderBy1S," +
                       "off._id," +
                       "off.vendorCode," +
                       "off.price," +
                       "off.currencyId," +
                       "off.thumbnail," +
                       "off.name," +
                       "IFNULL(crt.quantity,0)," +
                       "off.queryRest," +
                       "off.valueRest," +
                       "off.lastChangedRest," +
                       "off.price_uah_ir," +
                       "IFNULL(vlt.[rate],0) " + //валютный курс для currencyId товара
                       "FROM tOffer AS off " +
                       "LEFT JOIN tCart AS crt ON off.vendorCode=crt.offerId " +
                       "LEFT JOIN tCurrency AS vlt ON off.currencyId=vlt.valute " +
                       "WHERE  off.vendorName<>'" + BRAND_NAME + "' AND  (off.name LIKE " + nameFilter + ") " +
                       "ORDER BY  recType,orderBy1S ";

   Комменты говорят, что это поиск по бренду и названию

   makesense, 16 Мая 2018

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

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

   +8

   1. 1
   2. 2
   3. 3

   <a href="/login">
     <?php echo( 'В'. ($isLogged ? 'ы' : '') . 'ход'); ?>
   </a>

   bakesi, 16 Мая 2018

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

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

   −2

   1. 1

   https://vc.ru/37909-my-stroim-letnyuyu-shkolu-v-poselke-programmistov

   как вам мысль учить пхп на природе?

   roskomgovno, 16 Мая 2018

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

4. Куча / Говнокод #24262

   0

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

   ...
        F = fun(S) ->
                 ets:give_away(Table, NewOwner),
                 S
             end,
         sys:replace_state(OldOwner, F)
   ...

   Паттерн steal

   CHayT, 15 Мая 2018

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

5. Python / Говнокод #24255

   −2

   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

   secs = 0
   fsecs = 0.0
   mins = 40
   lastsecs = 0
   prevsecs = 0
   
   def getTimeStr() :
       def f ( x ) :
           if ( 10 <= x <= 19 ) : return ''
           y = x % 10
           if ( y == 1 ) : return 'а'
           if ( 2 <= y <= 4 ) : return 'ы'
           return ''
       return 'Время: %d минут%s %d секунд%s' % ( mins , f ( mins ) , secs , f ( secs ) )
   
   def updateTimer() :
       global timeString , root , secs , fsecs , lastsecs , prevsecs , mins
       if ( fin ) : return
       prevsecs , lastsecs = lastsecs , clock()
       t = secs
       if ( fsecs > 0 ) :
           fsecs -= lastsecs - prevsecs
           secs = int ( fsecs )
       elif ( mins > 0 ) :
           mins -= 1
           fsecs = 60 + fsecs - lastsecs + prevsecs
           secs = int ( fsecs )
       else :
           finish()
       if ( secs == t ) :
           timeString [ 'text' ] = getTimeStr()
       root.after ( 100 , updateTimer )

   Святая Питония и tkinter

   shit_again, 12 Мая 2018

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

6. Куча / Говнокод #24253

   +1

   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
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90

   // https://github.com/Navadvipa-Chandra-das/prabhupada_calendar/blob/master/source/Prabhupada_convert.d
   
       bool день_ли_это;
       Тип_строки вид_строки;
       
       Класс_Хитрый_накопитель Хитрый_накопитель = new Класс_Хитрый_накопитель( !Нужно_ли_удалять_пустые_дни, !Нужно_ли_удалять_пустые_дни_окончания_поста, Выходной_файл );
       // цикл не простой, а с меткой "цикл"
       цикл: while ( ( строка = Входной_файл.readln() ) !is null ) {
   
         if ( Нужно_ли_удалять_служебные_строки ) {
           if ( строка.length < 3 )
             // чудо, а не возможность! Можно делать любые переходы и вылеты из множества вложенных циклов!
             continue цикл;
           служебная_строка = matchFirst( строка, выражение_служебная_строка );
           if ( служебная_строка.captures.length > 0 )
             continue цикл;
         }
       
         день_месяца = matchFirst( строка, выражение_день );
         день_ли_это = день_месяца.captures.length > 0;
         
         if ( день_ли_это ) {
           вид_строки = Тип_строки.День;
           if ( Нужна_ли_табуляция ) {
             строка = "\t"c ~ день_месяца.captures[ 1 ] ~ "\t"c ~ день_месяца.captures[ 2 ]              ~ "\t"c ~ день_месяца.captures[ 3 ] ~
                      "\t"c ~ день_месяца.captures[ 4 ] ~ "\t"c ~ день_месяца.captures[ 6 ].stripRight() ~ "\t"c ~ день_месяца.captures[ 7 ] ~
                      "\t"c ~ день_месяца.captures[ 8 ] ~ "\t"c ~ день_месяца.captures[ 10 ];
             // обрабатываем "звездочку" поста
             if ( день_месяца.captures[ 11 ].indexOf( '*' ) != -1 )
               строка ~= "\t*"c;
             строка ~= "\n";
           }
           // Удаляем, при желании комментрий к Экадаши - подходит для поста и не додходит для поста. Все равно есть еще символ звездочка и
           // строка с именем Экадаши и явным упоминанием о посте!
           if ( Нужно_ли_удалять_комментарий_Экадаши ) {
             строка = replaceFirst( строка, выражение_для_чистки_комментария_Экадаши, ""c );
           }
         } else {
           проверка_месяца = matchFirst( строка, выражение_месяц_ли_это );
           if ( проверка_месяца.captures.length > 0 ) {
             вид_строки = Тип_строки.Месяц;
             // Просто удаляем цепочку начальных пробелов в строках месяца, если готовим табуляцию
             if ( Нужна_ли_табуляция )
               строка = replaceFirst( строка, выражение_цепочка_пробелов_в_начале_строки, ""c );
           } else {
             // Заменяем цепочку пробелов вначале строки на один единственный символ табуляции для праздников, но только если нужна табуляция
             пробелы_в_начале_строки = matchFirst( строка, выражение_цепочка_пробелов_в_начале_строки );
             if ( пробелы_в_начале_строки.captures.length > 0 ) {
               окончание_поста = matchFirst( строка, выражение_окончание_поста );
               if ( окончание_поста.captures.length > 0 )
                 вид_строки = Тип_строки.Окончание_поста;
               else
                 вид_строки = Тип_строки.Праздник;
               if ( Нужна_ли_табуляция )
                 строка = replaceFirst( строка, выражение_цепочка_пробелов_в_начале_строки, "\t"c );
             } else
               вид_строки = Тип_строки.Солнце;
           }
         }
   
         // Удаляем строки из тире и GCal, но только если нужна табуляци. В книге они не нужны.
         // Можно указать номер версии программы GCal в колонтитулах книги при желании
         if ( Нужна_ли_табуляция ) {
           if ( вид_строки == Тип_строки.Месяц )
             строка = replaceAll( строка, выражение_для_чистки_Месяца, ""c );
           if ( вид_строки == Тип_строки.Солнце )
             строка = replaceAll( строка, выражение_для_чистки_Солнца, ""c );
         }
         
         // Переводим сначала фразы. Длительная операция, так как переводятся все имеющиеся фразы, не зависимо от того, встречаются ли они в строке или нет
         foreach ( ref фраза; фразы )
           строка = replaceAll( строка, regex( фраза ), _Словарь_фраз[ фраза ] );
   
         // Переводим слова. Быстрее, так как переводятся только те слова, которые встречаются в данной конкретной строке
         слова = std.regex.split( строка, выражение_разделитель );
         // сортируем, чтобы короткие слова не позаменялись в более длинных словах внутри
         // Вдохнули поглубже! Такого чуда я еще не видел! Не иначе как mixin в действии!
         слова.sort!( "a.length > b.length" );
         foreach ( ref слово; слова )
           if ( слово.length > 0 )
             строка = replaceAll!( Чудо_слов )( строка, regex( слово ) );
         
         Хитрый_накопитель.В_копилку( строка, вид_строки );
       }
       // Пропихнем в файл последнюю строку
       Хитрый_накопитель.В_копилку( ""c, Тип_строки.День );
   
       window.showMessageBox( UIString.fromId( "ЧЕСТЬ_ИМЕЮ_ДОЛОЖИТЬ"c ),
                              UIString.fromId( "ЗАДАНИЕ_ВЫПОЛНЕНО!"c ) );
   }

   Кришнаит, пишущий на языке D в стиле 1С

   j123123, 12 Мая 2018

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

7. Java / Говнокод #24249

   +3

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

   if (x_sum_first_row == 3 || x_sum_second_row == 3 || x_sum_third_row == 3 ||
               x_sum_first_column == 3 || x_sum_second_column == 3 || x_sum_third_column == 3 ||
               x_sumDiagonalLR == 3 || x_sumDiagonalRL == 3 ||
               o_sum_first_row == 3 || o_sum_second_row == 3 || o_sum_third_row == 3 ||
               o_sum_first_column == 3 || o_sum_second_column == 3 || o_sum_third_column == 3 ||
               o_sumDiagonalLR == 3 || o_sumDiagonalRL == 3)
               someoneWon = true;
   
           return someoneWon;

   https://codereview.stackexchange.com/questions/125248/java-tic-tac-toe-game-implemented-through-mvc

   roskomgovno, 11 Мая 2018

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

8. Си / Говнокод #24247

   −3

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

   double m_sqrt (double p){
                double l = 1;
                double m = 0;
   	
          for(m  =  (l +  ( p / l )   )  / 2;          
   		m  !=  l ;  
   		m  =  (l +  ( p / l )   )  / 2){
   	 	l = m; 
   	}
   
   return m;
   }

   tyrin, 10 Мая 2018

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

9. Java / Говнокод #24245

   +3

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

   public static Date round(Date d) {
       try {
           SimpleDateFormat sdf = new SimpleDateFormat("ddMMyyyy");
           return sdf.parse(sdf.format(d));
       } catch (ParseException ex) {
           //This exception will never be thrown, because sdf parses what it formats
           return d;
       }
   }

   Простейший коробочный способ округления даты до дня.

   3.14159265, 10 Мая 2018

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

10. SQL / Говнокод #24243

    +1

    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

    select
      x1.airport, x1.amount, x1.exempt
    from
      Taxes x1
    where
      x1.code = 'departure'
      and x1.airport in (
        select
          xx1.airport
        from
          (select
            yx1.airport
          from
            Taxes yx1
          where
            yx1.airport = x1.airport
            and yx1.code = x1.code
          group by
            yx1.airport, yx1.amount, yx1.exempt
          ) xx1
        group by xx1.airport
        having count(xx1.airport) > 1
      )
    ;

    Давно я не копался в SQL. Вот что нагородил :(
    
    Суть такова: у аэропортов есть departure tax, который может быть разный в зависимости от некоторых критериев (нерелевантно каких).
    А может быть и одинаковый. Но все равно аэропорт может иметь несколько рядков в БД (с одинаковым amount). А еще может быть exempt = 'X',
    что тождественно amount = 0.
    
    Задача: выбрать все рядки с departure tax для аэропортов у которых taxamt/exempt разный в зависимости от некоторых других критериев.

    Elvenfighter, 10 Мая 2018

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

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

• 2095
• …
• 1790
• 1789
• 1788
• 1787
• 1786
• 1785
• 1784
• 1783
• 1782
• …
• 1