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

    +63.6

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    D3DXINLINE
D3DXVECTOR2::D3DXVECTOR2( CONST FLOAT *pf )
{
#ifdef D3DX_DEBUG
    if(!pf)
        return;
#endif

    x = pf[0];
    y = pf[1];
}

D3DXINLINE
D3DXVECTOR2::D3DXVECTOR2( CONST D3DXFLOAT16 *pf )
{
#ifdef D3DX_DEBUG
    if(!pf)
        return;
#endif

    D3DXFloat16To32Array(&x, pf, 2);
}

D3DXINLINE BOOL
D3DXVECTOR3::operator == ( CONST D3DXVECTOR3& v ) const
{
    return x == v.x && y == v.y && z == v.z;
}

D3DXINLINE BOOL
D3DXVECTOR3::operator != ( CONST D3DXVECTOR3& v ) const
{
    return x != v.x || y != v.y || z != v.z;
}

    Во-первых, классный "режим отладки". Если D3DX_DEBUG определён, то при нулевом указателе pf происходит тихий возврат. Совершенно, так сказать, по-английски. А вот анализ второй функции показывает, что данный исходник вообще не получится собрать с D3DX_DEBUG. Наглядная иллюстрация вреда тупого копипаста. Определения операторов == и != демонстрируют, как не надо сравнивать вещественные числа.
    Кто же автор этого безобразия? Имя не известно, зато известно место работы. Ибо этот "код" полностью представлен в файле d3dx9math.inl от компании Microsoft.

    Запостил: whiskey, 22 Марта 2010

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

    • ava SkaarjZR 22.03.2010 12:09 # +0.4

      Объясните, пожалуста, почему в Windows коде пишут такими большими буквами?
      Ответить

      • ava guest 22.03.2010 13:27 # +0.6

        Большими буквами пишут макросы. А наши мягкие друзья их очень любят.
        Ответить

        • ava SkaarjZR 22.03.2010 14:39 # +0.4

          Макросы, это понятно, но неужели у них так весь код написан?
          Читать же не удобно.
          Ответить

          • ava guest 22.03.2010 15:01 # +0.6

            у майкрософта имена типов пишутся большими буквами.

            макросов в коде как бы и нет
            Ответить

        • ava guest 22.03.2010 14:44 # 0

          мелкие любят венгерскую нотацию, от которой аж блевать хочется
          Ответить

          • ava guest 25.03.2010 03:36 # 0

            _системную_ венгерскую нотацию. Есть большая разница.
            Ответить

    • ava guest 22.03.2010 12:52 # +0.6

      краска для валос у них светлое слишком
      Ответить

      • ava guest 22.03.2010 14:26 # +0.6

        гэнать не надо, краска в индий разный очень на цвет, вкус и запах
        Ответить

    • ava guest 22.03.2010 15:04 # −0.4

      вот поэтому я юзаю опенгл всегда.

      его очень удобно инкапсулировать в какие угодно архитектуры.
      а директ3д - кривое негибкое месиво

      жалко, что производители видеокарты щас в тандеме с МС...
      Ответить

    • ava guest 22.03.2010 23:55 # 0

      Да плевать, кроме самих мс этими функциями никто не пользуется.
      Ответить

    • ava guest 23.03.2010 18:36 # +0.4

      Все нормально. Мелкиесофсткие ошибки у всех есть.
      Ответить

    • ava guest 23.03.2010 18:37 # −2

      DX - тру библиотека.
      Ответить

      • ava guest 31.03.2010 08:58 # +0.4

        Иди в жопу, быдло! Ничего не понимаешь! Восслався наш истинно верный OpenGL.
        Ответить

      • ava guest 31.03.2010 08:58 # +0.2

        DX - это CloseGL.
        Ответить

    • ava guest 31.03.2010 08:59 # +0.4

      Это библиотека только для куллхацкеров!
      Ответить

    • ava guest 01.04.2010 00:04 # +0.2

      Значит при определении D3DX_DEBUG туда разрешено передавать NULL, и оно не меняет координаты - логично.
      И почему исходник не получится собрать с D3DX_DEBUG, поясните пожалуйста.
      Сравнение вещественных чисел на точное совпадение - вполне допустимая операция, будет определять копии.
      Ответить

      • ava whiskey 01.04.2010 07:25 # 0

        Виноват, вкралась ошибка. Вот то, из-за чего не будет собираться:

        D3DXINLINE
        D3DXVECTOR2::D3DXVECTOR2( FLOAT fx, FLOAT fy )
        {
        #ifdef D3DX_DEBUG
        if(!pf)
        return;
        #endif

        x = fx;
        y = fy;
        }
        Ответить

        • ava guest 02.04.2010 21:56 # 0

          Да, косячок-с... Значит D3DX_DEBUG никогда не определялся и вреда не приносил :). Бдите!
          Ответить

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