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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2361)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (648)
9. Python (583)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. Куча / Говнокод #16340

   +126

   1. 1
   2. 2

   Если переменных в памяти потребуется слишком большое количество,
   которое не сможет вместить в себя сама аппаратная часть, произойдет перегрузка системы или её зависание.

   Милок сидит на досе и не умеет в gc.
   http://code-live.ru/post/cpp-pointers/

   gost, 15 Июля 2014

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

2. C++ / Говнокод #16339

   +28

   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

   #include <iostream>
   using namespace std;
   
   int main()
   {
       int a; // Объявление статической переменной
       int b = 5; // Инициализация статической переменной b
   
       a = 10;
       b = a + b;
       cout << "b is " << b << endl;
       return 0;
   }

   Комментарии доставляют.
   http://code-live.ru/post/cpp-pointers/

   gost, 15 Июля 2014

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

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

   +155

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

   $data = "<?php define('DSN', 'mysql:dbname={$_POST['db_name']};host={$_POST['host_mysql']}'); define('DBUSER', '{$_POST['login_mysql']}'); define('DBPASS', '{$_POST['password_mysql']}');";
   $mysql_string = file_get_contents( 'class/mysql.php' );
   $new_mysql = $data . ' ' . $mysql_string;
   if ( !file_put_contents( 'class/mysql.php', $new_mysql ) ) {
   	errorMessage( "Не удалось сохранить файлы конфигураций, пожалуйста обратитесь в службу технической поддержки клиентов" );
   }

   install.php

   kindofbear, 15 Июля 2014

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

4. 1C / Говнокод #16337

   −161

   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

   Для А =0 по Значение.Количество()-1 Цикл
   				Попытка	
   					Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4)+"*"+Строка(Значение[А].поле5)+"*"+Строка(Значение[А].поле6)+"*"+Строка(Значение[А].поле7)+"*"+Строка(Значение[А].поле8)+"*"+Строка(Значение[А].поле9)+"*"+Строка(Значение[А].поле10));
   					Табл.InsertParagraphAfter();
   				Исключение	
   					Попытка	
   						Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4)+"*"+Строка(Значение[А].поле5)+"*"+Строка(Значение[А].поле6)+"*"+Строка(Значение[А].поле7)+"*"+Строка(Значение[А].поле8)+"*"+Строка(Значение[А].поле9));
   						Табл.InsertParagraphAfter();
   					Исключение
   						Попытка	
   							Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4)+"*"+Строка(Значение[А].поле5)+"*"+Строка(Значение[А].поле6)+"*"+Строка(Значение[А].поле7)+"*"+Строка(Значение[А].поле8));
   							Табл.InsertParagraphAfter();
   						Исключение
   							
   							Попытка	
   								Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4)+"*"+Строка(Значение[А].поле5)+"*"+Строка(Значение[А].поле6)+"*"+Строка(Значение[А].поле7));
   								Табл.InsertParagraphAfter();
   							Исключение
   								Попытка	
   									Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4)+"*"+Строка(Значение[А].поле5)+"*"+Строка(Значение[А].поле6));
   									Табл.InsertParagraphAfter();
   								Исключение	
   									Попытка	
   										Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4)+"*"+Строка(Значение[А].поле5));
   										Табл.InsertParagraphAfter();
   									Исключение	
   										Попытка	
   											Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3)+"*"+Строка(Значение[А].поле4));
   											Табл.InsertParagraphAfter();
   										Исключение
   											Попытка	
   												Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+Строка(Значение[А].поле2)+"*"+Строка(Значение[А].поле3));
   												Табл.InsertParagraphAfter();
   											Исключение
   												Попытка	
   													Табл.InsertAfter(Строка(Значение[А].поле1)+"*"+(Строка(Значение[А].поле2)));
   													Табл.InsertParagraphAfter();
   												Исключение
   													Попытка	
   														Табл.InsertAfter(Строка(Значение[А].поле1));
   														Табл.InsertParagraphAfter();
   													Исключение
   														Попытка	
   															Табл.InsertAfter(Строка(Значение[А].поле0));
   															Табл.InsertParagraphAfter();
   														Исключение
   														КонецПопытки;
   													КонецПопытки;	
   												КонецПопытки;
   											КонецПопытки;	
   										КонецПопытки;
   									КонецПопытки;
   								КонецПопытки;	
   							КонецПопытки;	
   						КонецПопытки;	
   					КонецПопытки;
   				КонецПопытки;
   			КонецЦикла;

   Ninomia, 15 Июля 2014

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

5. C# / Говнокод #16336

   +135

   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

   Public CallGridRowModel(CallView callView)
   {
     . . .
     Caller = GetCallerOrCalleeNameColumn(callView, true)
     Callee = GetCallerOrCalleeNameColumn(callView, false)
     . . .
   }
     
     
   private object GetCallerOrCalleeNameColumn(CallView callView, bool isCallerNameColumn)
   {
     if(isCallerNameColumn)
     {
       if(. . .) return . . .;
     }
     if(!isCallerNameColumn)
     {
       if(. . .) return . . .;
     }
     
     if(isCallerNameColumn)
     {
       return . . .;
     }
     else
     {
       return . . .;
     }
   }

   onixar, 15 Июля 2014

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

6. JavaScript / Говнокод #16335

   +155

   1. 1

   Object.prototype.toString.call(o).match(/\[object (\w+)\]/)[1];

   Реализация typeof

   kissarat, 15 Июля 2014

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

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

   +157

   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

   <?php
   
   class ListWidget extends Widget
   {
   	public $itemsPerPage = 10;
   	private static $allowedItemsPerPage = array(10);
   	// ...
   
   	private function getTotalPages()
   	{
   		$this->itemsPerPage = in_array($this->itemsPerPage, self::$allowedItemsPerPage) ? $this->itemsPerPage : 10;
   		// ...
   	}
   }

   Надежная проверка.

   ftr, 14 Июля 2014

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

8. Ruby / Говнокод #16324

   −151

   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

   def get_team_choices_with_index
       school_ids = params[:school_id].split(",")
       @index = params[:index]
   
       @teams_array = Array.new
       school_ids.each do |s|
         next if s.eql? "0"
         School.find(s).teams.each do |t|
           @teams_array << t
         end
       end
   
       @teams_array.flatten!
       @teams_array
     end

   А должно быть что-то вроде
   @teams_array = Team.where(school_id: school_ids)

   dj_Andreus, 14 Июля 2014

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

9. 1C / Говнокод #16323

   −160

   1. 1
   2. 2
   3. 3

   Если Год (ТекДата) = 1 Тогда
     ТекДата = ТекущаяДата ();
    КонецЕсли;

   вот так надо проверять дату на заполненность!

   alexinzaz, 14 Июля 2014

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

10. C++ / Говнокод #16322

    +28

    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

    TLSOSI7Command &TLSLongTelegram::GetOSI7Command () const
    {
       if(!m_pOSI7Command) { // private pointer not initialized
                  // WARNING! Quick'n'dirty! is UNINITIALIZED althoug it should be - just for preventing abnormal end!
                  const_cast<TLSOSI7Command*>(m_pOSI7Command)=new TLSOSI7Command;
       } // private pointer not initialized
       else 
       { 
                 // NOT private pointer not initialized
       }; 
       // NOT private pointer not initialized
       return *m_pOSI7Command;
    }

    m)

    blackhearted, 14 Июля 2014

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

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

• 2105
• …
• 1383
• 1382
• 1381
• 1380
• 1379
• 1378
• 1377
• 1376
• 1375
• …
• 1