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

    +145

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    // Отправная точка новых изысканий
uint16_t Mask = 0x0000;
uint8_t i = 0;
for(i=0; i<255; i++)
	Mask |= (1 << i);

ResponseBuf = (VirtualMemory & (Mask << RequestedAddr)) >> RequestedAddr;

    Из кода firmware одного девайса.

    Запостил: Sushev, 14 Июня 2015

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

    • ava Sushev 14.06.2015 11:35 # +3

      +1, если цикл понравился.
      Ответить

      • ava inkanus-gray 14.06.2015 12:43 # +2

        Я джва года ждал такой цикл!
        Ответить

        • ava laMer007 15.06.2015 14:37 # +1

          Отличная идея. Пойду заведу проект на кикстартере.
          Ответить

          • ava inkanus-gray 15.06.2015 19:24 # +1

            А цикл-то на самом деле нетривиальный. Гэцэцэ отказывается его оптимизировать даже с параметром -O3. Шланг пытается использовать SSE, но генерирует всё тот же цикл.
            Ответить

            • ava Sushev 16.06.2015 17:19 # 0

              Интересное наблюдение, сейчас проверил. Кстати, так и не понял до сих пор, для чего автор этот код породил.
              Ответить

          • ava Alina_Poksenova 04.12.2022 00:45 # 0

            Ну как, завёл?
            Ответить

        • ava guest 16.06.2015 02:40 # 0

          Давно хочу такой цикл чтобы было можно корованы
          Ответить

    • ava Elvenfighter 14.06.2015 12:02 # +1

      Зато надёжно! И можно тип маски менять вплоть до 256 бит
      Ответить

      • ava inkanus-gray 14.06.2015 12:44 # +2

        Нельзя. Тогда цикл старший бит не заполнит (потому что там знак меньше вместо меньше или равно). Так что только до 255 бит.
        Ответить

    • ava Sushev 14.06.2015 12:50 # 0

      А комментарий в начале кода, по видимому, является троллингом (
      Ответить

    • ava inkanus-gray 14.06.2015 15:13 # +2 

      uint16_t i = 0;
for(i=0; i<=65535; i++) {
	Mask |= i;
	if (!~Mask) break;
}
      Ответить

    • ava TarasB 15.06.2015 15:26 # 0

      Оптимизация:
      uint16_t Mask = 0x0001; // инициализируем единичкой
uint16_t i = 0;
for(i=0; i<=65535; i++) {
	Mask |= ((1<<(1<<i))-1)<<(1<<i);
	if (!~Mask) break;
}
      Ответить

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