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

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

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

1. PHP (5714)
2. C++ (2747)
3. Куча (2432)
4. JavaScript (2035)
5. C# (1931)
6. Java (1503)
7. Си (1126)
8. Pascal (649)
9. Python (594)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. bash (203)
14. Objective C (202)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (48)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

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

   +147

   1. 1
   2. 2

   //вынести куда-нибудь в index.php что-ли... а то используется уже в трёх местах
   //a: по всему порталу так, не парься

   Комментарий к коду где-то в середине файла.

   franzose, 08 Августа 2013

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

3. C++ / Говнокод #13556

   +12

   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

   if  ( (_gen_statP==tm_Manual)  ||
                             (_gen_statP==tm_OK)          && (_cur_statP!=tm_Manual)  ||
                             (_gen_statP==tm_ValidTM)     && (_cur_statP!=tm_Manual)  &&  (_cur_statP!=tm_OK) ||
                             (_gen_statP==tm_Rejected)    && (_cur_statP!=tm_Manual)  &&  (_cur_statP!=tm_OK) && (_cur_statP!=tm_ValidTM) ||
                             (_gen_statP==tm_Bad)         && (_cur_statP!=tm_Manual)  &&  (_cur_statP!=tm_OK) && (_cur_statP!=tm_ValidTM) && (_cur_statP!=tm_Rejected) ||
                             (_gen_statP==tm_Statistic)   && (_cur_statP!=tm_Manual)  &&  (_cur_statP!=tm_OK) && (_cur_statP!=tm_ValidTM) && (_cur_statP!=tm_Rejected)  && (_cur_statP!=tm_Bad)
                           ) {
                               _cur_statP = _gen_statP;
                             }
   
   
                       if  ( (_gen_statQ==tm_Manual)  ||
                             (_gen_statQ==tm_OK)          && (_cur_statQ!=tm_Manual) ||
                             (_gen_statQ==tm_ValidTM)     && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) ||
                             (_gen_statQ==tm_Rejected)    && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) && (_cur_statQ!=tm_ValidTM) ||
                             (_gen_statQ==tm_Bad)         && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) && (_cur_statQ!=tm_ValidTM) && (_cur_statQ!=tm_Rejected) ||
                             (_gen_statQ==tm_Statistic)   && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) && (_cur_statQ!=tm_ValidTM) && (_cur_statQ!=tm_Rejected)  && (_cur_statQ!=tm_Bad)
                           ) {
                               _cur_statQ = _gen_statQ;
                             }

   Вроде работает, но уж больно некрасиво) по идее вычисляет новый статус в соответствии с "приоритетом"

   mapron, 06 Августа 2013

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

4. PHP / Говнокод #13552

   +155

   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

   /**
   * Конструктор класса
   * @param mixed $db
   */   
   public function __construct($db = null)
   {
           if ($db instanceof db) {
               self::$_db = & $db;
           } // передан объект БД
           elseif ($db !== null) {
               self::$_db = new db($db);
           } // создание по схеме БД
           else {
               self::$_db = new db();
           } // создание по умолчанию
   
           return $this;
   }

   dimkich, 05 Августа 2013

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

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

   +149

   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

   // Проверка наличия интернета по доступности яндекса и рамблера
     public function is_inet($wait=false) {
       if ($wait) {
   
         while(!($this->ping('yandex.ru')&&$this->ping('rambler.ru'))) {
           echo date('Y/m/d H:i:s >> ').'Internet disconnect, wait...'."<br/>\n";
           }
         return true;
         }
   
         elseif ($this->ping('yandex.ru')&&$this->ping('rambler.ru')) return true;
         else return false;
       }

   madfriend, 02 Августа 2013

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

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

   +125

   1. 1
   2. 2
   3. 3

   <a class="buy_btn" href="javascript://228;" onclick="$.l_basket_modify(this.href.replace(/\D+/g, ''), 1);">
      <img src="/images/addToBasket.png" alt="Добавить в корзину">
   </a>

   Сначала привлёк хинт у ссылки при наведении...

   Crazyzubr, 02 Августа 2013

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

7. C++ / Говнокод #13503

   +21

   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

   if (Utility::compareVersions(myVersionStr, latestVersionStr) < 0)
   {
       // New version is available. Download and install the new version.
       performUpdate();
   }		
   else
   {
       bool isUpdateRunning = Utility::isUpdateRunning();
   				
       if (!isUpdateRunning)
       {
           performUpdate();
       }
   }

   Нашел в коде после того как у тестера версия 1.0.0.401 обновилась на 1.0.0.395

   letheriem, 29 Июля 2013

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

8. Python / Говнокод #13496

   −101

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

   <div class="book-info">
                   <strong class="book">${label}</strong>
                   {{!if desc && desc != "None None"}}
                   <span class="author">${desc}</span>
                   {{!/if}}
               </div>

   Это кусочек из JQuery template. Ему на вход приходит JSON, сформированный в Python.
   Кто бы догадался, что означает это "None None"?
   
   Да, это случай, когда имя и фамилия автора книги неизвестны. Вместо того, чтобы в классе Book сделать нормальный метод getAuthor, чуваки предпочли наговнякать такой вот IF в клиентской части.

   asolntsev, 28 Июля 2013

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

9. C# / Говнокод #13480

   +110

   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

   private void Save(string ThreadID, string Board)
           {
               string pathL;
               if (cbGIF.Checked)
               {
                   pathL = String.Format(path, Board, ThreadID, "-gif");
               }
               else
               {
                   pathL = String.Format(path, Board, ThreadID, "");
               }
               string htmlPath = String.Format(threadPath, Board, ThreadID);
   
               WebClient wcli = new GZipWebClient();
               string cThread = wcli.DownloadString(htmlPath);
               
               var rx = new Regex(cbGIF.Checked ? regExGif : regEx);
               var ms = rx.Matches(cThread);
               imgSaved = 0;
               imgCount = ms.Count;
               saveProgress.Minimum = 0;
               saveProgress.Maximum = imgCount;
               saveProgress.Value = 0;
               if (!Directory.Exists(pathL))
               {
                   Directory.CreateDirectory(pathL);
               }
               try
               {
                   foreach (Match m in ms)
                   {
                       WebClient ccl = new WebClient();
                       ccl.DownloadFileCompleted += new AsyncCompletedEventHandler(ccl_DownloadFileCompleted);
                       string[] v = m.Value.Split('"');
                       string sd = v[1].Split('/').Last();
                       string a = url + v[1];
                       string b = pathL + sd;
                       if (File.Exists(b))
                       {
                           imgSaved++;
                           saveProgress.Value = imgSaved;
                           lblSaveProgress.Text = imgSaved.ToString() + "/" + imgCount.ToString();
                           if (imgSaved == imgCount)
                           {
                               btnSave.Enabled = true;
                               lblSaveProgress.Text = "FUKKEN SAVED!";
                           }
                       }
                       else ccl.DownloadFileAsync(new Uri(a),b);
                   }
               }
               catch (WebException e)
               {
                   MessageBox.Show(e.Message + e.StackTrace);
               }          
           }

   В пределах одного метода бросаемся из крайности в крайность в именовании переменных.

   A1mighty, 26 Июля 2013

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

10. JavaScript / Говнокод #13465

    +147

    1. 1

    eval("document.getElementById('bmnu_rbt').onclick = function () { menu_to_left(); }");

    Без комментанриев

    kostoprav, 22 Июля 2013

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

11. PHP / Говнокод #13448

    +149

    1. 1

    if('null'==$this->user_data['e_mail']) $this->user_data['e_mail']=null;

    clgs, 18 Июля 2013

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

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

• 1983
• …
• 177
• 176
• 175
• 174
• 173
• 172
• 171
• 170
• 169
• …
• 1