1. Java / Говнокод #11451

    +75

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    private void executeUiOperation(final UiOperation operation, final Boolean documentReadOnly) {
        boolean readOnly = !edit;
        if (documentReadOnly != null) {
            readOnly |= documentReadOnly;
        }
//....

    Кручу-верчу запутать хочу...

    Запостил: ingenuus, 20 Июля 2012

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

    • ava ingenuus 20.07.2012 13:46 # 0

      Причем при использовании documentReadOnly передается boolean.
      final boolean viewOnly
...
executeUiOperation(operation, viewOnly);
      Ответить

    • ava bormand 20.07.2012 14:02 # +8

      Boolean vs boolean

      Джавапроблемы ;)
      Ответить

      • ava absolut 20.07.2012 19:44 # 0

        Однажды в .net p/invoke нарвался на то, что типы bool по умолчанию маршалятся как BOOL из WinAPI, т.е. 4 байта, а нужен был однобайтовый плюсовый bool. Так что не только джавапроблемы.
        Ответить

        • ava someone 21.07.2012 18:20 # −3

          Стоп... в плюсах однобайтовы...

          Убиться, и правда, в GCC и MSVC sizeof(bool) == 1.

          Сколько живу и не знаю...
          Ответить

          • ava Lure Of Chaos 22.07.2012 00:05 # 0

            странно, по-моему везде, кроме джавы, бул однобайтен.
            Ответить

            • ava Fai 22.07.2012 12:27 # 0

              Однобайтовый медленнее же не?
              Ответить

              • ava Lure Of Chaos 22.07.2012 12:30 # +5

                жаба медленее по определению, при чем тут бул?
                Ответить

                • ava Fai 22.07.2012 12:32 # 0

                  Однобайтовый медленнее из-за архитектуры, при чём тут жаба?
                  Ответить

                  • ava Lure Of Chaos 22.07.2012 12:41 # +2

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

                  • ava bormand 22.07.2012 15:29 # +2

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

                    Однобайтовость же при размещении в памяти скажется только положительно - за счет меньшего размера структур с таким булом будет меньше промахов при обращениях к кешу.
                    Ответить

                    • ava USB 23.07.2012 00:42 # −1

                      >Однобайтовость же при размещении в памяти скажется только положительно - за счет меньшего размера структур с таким булом будет меньше промахов при обращениях к кешу.

                      #pragma pack 1
struct{
bool a;
int b;

                      В данном случае b соснет. Обращение к нему будет не выравненным и может печально закончится. Так что однобафтовый bull плохо. В любом случае это медленнее.
                      Ответить

                      • ava bormand 23.07.2012 05:29 # +1

                        Соснет b... но, как не странно, не сам бул...

                        И раз тут применили #pragma pack, то был вполне определенный протокол, в котором было написано 1 байт того-то и 4 байта сего-то. Поэтому все претензии не к однобайтовому булу, а к автору протокола.

                        P.S. Не надоело еще грызть кактусы и реализовывать сериализацию накладыванием структур? Ну я понимаю в embedded это актуально, и, возможно, в движках СУБД где надо быстро сохранять\загружать данные в непереносимом на другие машины формате (но там #pragma pack можно с чистой совестью выкинуть).
                        Ответить

            • ava 3.14159265 23.07.2012 16:01 # +1

              Странно, но в бейсиках (QB, VB) от MS (до .NET) - бул если и был, то был двухбайтным.
              Логикой это было никак не объяснить.

              А суть была вот в чем - false=0, а вот true= -1.
              Бейсиковский Byte был unsigned, то есть 0..255. Потому использовался 2-байтный Integer, в который можно было записать -1.
              Вот такое вот говно.
              Ответить

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