1. Си / Говнокод #17286

    +136

    1. 1
    2. 2
    /* To the unenlightened: This sets the 20 MSBs to 0 for sanity's sake. */
return four_bytes_to_uint32(four_byte_array) & ~(~0 << 12);

    Так приказали K&R.

    Запостил: codemonkey, 10 Декабря 2014

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

    • ava Pythoner 10.12.2014 19:24 # +2

      А почему не &0xfff?
      Ответить

      • ava codemonkey 11.12.2014 10:28 # +1

        Чтобы если/когда понадобиться 16, 20, 3.1415 бит урезать, не ебаться с вычислением константы, а заменить число сдвигов.
        Ответить

        • ava kipar 11.12.2014 13:07 # 0

          А прикольно. Я обычно (1<<12)-1 делаю, но для 16 бит мой вариант не прокатит, а этот прокатит.
          Ответить

          • ava TarasB 11.12.2014 13:32 # 0

            почему для 16 бит не прокатит?
            Ответить

            • ava kipar 11.12.2014 16:32 # 0

              хм. я уже привык что int=16 бит. на 32-битных архитектурах - для 32-х бит.
              ---
              С другой стороны, сдвиг на 32 это наверное UB, так что все равно не пойдет.
              Ответить

              • ava bormand 11.12.2014 18:37 # 0

                > сдвиг на 32 это наверное UB
                Implementation defined вроде, но не особо лучше UB'а. Где-то зануляет регистр, как и положено (вроде бы, если числа - константы, то так и будет), где-то вообще ничего не делает (barrel shifter'ы x86 и многих других процов тупо обрезают сдвиг на 5 (32-битный регистр) или 6 (64-битный регистр) бит, поэтому в 32-битном режиме a << 32 == a << 0, a << 33 == a << 1 и т.п.).
                Ответить

                • ava kipar 11.12.2014 18:45 # 0

                  >The integer promotions are performed on each of the operands. The type of the result is that of the promoted left operand. If the value of the right operand is negative or is greater than or equal to the width of the promoted left operand, the behavior is undefined

                  Так что UB, компилятор может смело форматировать диск.
                  Ответить

                  • ava bormand 11.12.2014 21:12 # 0

                    > компилятор может смело форматировать диск
                    Надо написать такой компилятор для отучения от UB'ов.
                    Ответить

                    • ava 3.14159265 11.12.2014 22:20 # 0

                      Кто им будет пользоваться?
                      План должен быть чуть сложнее:
                      а) пишем компилятор, которым все хотят и будут пользоваться
                      б) в (авто!)обновлении добавляем UB, который ворует и высылает нам пароли и номера кредиток
                      Ответить

        • ava Pythoner 11.12.2014 15:16 # 0

          Ебаться?
          С вычислением?
          Да ладно, эти константы за секунду в уме просчитываются.
          Ответить

    • ava someone 10.12.2014 19:34 # +1

      > four_bytes_to_uint32

      Big-endian или little-endian?
      Ответить

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