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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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 / Говнокод #8566

   +162

   1. 1
   2. 2

   $id = $this->db->fetchCol('SELECT id FROM data;');
   $names = $this->db->fetchCol('SELECT name FROM points INNER JOIN data ON points.id = data.point_id_from;');

   Zend Db usage

   jf4ever, 18 Ноября 2011

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

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

   +162

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

   $tmp_arr=array();
   $id1_str='';
   $root_str='';
   foreach($id1 as $id){
   	$id1_str.=$id['id'].' , ';
   	if(!isset($tmp_arr[$id['root']])){
   		$root_str.=$id['root'].' , ';
   		$tmp_arr[$id['root']]=true;
   	}
   }

   это типа такой способ собрать distinct root ids в строку!
   не забыть потом откусить хвост ' , '
   нормальные герои не ищут легких путей

   shitcoder, 17 Ноября 2011

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

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

   +162

   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

   //create image array
            $images = array();
            //fill it
            for ($i = 1; $i <= 180; $i++) 
               if (($i != $passimage1)&&($i != $passimage2)&&($i != $passimage3)&&($i != $passimage4)) 
               $images[] = $i;
            //randomize it
            for ($i = 0; $i < 176; $i++) {
               $r = rand(0,175);
               $swap = $images[$r];
               $images[$r] = $images[$i];
               $images[$i] = $swap;
            }

   denis90, 16 Ноября 2011

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

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

   +162

   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

   $er = array();
   
   $file1=$config['tpl_dir'].$config['general']['site_template'].'/style/catalog.css';
   $file2=$config['tpl_dir'].$config['general']['site_template'].'/module/catalog.html';
   $file3=$config['tpl_dir'].$config['general']['site_template'].'/module/catalog_read.html';
   $file4='modules/catalog/setup_catalog/table_catalog.MYD';
   $file5='modules/catalog/setup_catalog/table_catalog_cats.MYD';
   $file6='modules/catalog/setup_catalog/table_catalog_comments.MYD';
   $file7='modules/catalog/setup_catalog/table_catalog_rating.MYD';
   
   
   if(file_exists($file1)==True) {}else{$er[]="нет файла стилей <b>\"catalog.css\"</b> <br> он должен находиться в<b> $file1</b>"; }
   if(file_exists($file2)==True) {}else{$er[]="нет файла дизайна <b>\"catalog.html\"</b> <br> он должен находиться в<b> $file2</b>"; }
   if(file_exists($file3)==True) {}else{$er[]="нет файла дизайна <b>\"catalog_read.html\"</b> <br> он должен находиться в<b> $file3</b>"; }
   if(file_exists($file4)==True) {}else{$er[]="нет файла таблицы <b>\"table_catalog.MYD\"</b> <br> он должен находиться в<b> $file4</b>"; }
   if(file_exists($file5)==True) {}else{$er[]="нет файла таблицы <b>\"table_catalog_cats.MYD\"</b> <br> он должен находиться в<b> $file5</b>"; }
   if(file_exists($file6)==True) {}else{$er[]="нет файла таблицы <b>\"table_catalog_comments.MYD\"</b> <br> он должен находиться в<b> $file6</b>"; }
   if(file_exists($file7)==True) {}else{$er[]="нет файла таблицы <b>\"table_catalog_rating.MYD\"</b> <br> он должен находиться в<b> $file7</b>"; }

   Установщик модуля. Условия проверки доставляют: бул сравнимаем с булом (и получаем бул), вместо отрицания держим еще одну ветку условия.

   Мартин, 12 Ноября 2011

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

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

   +162

   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

   function section($sct,$grp = 'all') {
   		global $g,$a,$d,$lists,$master_list,$currency;
   		$access = ac_section($sct,$this->adminmode);
   		if ($access['r'] == 'no' )  {header("Location: /errors/all/403");}
   		$master = q1("SELECT * FROM `_s` WHERE `_id`='".$sct."' AND `_l`='".GetLanguage()."'");
   		if (isset($_POST['pcounts'])) {
   			$_SESSION['_pcount']=(int)$_POST['pcounts'];
   		} elseif (isset($_GET['pcounts'])) {
   			$_SESSION['_pcount']=(int)$_GET['pcounts'];
   		}
   		if (isset($_SESSION['_pcount'])) $master['_pcount']=$_SESSION['_pcount'];
           $master_struct = $this->Struct_fields(qa("SHOW COLUMNS FROM `_s`"));
   		$grps = qa("SELECT * FROM `_g` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')).((!empty($a[0]['gby']))?(" ORDER BY `".$a[0]['gby']."`".((!empty($a[0]['gdir']))?(' DESC'):(' ASC'))):(" ORDER BY ".$master['_sorting'])).((!empty($a[0]['gfrom']))?(' LIMIT '.$a[0]['gfrom'].','.((!empty($a[0]['gper']))?($a[0]['gper']):((!empty($master['_gcount']))?($master['_gcount']):('10')))):(' LIMIT 0,'.((!empty($a[0]['gper']))?($a[0]['gper']):((!empty($master['_gcount']))?($master['_gcount']):('10'))))));
   		$this->cq++;
   		$grps_nav_count = qa("SELECT count(*) `c` FROM `_g` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')));
   		$grps_len = sizeof($grps);
   		$grps_struct = $this->Struct_fields(qa("SHOW COLUMNS FROM `_g`"));
   		$pgs = qa("SELECT * FROM `_p` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')).((!empty($a[0]['by']))?(" ORDER BY `".$a[0]['by']."`".((!empty($a[0]['dir']))?(' DESC'):(' ASC'))):(" ORDER BY ".$master['_sorting'])).((!empty($a[0]['from']))?(' LIMIT '.$a[0]['from'].','.((!empty($a[0]['per']))?($a[0]['per']):((!empty($master['_pcount']))?($master['_pcount']):('10')))):(' LIMIT 0,'.((!empty($a[0]['per']))?($a[0]['per']):((!empty($master['_pcount']))?($master['_pcount']):('10'))))));
   		$this->cq++;
   		$pgs_nav_count = qa("SELECT count(*) `c` FROM `_p` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')));
   		$this->cq++;
   		$pgs_len = sizeof($pgs);
   		$pgs_struct = $this->Struct_fields(qa("SHOW COLUMNS FROM `_p`"));
   		$list_g = '';
   		$list_p = '';
   // Дальше еще строк 200 подобного кода

   Нашел бэкап сайта, который надо было дорабатывать
   И это только одна функция одного из классов. Всего около 15 классов, в каждом по 30-40 функций.
   При этом есть еще куча файлов вызываемых напрямую(отправка сообщений, добавление/редактирование/удаление данных в админке), хотя и используется mod_rewrite
   
   долго думал, потом переписал нужные куски заново

   Bagration, 11 Ноября 2011

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

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

   +162

   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

   /**/
   	TCHAR* szFileType = _T("");
   	int ftype = JGetByte( hContact, "AvatarType", PA_FORMAT_UNKNOWN );
   	if( ftype == PA_FORMAT_UNKNOWN ) {
   		WIN32_FIND_DATA ffd;
   		HANDLE hFind = FindFirstFile( ptszDest, &ffd );
   		if ( hFind != INVALID_HANDLE_VALUE ) {
   			wchar_t* szExt = m_strrchrs( ffd.cFileName, ".\\" );
   			if( szExt ) {
   				if( !_tcsicmp( szExt, _T(".jpg") ) )
   					ftype = PA_FORMAT_JPEG;
   				else if( !_tcsicmp( szExt, _T(".png") ) )
   					ftype = PA_FORMAT_PNG;
   				else if( !_tcsicmp( szExt, _T(".gif") ) )
   					ftype = PA_FORMAT_GIF;
   				else if( !_tcsicmp( szExt, _T(".bmp") ) )
   					ftype = PA_FORMAT_BMP;
   				if( ftype != PA_FORMAT_UNKNOWN )
   					JSetByte( hContact, "AvatarType", ftype );
   			}
   			FindClose( hFind );
   		}
   	}
   	switch( ftype ) {
   		case PA_FORMAT_JPEG: szFileType = _T("jpg");   break;
   		case PA_FORMAT_PNG:  szFileType = _T("png");   break;
   		case PA_FORMAT_GIF:  szFileType = _T("gif");   break;
   		case PA_FORMAT_BMP:  szFileType = _T("bmp");   break;
   	}

   http://infium.googlecode.com/svn/trunk/Jabber/jabber_misc.cpp

   bot-minurast, 03 Ноября 2011

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

8. JavaScript / Говнокод #8384

   +162

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

   $.ajax({
          url: host + "json/" + action + "/?event-name=" + 
                  $("#event-name").val() +  
                  "&event-date-from=" + $("#event-date-from").val() + 
                  "&event-date-to=" + $("#event-date-to").val() + 
                  "&event-place-id=" + $("#event-place-id").val() +
                  "&event-id=" + $("#event-id").val() +
                  "&event-pls=" + $("#query").val(),

   Вот, что осталось в наследство от предшественника... И так во всем проекте FUUUUU~~~~~~

   unkwua, 01 Ноября 2011

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

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

   +162

   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

   //записали значения, от которых отталкиваться будем. Типа первые значения. Почти повторяют столбцы в таблице
   $maxv1=303268.26141;
   $maxv2=0;
   $maxv3=0;
   $maxm1=294798.84277;
   $maxm2=0;
   $maxm3=0;
   $maxq=10067.89169;
   
   //а теперь финт ушами - находим сумму полей в таблице
   $thisres = mysql_db_query($base, "select * from $Tablename[3]", $link);
   while ($thisrow = mysql_fetch_array($thisres)) {
   	$maxv1+=0+(float)$thisrow['v1'];  //самая прекрасная часть
   	$maxv2+=0+(float)$thisrow['v2'];
   	$maxv3+=0+(float)$thisrow['v3'];
   	$maxm1+=0+(float)$thisrow['m_in'];
   	$maxm2+=0+(float)$thisrow['m_out'];
   	$maxm3+=0+(float)$thisrow['m3'];
   	$maxq+=0+(float)$thisrow['q'];
   }
   //и так еще 2 таблицы складываются

   Шедевр выдран из творения моего коллеги. Комментарии мои. Задача - найти разницу по каждому полю между двумя последними записями в таблице и результат записать в другую таблицу.

   zloiia, 30 Октября 2011

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

10. PHP / Говнокод #8313

    +162

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

    public function action_delete(){   
            if (ORM::factory($_GET['type'], $_GET['id'])->delete())        
                $arr[0] = array("text" => "ok");
            else
                $arr[0] = array("text" => "no");
                                                   
            echo json_encode($arr);                               
    }

    Метод контролера (kohana), который используется для ajax запросов. Хоть в нем ничего не проверяется, за то этот метод очень функциональный. С помощью GET request можно удалить из БД практически все!

    k1011, 27 Октября 2011

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

11. C++ / Говнокод #8270

    +162

    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

    template <
    	typename PropertyOwner // Класс владельца
    >
    class properties {
    public:
    	// Получить указатель на владельца по указателю на свойство
    	static PropertyOwner * owner( void * property ) { 
    		int aai = (int)&(((PropertyOwner*)0)->__properties);
    		return (PropertyOwner *)((char*)property - aai);
    	}
    };
     
    /**
     * Шаблон класса свойства
     */
    template <
    	typename PropertyOwner, // Класс владельца
    	typename PropertyType, // Тип свойства
    	PropertyType (PropertyOwner::*getter)(), // Геттер
    	void (PropertyOwner::*setter)(PropertyType) > // Сеттер
    class property {
    public:
     
    	/**
    	 * Чтение свойства - вызов геттера
    	 */
    	operator PropertyType() {
    		return (properties<PropertyOwner>::owner( this )->*getter)();
    	}
     
    	/**
    	 * Запись в свойство - вызов сеттера
    	 */
    	void operator = ( const PropertyType & value ) {
    		(properties<PropertyOwner>::owner( this )->*setter)( value );
    	}
    };
     
    // Макросы для удобного определения свойств /////////
     
    /**
     * Начать объявления свойств в классе cls
     */
    #define properties_start(cls) union { properties<cls> __properties;
     
    /**
     * Закончить объявление свойств в классе cls
     */
    #define properties_end() };
     
    /**
     * Объявить свойство в классе cls типа type c геттером getter и сеттером setter
     */
    #define property(cls,type,getter,setter) property<cls,type,&cls::getter,&cls::setter>

    http://www.codenet.ru/progr/cpp/cpp-properties.php

    AnimeGovno-_-, 22 Октября 2011

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

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

• 1573
• …
• 1364
• 1363
• 1362
• 1361
• 1360
• 1359
• 1358
• 1357
• 1356
• …
• 1