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

    +159

    1. 1
    const TWindow window(TWindowKindController::TWindowID WindowName, bool& isMainWindow=*(bool*)NULL)

    Крупный проЭкт.
    Мне кажется или с isMainWindow что-то не чисто?

    Запостил: Говногость, 14 Марта 2012

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

    • ava Говногость 14.03.2012 10:28 # 0

      Говно справа.
      Ответить

    • ava Steve_Brown 14.03.2012 13:05 # −1

      А почему бы и нет, кстати? Необязательный [out] параметр.
      Другой вопрос, что указателем все-таки, наверное, было лучше - при вызове придется брать адрес, но амперсанд только добавит наглядности.
      Ответить

      • ava absolut 14.03.2012 13:43 # +1

        Можно бы и иерархию классов окон построить нормальную. А не заниматься извращениями.
        Ответить

      • ava Kirinyale 14.03.2012 18:19 # +2

        Можно ещё boost::optional<bool&> тогда уже. Немного оверхеда, зато работать потом - один в один (кроме неявных преобразований, конечно) как с указателем, только при попытке разыменовать NULL (точнее, boost::none) будет ассерт.

        А конкретно за NULL в ссылке - руки отрывать и обратно в задний проход засовывать. С проворотом до щелчка. Т.к. у нормальных людей функция, принимающая ссылку, как раз и символизирует, что этот аргумент обязателен, и неинициализированным не должен быть никогда.

        А то так можно докатиться и до кодстайла с обязательными проверками ссылок на NULL...
        Ответить

        • ava Говногость 14.03.2012 19:08 # 0

          >А то так можно докатиться и до кодстайла с обязательными проверками ссылок на NULL...
          У нас в этом проекте так и есть. :(
          Ответить

          • ava Kirinyale 14.03.2012 19:11 # +1

            Виновника уже заставили сделать харакири тупым ножом?
            Ответить

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