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

    +139

    1. 1
    2. 2
    3. 3
    4. 4
    inline Gdiplus::Color colorrefToGdiColor(COLORREF col, char alpha)
{
	return (static_cast<unsigned long>(static_cast<unsigned char>((col & Gdiplus::Color::RedMask) >> Gdiplus::Color::RedShift)) <<   Gdiplus::Color::BlueShift) | (static_cast<unsigned long>(static_cast<unsigned char>((col & Gdiplus::Color::GreenMask) >> Gdiplus::Color::GreenShift)) << Gdiplus::Color::GreenShift) | (static_cast<unsigned long>(static_cast<unsigned char>((col & Gdiplus::Color::BlueMask) >> Gdiplus::Color::BlueShift)) << Gdiplus::Color::RedShift) | (static_cast<unsigned long>(alpha) << Gdiplus::Color::AlphaShift);
}

    Тихо себя ненавижу. Слава Б-г'у это всё выкидывается.

    Запостил: Altravert, 23 Июля 2010

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

    • ava Altravert 23.07.2010 05:58 # +3

      Ой бляттть, это ж плюсы. Позор мне.
      Ответить

    • ava AxisPod 23.07.2010 06:52 # 0

      Хы, а причем БГ то тут, он менеджер, а не программист. Да и когда тысячи людей делают одно, подобные казусы будут.
      Ответить

    • ava absolut 23.07.2010 07:15 # 0

      BlueShift ... сразу вспомнил про Half-Life
      Ответить

      • ava nil 23.07.2010 08:24 # 0

        Shift:

        2 a chiefly dialect : a change of clothes b (1) chiefly dialect : shirt (2) : a woman's slip or chemise (3) : a usually loose-fitting or semifitted dress

        И еще маски какие-то, чем они там вообще занимаются?
        Ответить

        • ava Altravert 23.07.2010 08:27 # 0

          Там цвет хранится одной цифрой типа 0xAARRGGBB. Маски соответственно

          enum
          {
          AlphaShift = 24,
          RedShift = 16,
          GreenShift = 8,
          BlueShift = 0
          };

          enum
          {
          AlphaMask = 0xff000000,
          RedMask = 0x00ff0000,
          GreenMask = 0x0000ff00,
          BlueMask = 0x000000ff
          };

          Также там есть методы типа
          BYTE GetGreen() const { return (BYTE) (Argb >> GreenShift); }
          на которые студия естественно громко матерится (типа ты че, баран, большие типы режешь).
          Убожество в общем.
          Ответить

      • ava Говногость 23.07.2010 13:12 # +1

        >BlueShit
        Что-же такое нужно сьесть, что-бы его увидеть...
        Ответить

    • ava nil 23.07.2010 08:27 # −1

      Вот в нашем Си такого не было.
      А что, #define уже отменили? Стандартная практика же:

      #define blabla (expression)
void func()
{
// Use it
}
#undef blabla
      Ответить

      • ava Altravert 23.07.2010 08:35 # 0

        Дефайны я не люблю. Использую, конечно, но стараюсь обходиться. Ну как-то отпадает желание, когда в WinGDI.h видишь
        #define RGB(r,g,b) ((COLORREF)(((BYTE)(r)|((WORD)((BYTE)(g) )<<8))|(((DWORD)(BYTE)(b))<<16)))
        Ответить

      • ava Altravert 23.07.2010 08:37 # −1

        А что дает это? Что за expression? И зачем undef?
        Ответить

        • ava nil 23.07.2010 08:43 # +1

          Дает, что можно немного сократить код, поскольку у нас тут три с половиной одинаковых куска.
          expression в данном случае будет что-то вроде того куска со статик кастами. Параметром будут слова с масками и шифтами.

          андеф чтобы прибрать за собой. На всякий случай (особенно важно в заголовочных фйайлах). В большинстве случаев не укусит, но, имхо, все равно стóит.
          Ответить

    • ava govnokoder777 23.07.2010 12:58 # 0

      Вообще, у Gdiplus::Color есть конструктор принимающий 4 BYTE
      статик касты можно смело выкидывать)

      inline Gdiplus::Color COLORREFToColor(COLORREF clr, int alpha = 255)
      {
      return Gdiplus::Color(alpha, GetRValue(clr), GetGValue(clr), GetBValue(clr));
      }
      Ответить

      • ava Altravert 23.07.2010 16:22 # 0

        Я в курсе. Но -- но -- но -- ноубадикэрс) Не для гдишного цвета это должно было быть, а макросы я не помню почему отбросил.
        Черт, это было очень давно и мне давно очень стыдно)
        Ответить

    • ava Bjarne_Stroustrup 23.07.2010 15:44 # 0

      Мне одному название COLORREF кажется идиотским?
      Ответить

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