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

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

   +117

   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

   /// <summary>
   /// Преобразование элемента перечисления в строку
   /// </summary>			
   private String GetEnumElementString(Object e, Type t)
   {
   	string result = string.Empty;
   
   	if (t == typeof(AccessMode))
   	{
   		if ((AccessMode)e == AccessMode.CacheOnly)
   		{
   			result = LocalStrings.AccessMode_CacheOnly_RU;
   		}
   		else if ((AccessMode)e == AccessMode.ServerOnly)
   		{
   			result = LocalStrings.AccessMode_ServerOnly_RU;
   		}
   		else if ((AccessMode)e == AccessMode.ServerAndCache)
   		{
   			result = LocalStrings.AccessMode_ServerAndCache_RU;
   		}
   		else
   		{
   			throw new Exception(string.Format("Отсутствует конвертация перечисления {0} к списку строк.", e));
   		}
   	}
   	else if (t == typeof(MapImageType))
   	{
   		if ((MapImageType)e == MapImageType.Map)
   		{
   			result = LocalStrings.MapImageType_Map_RU;
   		}
   		else if ((MapImageType)e == MapImageType.Satellite)
   		{
   			result = LocalStrings.MapImageType_Sattelite_RU;
   		}
   		else if ((MapImageType)e == MapImageType.Hybrid)
   		{
   			result = LocalStrings.MapImageType_Hybrid_RU;
   		}
   		else
   		{
   			throw new Exception(string.Format("Отсутствует конвертация перечисления {0} к списку строк.", e));
   		}
   	}
   	else
   	{
   		throw new Exception(string.Format("Отсутствует конвертация перечисления {0} к списку строк.", e));
   	}
   
   	return result;
   }

   Конвертация Enum в строку. Аффтар уникален, и это лишь одно из проявлений его уникальности.
   Swith - для трусов, статические тулзы и Extension - для дураков. У нас для этого будут методы классов, между прочим класс, в который входит этот метод - UserControl.
   Кстати, поля LocalStrings - не константы, а статика:
   
   public static string AccessMode_CacheOnly_RU = "Только кэш"; ....

   ICELedyanoj, 08 Февраля 2012

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

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

   +153

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

   QSqlQuery* query = new QSqlQuery(db2);
   QDate curDate = QDate::currentDate();
   query->prepare("select id,custom,actual0,actual1,division,fili,author,dogo,sumo_dogo,sumo_proshe,sumo_penis,sumo_shtrafa,state,mark,anka from clbalance where actual0 <= ? and actual1 > ?");
   query->addBindValue(QDate::currentDate().addDays(-1));
   query->addBindValue(QDate::currentDate().addDays(-1));

   А вдруг дата уже изменилась ?)
   Отдельно доставляют названия столбцов.

   ir4y, 08 Февраля 2012

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

3. JavaScript / Говнокод #9348

   +148

   1. 1

   http://digest.subscribe.ru/inet/review/n763622828.html

   Статья прелесть. document.write() в наше-то время...

   0rt, 08 Февраля 2012

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

4. Куча / Говнокод #9347

   +127

   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

   Lab1
    s x(2,2)=1
    s x(2,2,9)=0
    s y(3,6,7)=3
    s y(3,6,8)=4
    s y(3,6,7,8,4)=5
    s y(3,6,7,8,9)=6
    m x(2,2)=y(3,6,7,8)
    d Out("x(2,2)")
    
   Out(l)
    i $d(l)#10{
   	 i l
    	w l_" =",?15,@l,!
    	i $d(@l)\10{
   	 	f {
   		 	s c=$q(@l,1)
   		 	q:c=""
   		 	d Out(c)
   		 	k @c
   	 	}
    	}
    	q
    }

   Вот так в Cache Object Script можно вывести ветку многомерного массива....

   Taru4, 08 Февраля 2012

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

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

   +174

   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
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90

   //присваиваем массиву значения
   	for ($i=0; $i<7; $i++) 
   		{
   		$rand[$i]=rand(1,52);
   		}
   
   //проверяем 50 раз что бы исключить повторы
   for ($i=0; $i<50; $i++) 
   	{
   //проверяем $rand[0] на совпадения
   	if ($rand[0]==$rand[1])
   		{
   		$rand[0]=rand(1,52);
   		}
   	else
   		{
   		if ($rand[0]==$rand[2])
   			{
   			$rand[0]=rand(1,52);
   			}
   		else
   			{
   			if ($rand[0]==$rand[3])
   				{
   				$rand[0]=rand(1,52);
   				}
   			else
   				{
   				if ($rand[0]==$rand[4])
   					{
   					$rand[0]=rand(1,52);
   					}
   				else
   					{
   					if ($rand[0]==$rand[5])
   						{
   						$rand[0]=rand(1,52);
   						}
   					else
   						{
   						if ($rand[0]==$rand[6])
   							{
   							$rand[0]=rand(1,52);
   							}
   						}
   					}
   				}
   			}
   		}
   
   //проверяем $rand[1] на совпадения
   	if ($rand[1]==$rand[0])
   		{
   		$rand[1]=rand(1,52);
   		}
   	else
   		{
   		if ($rand[1]==$rand[2])
   			{
   			$rand[1]=rand(1,52);
   			}
   		else
   			{
   			if ($rand[1]==$rand[3])
   				{
   				$rand[1]=rand(1,52);
   				}
   			else
   				{
   				if ($rand[1]==$rand[4])
   					{
   					$rand[1]=rand(1,52);
   					}
   				else
   					{
   					if ($rand[1]==$rand[5])
   						{
   						$rand[1]=rand(1,52);
   						}
   					else
   						{
   						if ($rand[1]==$rand[6])
   							{
   							$rand[1]=rand(1,52);
   							}
   						}
   					}
   				}
   			}
   		}

   Генератор "не повторяющихся цифр от 1 до 52"
   Влезло 2 if-блока из семи в оригинале :)

   govnokod-observer, 07 Февраля 2012

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

6. Java / Говнокод #9344

   +67

   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

   @Transactional(readOnly = true)
       public Employee getByUsername(String username) throws SecurityException {
           Employee employee = employeeDaoDb.getByUsername(username);
   
           if (employee != null) {
               return employee;
           }
   
           employee = employeeDaoLdap.getByUsername(username);
   
           if (employee != null) {
               if (employee.getFirstName() == null && employee.getFamilyName() == null) {
                   employee.setFirstName(employee.getUsername());
               }
               return employeeDaoDb.save(employee);
           }
   
           return null;
       }

   Grig, 07 Февраля 2012

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

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

   +150

   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

   function select_district_area_city($selctor1,$selector2,$url,$bind_after_selector="", $bind_after_function="",$url2="", $pref=""){
   	$add_url=(empty($pref)?'""':'"&pref='.$pref.'"');
   	$main_page=is_home() && empty($_GET);
   		?>
   	<script type="text/javascript">
   		$(document).ready(function(){
   			$("<?=$selctor1?>").bind("change",new_chenge);
   			new_chenge();
   			//функция для отображения на карте
   			//alert("44444");
   
   
   			function new_chenge(){
   				params=$("<?=$selctor1?>").val();
   				//;
   				<?=($main_page?'showAddress(params);':'');?>
   				if (params.length > 0){
   					params='<?=$url?>='+params+<?=$add_url?>;
   					$("<?=$selector2?>").load(params,"",function(){
   						hidden_area=$("#<?=$pref?>hidden_area").val();
   						if (hidden_area.length > 0){
   							//$("#for_select_area")
   							$("#<?=$pref?>for_select_area [value='"+hidden_area+"']").attr("selected", "selected");
   							//showAddress_7(hidden_area);
   							<?=($main_page?'showAddress_7(hidden_area);':'');?>
   								<?=$pref?>new_chenge2();
   						}
   						if('<?=$bind_after_selector?>'.length > 0 && '<?=$bind_after_function?>'.length > 0){
   							if ($('<?=$bind_after_selector?>').length > 0){
   								$('<?=$bind_after_selector?>').bind(<?=$bind_after_function?>);
   							}
   						}
   					});
   				}
   			}
   			function <?=$pref?>new_chenge2(){
   				//alert("qqqqqq");
   				params2=$('<?=$bind_after_selector?>').val();
   				$("#<?=$pref?>hidden_area").val(params2);
   				//showAddress_7(params2);
   				<?=($main_page?'showAddress_7(params2);':'');?>
   				hidden_sity=$("#<?=$pref?>hidden_sity").val();
   				params2=params2+"&l1="+$("<?=$selctor1?>").val();
   				params2='<?=$url2?>'+"/?ajax_query&l2="+params2+<?=$add_url?>;
   				$(".<?=$pref?>for_select_city_load").load(params2,"",function(){
   
   					hidden_sity=$("#<?=$pref?>hidden_sity").val();
   					if (hidden_sity.length > 0){
   						$("#<?=$pref?>for_select_sity [value='"+hidden_sity+"']").attr("selected", "selected");
   						//showAddress_13(hidden_sity);
   						<?=($main_page?'showAddress_13(hidden_sity);':'');?>
   					}
   					$('#<?=$pref?>for_select_sity').bind("change",function(){
   						$("#<?=$pref?>hidden_sity").val($('#<?=$pref?>for_select_sity').val());
   						//showAddress_13($('#for_select_sity').val());
   						<?=($main_page?'showAddress_13($("#'.$pref.'for_select_sity").val());':'');?>
   					});
   				});
   			}
   			//////////////////////////////////////////////
   
   		});
   	</script>
   	<?
   	}

   кто поймет что этот кот делает тому медаль за отвагу

   randombot, 07 Февраля 2012

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

8. Java / Говнокод #9342

   +76

   1. 1
   2. 2

   Map<String,Object> status = service.getCurStatus(id);
   Date actionDate = status.get("actionDate") != null ? (Date) status.get("actionDate") : null;

   wild_beaver, 07 Февраля 2012

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

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

   +159

   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

   switch ($_SERVER['REQUEST_URI']) {
   
   	case "/technics/items.php?SECTION_ID=2395":
   			 LocalRedirect('/tele_video/televizory_led/televizor_led', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2395&brend=94279":
   			 LocalRedirect('/tele_video/televizory_led/televizor_led-lg', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2396&brend=94292":
   			 LocalRedirect('/tele_video/televizory_zhk/televizor_zhk-philips', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2397":
   			 LocalRedirect('/tele_video/televizory_plazmennye/televizor_plazmennyy', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2407&brend=94413":
   			 LocalRedirect('/tekhnika_dlya_doma/stiralnye_mashiny/stiralnaya_mashina-lg', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2407&brend=99348":
   			 LocalRedirect('/tekhnika_dlya_doma/stiralnye_mashiny/stiralnaya_mashina-bosch', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2407&brend=99402":
   			 LocalRedirect('/tekhnika_dlya_doma/stiralnye_mashiny/stiralnaya_mashina-indesit', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2407&brend=99413":
   			 LocalRedirect('/tekhnika_dlya_doma/stiralnye_mashiny/stiralnaya_mashina-samsung', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2407&brend=99518":
   			 LocalRedirect('/tekhnika_dlya_doma/stiralnye_mashiny/stiralnaya_mashina-hotpoint_ariston', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2408&brend=94434":
   			 LocalRedirect('/tekhnika_dlya_doma/kholodilniki/kholodilnik-lg', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2408&brend=94440":
   			 LocalRedirect('/tekhnika_dlya_doma/kholodilniki/kholodilnik-atlant', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2408&brend=99353":
   			 LocalRedirect('/tekhnika_dlya_doma/kholodilniki/kholodilnik-bosch', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2408&brend=99407":
   			 LocalRedirect('/tekhnika_dlya_doma/kholodilniki/kholodilnik-indesit', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2408&brend=99448":
   			 LocalRedirect('/tekhnika_dlya_doma/kholodilniki/kholodilnik-samsung', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2408&brend=99513":
   			 LocalRedirect('/tekhnika_dlya_doma/kholodilniki/kholodilnik-hotpoint_ariston', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?group=94355&SECTION_ID=2401":
   			 LocalRedirect('/kukhnya/vstraivaemaya_tekhnika/kholodilnik', false, '301 Moved Permanently'); die(); 
   			break;
   	case "/technics/items.php?SECTION_ID=2396":
   			 LocalRedirect('/tele_video/televizory_zhk/televizor_zhk-philips', false, '301 Moved Permanently'); die(); 
   			break;
   		
   	}

   Вот такое чудо сегодня выкатил разработчик со стороны заказчика ))
   и это еще не все адреса...

   ast, 07 Февраля 2012

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

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

    +143

    1. 1

    $fullness = $what ? '1' : '0';

    $what это boolean
    наверняка уже было, но такие конструкции меня просто убивают

    ghost404, 07 Февраля 2012

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

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

• 2105
• …
• 901
• 900
• 899
• 898
• 897
• 896
• 895
• 894
• 893
• …
• 1