1. C# / Говнокод #19722

    +1

    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
    private int GetSroOper(MiscTransactionParams param)
		{
			if (param.ExpenseItem != null)
			{
				switch (param.ContractType)
				{
					case Entities.AT.ContractType.Undefined:
					case Entities.AT.ContractType.TimeAndMaterials:
					case Entities.AT.ContractType.Retainer:
						switch ((int)param.ExpenseItem.ExpenseCategory)
						{
							case 2:
							case 3:
							case 29684252:
							case 29684285:
							case 29684805:
								return param.BillingItem.IsNonBillable() ? Helper.SLServiceOrderOperation600 : Helper.SLServiceOrderOperation900;
							default:
								throw new InvalidOperationException("Unsupported Expense Category value");
						}
					case Entities.AT.ContractType.RecurringService:
						switch ((int)param.ExpenseItem.ExpenseCategory)
						{
							case 2:
							case 3:
							case 29684252:
							case 29684285:
							case 29684805:
								return param.BillingItem.IsNonBillable() ? Helper.SLServiceOrderOperation550 : Helper.SLServiceOrderOperation950;
							default:
								throw new InvalidOperationException("Unsupported Expense Category value");
						}
					case Entities.AT.ContractType.FixedPrice:
						switch ((int)param.ExpenseItem.ExpenseCategory)
						{
							case 2:
							case 3:
							case 29684252:
							case 29684285:
							case 29684805:
								return param.BillingItem.IsNonBillable() ? Helper.SLServiceOrderOperation500 : Helper.SLServiceOrderOperation950;
							default:
								throw new InvalidOperationException("Unsupported Expense Category value");
						}
					case Entities.AT.ContractType.Incident:
						switch ((int)param.ExpenseItem.ExpenseCategory)
						{
							case 2:
							case 3:
							case 29684252:
							case 29684285:
							case 29684805:
								return param.BillingItem.IsNonBillable() ? Helper.SLServiceOrderOperation600 : Helper.SLServiceOrderOperation950;
							default:
								throw new InvalidOperationException("Unsupported Expense Category value");
						}
					default:
						throw new InvalidOperationException("Unsupported ContractType value");
				}
			}
			else
			{
				switch (param.ContractType)
				{
					case Entities.AT.ContractType.Undefined:
					case Entities.AT.ContractType.TimeAndMaterials:
					case Entities.AT.ContractType.Retainer:
						return param.BillingItem.IsNonBillable() ? Helper.SLServiceOrderOperation600 : Helper.SLServiceOrderOperation900;
					case Entities.AT.ContractType.RecurringService:
						return param.BillingItem.IsNonBillable() ? Helper.SLServiceOrderOperation550 : Helper.SLServiceOrderOperation950;
					case Entities.AT.ContractType.FixedPrice:
					case Entities.AT.ContractType.Incident:
						int sroOper = (int)param.OrderOperation.EntityRow()[Constants.ServiceOrderOperation.Field.SroOper];
						switch (sroOper)
						{
							case 500:
							case 600:
							case 800:
							case 850:
							case 950:
								return sroOper;
							default:
								throw new InvalidOperationException("Unsupported Operation value");
						}
					default:
						throw new InvalidOperationException("Unsupported ContractType value");
				}
			}
		}

    Это наш сеньор написал

    Запостил: kagetoki, 30 Марта 2016

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

    • ava wvxvw 30.03.2016 15:03 # +2

      А что такое / кто такой срупер?
      Ответить

      • ava kagetoki 30.03.2016 19:08 # 0

        SroOper - это сущность из стороннего API, которое тоже написали какие-то сеньоры. Этот API умеет работать только с нетипизированными датасетами, что-то типа LoadDataSet(string fields, string filter, string orderby, int maxCount)
        Ответить

    • ava tucvbif 30.03.2016 15:05 # 0

      Хоть бы лишние отступы убрал.
      А вообще, в C# можно как-то кратко проверить, принадлежит ли переменная множеству значений? что-то типа
      if ({2,3,29684252,29684285,29684805}.contains((int)param.ExpenseItem.ExpenseCategory)) ...

      ?
      Ответить

      • ava Vindicar 30.03.2016 15:12 # 0

        Судя по всему, только костыль в виде extension method писать.
        Ответить

        • ava roman-kashitsyn 30.03.2016 15:24 # 0

          А какой-нибудь
          new[] {2,3,29684252,29684285,29684805}.Contains(x)
          не сработает?
          Ответить

          • ava tucvbif 30.03.2016 15:32 # +1

            Нет. Разве что как-то так, но это вообще через одно место:
            if (Array.Exists(new[] {2,3,29684252,29684285,29684805},(int s)=>(s==x)))…
            Ответить

            • ava roman-kashitsyn 30.03.2016 15:45 # +1

              УМВР 
              http://ideone.com/DQbyW9
              Ответить

              • ava tucvbif 30.03.2016 16:19 # +1 

                using System.Linq;

                Вот оно чё!
                Ответить

                • ava nihau 30.03.2016 19:02 # +1

                  Можно было бахнуть 
                  new Hashset <int>{2,3,29684252,29684285,29684805}.Contains(x)

                  и Enumerable бы не понадобился
                  Ответить

                  • ava Serega9I 31.03.2016 21:25 # 0

                    2, 3, 2968... - магические числа, через месяц-другой уже и не вспомнить о чем речь шла... нельзя так...
                    Ответить

                    • ava nihau 01.04.2016 12:22 # 0

                      Никто не спорит, это отдельная проблема.
                      Ответить

          • ava Vindicar 30.03.2016 19:25 # 0

            Вообще такой экстеншен удобнее вешать на int и/или Enum из-за комбо с переменным числом параметров метода.
            if (EnumValue.In(Value1, Value2, Value3)) DoStuff();
            Ответить

      • ava j123123 30.03.2016 15:43 # 0

        >А вообще, в C# можно как-то кратко проверить, принадлежит ли переменная множеству значений?

        битовым вектором через RLE
        Ответить

    • ava kegdan 30.03.2016 17:32 # 0

      Вот оно, будущее! Case-oriented programming!
      Ответить

      • ava bot 31.03.2016 00:25 # +3

        Чё до кейсов доебался? Почти такой же индекс. А конпелятор сделает из вореций как надо. Хуле там. На то он и конпелятор, чтобы делать как надо. На хуя кодеру за каждую ворецию самому думать? Вон конпелятор есть, пусть он и думает, шерсть на носу. Хули нахуй блять. И вообще, ты чё дерзкий какой? Кто ты вообще такой? С какого раёна?
        Ответить

        • ava kegdan 31.03.2016 00:37 # +1

          Чё до кейсов доебался? С какого раёна? Вон кароч блеать конпелятор есть, нах пусть он и конпелятор, ептить чтобы делать как надо. Почти такой нах типа же индекс. Хуле там. Чё до кейсов доебался?
          Ответить

        • ava kerman 31.03.2016 00:55 # +1

          Я прошу всех присутствующих отметить этот провокационный вопрос, от
          которого за версту разит мальтузианством, неомальтузианством,
          прагматизмом, экзистенцио...оа...нализмом и неверием, товарищи, в
          неисчерпаемую мощь человечества. Вы что же хотите сказать этим вопросом,
          товарищ %username%? Что в деятельности нашего научного учереждения может
          наступить момент, кризис, регресс, когда нашим потребителям не хватит
          продуктов потребления? Нехорошо, товарищ %username%! Не подумали вы! А мы
          не можем допустить, чтобы на нашу работу навешивали ярлыки и бросали тень.
          И мы этого, товарищи, не допустим.
          Ответить

    • ava 3_dar 30.03.2016 21:10 # 0

      Ис нот БИЛЛАБЛЕ
      Ответить

    • ava guest 01.04.2016 01:44 # +1

      сеньор помедор
      Ответить

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