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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2380)
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 / Говнокод #7850

   +156

   1. 1

   RewriteCond %{REQUEST_URI} !\.(cssіjsіjpgіgifіpng)$

   Творение удаленного сотрудника. Долго не мог понять, почему сервер кидает 500 ошибку.

   PrimeBIT, 13 Сентября 2011

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

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

   +168

   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

   ...
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=6&cat=i_octa">'.gettags("str_octa").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=6&cat=i_accident">'.gettags("str_sluch").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=6&cat=i_travel">'.gettags("str_travel").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=6&cat=i_history">'.gettags("str_history").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="8" class="lnk"><img src="images/spacer.gif" width="10" height="8" /></td>';
           echo '</tr>';
           echo '</table>';
           echo '</DIV>';
           echo '<!---  end of left menu area ---></td></tr>';
           break;        
   // Spravochniki    
       case 7:
           echo '<td width="5"   bgcolor="#FFFFFF"><img src="images/spacer.gif" width="5" height="700" /></td>';
           echo '<td width="219" valign="top" ><!---  left area --->';
           echo '<table width="219" height="100%" border="0" cellpadding="0" cellspacing="0" bgcolor="#FFFFFF">';
           echo '<tr>';
           echo '<td width="219" height="89"><!---  left menu area --->            <DIV class="left-menu">';
           echo '<table width="219" cellpadding="0" cellspacing="0" class="left-menu" border="0">';
           echo '<tr>';
           echo '<td width="219" height="27" background="images/LMenuBG.gif" class="txtTitleWhite">'.gettags("spr_notarius").'</td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="3" class="lnk"><img src="images/spacer.gif" width="10" height="3" /></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=7&cat=s_help">'.gettags("spr_nothelp").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=7&cat=s_functions">'.gettags("spr_funct").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="18" class="lnk" ><a href="?ierh=7&cat=s_when">'.gettags("spr_notwhen").'</a></td>';
           echo '</tr>';
           echo '<tr>';
           echo '<td width="219" height="27" background="images/LMenuBG.gif" class="txtTitleWhite">'.gettags("spr_zemsl").'</td>';
   ...

   Переделываем сайт одной компании. Пока суд да дело, их "текущий" сайт переехал к нам на хостинг. Из любопытства залез в код.
   Лучше бы я этого не делал.
   Собственно, ЭТО (и сотни, если не тысячи строк в том же духе) и выводят все странички.

   havelock, 12 Сентября 2011

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

3. Куча / Говнокод #7848

   +85

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

   Да обойдут вас стороной фатал эрроры, и ужас от переписывания говнокода нубов!=) 
   Пишите так, чтобы программа не требовала дополнительной плашки памяти на 2 гектара!
   Желаю успешного развития в этой нелегкой сфере IT!
   
   Мои искренние поздравления с днем программиста!

   1_and_0, 12 Сентября 2011

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

4. C++ / Говнокод #7847

   +167

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

   [code=cpp]
   #include <iostream.h> 
   void main(void) 
   { 
   cout << Заключайте сообщение в кавычки; 
   } 
   [/code]

   http://programmersclub.ru/01/

   ReallyBugMeNot, 12 Сентября 2011

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

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

   +160

   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

   $id = $_POST['id'];
   //$id = 194;
   $temp = $id;
   $str = "";
   for ( $incr = 0; $incr < 100; $incr++ ) {
   	$result = $dbs -> query("SELECT * FROM 
   								market_categs 
   							WHERE 
   								categid={$temp} AND 
   								published = 1 AND 
   								status = 1");
   	if ( mysql_num_rows($result) > 0 ) {
   		$categ = mysql_fetch_array($result);
   		$result2 = $dbs -> query("SELECT * FROM 
   									market_categs_trans 
   								WHERE 
   									categid={$categ[categid]}");
   		$categ_title = mysql_fetch_array( $result2 );
   		$str2 = "";
   		$temp2 = $categ[categid];
   		for ( $incr2 = 0; $incr2 < 100; $incr2++ ) {
   			$result3 = $dbs -> query("SELECT * FROM 
   										market_categs 
   									WHERE 
   										categid={$temp2} AND 
   										published = 1 AND 
   										status = 1");
   			if ( mysql_num_rows($result3) > 0 ) {
   				$categ2 = mysql_fetch_array($result3);
   				$str2 .= "/".$categ2[url];
   				$temp2 = $categ2[parent];
   			}
   		}
   		$str2 = substr($str2,1);
   		$nb2 = substr_count($str2,"/");
   		$temp_str2 = explode ("/",$str2);
   		$str2 = "";
   		$back = "";
   		for ($i2 = $nb2; $i2 >= 0; $i2--) {
   			$str2 .="/".$temp_str2[$nb2];
   			$nb2--;
   		}
   		$str .= " / <a href='/".substr($str2,1)."'>".$categ_title[title]."</a>";
   		$temp = $categ[parent];
   	}
   }

   Рекурсия... мать его

   cephuo, 12 Сентября 2011

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

6. SQL / Говнокод #7845

   −112

   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

   CREATE TRIGGER after_update_limit AFTER UPDATE ON limits
   FOR EACH ROW
   BEGIN
   	SET @annual_normative = NULL;
   	SET @ud_use = NULL;
   	SET @ud_disposal = NULL;
   	SET @placed_deposited = NULL;
   	SET @placed_disposal = NULL;
   	SET @custom1_disposal = NULL;
   	SET @custom1_deposited = NULL;
   	SET @custom1_use = NULL;
   	SET @custom1_neutralization = NULL;
   	SET @custom2_disposal = NULL;
   	SET @custom2_deposited = NULL;
   	SET @custom2_use = NULL;
   	SET @custom2_neutralization = NULL;
   	SET @custom3_disposal = NULL;
   	SET @custom3_deposited = NULL;
   	SET @custom3_use = NULL;
   	SET @custom3_neutralization = NULL;
   	SET @custom4_disposal = NULL;
   	SET @custom4_deposited = NULL;
   	SET @custom4_use = NULL;
   	SET @custom4_neutralization = NULL;
   	SET @custom5_disposal = NULL;
   	SET @custom5_deposited = NULL;
   	SET @custom5_use = NULL;
   	SET @custom5_neutralization = NULL;
   	SELECT
   		SUM(annual_normative), SUM(ud_use), SUM(ud_disposal), SUM(placed_deposited), SUM(placed_disposal),
   		SUM(custom1_disposal), SUM(custom1_deposited), SUM(custom1_use), SUM(custom1_neutralization),
   		SUM(custom2_disposal), SUM(custom2_deposited), SUM(custom2_use), SUM(custom2_neutralization),
   		SUM(custom3_disposal), SUM(custom3_deposited), SUM(custom3_use), SUM(custom3_neutralization),
   		SUM(custom4_disposal), SUM(custom4_deposited), SUM(custom4_use), SUM(custom4_neutralization),
   		SUM(custom5_disposal), SUM(custom5_deposited), SUM(custom5_use), SUM(custom5_neutralization)
   	INTO
   		@annual_normative, @ud_use, @ud_disposal, @placed_deposited, @placed_disposal,
   		@custom1_disposal, @custom1_deposited, @custom1_use, @custom1_neutralization,
   		@custom2_disposal, @custom2_deposited, @custom2_use, @custom2_neutralization,
   		@custom3_disposal, @custom3_deposited, @custom3_use, @custom3_neutralization,
   		@custom4_disposal, @custom4_deposited, @custom4_use, @custom4_neutralization,
   		@custom5_disposal, @custom5_deposited, @custom5_use, @custom5_neutralization
   	FROM limits
   	WHERE id_enterprise = NEW.id_enterprise;
   	UPDATE limits_total
   	SET
   		annual_normative = @annual_normative, ud_use = @ud_use, ud_disposal = @ud_disposal, placed_deposited = @placed_deposited, placed_disposal = @placed_disposal,
   		custom1_disposal = @custom1_disposal, custom1_deposited = @custom1_deposited, custom1_use = @custom1_use, custom1_neutralization = @custom1_neutralization,
   		custom2_disposal = @custom2_disposal, custom2_deposited = @custom2_deposited, custom2_use = @custom2_use, custom2_neutralization = @custom2_neutralization,
   		custom3_disposal = @custom3_disposal, custom3_deposited = @custom3_deposited, custom3_use = @custom3_use, custom3_neutralization = @custom3_neutralization,
   		custom4_disposal = @custom4_disposal, custom4_deposited = @custom4_deposited, custom4_use = @custom4_use, custom4_neutralization = @custom4_neutralization,
   		custom5_disposal = @custom5_disposal, custom5_deposited = @custom5_deposited, custom5_use = @custom5_use, custom5_neutralization = @custom5_neutralization
   	WHERE id_enterprise = NEW.id_enterprise
   	LIMIT 1;
   END;

   Сперва можно посмеяться, а затем подскажите, пжл, как тоже самое написать по человечески ???

   DarkThinker, 12 Сентября 2011

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

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

   +163

   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

   /*
    См. маткад файл по нерекурсивным цифровым фильтрам.
    Рачёт дискретной последовательности отфильтрованного сигнала
    У меня то в файликах уже значения, функции нету,а в маткаде такая строка
       lpf(k)*x(Ta*(n-k)) - дык тут идёт расчёт функции х(т). Идея!!!
       *барабанная дробь*
       Сигнал повторяется с периодом 2Пи, тем более данный сигнал
       парный. Знач делаем следующее
       1. Вычисляем Step = |Ta*(n-k)|.
       2. Так как сигнал парный и симетричный на отрезке 2Пи
          то значение функции в Step = |step - 2*ПИ| тоже самое, что и
          значение функции в Step. Зачем такой геморой? Ибо в файле step.dat
          хначения идут до 2*Пи, тобишь до 6.283
       3. В файле data.dat на строке Step ищем искомое нам значение,
          по сути тоже самое, что и x(Ta*(n-k)), где х - сигнал.
   
       Сл. функция и возвращает значение Step = |step - 2*ПИ|
   
   */
   int FindStep(int k)
   {
    float step = fabs(fabs((Ta * k)) - 6.283); // (n-k) in main program
    return step*1000;
   }

   Комментарий умилил

   denis90, 12 Сентября 2011

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

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

   +166

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

   <!--CAPCHA-->
     <?php
      $input = array ("0283", "0830", "1541", "3510", "7269", "0110", "2500", "4998", "7203", "9480", 
                               "1540", "1581", "1891", "0064", "0128", "0256", "0512", "1024", "2048", "4096");
      $rand_keys = array_rand($input);
      $img = $input[$rand_keys];
      $_SESSION['img'] = $img;
      print ("<img src=\"/images/$img.gif\">"); 
     ?>
            <!--CAPHA answer-->

   понятно и без слов ;)

   tranirk, 12 Сентября 2011

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

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

   −139

   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

   Функция ПолучитьЕдиницу(ТекЕдиница)
   	Тек = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию(ТекЕдиница.Наименование);
   	Если Тек = Справочники.КлассификаторЕдиницИзмерения.ПустаяСсылка() Тогда
   		Тек = Справочники.КлассификаторЕдиницИзмерения.СоздатьЭлемент();
   		Тек.Наименование = ТекЕдиница.Наименование;
   		Тек.НаименованиеПолное = ТекЕдиница.Наименование;
   		СтроковыйКод = "001";
   		Запрос = Новый Запрос;
   		Запрос.Текст = "ВЫБРАТЬ ПЕРВЫЕ 1
   		               |	КлассификаторСтранМира.Код КАК Код
   		               |ИЗ
   		               |	Справочник.КлассификаторСтранМира КАК КлассификаторСтранМира
   		               |
   		               |УПОРЯДОЧИТЬ ПО
   		               |	Код УБЫВ";
   					   
   					   Результат = Запрос.Выполнить().Выгрузить();
   					   ЧисловойКод = Число(Результат[0].Код) + 1;
   					   Если ЧисловойКод < 10 Тогда
   						   СтроковыйКод = "00"+ЧисловойКод
   					   ИначеЕсли ЧисловойКод < 100 Тогда
   						   СтроковыйКод = "0"+ЧисловойКод
   					   Иначе
   						   СтроковыйКод = ""+ЧисловойКод
   					   КонецЕсли;
   					   
   		Тек.Код = СтроковыйКод;
   		
   		Попытка
   			Тек.Записать();
   		Исключение
   			ЕстьОшибка = Истина;
   			Сообщить("Не могу записать единицу измерения "+ТекЕдиница.Наименование+" !");
   			Сообщить(ОписаниеОшибки());
   			Возврат "";
   		КонецПопытки;
   	КонецЕсли;
   	Если Тек = Справочники.КлассификаторЕдиницИзмерения.ПустаяСсылка() Тогда
   		ЕстьОшибка = Истина;
   		Возврат "";
   	Иначе
   		Возврат Тек;
   	КонецЕсли;
   КонецФункции

   Получила в наследство от команды разработчиков во главе с неким Леонидом. Порадовало формирование кода для нового элемента классификатора единиц измерения. И правда, почему бы не поискать максимальный код в каком-нибудь неожиданном месте? А такое оригинальное формирование строки с лидирующими нулями просто убило до смерти.

   Shapokljak, 12 Сентября 2011

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

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

    +164

    1. 1

    $arParams["FILES_COUNT"] = intVal(intVal($arParams["FILES_COUNT"]) > 0 ? $arParams["FILES_COUNT"] : 5);

    Битрикс. Количество, так сказать, файлов.

    franzose, 12 Сентября 2011

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

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

• 2109
• …
• 772
• 771
• 770
• 769
• 768
• 767
• 766
• 765
• 764
• …
• 1