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

    +178

    1. 1
    e.voiceMail = !!e.voiceMail;

    поле voiceMail имеет тип bool. Привет из солнечного кетая

    Запостил: dzagy, 25 Февраля 2011

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

    • ava absolut 25.02.2011 15:21 # +5

      Обычно такой прием используется для приведения int'а к 0/1.
      Ответить

    • ava Dummy00001 25.02.2011 17:19 # +1

      это было нужно на старых компилерах, которые не давали гарантии что true == bool(1).
      или если bool меняется каким другим (кривым) способом.

      меня тоже как то народ на современный уровень вытянул, тыкнув в С++ стандарт где черным по белому написано что бул переменную можно сравнивать с true. тоже самое применимо к Cшному типу _Bool.
      Ответить

      • ava gegMOPO4 26.02.2011 10:21 # +1

        А вот не нужно булевскую переменную сравнивать с true.
        Ответить

        • ava absolut 26.02.2011 10:50 # 0

          А с false?
          Ответить

          • ava gegMOPO4 26.02.2011 11:33 # 0

            В PHP -- можно (иногда даже нужно). Делайте выводы.
            Ответить

        • ava Dummy00001 26.02.2011 12:48 # 0

          я лично протестировал кучу (Юниховых) компилеров: можно сравнивать с true.
          Ответить

          • ava absolut 26.02.2011 13:02 # +1

            Я думаю, gegMOPO4 хотел сказать, что бессмысленно писать что-то типа: if( variable == true ). Достаточно просто if( variable ).
            Ответить

            • ava Dummy00001 26.02.2011 13:19 # 0

              `== true`/`== false` в некоторых местах добавляет читабельности.
              Ответить

              • ava absolut 26.02.2011 14:22 # 0

                С ==false еще может быть. Но с ==true при удачном именовании сравниваемого значения - довольно сомнительно.
                Ответить

              • ava gegMOPO4 27.02.2011 12:58 # 0

                А if(variable == false == false) наверное вообще суперчитабельный код?
                Ответить

    • ava dzagy 25.02.2011 20:12 # 0

      Изначально поле булево и задается везде как true/false, никаких интов туда не передается, я сначала так же подумал, только потом полез глубже
      Ответить

    • ava Мартин 26.02.2011 15:18 # −4

      Разве в СПП не "->" вместо "."?
      Ответить

      • ava absolut 26.02.2011 16:58 # 0

        "->" для обращения к членам объекта через указатель;
        "." - через ссылку (в частном случае сам объект)
        Ответить

        • ava ctm 01.03.2011 08:28 # +2

          еще "->" - оператор, который можно перегрузить (например в смарт-поинтерах http://www.firststeps.ru/theory/c/r.php?27) - в этом случае будет вызов метода класса operator->.
          "." это доступ к полям у самого объекта - с ним ничего не сделаешь - это именно то, что написано.
          Ответить

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