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

    +133

    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
    abstract class A { }

abstract class B
{
     void CallValidate(A objA)
     {
          MethodInfo info = base.GetType().GetMethod("Validate", new[] { objA.GetType() });

          if (info == null)
          {
               throw new ApplicationException("The method 'Validate' with parameter type '" + objA.GetType() + "' is not implemented.");
          }

          info.Invoke(this, new object[] { objA });
     }
}

    Метод CallValidate базового класса проверяет наличие метода "Validate" с нужной сигнатурой в своих дочерних классах и вызывает его, если таковой есть, иначе - кидает исключение.

    Запостил: Guid, 02 Апреля 2013

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

    • ava vse_govno 02.04.2013 14:00 # −1

      zogчем?
      Ответить

      • ava Guid 02.04.2013 14:57 # −1

        "Зачем?" - не правильный вопрос :) "Зачем делать это нужно было делать так?" :)
        Ответить

    • ava taburetka 02.04.2013 14:05 # 0

      Какой класс базовый у класса Б?
      Ответить

      • ava Guid 02.04.2013 15:11 # 0

        "Метод CallValidate базового класса" - хотел сказать сказать: "Метод CallValidate абстрактного класса B". Вообще, B - это некий абстрактный "валидатор" чего-то там. Еще раз пересмотрел код, оказалось все хуже. Вариант использования предполагался такой:

        abstract class A { ... }
        class A1 : A { ... }
        class A2 : A { ... }
        class A3 : A { ... }

        ...

        abstract class B
        {
        void CallValidate(A objA) { ... }

        void Validate(A1 a) { ... }
        void Validate(A2 a) { ... }
        }

        ...

        B validator;
        validator.CallValidate(new A3()); // бросит исключение, т.к. в B не определен метод "void Validate(A3 a) { ... }"
        Ответить

        • ava neeedle 02.04.2013 15:53 # +1 

          B validator;
validator.CallValidate(new A3()); // бросит исключение, т.к. в B не определен метод "void Validate(A3 a) { ... }"


          Это как это вы так хитро инстанцировали абстрактный класс?
          Ответить

          • ava Guid 03.04.2013 06:30 # 0

            Это псевдокод, который демонстрирует сценарий воспроизведения ошибки, не более того.
            Ответить

    • ava mangyst 02.04.2013 14:58 # −1

      как говорится "гибкий клиент")))
      Ответить

    • ava someone 02.04.2013 17:41 # −1

      Мне кажется, лучше сделать интерфейс IValidator<T> из одного метода, а в классе B сделать dictionary от типов аргументов к валидаторам. И метод типа Register(T, IValidator<T>).

      Или даже проще - передавать аргументом делегата и прямо при вызове Register подсовывать ему лямбду, если в .NET делегаты могут быть дженериками.
      Ответить

      • ava neeedle 02.04.2013 18:54 # 0

        Да просто сделать этот интерфейс, IValidator и проверять можно ли превести ссылку к ссылке типа этого интерфейса.
        И все.
        not implemented exeption там есть.
        Ответить

    • ava Dummy00001 02.04.2013 22:52 # +2

      шарпов не знаю, но если бы С++, я бы сказал что кто-то опять объелся паттернов и маструбирует визитором.
      Ответить

      • ava roman-kashitsyn 02.04.2013 22:59 # +1

        Нет, кто-то тоскует по привычному жабоскриптику.
        Ответить

        • ava neeedle 03.04.2013 04:33 # 0

          Почему так решили, можно узнать?
          Ответить

          • ava roman-kashitsyn 03.04.2013 07:49 # +1

            Динамический вызов там, где, видимо, можно обойтись введением интерфейса.
            Ответить

            • ava neeedle 03.04.2013 10:44 # 0

              Тоже заметил, что тут просто через жопу вводится интерфейс.
              Но с яваскриптом не связал.
              Ответить

              • ava Yuuri 04.04.2013 00:01 # 0

                Там же уже есть dynamic?
                Ответить

                • ava neeedle 04.04.2013 06:44 # 0

                  К чему этот вопрос?
                  Ответить

                  • ava Yuuri 04.04.2013 11:58 # 0

                    Пардон, надо было вопрос в корне треда задавать. Имею в виду, что на мой достаточно далёкий от дотнета взгляд автор кода пытается переизобрести dynamic.
                    Ответить

                    • ava neeedle 04.04.2013 15:18 # 0

                      Нет, он пытается с помощью асбтрактного класса получить not implemented exeption который означает, что в этом классе не переопределен метод абстрактного класса предка.
                      Смысл говнокода в том, что для таких вещей существует интерфейс.
                      Ответить

    • ava Ccik 04.04.2013 10:00 # 0

      Ну сделал бы виртуальный метод, который по умолчанию кидает исключение, раз уж так хочется
      Ответить

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