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

    +1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    if (test)
{
_inquiryLineRepository.RequestToOrderAPI(model, idetails, true);

}
else
{
_inquiryLineRepository.RequestToOrderAPI(model, idetails);
}

    индусы радуются и танцуют групповые танцы, когда видят, что подобный код пишет кто то кроме них

    Запостил: and1dufresn3, 04 Августа 2016

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

    • ava guest 04.08.2016 23:25 # +16

      а вдруг там без параметра по умолчанию, а это 2 разных метода?
      Ответить

      • ava Lokich 05.08.2016 14:57 # +17

        вангую, что там определение метода public voind RequestToOrderAPI(object model, object idetails, bool test=false){...}
        и можно было бы поменять на inquiryLineRepository.RequestToOrderAPI( model, idetails, test);
        Ответить

      • ava and1dufresn3 05.08.2016 15:00 # +14

        Если бы это было еще и два разных метода, то индусы бы фильм сняли по этому поводу.
        Ответить

        • ava bormand 05.08.2016 18:39 # +17

          А в джаве это и были бы 2 разных метода.
          Ответить

          • ava and1dufresn3 06.08.2016 02:09 # +14

            А в хаскеле этот код вообще бы не запустился. Продолжим сравнения?
            Суть в том что даже если бы это были два разных метода типа перегрузки, то все равно лаконично было бы вызвать
            inquiryLineRepository.RequestToOrderAPI( model, idetails, test);
            Даже в джаве, блджад.
            Ответить

          • ava 3_14dar 08.08.2016 17:40 # +13

            А в сисярпе нет?
            Ответить

            • ava bormand 08.08.2016 17:56 # +14

              В сисярпе дефолты вроде есть.
              Ответить

              • ava guesto 08.08.2016 18:14 # +14

                сисярпа не существует
                существует CLR. И там это были бы разныеметоды
                Ответить

                • ava bormand 08.08.2016 18:17 # +13

                  Хм, т.е. сисярповые дефолты в CLR разваливаются на пачку методов (а не просто подставляются значения в точку вызова, как в крестах)?
                  Ответить

                  • ava 3_14dar 08.08.2016 18:17 # +14

                    Ага, на 2**N дефолтов.
                    Ответить

                    • ava bormand 08.08.2016 18:18 # +14

                      Ну там же именованных параметров нету? Так что просто на N.
                      Ответить

                    • ava dm_fomenok 09.08.2016 16:00 # +14

                      Хуй там, опциональный параметр в IL имеет аттрибут [Opt], никакого развала не происходит
                      Ответить

                  • ava 3_14dar 08.08.2016 19:22 # +14

                    Декомпилятор/дизассемблер в помощь.
                    Ответить

                  • ava Lokich 08.08.2016 22:32 # +15

                    >Хм, т.е. сисярповые дефолты в CLR разваливаются на пачку методов (а не просто подставляются значения в точку вызова, как в крестах)?
                    возможно это ответит на твои вопросыhttp://codebetter.com/2011/01/11/c-in-depth-optional-parameters-and-named-arguments-2/
                    Ответить

                    • ava bormand 08.08.2016 22:37 # +14

                      The generated IL acts exactly as if your original source code had contained the default value. This means if you ever change the default value without recompiling everything that references it, the old callers will still be using the old default value.

                      Ну короче всё-таки как в крестах: существует только одна копия метода, которая принимает все аргументы, а дефолтные значения тупо подставляются в точках вызова, как-будто их руками передали.
                      Ответить

                • ava kegdan 08.08.2016 19:34 # +15

                  >> сисярпа не существует
                  >> существует CLR.

                  Ты долбоеб? Языков программирования не существует, существует только бинарный код
                  Ответить

                • ava Shamill 09.08.2016 16:08 # +15

                  Ха, ты пернул в лужу
                  Ответить

    • ava guest 05.08.2016 21:44 # +40

      Код говно.
      ВНИМАНИЕ. ГОВНОКОДНАДЗОР ПРЕДУПРЕЖДАЕТ! ЧЕРЕЗМЕРНОЕ ЗЛОУПОТРЕБЛЕНИЕ ОДИНАКОВЫМИ ДЕЙСТВИЯМИ ПРИВОДИТ К ОЖИРЕНИЮ ДАННЫХ НА ЖЕСТКОМ ДИСКЕ!!!
      Ответить

    • ava dm_fomenok 08.08.2016 13:23 # +13

      public void RequestToOrderAPI(object model, object idetails, bool test)
      {
      if (test == true) throw new ArgumentException("test");
      ***
      }

      public void RequestToOrderAPI(object model, object idetails)
      {
      bool test = false;
      ***
      }
      Ответить

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