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

    +119

    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
    public  static  string      LoadGridDT          (TObjects pList, out DataTable Dt, params object[] pArgs)
...              
                switch (pList)
                {
                    case TObjects.Radio                 :
                        if (pArgs.Length == 1 && pArgs[0] is int)
                            vLoadResult = Protocol.Radionuclide.Load((int)pArgs[0], out Dt);
                        else if (pArgs.Length > 1 && pArgs[0] is int && pArgs[1] is bool)
                            vLoadResult = Part2.Radionuclide.Load((int)pArgs[0], out Dt);
                        else vLoadResult = Radionuclide.Load(out Dt);
                        break;
                    case TObjects.Protocol              :
                        // [0](int): Тип протокола; [1](int): Очередь; [2](bool): Активность; [3](int): Объект
                if (pArgs.Length > 3 && pArgs[0] is int && pArgs[1] is int && pArgs[2] is bool && pArgs[3] is int)
                    if (pArgs.Length > 4 && pArgs[4] is int)
                        vLoadResult = Protocol.LoadList((int)pArgs[0], (int)pArgs[1], (int)pArgs[3], (bool)pArgs[2], out Dt, (int)pArgs[4]);
                    else
                    {
                        if (pArgs.Length > 4 && pArgs[4] is bool && (bool)pArgs[4] == true) // 5-й аргумент для загрузки детализированного списка протоколов с учетом объекта отбора
                            vLoadResult = Protocol.Load((int)pArgs[0], (int)pArgs[1], (int)pArgs[3], (bool)pArgs[2], out Dt);
                        else // недетализированный список протоколов с учетом объекта отбора
                            vLoadResult = Protocol.LoadList((int)pArgs[0], (int)pArgs[1], (int)pArgs[3], (bool)pArgs[2], out Dt);
                    }
                else
                // [0](List<int>): Типы протокола; [1](int): Очередь; [2](int): Id место отбора; [3](bool): Активность
                if (pArgs.Length > 3 && pArgs[0] is List<int> && pArgs[1] is int && pArgs[2] is int && pArgs[3] is bool)
                    if (pArgs.Length > 4 && pArgs[4] is int)
                        vLoadResult = Protocol.LoadList((List<int>)pArgs[0], (int)pArgs[1], (int)pArgs[2], (bool)pArgs[3], out Dt, (int)pArgs[4]);
                    else
                        vLoadResult = Protocol.Load((List<int>)pArgs[0], (int)pArgs[1], (int)pArgs[2], (bool)pArgs[3], out Dt);
                else
                // [0](List<int>): Типы протокола; [1](int): Очередь; [2](bool): Активность
                if (pArgs.Length > 2 && pArgs[0] is List<int> && pArgs[1] is int && pArgs[2] is bool)
                    if (pArgs.Length > 3 && pArgs[3] is int)
                        vLoadResult = Protocol.LoadList((List<int>)pArgs[0], (int)pArgs[1], (bool)pArgs[2], out Dt, (int)pArgs[3]);
                    else
                        vLoadResult = Protocol.LoadList((List<int>)pArgs[0], (int)pArgs[1], (bool)pArgs[2], out Dt);
                else
                // [0](int): Тип протокола; [1](int): Очередь; [2](bool): Активность;
                if (pArgs.Length > 2 && pArgs[0] is int && pArgs[1] is int && pArgs[2] is bool)
                    if (pArgs.Length > 4 && pArgs[3] is bool && pArgs[4] is int)
                        vLoadResult = Protocol.LoadList((int)pArgs[0], (int)pArgs[1], (bool)pArgs[2], out Dt, (int)pArgs[4]);
                    else
                        vLoadResult = Protocol.Load((int)pArgs[0], (int)pArgs[1], (bool)pArgs[2], out Dt);
                else
                // [0](int): Тип протокола; [1](int): Очередь
                if (pArgs.Length > 1 && pArgs[0] is int && pArgs[1] is int)
                    if (pArgs.Length > 2 && pArgs[2] is int)
                        vLoadResult = Protocol.LoadList((int)pArgs[0], (int)pArgs[1], out Dt, (int)pArgs[2]);
                    else
                        vLoadResult = Protocol.LoadList((int)pArgs[0], (int)pArgs[1], out Dt);
                else
...

    Метод 500+ вот такого рода строк...

    Запостил: gcoder, 08 Февраля 2012

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

    • ava kyzi007 08.02.2012 10:50 # 0

      Ой *ля, сочувствую, можно вешаться если придется что то править.
      Ответить

    • ava gcoder 08.02.2012 10:58 # 0

      Фишка еще в том, что этот метод вызывается из другого:
      public static int LoadListData (UIComboBox pEdit, TObjects pList, int pFocusID, bool pShowErr, params object[] pArgs),
      часть элементов pArgs используется в нем, а часть в вышеописанном...
      Ответить

      • ava kyzi007 08.02.2012 11:02 # 0

        А это все работает вообще?
        Ответить

        • ava gcoder 08.02.2012 11:08 # 0

          Да, года 4 проекту... но править это некомфортно как-то. =)
          Ответить

    • ava wvxvw 08.02.2012 11:11 # 0

      Ну, видимо человк не знал, что метод можно перегрузить. Хотя и отрефакторить будет не так сложно - вынести каждый if в отдельную функцию, а из условия if'a сделать список аргументов.
      Ответить

      • ava gcoder 08.02.2012 11:31 # 0

        > не знал, что метод можно перегрузить
        Знал он, что можно перегружать, он, видимо, хотел написать 1 супер метод загрузки всего везде. =) А частные перегруженные методы загрузки конкретного справочника с конкретными параметрами как раз и вызываются...
        "Хотя и отрефакторить будет не так сложно"
        Угу, да и дописать / поправить тоже...
        Ответить

        • ava gcoder 08.02.2012 11:39 # 0

          Да и вообще, такого кода и подхода полно. В формах, например, кроме прочих конструкторов, бывает 1 с "params object[]", в который может передаваться переменное кол-во разнообразных параметров штук до 10... =)
          Ответить

    • ava roman-kashitsyn 08.02.2012 12:00 # +3

      write once fix all your life
      Ответить

    • ava Govnocoder#0xFF 08.02.2012 18:43 # 0

      Типун ему на Ctrl+C!
      Ответить

    • ava guest8 09.04.2019 12:39 # −999

      показать все, что скрытоvanished
      Ответить

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