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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2389)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
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 (37)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

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

   +144.5

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

   <?php
   /* КУЧА ГОВНОКОДА */
   ?>
   
   <TITLE><?php echo $m_feld[0];?></TITLE>
   
   <?php
   /* КУЧА ГОВНОКОДА */
   ?>

   ...вот так вот завернуть...навыворот можна ))

   v01t, 21 Января 2010

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

3. Java / Говнокод #2464

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

   protected static Map<Long, List<Bank.BankMoney>> getWinnersBanksMap(Map<Long, List<Bank.BankMoney>> sharedBank) {
           Map<Long,List<Bank.BankMoney>> winnersWithBanks = new HashMap<Long, List<Bank.BankMoney>>();
   
           for (Map.Entry<Long, List<Bank.BankMoney>> longListEntry : sharedBank.entrySet()) {
                   Long playerId = longListEntry.getKey();
                   List<Bank.BankMoney> banksWonOnThisIteration = longListEntry.getValue();
   
                   List<Bank.BankMoney> banks1 = winnersWithBanks.get(playerId);
                   if (banks1 == null) {
                       banks1 = new ArrayList<Bank.BankMoney>();
                       winnersWithBanks.put(playerId, banks1);
                   }
                   banks1.addAll(banksWonOnThisIteration);
               }
           return winnersWithBanks;
       }

   В результате серии рефакторингов чужого кода обнаружил следующий метод, над которым долго ломал голову - что он делает... Оказалось, ничего. Возвращает копию исходного объекта.

   dvb, 21 Января 2010

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

4. SQL / Говнокод #2451

   −861.2

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

   SELECT * FROM obj WHERE telephon LIKE '***' AND off = 0;
   
   SELECT obj.* FROM 
   obj 
   INNER JOIN obj2ls ON obj.ObjID = obj2ls.ObjKod AND obj2ls.Bdate <= CURDATE() AND obj2ls.Edate >= CURDATE() AND obj2ls.off = 0 
   INNER JOIN ls ON obj2ls.LsKod = ls.lsid AND ls.off = 0
   INNER JOIN ls2contr ON ls.lsid = ls2contr.lskod AND ls2contr.bdate <= CURDATE() AND ls2contr.edate >= CURDATE() AND ls2contr.off = 0
   INNER JOIN contr ON ls2contr.contrkod = contr.contrid AND contr.off = 0
   WHERE obj.telephon LIKE '***' AND obj.off = 0 AND contr.contr_type = 2;

   О плохом проектировании базы данных.
   Был простой запрос, к нему добавили проверку 1го(!) доп. условия.
   Еще умиляют JOINы в духе *Kod = *Id.

   Mihard, 19 Января 2010

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

5. Куча / Говнокод #2413

   +126.6

   1. 1

   <meta млн="" $42="" за="" рынке,="" масличном="" на="" оператора="" украинского="" крупнейшего="" group,="" allseeds="" доли="" контрольной="" приобретении="" о="" соглашение="" подписала="" Украины,="" групп="" агропромышленных="" крупнейших="" из="" одна="" ,="" Групп="" Кернел="" content="" name="description"/>

   Говнодвижок слепил описание странички.

   condemetrius, 14 Января 2010

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

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

   +144.5

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

   <?php if (!$mode || true) { ?>
   <div class="yiiForm">
   <?php } else { ?>
   <div class="yiiForm">
   <?php } ?>

   последствия удаления ненужных частей кода

   Bethrezen, 28 Декабря 2009

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

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

   +153.5

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

   private function _createPath() {
         $_url = DEFAULT_URL;
         $_url = false === strpos($_url, 'http://') ? 'http://' . $_url : $_url;
         $_url = false === strpos($_url, '/', strlen($_url) - 1) ? $_url . '/' : $_url;
         return $_url . 'gateway.php?';
    }

   Давайте скидываться на киллеров. Заработаем больше :)

   asserte, 11 Декабря 2009

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

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

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

   function onbot($who) {
       $crime = 5;
       $crime_ar=array();
       for($i=0;$i<=$crime;$i++) {
           $crime_ar[$i]=$i;}
       for($i=$crime+1;$i<=100;$i++) {
           $crime_ar[$i]="empty";
       }
       $crime_numer=rand(0,100);
       $is_crime="0";
       for($i=0;$i<=100;$i++) {
           if($crime_numer==$crime_ar[$i]) {
               $is_crime="1";
           }
       }
       if($is_crime == 1) {
           $at = rand(1,4);
           if($at == 1) {$attacker = 'gorg1';}
           if($at == 2) {$attacker = 'gorg2';}
           if($at == 3) {$attacker = 'gorg3';}
           if($at == 4) {$attacker = 'gorg4';}
           attack($who,$attacker,'1');
           die();
       }
   }
   
   function attack($who,$attacker,$dif) {
       ..........................................
       if($attacker == "gorg1") {
           $prototype = "Рядовой Горг";
           $name = "Рядовой Горг";}
       if($attacker == "gorg2") {
           $prototype = "Горг-Лейтенант";
           $name = "Горг-Лейтенант";}
       if($attacker == "gorg3") {
           $prototype = "Горг-Капитан";
           $name = "Горг-Капитан";}
       if($attacker == "gorg4") {
           $prototype = "Горг Подполковник";
           $name = "Горг Подполковник";}
       $mine_id=$db["id"];
       // далее идет заполнение таблиц, для "блоя"
       }
   }

   Разбираюсь в онлайн игре. Архитектура самой игры на высшем уровне!

   dicos, 11 Декабря 2009

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

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

   +165.4

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

   <script language="JavaScript">
   		function change_city() 
   		{
   			document.getElementById("header_18").style.display='none';
   			document.getElementById("header_18_add").style.display='block';
   			document.getElementById("header_18_add").id='header_18'; //т.к. стили прописаны для id
   		}
   	</script>
   </head>
   <body>
   
   <div id="mega_scroller">

   <div id="mega_scroller">
   document.getElementById("header_18_add") .id='header_18'; //т.к. стили прописаны для id
   http://www.cinemapark.ru/ - гении делали

   pompomru, 07 Декабря 2009

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

10. C# / Говнокод #2238

    +96.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
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54

    public class zTimer
    {
        object SomeTag;
        int StartTime;
        int TimeOut;
    
        public int Start(int Value, object tag)
        {
            SomeTag = tag;
            TimeOut = Value;
            StartTime = MakeLngTime(DateTime.Now);
            return StartTime;
        }
    
        public int MakeLngTime(DateTime Tm)
        {
           return ((Tm.Day * 24 + Tm.Hour) * 60 + Tm.Minute) * 60 + Tm.Second;
        }
    
        public bool IsTimeOut(object tag)
        {
            if (tag != null && SomeTag != null)
            {
                if (tag == SomeTag)
                {
                    if (MakeLngTime(DateTime.Now) > StartTime + TimeOut)
                    {
                        return true;
                    }
                    else
                    {
                        return false;
                    }
                }
                else
                {
                    StartTime = MakeLngTime(DateTime.Now);
                    SomeTag = tag;
                    return false;
                }
            }
            else
            {
                if (MakeLngTime(DateTime.Now) > StartTime + TimeOut)
                {
                    return true;
                }
                else
                {
                    return false;
                }
            }
        }
    }

    Могучий таймер. Особое внимание стоит уделить методу MakeLngTime.

    crelian, 07 Декабря 2009

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

11. Pascal / Говнокод #2151

    +92.7

    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

    unit CelFahKel;
    
    interface
    
    uses
      Windows, Messages, SysUtils, Variants, Classes, Graphics, Controls, Forms,
      Dialogs, StdCtrls, ExtCtrls;
    
    type
      TForm1 = class(TForm)
        Button1: TButton;
        Edit1: TEdit;
        Edit2: TEdit;
        RadioGroup1: TRadioGroup;
        RadioGroup2: TRadioGroup;
        procedure Button1Click(Sender: TObject);
      private
        { Private declarations }
      public
        { Public declarations }
      end;
    
    var
      Form1: TForm1;
    
    implementation
    
    {$R *.dfm}
    
    procedure TForm1.Button1Click(Sender: TObject);
    var
      F: Extended;
      C: Integer;
    begin
      if RadioGroup1.ItemIndex=0 then
    
        if RadioGroup2.ItemIndex=0 then
          Edit2.Text := Edit1.Text
        else if RadioGroup2.ItemIndex=1 then
        begin
          F := StrToFloat(Edit1.Text) * (9/5) + 32;
          Edit2.Text := FloatToStr(F)
        end
        else if RadioGroup2.ItemIndex=2 then
        begin
          C := StrToInt(Edit1.Text) + 273;
          Edit2.Text := FloatToStr(C);
        end;
    
      if RadioGroup1.ItemIndex=1 then
        if RadioGroup2.ItemIndex=0 then
        begin
          F := StrToFloat(Edit1.Text) - 32 * (5/9);
          Edit2.Text := FloatToStr(F);
        end
        else if RadioGroup2.ItemIndex=1 then
          Edit2.Text := Edit1.Text
        else if RadioGroup2.ItemIndex=2 then
        begin
          F := StrToFloat(Edit1.Text) - 32 * (5/9) + 273;
          Edit2.Text := FloatToStr(F);
        end;
    
      if RadioGroup1.ItemIndex=2 then
        if RadioGroup2.ItemIndex=0 then
        begin
          C := StrToInt(Edit1.Text) - 273;
          Edit2.Text := IntToStr(C);
        end
        else if RadioGroup2.ItemIndex=1 then
        begin
          F := StrToFloat(Edit1.Text) + 32 * (9/5) - 273;
          Edit2.Text := FloatToStr(F);
        end
        else if RadioGroup2.ItemIndex=2 then
           Edit2.Text := Edit1.Text
    end;
    end.

    Всего-то навсего перевод градусных величин: Сельсий <-> Фаренгейт <-> Кельвин и вывод в Эдит

    sevenfourk, 18 Ноября 2009

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

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

• 1974
• …
• 26
• 25
• 24
• 23
• 22
• 21
• 20
• 19
• 18
• …
• 1