- 001
- 002
- 003
- 004
- 005
- 006
- 007
- 008
- 009
- 010
- 011
- 012
- 013
- 014
- 015
- 016
- 017
- 018
- 019
- 020
- 021
- 022
- 023
- 024
- 025
- 026
- 027
- 028
- 029
- 030
- 031
- 032
- 033
- 034
- 035
- 036
- 037
- 038
- 039
- 040
- 041
- 042
- 043
- 044
- 045
- 046
- 047
- 048
- 049
- 050
- 051
- 052
- 053
- 054
- 055
- 056
- 057
- 058
- 059
- 060
- 061
- 062
- 063
- 064
- 065
- 066
- 067
- 068
- 069
- 070
- 071
- 072
- 073
- 074
- 075
- 076
- 077
- 078
- 079
- 080
- 081
- 082
- 083
- 084
- 085
- 086
- 087
- 088
- 089
- 090
- 091
- 092
- 093
- 094
- 095
- 096
- 097
- 098
- 099
- 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...
Follow us!
В SQL это делается с помощью pivot / unpivot. Пусть погуглит по "pivot поворот таблицы".
MSSQLизм?
Вы так говорите, как будто это мой код ;)
> таком виде
Ну вот собственно я и пишу - или выпрямлять саму архитектуру. Проблема то зарыта совсем не в коде, который запостил ОП, а намного глубже, в архитектуре проекта, которая вынудила кодера написать это говнище.
> Может заменить на Dictionary?
Скорее всего да... Или, возможно, на массив структурок. Я не видел остального кода, но думаю что подобная замена уберет и кучу копипасты из кода, который потом всю эту инфу выводит.
есть 1 момент, он сам делал структуру данных в базе и, как следствие, ее обработку в коде ;)
2. по его словам у него 10+ лет опыта в c#
3. это еще не самый говнокод из проекта...был случай, когда он проблемы многопоточности Thread.Sleep()-ми решал...но, алилуя, потом почему-то переделал, видимо не мопогло)))))