1. C++ / Говнокод #4729

    +174

    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
    // used to set or reset batch mode
void SetIsBatch(bool bSet)
{
    if(bSet)
    {
        if(m_Chart.GetIsBatched())
            return;
        else
        {
            m_Chart.SetIsBatched(TRUE);
        }
    }
    else
    {
       if(m_Chart.GetIsBatched())
       {
           m_Chart.SetIsBatched(FALSE);
       }
       else
           return;
    }
}

    Запостил: Shumway, 24 Ноября 2010

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

    • ava striker 24.11.2010 10:53 # +2 

      void SetIsBitch(bool bSet)


      не удержался
      Ответить

      • ava andremacareno 24.11.2010 11:43 # +3

        BITCHES
        DON'T KNOW
        BOUT MY
        PROGRAM
        Ответить

      • ava absolut 24.11.2010 18:10 # 0

        striker, когда обновления ждать ? столько холиваров уже не состоялось из-за канувших в Лету ГК с рейтингом -11.
        Ответить

        • ava striker 25.11.2010 01:37 # +2

          Ох скоро скоро :)
          Ответить

        • ava andremacareno 25.11.2010 07:52 # 0

          да ладно, дождемся еще :)
          Я бы сильно удивился, если бы у страйкера отводилось бы 25 часов в день на сайт (как это происходит у Дурова с его ВКонтакте) .
          Ответить

          • ava istem 25.11.2010 08:31 # 0

            время на сайт, кажется, пропорционально сотням денег с него...
            пример товарища Павла Задова Дурова должен нас всех учить и, черт подери, вдохновлять на доблестнейший плагиат подвиг.
            Ответить

    • ava TarasB 24.11.2010 12:11 # 0

      if (bSet != (bool)m_Chart.GetIsBatched()) {
      m_Chart.SetIsBatched(bSet);
      } else {
      return;
      }
      Ответить

      • ava absolut 24.11.2010 13:43 # −1

        (bool) не нужен.
        ветка else не нужна.
        m_Chart.SetIsBatched(bSet); может быть нужно трансформировать в m_Chart.SetIsBatched(bSet?TRUE:FALSE);
        Ответить

        • ava TarasB 24.11.2010 14:06 # +1

          Нет, надо трансформировать в
          (bSet?TRUE:(bSet?TRUE:(bSet?TRUE:(bSet?T RUE:(bSet?TRUE:(bSet?TRUE:(bSet?TRUE:(bS et?TRUE:(bSet?TRUE:(bSet?TRUE:(bSet?TRUE :(bSet?TRUE:(bSet?TRUE:(bSet?TRUE:(bSet? TRUE:(bSet?TRUE:(bSet? итд
          и в конце :false))))))))))))))))))))));
          (типа ржёт и говорит "неправда, неправда")
          Ответить

          • ava absolut 24.11.2010 14:21 # 0

            Можно и так. Просто TRUE и FALSE это от лукавого. Должно быть true/false соответственно.
            Ответить

            • ava TarasB 24.11.2010 14:26 # 0

              Я не понял, ты прикалываешься, или всерьёз? После ?TRUE:FALSE я решил, что прикалываешься, но этот твой пост вроде серьёзный.
              Ответить

              • ava absolut 24.11.2010 14:42 # −1

                TRUE/FALSE - именно большими буквами ввели ещё до появления булева типа в стандарте.
                Поэтому преобразование в/из них в нормальные true/false может быть не очевидно.
                И если например SetIsBatched( arg ) принимает в качестве аргумент FALSE/TRUE (которые реально, скорее всего есть int),
                а внутри делает что-то вида: if( arg == TRUE ), то если TRUE например объявлен как #define TRUE 0xFFFFFFFF, при передаче
                в метод значение true (именно маленькими буквами) условие не выполнится.
                Понятно, что если SetIsBatched() внутри так написана, то это уже говнокод, но чтобы от этого абстрагироваться нужно явно привести true/false к TRUE/FALSE.
                Ответить

                • ava TarasB 24.11.2010 14:50 # 0

                  Городить велосипеды ради говнокодеров? Нафиг надо, чё я, микрософт штоле?
                  Ответить

                  • ava absolut 24.11.2010 14:55 # 0

                    Велосипед тут не причем. Если SetIsBatched работает с TRUE/FALSE, то и передавать в нее надо TRUE/FALSE. Иначе можно нарваться на неожиданное поведение. Или переписать под bool тип и не париться в дальнейшем. Всё зависит от того насколько много раз придется ее использовать снова и сколько и где она используется уже.
                    Ответить

        • ava TarasB 24.11.2010 14:08 # 0

          Насчёт bool не нужен - если GetIsBathed не булевый, а целый, то приведётся к булеву?
          Ветку "иначе" я добавил, если после там ещё что-то есть, сюда же ГК урезанные часто выкладывают.
          Ответить

          • ava absolut 24.11.2010 14:25 # 0

            Если целый, то привести к bool придется. Твоя правда.
            На счет return; - функция из ГК выглядит законченной (фигурная скобка на 22 строке), поэтому нет необходимости в дополнительном return;
            Ответить

            • ava TarasB 24.11.2010 14:51 # 0

              > функция из ГК выглядит законченной

              Это ничего не значит, возможно, из неё вырезали то, что не относится к ГК.
              Ответить

              • ava absolut 24.11.2010 14:58 # 0

                >возможно, из неё вырезали то, что не относится к ГК.
                Возможно. Но я говорю о том, что вижу, а не о том, что возможно.
                Ответить

    • ava Pauchok-Anaynckiy 24.11.2010 12:19 # 0

      полифонические вариации на тему:
      if (result == true) return true; else return false;

      поржал! )))))) день удался ))))
      Ответить

      • ava ctm 25.11.2010 07:48 # 0

        ага, в довольно запущенном виде только%)
        в коде есть еще else return - в том коде, что представлен тоже не нужно
        Ответить

        • ava Pauchok-Anaynckiy 25.11.2010 11:15 # 0

          "в том коде, что представлен тоже не нужно" - это в смысле в той строчке, которую я показал как тему для полифионичесчких вариаций?

          конечно не нужно )))))) эта строчка же сама ГК! и прародитель все х ГК подобного рода )))) она здесь уже давно поселилась под номером 56
          Ответить

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