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

    +152

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if (getMaterialDom().ztest_)
 {
  device.SetRenderState(D3DRS_ZENABLE, TRUE);
  device.SetRenderState(D3DRS_ZFUNC, D3DCMP_LESSEQUAL);
 }
 else
  device.SetRenderState(D3DRS_ZENABLE, FALSE);

if (getMaterialDom().zwrite_)
	device.SetRenderState(D3DRS_ZWRITEENABLE, TRUE);
else
	device.SetRenderState(D3DRS_ZWRITEENABLE, FALSE);

    Реализация параметров материала z-test и z-write в 3D-движке.

    Запостил: Kirinyale, 04 Августа 2011

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

    • ava guest 04.08.2011 19:44 # −1

      что не так?
      Ответить

      • ava absolut 04.08.2011 20:53 # +2

        По крайней мере 9-12 можно превратить в одну.
        И, кстати, почему TRUE/FALSE, а не ZTRUE/ZFALSE true/false ?
        Ответить

        • ava guest 04.08.2011 21:01 # −1

          мой союзник тупит
          const auto material=getMaterialDom();
const bool zt=material.ztest_;
device.SetRenderState(D3DRS_ZENABLE, zt);
if(zt) device.SetRenderState(D3DRS_ZFUNC, D3DCMP_LESSEQUAL);
device.SetRenderState(D3DRS_ZWRITEENABLE, material.zwrite_);
          Ответить

          • ava SmackMyBitchUp 04.08.2011 22:22 # 0

            а почему не сразу const bool zt = getMaterialDom().ztest?
            Ответить

            • ava bugmenot 04.08.2011 22:24 # +3

              explaining var головного мозга
              Ответить

            • ava guest 05.08.2011 07:44 # 0

              Я написал так, чтобы когда придёт тарас - мне не пришлось бы оправдываться.
              Его дельфи неумеет elimination expression, поэтому, как он говорит: "Всегда нужно кешировать в переменные".
              Ответить

          • ava guest 05.08.2011 21:13 # −1

            >мой союзник тупит
            гусь свинье не товарищ
            Ответить

          • ava guest 08.08.2011 15:14 # −2

            >const auto
            Разве можно const ставить перед auto в С++0х?
            Ответить

            • ava Kirinyale 09.08.2011 10:57 # 0

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

              • ava absolut 10.08.2011 10:22 # −1

                кстати, интересно, если функция возвращает const int&, и ее результат присваивается переменной, объявленной как const auto, получается как бы двойная константность :)
                Ответить

        • ava guest 04.08.2011 21:05 # −1

          >почему TRUE/FALSE?
          Микрософт эту глупость со времен винапи пропагандируют до сих пор во всех своих геймдейвских примерах для DX.
          Ответить

        • ava interested 05.08.2011 14:50 # 0

          Ваш второй вопрос даёт ясно понять, что 9-12 объединить нельзя.
          Что такое TRUE и FALSE по выдранному постером куску кода нам не ясно. Это могут быть константы неизвестного типа, это могут быть ключевые слова перечисления и даже макросы. Кажется, что последнее наиболее вероятно и за TRUE/FALSE кроются страшные, "изпальцавысосанные" целочисленные коды.
          Ответить

          • ava absolut 05.08.2011 15:32 # +3

            а теперь вспоминаем про тернистый оператор: 
            device.SetRenderState(D3DRS_ZWRITEENABLE, getMaterialDom().zwrite_ ? TRUE : FALSE);
            Ответить

          • ava guest 05.08.2011 21:11 # −1

            да, в winbase.h TRUE и FALSE определены, как int :(
            но SetRenderState принимает BOOL, что определён как int, да :)
            Из этого вывод: :(

            А сделали они это для поддержки WIN API в pure Си.

            Я слышал, что DX можно программировать под pure Си без С++ враперов. Кто-нибудь пробовал?
            Ответить

        • ava RaZeR 05.08.2011 14:54 # 0

          01-07 также можно сократить:

          device.SetRenderState(D3DRS_ZENABLE, getMaterialDom().ztest_);
          if (getMaterialDom().ztest_) device.SetRenderState(D3DRS_ZFUNC, D3DCMP_LESSEQUAL);
          Ответить

        • ava gegMOPO4 06.08.2011 21:58 # 0

          1-7 объясняют, почему это делать не стоит.
          Ответить

      • ava Kirinyale 08.08.2011 11:45 # +1

        z-write без z-test работать не будет. Потому что D3DRS_ZENABLE при выключении убивает вообще всё, связанное с буфером глубины.
        Вместо 07 нужно было:
        device.SetRenderState(D3DRS_ZFUNC, D3DCMP_ALWAYS);

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

    • ava guest 05.08.2011 21:24 # −1

      Что-за игра то?
      Ответить

      • ava eth0 05.08.2011 21:32 # +1

        Домики деревянные, эльфы набигают, я уже джва года жду такую игру. :(
        Можно грабить корованы.
        Ответить

        • ava guest 05.08.2011 21:35 # −1

          грабить корованы можно?
          .
████████████████████
░░░░░░░░░░░░░░░█
░░███░░░░░░░███░█
░█░░░█░░░░░█░░░█░█
░░░░░░░░░░░░░░░░░█
░░███░░░░░░░███░░█
░█░░░█░░░░░█░░░██
█░░░███░░░█░░░███
█░░████░░░█░░████
█░░░███░░░█░░░███
░█░░░█░░░░░█████
░░███░░░░░█░░░░░██
░░░░░░░░░░░░░░░░░█ █
░░██░░░░██░░░░░░░░███
░░░████████░░░░░░░███
░░░░████████░░░░░███
░░░░░░████████████
░░░░░░░░░░░░░░░█
░░░░░░░░░░░████
░░░░░░░░░░░░█
░░░░░░░░░░░█
░░░░░░░
░░░░░░░░░▄▄▄▄
          Ответить

      • ava Kirinyale 08.08.2011 11:47 # 0

        очередное HO/Adventure для бигфиша
        Ответить

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