Си диез / Говнокод #12223 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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
  91. 91
  92. 92
  93. 93
  94. 94
  95. 95
  96. 96
  97. 97
  98. 98
  99. 99
  100. 100
switch (occupation.name)
{
	case "architecture and engineering occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.ArchitectureAndEngineeringOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.ArchitectureAndEngineeringOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.ArchitectureAndEngineeringOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.ArchitectureAndEngineeringOccupationsFemalePercentageCity = occupation.percent;
			}
		}
	break;

	case "arts, design, entertainment, sports, and media occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsFemalePercentageCity = occupation.percent;
			}
		}
	break;

	case "business and financial operations occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.BusinessAndFinancialOperationsOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.BusinessAndFinancialOperationsOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.BusinessAndFinancialOperationsOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.BusinessAndFinancialOperationsOccupationsFemalePercentageCity = occupation.percent;
			}
		}
	break;

	case "community and social services occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.CommunityAndSocialServicesOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.CommunityAndSocialServicesOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.CommunityAndSocialServicesOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.CommunityAndSocialServicesOccupationsFemalePercentageCity = occupation.percent;
			}
		}

эх жаль здесь ограничение на 100 строк всего лишь...не поместилось всего 200+ отборного ветвления. известный до боли мегапрограммер наваял очередной шедевр)))) по мотивам http://govnokod.ru/10448 и http://govnokod.ru/10313 и еще http://govnokod.ru/12136...

Запостил: mangyst mangyst, (Updated )

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

  • ava vladthesparrow # Ссылка на оригинал 0

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

    • ava mangyst # Ссылка на оригинал 0 ⇈

      да, у меня тоже такая мысль была, когда мне первый раз дали этот проект...там вместо классов [,] использовались(не везде правада, но факт), а перед их заполнением была комменты под каким индексом какое значение)))
      Ответить

  • ava santa_microbe # Ссылка на оригинал 0

    Так научите человека, как надо, а то он вам бесконечно подобный код будет генерировать
    Ответить

    • ava vladthesparrow # Ссылка на оригинал 0 ⇈

      Судя по коду, чувак транспонирует таблицу - строки справочника профессий превращает в столбцы.
      В SQL это делается с помощью pivot / unpivot. Пусть погуглит по "pivot поворот таблицы".
      Ответить

        • ava vladthesparrow # Ссылка на оригинал 0 ⇈

          Ну попробуйте на LINQ сделать pivot. Может даже в 30 строк кода уложитесь.
          Ответить

          • ava bormand # Ссылка на оригинал 0 ⇈

            Да тут больше проблем не с транспонированием, а с тем, как засунуть значение в поле ArchitectureAndEngineeringOccupationsMal ePercentageNB. Собрать имя из кусков, да через рефлекшен засунуть разве что... Или надо выпрямлять саму архитектуру.
            Ответить

            • ava vladthesparrow # Ссылка на оригинал 0 ⇈

              Для чего храните значения в таком виде? Может заменить на Dictionary? И использовать id профессий вместо их названий?
              Ответить

              • ava bormand # Ссылка на оригинал 0 ⇈

                > храните
                Вы так говорите, как будто это мой код 😉

                > таком виде
                Ну вот собственно я и пишу - или выпрямлять саму архитектуру. Проблема то зарыта совсем не в коде, который запостил ОП, а намного глубже, в архитектуре проекта, которая вынудила кодера написать это говнище.

                > Может заменить на Dictionary?
                Скорее всего да... Или, возможно, на массив структурок. Я не видел остального кода, но думаю что подобная замена уберет и кучу копипасты из кода, который потом всю эту инфу выводит.
                Ответить

                • ava mangyst # Ссылка на оригинал 0 ⇈

                  >или выпрямлять саму архитектуру
                  есть 1 момент, он сам делал структуру данных в базе и, как следствие, ее обработку в коде 😉
                  Ответить

    • ava mangyst # Ссылка на оригинал 0 ⇈

      1. он нечто подобное фрилансеру, удаленно работает (нанял его кастомер) - не получится учить
      2. по его словам у него 10+ лет опыта в c#
      3. это еще не самый говнокод из проекта...был случай, когда он проблемы многопоточности Thread.Sleep()-ми решал...но, алилуя, потом почему-то переделал, видимо не мопогло)))))
      Ответить

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓