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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2386)
4. JavaScript (2034)
5. C# (1930)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
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 (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. 1C / Говнокод #23173

   +2

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

   defmodule Utils.String do
     @moduledoc false
     def comma_split(str), do: String.split(str, " ", trim: true)
   end

   AndrewDryga, 10 Июля 2017

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

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

   +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
   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

   class StreamRailBase:
       __metaclass__ = ABCMeta
   
       def __init__(self): pass
   
       # @abstractmethod
       # def create_connection(self): pass
       #
       # @abstractmethod
       # def send_to_sr(self, method, path, params=None): pass
   
       # @abstractmethod
       # def get_advertisers(self): pass
       #
       # @abstractmethod
       # def get_targeting_conditions(self, env, geos, os, size, white_list, black_list): pass
       #
       # @abstractmethod
       # def create_ad_source(self, name, price, partner, tag_url, env, geos, os, size, req_cap, imp_cap, white_list=None, black_list=None): pass
       #
       # @abstractmethod
       # def create_sr_tag(self, tag_instance): pass
       #
       # @abstractmethod
       # def create_domain_list(self, f, list_type, adsource_name): pass
       #
       # @abstractmethod
       # def activate_adsource(self, adsource_id): pass
       #
       # @abstractmethod
       # def deactivate_adsource(self, adsource_id): pass
       #
       # @abstractmethod
       # def set_caps(self, params, new_req_cap, new_imp_cap): pass
       #
       # @abstractmethod
       # def set_geos(self, params, new_geos): pass
       #
       # @abstractmethod
       # def edit_domain_list(self, f, list_id): pass
       #
       # @abstractmethod
       # def set_list(self, params, new_list, list_type): pass
       #
       # @abstractmethod
       # def set_size(self, params, size): pass
       #
       # @abstractmethod
       # def edit_adsource(self, tag_instance, form_data): pass
       #
       # @abstractmethod
       # def create_new_partner(self, name): pass
       #
       # @abstractmethod
       # def sync_tag(self, tag_instance): pass
       #
       # @abstractmethod
       # def add_new_tag_to_template(self, tag_name, price, tag_id, adv_id): pass

   В человеке умер рахитектор, и в отчаянии, он закомитил эту хуйню в мастер.

   wvxvw, 09 Июля 2017

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

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

   +2

   1. 001
   2. 002
   3. 003
   4. 004
   5. 005
   6. 006
   7. 007
   8. 008
   9. 009
   10. 010
   11. 011
   12. 012
   13. 013
   14. 014
   15. 015
   16. 016
   17. 017
   18. 018
   19. 019
   20. 020
   21. 021
   22. 022
   23. 023
   24. 024
   25. 025
   26. 026
   27. 027
   28. 028
   29. 029
   30. 030
   31. 031
   32. 032
   33. 033
   34. 034
   35. 035
   36. 036
   37. 037
   38. 038
   39. 039
   40. 040
   41. 041
   42. 042
   43. 043
   44. 044
   45. 045
   46. 046
   47. 047
   48. 048
   49. 049
   50. 050
   51. 051
   52. 052
   53. 053
   54. 054
   55. 055
   56. 056
   57. 057
   58. 058
   59. 059
   60. 060
   61. 061
   62. 062
   63. 063
   64. 064
   65. 065
   66. 066
   67. 067
   68. 068
   69. 069
   70. 070
   71. 071
   72. 072
   73. 073
   74. 074
   75. 075
   76. 076
   77. 077
   78. 078
   79. 079
   80. 080
   81. 081
   82. 082
   83. 083
   84. 084
   85. 085
   86. 086
   87. 087
   88. 088
   89. 089
   90. 090
   91. 091
   92. 092
   93. 093
   94. 094
   95. 095
   96. 096
   97. 097
   98. 098
   99. 099
   100. 100

   switch (address & 0xF000)
       {
           case 0x0000:
           case 0x1000:
           case 0x2000:
           case 0x3000:
               return rom[address];
           case 0x4000:
           case 0x5000:
           case 0x6000:
           case 0x7000:
               return rom[address + 0x4000 * (bank_index - 1)];
           case 0x8000:
           case 0x9000:
               return vram[address - 0x8000];
           case 0xA000:
           case 0xB000:
               return sram[address - 0xA000];
           case 0xC000:
           case 0xD000:
               return wram[address - 0xC000];
           case 0xE000:
           case 0xF000:
               switch (address & 0x0F00)
               {
                   case 0x0000:
                   case 0x0100:
                   case 0x0200:
                   case 0x0300:
                   case 0x0400:
                   case 0x0500:
                   case 0x0600:
                   case 0x0700:
                   case 0x0800:
                   case 0x0900:
                   case 0x0A00:
                   case 0x0B00:
                   case 0x0C00:
                   case 0x0D00:
                       return wram[address - 0xF000];
                   case 0x0E00:
                       switch (address & 0x00F0)
                       {
                           case 0x0000:
                           case 0x0010:
                           case 0x0020:
                           case 0x0030:
                           case 0x0040:
                           case 0x0050:
                           case 0x0060:
                           case 0x0070:
                           case 0x0080:
                           case 0x0090:
                               return oam[address - 0xFE00];
                           case 0x00A0:
                           case 0x00B0:
                           case 0x00C0:
                           case 0x00D0:
                           case 0x00E0:
                           case 0x00F0:
                               return 0b00;
                       }
                   case 0x0F00:
                       switch (address & 0x00F0)
                       {
                           case 0x0000:
                           case 0x0010:
                           case 0x0020:
                           case 0x0030:
                           case 0x0040:
                           case 0x0050:
                           case 0x0060:
                           case 0x0070:
                               return io_regs[address - 0xFF00];
                           case 0x0080:
                           case 0x0090:
                           case 0x00A0:
                           case 0x00B0:
                           case 0x00C0:
                           case 0x00D0:
                           case 0x00E0:
                           case 0x00F0:
                               switch (address & 0x000F)
                               {
                                   case 0x0000:
                                   case 0x0001:
                                   case 0x0002:
                                   case 0x0003:
                                   case 0x0004:
                                   case 0x0005:
                                   case 0x0007:
                                   case 0x0008:
                                   case 0x0009:
                                   case 0x000A:
                                   case 0x000B:
                                   case 0x000C:
                                   case 0x000D:
                                   case 0x000E:
                                       return hram[address - 0xFF80];
                                   case 0x000F:

   jangolare, 06 Июля 2017

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

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

   +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

   $value = $grandTotal  - $tax;
           $total_name = $this->__('Grandtotal_excl_tax')." (".$this->__('excl. tax').")";
           $row = '<div class="columns xsmall-6 small-6">';
           $row .= $total_name;
           $row .= '</div><div class="columns xsmall-6 small-6 align-right">'.$this->helper('checkout')->formatPrice($value).'</div>';
   
           $totals["grand_total_excl_tax"] = $row;
   
   
           //$orderedTotals = Mage::getStoreConfig('onestepcheckout/general/summary_totals');
           $orderedTotals =  "subtotal,aw_giftcard_duty,aw_giftcard_duty_discount,shipping,cashondelivery,shipping_discount,discount,aw_giftcard,grand_total_excl_tax,tax,grand_total";
           $orderedTotals = explode(',', $orderedTotals);
           foreach($orderedTotals as $total) {
               $total = trim($total);
               if (isset($totals[$total])) {
                   echo $totals[$total];
               }
           }
           ?>
       </div>

   Разгребаем проект на Magento доставшийся в наследство. Умеете ли вы готовить Array() так как готовят его в строках 11-12 ?
   
   #littleitaly

   mrdink, 06 Июля 2017

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

6. Python / Говнокод #23145

   +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

   if hasattr(query, "items"):
           query = query.items()
       else:
           # It's a bother at times that strings and string-like objects are
           # sequences.
           try:
               # non-sequence items should not work with len()
               # non-empty strings will fail this
               if len(query) and not isinstance(query[0], tuple):
                   raise TypeError
               # Zero-length sequences of all types will get here and succeed,
               # but that's a minor nit.  Since the original implementation
               # allowed empty dicts that type of behavior probably should be
               # preserved for consistency
           except TypeError:
               ty, va, tb = sys.exc_info()
               raise TypeError("not a valid non-string sequence "
                               "or mapping object").with_traceback(tb)

   https://github.com/python/cpython/blob/master/Lib/urllib/parse.py#L848
   
   Зачем генерировать TypeError, а потом ее ловить и снова кидать?

   Admina_mamu_ebal, 23 Июня 2017

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

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

   +2

   1. 1

   return ( empty( $syndication_meta ) ? false : true );

   ldv_a, 16 Июня 2017

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

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

   +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

   public function sql ($sql) {
   	$r = array(
   		"'" => "", //ору с этого
   		'"' => '',
   		'DROP' => '',
   		'TRUNCATE' => '',
   		'SELECT' => '',
   		'UPDATE' => '',
   		'INSERT' => '',
   		'DELETE' => '',
   		'INSERT' => '',
   		'UNION' => ''
   	);
   
   	if($this->strposa(strtoupper($sql), array_keys($r))){
   		mail('[email protected]', 'Внимание!', 'Обнаружена попытка SQL инъекции: "'.$sql.'" с IP адреса: '.$_SERVER['REMOTE_ADDR']);
   	}
   
   	$sql = str_ireplace(array_keys($r), $r, $sql);
   	$sql = htmlspecialchars($sql);
   	$sql = strip_tags($sql);
   	$sql = stripslashes($sql);
   
   	return $sql;
   }

   enly1, 14 Июня 2017

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

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

   +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
   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

   /**
        * Установка условий проверки корректности дат на начальную дату
        * */
       setConditionOne(date1, date2) {
           if ( (date1[2] > date2[2]) && (date1[1] >= date2[1]) && (date1[0] < date2[0]) )
               return false;
           else if ( (date1[2] > date2[2]) && (date1[1] >= date2[1]) && (date1[0] >= date2[0]) )
               return true;
           else if ( (date1[2] == date2[2]) && (date1[1] > date2[1]) && (date1[0] == date2[0]) )
               return true;
           else if ( (date1[2] > date2[2]) && (date1[1] > date2[1]) )
               return true;
           else if (date1[2] > date2[2] && (date1[1] > date2[1]) && (date1[0] > date2[0]) )
               return true;
           else if ( (date1[2] < date2[2] ) && (date1[1] > date2[1]) && (date1[0] < date2[0]))
               return false;
           else if ( (date1[2] <= date2[2] ) && ( date1[1] > date2[1]) && (date1[0] <= date2[0]) )
               return true;
           else if ( (date1[2] > date2[2] ) && ( date1[1] >= date2[1]) && (date1[0] < date2[0]) )
               return false;
           else if ( (date1[0] > date2[0]) )
               return true;
           else if ( (date1[2] == date2[2] ) && ( date1[1] > date2[1]) && (date1[0] == date2[0]) )
               return true;
           else if ( (date1[2] > date2[2] ) && ( date1[1] > date2[1]) && (date1[0] == date2[0]) )
               return true;
           else if ( (date1[2] > date2[2] ) && ( date1[1] == date2[1]) && (date1[0] == date2[0]) )
               return true;
           else
               return false;
       }
   
      /**
        * Установка условий проверки корректности дат на конечную дату
        * */
       setConditionTwo(date1, date2) {
           if ( (date1[2] > date2[2]) && (date1[1] >= date2[1]) && (date1[0] < date2[0]) )
               return false;
           else if ( (date1[2] > date2[2]) && (date1[1] >= date2[1]) && (date1[0] >= date2[0]) )
               return true;
           else if ( (date1[2] == date2[2]) && (date1[1] > date2[1]) && (date1[0] >= date2[0]) )
               return true;
           else if ( (date1[2] > date2[2]) && (date1[1] > date2[1]) )
               return true;
           else if ( date1[2] > date2[2] && (date1[1] > date2[1]) && (date1[0] > date2[0]) )
               return true;
           else if ( (date1[2] < date2[2] ) && (date1[1] > date2[1]) && (date1[0] < date2[0]) )
               return false;
           else if ( (date1[2] > date2[2] ) && ( date1[1] >= date2[1]) && (date1[0] < date2[0]) )
               return false;
           else if ( (date1[2] <= date2[2] ) && ( date1[1] > date2[1]) && (date1[0] <= date2[0]) )
               return true;
           else if ( (date1[0] > date2[0]) )
               return true;
           else if ( (date1[2] > date2[2] ) && ( date1[1] > date2[1]) && (date1[0] == date2[0]) )
               return true;
           else
               return false;
       }

   Сравнение двух дат.

   kliment1986, 01 Июня 2017

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

10. 1C / Говнокод #23081

    +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

    //Модуль основной формы внешней обработки
    
    &НаКлиенте
    Процедура ПродатьСлона(Оферта)
    	
    	Перем Акцепт;
    	
    	Если ПустаяСтрока(Оферта) Тогда
    		
    		Оферта = "Купи слона!";
    		
    	Иначе
    		
    		Оферта = "Все говорят """+Оферта+""" а ты купи слона!";
    		
    	КонецЕсли;	
    		
    	ВвестиСтроку(Акцепт,Оферта,0, Истина); 	
    		
        ПродатьСлона(Акцепт);
    	
    КонецПроцедуры	
    
    &НаКлиенте
    Процедура ПриОткрытии(Отказ)
    	
    	ПродатьСлона("");
    	
    КонецПроцедуры

    Продаём слона!

    informatikmarazmatik, 30 Мая 2017

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

11. Си / Говнокод #23080

    +2

    1. 1
    2. 2
    3. 3

    for(i==1;i<=1000;i++)
    
    https://youtu.be/Tzl0ELY_TiM?t=89

    И эти люди хотят кого-то учить программированию?

    j123123, 30 Мая 2017

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

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

• 1573
• …
• 114
• 113
• 112
• 111
• 110
• 109
• 108
• 107
• 106
• …
• 1