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

    +145.4

    1. 1
    ViewState["Action"] = result.client == "merchantName" ? false : true;

    Запостил: nettrash, 18 Февраля 2010

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

    • ava guest 18.02.2010 19:37 # −1.2

      и чего? написать чуть длиннее чем кратчайшим способом == говнокод?
      Ответить

    • ava nettrash 18.02.2010 19:55 # 0

      считаю что да ))))
      Ответить

    • ava Dreyk 18.02.2010 19:57 # +5.2

      Бл*, да скока можно постить эти бойаны с "? true : false", "? false : true" и иже с ними? Не смешно уже
      Ответить

      • ava nettrash 18.02.2010 20:00 # −4

        Хорошо! Предложи свой вариант написания этой строки! Обсудим! ))))
        Ответить

        • ava guest 18.02.2010 20:41 # +5.4

          Кагбе
          ViewState["Action"] = result.client != "merchantName";
          было бы вполне достаточно. Да и менше путаницы при чтении кода.
          Ответить

          • ava Dummy00001 21.02.2010 19:33 # −0.4

            и все равно я иногда пишу "? true : false;" просто потому что это более читабельно.

            не часто - но в некоторых местах где куча логики мне это реально помогает.
            Ответить

    • ava pushkoff 18.02.2010 21:37 # 0

      я так понимаю в C# это просаживает производительность, из-за того что компилятор байткода не выкинет лишние команды?
      Ответить

      • ava guest 18.02.2010 21:46 # 0

        ваши познания про С диез меня пугают.
        Байткод в шарпе - рофлмао
        Ответить

        • ava pushkoff 18.02.2010 23:10 # 0

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

          • ava guest 19.02.2010 17:14 # 0

            ну меня учил рихтер что то что вы назвали байткодом является IL(Intermediate Language)
            Ответить

            • ava pushkoff 19.02.2010 17:21 # 0

              сори, не силен в терминологии...
              Ответить

            • ava guest 27.02.2010 04:10 # −1

              лол, IL это и есть байткод

              Майкрософт любит свои понятия придумывать по жизни и плевать на существующие. Не байткод, с*ка, а "промежуточный язык". Не library, с*ка, а assembly. В ООП уже было понятия делегата, но МС наплевали на это и так обозвали лямбды. ит. д.
              Ответить

              • ava guest 06.03.2010 17:27 # 0

                Делегаты там есть. А лямбда это из ФП.
                Ответить

              • ava guest 06.03.2010 18:09 # 0

                лямбды к делегатам не имеют отношения
                Ответить

      • ava guest 27.02.2010 04:04 # 0

        скомпилировал подобное в релиз, посмотрел байткод - нихрена компилятор не соптимизировал.

        мсил в итоге вышел примерно такой:

        1) сравнить
        2) если true, перейти к п. 5
        3) поставить на стек true
        4) возвратить
        5) поставить на стек false
        6) возвратить

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

    • ava guest 19.02.2010 04:57 # +3

      bool-shit
      Ответить

    • ava guest 19.02.2010 12:19 # −4.2

      !result.client.Equals("merhcantName", StringComparison.InvariantCultureIgnoreC ase);
      мне кажется так вернее
      Ответить

    • ava guest 20.02.2010 15:17 # −2.2

      наверно, так надо (с сохранением паттерна bool-shit):
      ViewState["Action"] = strcmp(result.client,"merchantName") ? true: false;
      Ответить

    • ava guest 17.03.2010 11:51 # 0

      Тут "? true : false" - обычный говнокод простительный для студентов.

      Но в шарпе можно и перегрузить оператор true:
      class RangeZeroToTen
    {

        public int Value { get;private  set; }

        public static bool operator true(RangeZeroToTen a)
        {
            return a.Value <=10;
        }
        public static bool operator false(RangeZeroToTen a)
        {
            return a.Value>10;
        }
        public void MoveNext()
        {
            Value++;
        }
    }

      ни потом юзается так

      var range = new RangeZeroToTen();
            while (range)
            {
                range.MoveNext();
            }


      Вот уж где говнокод при чем на уровне спецификации шарпа
      Ответить

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