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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2378)
4. JavaScript (2032)
5. C# (1929)
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 (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   +151.7

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

   for ($i = 0; $i < count($oldCLI); $i++) {
   	if ($oldCLI[$i] == null) {
   		unset($oldCLI[$i]);
   	}
   }

   unset($oldCLI[$i]); - это как я понимаю контрольный выстрел в голову

   Sadie, 09 Сентября 2009

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

2. JavaScript / Говнокод #1795

   +156.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
   60. 60
   61. 61

   function ChangeDisc(n, size, l_x, l_y, r_x, r_y)//а-а-а-а-а-а убейте меня
   {
       var disc_left = document.getElementById('sDisc1');
       var disc_right = document.getElementById('sDisc2');
       var disc_left_div = document.getElementById('sDisc1_div');
       var disc_right_div = document.getElementById('sDisc2_div');
       switch (size)
       {
           case 13:
               disc_left_div.style.paddingLeft = (l_x+3)+'px';
               disc_left_div.style.paddingTop = (l_y+2)+'px';
               disc_right_div.style.paddingLeft = (r_x+6)+'px';
               disc_right_div.style.paddingTop = (r_y+2)+'px';
               break;
           case 14:
               disc_left_div.style.paddingLeft = (l_x+1)+'px';
               disc_left_div.style.paddingTop = (l_y+1)+'px';
               disc_right_div.style.paddingLeft = (r_x+1)+'px';
               disc_right_div.style.paddingTop = (r_y+1)+'px';
               break;
           case 15:
               disc_left_div.style.paddingLeft = l_x+'px';
               disc_left_div.style.paddingTop = l_y+'px';
               disc_right_div.style.paddingLeft = r_x+'px';
               disc_right_div.style.paddingTop = r_y+'px';
               break;
           case 16:
               disc_left_div.style.paddingLeft = (l_x-2)+'px';
               disc_left_div.style.paddingTop = (l_y-2)+'px';
               disc_right_div.style.paddingLeft = (r_x-2)+'px';
               disc_right_div.style.paddingTop = (r_y-2)+'px';
               break;
           case 17:
               disc_left_div.style.paddingLeft = (l_x-3)+'px';
               disc_left_div.style.paddingTop = (l_y-4)+'px';
               disc_right_div.style.paddingLeft = (r_x-5)+'px';
               disc_right_div.style.paddingTop = (r_y-4)+'px';
               break;
           case 18:
               disc_left_div.style.paddingLeft = (l_x-5)+'px';
               disc_left_div.style.paddingTop = (l_y-5)+'px';
               disc_right_div.style.paddingLeft = (r_x-8)+'px';
               disc_right_div.style.paddingTop = (r_y-6)+'px';
               break;
           case 19:
               disc_left_div.style.paddingLeft = (l_x-8)+'px';
               disc_left_div.style.paddingTop = (l_y-8)+'px';
               disc_right_div.style.paddingLeft = (r_x-8)+'px';
               disc_right_div.style.paddingTop = (r_y-8)+'px';
               break;
           case 20:
               disc_left_div.style.paddingLeft = (l_x-10)+'px';
               disc_left_div.style.paddingTop = (l_y-10)+'px';
               disc_right_div.style.paddingLeft = (r_x-10)+'px';
               disc_right_div.style.paddingTop = (r_y-10)+'px';
               break;
       }
       disc_left.src = Images[n].src;
       disc_right.src = Images[n].src;
       scroll(0,0);
   }

   копипаста процветает... правильно, зачем структурировать код? проще так.

   gorky, 09 Сентября 2009

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

3. SQL / Говнокод #1794

   −863

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

   select ...
   from ...
   where ...
                    and NVL2(l_date_from, trunc(mmt.transaction_date), l_date_from) >= NVL(l_date_from, l_date_from) 
                    and NVL2(l_date_to, trunc(mmt.transaction_date), l_date_to) <= NVL(l_date_to, l_date_to)          
   ...

   из реально работающей системы

   nxx, 09 Сентября 2009

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

4. Perl / Говнокод #1793

   −103

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

   foreach my $current_item(@catalog_items)
   {
       foreach($current_item)
       {
           my $id = $_->[0];
           ...

   Suor, 09 Сентября 2009

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

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

   +161.3

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

   $CMS_VERSION = "1.3.1";
   $CMS_VERSION_NAME = "Havana";
   $CMS_SCHEMA_VERSION = "29";
   
   define('CMS_VERSION', $CMS_VERSION);
   define('CMS_VERSION_NAME', $CMS_VERSION_NAME);
   define('CMS_SCHEMA_VERSION', $CMS_SCHEMA_VERSION);

   На работе имеем дело с CMSMS (CMSMadeSimple).
   С модой у создателей неплохо, тут тебе и рекурсивный акроним в названии, и именнованные релизы... Но для чего делать вот так?
   
   p.s. говнокод откопал мой коллега. Привет ему ;)

   striker, 09 Сентября 2009

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

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

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

   C:\>ping 10.0.0.3 -l 1472 -f -n 100
   
   Обмен пакетами с 10.0.0.3 по с 1472 байт данных:
   
   Ответ от 10.0.0.3: число байт=1472 время=2мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=2мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=2мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=2мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Превышен интервал ожидания для запроса.
   Превышен интервал ожидания для запроса.
   Превышен интервал ожидания для запроса.
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128
   Ответ от 10.0.0.3: число байт=1472 время=1мс TTL=128

   VT

   ESGR, 09 Сентября 2009

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

7. C# / Говнокод #1790

   +144

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

   public void onApplicationStateChanged(int state)
   {
               if (state == Application.APPSTATE_UNINITIALIZED)
               {
                          // TODO: what to do?
               }
   }

   Чувак начал писать и забыл о чем...

   vassabi, 08 Сентября 2009

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

8. 1C / Говнокод #1789

   −176.1

   1. 1

   ДатаНачалаПериода = НачалоМесяца(Дата(Строка(Формат(ТекущийГод, "ЧГ=0")) + Строка(ТекущийМесяц) + "01"));

   Фрагмент кода от разработчиков конфигурации. Дело в том, что дату начала текущего месяца можно получить как НачалоМесяца(ТекущаяДата()). Даже если предположить, что переменные "ТекущийГод" и "ТекущийМесяц" содержат не актуальные значения (например, при перерасчете прошлых документов), достаточно выполнить НачалоМесяца(Дата(ТекущийГод, ТекущийМесяц, 1))

   polestnize, 08 Сентября 2009

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

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

   +166.8

   1. 1

   unset($keys[count($keys)-1][count($keys[count($keys)-1])-1]);

   stepushyn, 07 Сентября 2009

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

10. Java / Говнокод #1787

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

    public static ru.project.subpackage.PersonDTO convertOshPersonDtoToPersonDto(OshPersonDTO person){
            ru.project.subpackage.PersonDTO dto = new ru.project.subpackage.PersonDTO();
            dto.setPersonid(person.getPersonid().longValue());
            dto.setNamelast(person.getNamelast());
            dto.setNamefirst            (person.getNamefirst            ());
            dto.setNamesec              (person.getNamesec              ());
            dto.setInitials             (person.getInitials             ());
            dto.setSex                  (convertSkVocValue(person.getSex()));
            dto.setReason               (person.getReason               ());
            dto.setWorkphone            (person.getWorkphone            ());
            dto.setWorkphonedigit       (person.getWorkphonedigit       ());
            dto.setLocalphone           (person.getLocalphone           ());
            dto.setLocalphonedigit      (person.getLocalphonedigit      ());
            dto.setHomephone            (person.getHomephone            ());
            dto.setHomephonedigit       (person.getHomephonedigit       ());
            dto.setMobilephone          (person.getMobilephone          ());
            dto.setMobilephonedigit     (person.getMobilephonedigit     ());
            dto.setFax                  (person.getFax                  ());
            dto.setFaxdigit             (person.getFaxdigit             ());
            dto.setPager                (person.getPager                ());
            dto.setEmail                (person.getEmail                ());
            dto.setWeb                  (person.getWeb                  ());
            dto.setNamelastdative       (person.getNamelastdative       ());
            dto.setNamefirstdative      (person.getNamefirstdative      ());
            dto.setNamesecdative        (person.getNamesecdative        ());
            dto.setNamelastaccusative   (person.getNamelastaccusative   ());
            dto.setNamefirstaccusative  (person.getNamefirstaccusative  ());
            dto.setNamesecaccusative    (person.getNamesecaccusative    ());
            dto.setNamelastgenitive     (person.getNamelastgenitive     ());
            dto.setNamefirstgenitive    (person.getNamefirstgenitive    ());
            dto.setNamesecgenitive      (person.getNamesecgenitive      ());
            dto.setNamelastinstrumental (person.getNamelastinstrumental ());
            dto.setNamefirstinstrumental(person.getNamefirstinstrumental());
            dto.setNamesecinstrumental  (person.getNamesecinstrumental  ());
            dto.setNamelastprepositional(person.getNamelastprepositional());
            dto.setNamefirstprepositional(person.getNamefirstprepositional());
            dto.setNamesecprepositional (person.getNamesecprepositional ());
            return dto;
        }

    И так далее еще несколько сотен строк. А главное переупаковка из одних объектов в другие и обратно бессмысленна, т.к. можно использовать исходные (они доступны в приложении)

    johnsoft, 07 Сентября 2009

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

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

• 2109
• …
• 176
• 175
• 174
• 173
• 172
• 171
• 170
• 169
• 168
• …
• 1