Проблематика говноAPI (2 стр) / Общее / Форум / Флейм / GameDev.ru

monobogdan
> Не совсем понятно, что именно тебя смутило именно в самом DX.

Ну просто вместо вызова одной функции
InitMouse()
мы наблюдаем весь этот ужасный, суперперегруженный и ублюдский код пестрящий заполнением промежуточных структур.
Он весь практически на 99% не нужен.
Такое вот API.

monobogdan
Забанен

#16
12:32, 3 июля 2024

=A=L=X=
Значит вулкан/dx12 ещё более убогое API? Так-то DX9 был значительно проще и треугольник там можно было вывести за пару десятков строк с учетом создания окна, а ретейнед режим в упомянутом тобой DX6 был еще короче (кроме создания контекста DDraw).

В примере с гба не совсем ясно, какой уровень "простоты" ты ожидаешь от чего-то более комплексного типа гапи. Приведи пример идеальной архитектуры для 3D API, по твоему мнению.

entryway
Участник

#17
(Правка: 13:14) 12:54, 3 июля 2024

Из zdoom:

+ FDInputMouse::GetDevice

− Скрыть

bool FDInputMouse::GetDevice()
{
  HRESULT hr;

  if (g_pdi3 != NULL)
  { // DirectInput3 interface
    hr = g_pdi3->CreateDevice(GUID_SysMouse, (LPDIRECTINPUTDEVICE*)&Device, NULL);
  }
  else if (g_pdi != NULL)
  { // DirectInput8 interface
    hr = g_pdi->CreateDevice(GUID_SysMouse, &Device, NULL);
  }
  else
  {
    hr = -1;
  }
  if (FAILED(hr))
  {
    return false;
  }
  
  // How many buttons does this mouse have?
  DIDEVCAPS_DX3 caps = { sizeof(caps) };
  hr = Device->GetCapabilities((DIDEVCAPS *)&caps);
  // If that failed, just assume four buttons.
  if (FAILED(hr))
  {
    caps.dwButtons = 4;
  }
  // Now select the data format with enough buttons for this mouse.
  // (Can we use c_dfDIMouse2 with DirectInput3? If so, then we could just set
  // that unconditionally.)
  hr = Device->SetDataFormat(caps.dwButtons <= 4 ? &c_dfDIMouse : &c_dfDIMouse2);
  if (FAILED(hr))
  {
ufailit:
    Device->Release();
    Device = NULL;
    return false;
  }
  // Set cooperative level.
  hr = Device->SetCooperativeLevel(Window, DISCL_EXCLUSIVE | DISCL_FOREGROUND);
  if (FAILED(hr))
  {
    goto ufailit;
  }
  // Set buffer size so we can use buffered input.
  DIPROPDWORD prop;
  prop.diph.dwSize = sizeof(prop);
  prop.diph.dwHeaderSize = sizeof(prop.diph);
  prop.diph.dwObj = 0;
  prop.diph.dwHow = DIPH_DEVICE;
  prop.dwData = DINPUT_BUFFERSIZE;
  hr = Device->SetProperty(DIPROP_BUFFERSIZE, &prop.diph);
  if (FAILED(hr))
  {
    goto ufailit;
  }
  return true;
}

+ FWin32Mouse::GetDevice

+ FDInputMouse::ProcessInput

− Скрыть

void FDInputMouse::ProcessInput()
{
  DIDEVICEOBJECTDATA od;
  DWORD dwElements;
  HRESULT hr;
  int dx, dy;

  dx = 0;
  dy = 0;

  if (!Grabbed || !use_mouse)
    return;

  event_t ev = { 0 };
  for (;;)
  {
    DWORD cbObjectData = g_pdi3 ? sizeof(DIDEVICEOBJECTDATA_DX3) : sizeof(DIDEVICEOBJECTDATA);
    dwElements = 1;
    hr = Device->GetDeviceData(cbObjectData, &od, &dwElements, 0);
    if (hr == DIERR_INPUTLOST || hr == DIERR_NOTACQUIRED)
    {
      Grab();
      hr = Device->GetDeviceData(cbObjectData, &od, &dwElements, 0);
    }

    /* Unable to read data or no data available */
    if (FAILED(hr) || !dwElements)
      break;

    /* Look at the element to see what happened */
    // GCC does not like putting the DIMOFS_ macros in case statements,
    // so use ifs instead.
    if (od.dwOfs == (DWORD)DIMOFS_X)
    {
      dx += od.dwData;
    }
    else if (od.dwOfs == (DWORD)DIMOFS_Y)
    {
      dy += od.dwData;
    }
    else if (od.dwOfs == (DWORD)DIMOFS_Z)
    {
      WheelMoved(0, od.dwData);
    }
    else if (od.dwOfs >= (DWORD)DIMOFS_BUTTON0 && od.dwOfs <= (DWORD)DIMOFS_BUTTON7)
    {
      /* [RH] Mouse button events mimic keydown/up events */
      if (!GUICapture)
      {
        PostButtonEvent(od.dwOfs - DIMOFS_BUTTON0, !!(od.dwData & 0x80));
      }
    }
  }
  PostMouseMove(dx, -dy);
}

+ FWin32Mouse::ProcessInput

monobogdan
Забанен

#18
12:56, 3 июля 2024

Хехе, занимательно. Но динпут то еще и геймпады всякие поддерживает.

kipar
Участник

#19
13:16, 3 июля 2024

monobogdan
в winapi тоже есть joyGetPos

true_bump
Постоялец

#20
20:15, 3 июля 2024

=A=L=X=
>Оказалось, что все эти годы можно было сделать ясные понятные краткие и лаконичные функции...
Вероятно, избыточность кода может быть связана:
- с многозадачностью Windows, ОС постоянно озирается на 360 ºС, вдруг возникла другая задача с высоким приоритетом. Поэтому надо ОС постоянно контролировать, не потеряла ли она фокус, какое-нибудь устройство и т.д.
- с мультиплатформенностью
- разработчик заранее заложил некие возможности расширения функционала, которые пока не востребованы
- с возможностями и особенностями никроуровневого доступа
- с тем, что разработчикам DirectX было глубоко фиолетово до среднего пользователя. Типа, кому надо - выплывет, а, остальные могут сами решать свои проблемы. Лично я это понял, когда разбирал примеры DirectX.

Имбирная Ведьмочка
Забанен

#21
22:18, 3 июля 2024

Всё очень просто, DirectX и OpenGL — это апи не для написания игр, а для написания движков.
Если хотите, чтобы было легко и просто для написания игры — берите Хрюнити, Сранрил или Годо.

kipar
Участник

#22
23:43, 3 июля 2024

Имбирная Ведьмочка
> Если хотите, чтобы было легко и просто для написания игры
думаю, здесь обсуждают какое апи легче и проще для написания движков.

nes
Постоялец

#23
0:05, 4 июля 2024

Имбирная Ведьмочка
Нельзя просто так взять и оправдать говенность АПИ его предназначением для элитных разработчиков.

Имбирная Ведьмочка
Забанен

#24
(Правка: 0:21) 0:19, 4 июля 2024

kipar
> думаю, здесь обсуждают какое апи легче и проще для написания движков.
Апи для движков не предназначены для того, чтобы быть лёгкими и простыми. Они предназначены предоставить максимум возможностей системы при минимальном оверхеде.

Какая вообще претензия, конкретно, у ОПа к графическим апи? Что там нужно передавать много параметров? А ему не приходило в голову, что это может быть затем, что разные приложения будут передавать разные значения в эти параметры в зависимости от своих задач?

nes
> Нельзя просто так взять и оправдать говенность АПИ его предназначением для элитных разработчиков.
"Говёность" зависит от задачи, для которой оно применяется. Молоток плохо подходит для того, чтобы закручивать гайки — но это не значит, что молоток является "говёным гаечным ключом", это значит только то, что ты применяешь не тот инструмент не для той задачи. Т.е. проблема не в апи, а в прокладке.

nes
Постоялец

#25
0:32, 4 июля 2024

Имбирная Ведьмочка
Ну да, ну да, пример:

Имеем:

glGenTextures(...);
glBindTexture..D(...);
glTexImage..D(...);

Могло бы быть:

glCreateTexture..D(...);

Будешь дальше оправдывать очевидные косяки в говно АПИ?

MrShoor
Постоялец

#26
3:18, 4 июля 2024

Имбирная Ведьмочка
> Какая вообще претензия, конкретно, у ОПа к графическим апи?
Ну если брать OGL - то есть вполне конкретная претензия.
Глобальное состояние на каждый пук. Чтобы поменять данные в буфере надо:

glGetIntegerv(GL_ARRAY_BUFFER_BINDING, &oldbuffer);
glBindBuffer(GL_ARRAY_BUFFER, buffertochange);
glBufferData(GL_ARRAY_BUFFER,...);
glBindBuffer(GL_ARRAY_BUFFER, oldbuffer);

А если вдруг мы начем использовать VAO - не забудь код выше попатчить до:

glGetIntegerv(GL_VERTEX_ARRAY_BINDING, &oldvao);
if (oldvao) {
  glBindVertexArray(0);
  glBindBuffer(GL_ARRAY_BUFFER, buffertochange);
  glBufferData(GL_ARRAY_BUFFER,...);
  glBindVertexArray(oldvao);
} else {
  glGetIntegerv(GL_ARRAY_BUFFER_BINDING, &oldbuffer);
  glBindBuffer(GL_ARRAY_BUFFER, buffertochange);
  glBufferData(GL_ARRAY_BUFFER,...);
  glBindBuffer(GL_ARRAY_BUFFER, oldbuffer);
}

Зашибись GAPI, рекомендую!

Куча констант, которые по хорошему должны быть типизированными энамами, но они просто чиселко. При этом весь спейс засран, в результате с виду всё верно:

glTexImage2D(GL_TEXTURE_2D, 0, GL_R, 512, 512, 0,  GL_R,  GL_UNSIGNED_BYTE, mydata);

Ан нет, обосрамс.
Кажется верно:

glDrawArrays(GL_POINT, 0, numpoints);

Ан нет, обосрамс.

И вишенка на торте - это стековый glGetError! Кстати этот самый glGetError настолько печальный, что разрабы драйверов кладут на него болт, и он буквально тормозит (даже если ошибок нет и он всегда возвращает 0 - он тупо может уронить фпс в несколько раз)

MrShoor
Постоялец

#27
3:28, 4 июля 2024

А, кстати. Был один момент, когда мне нужно было похукать рендер на OGL, чтобы в определенный момент дорисовывать свою графику в чужой кадр. Так вот, я святая простота, использовал glGet функции чтобы получить текущее состояние рендера (чтобы знать пора моему коду рисоваться или нет). Так вот, всё завелось на NVidia, а на ATI - часть нужных мне glGet просто не работали, часть работала неправильно (помню точно что не работал GL_TEXTURE_BINDING_BUFFER например, и еще штуки 3-4 других glGet). В результате мне пришлось хукать glBindBuffer и всё остальное, чтобы самому запоминать состояние.
К чему я это всё? К тому, что код выше с glGetIntegerv - он более менее работает, но шаг в сторону - и аналогичный подход может запросто перестать работать.
И как после этого не называть OGL убогим апи?

=A=L=X=
Постоялец

#28
4:23, 4 июля 2024

Справедливости ради MS сами опомнились и отрекомендовали использовать DirectInput.
Например к геймпадам теперь есть XInput API и посмотрите какая это красота и правильность кода: https://learn.microsoft.com/en-us/windows/win32/xinput/functions
Никаких COM-интерфейсов - тупо восемь функций, потому что ну не нужны тут объекты и вся эта мишура, всё просто же.
По каждому случаю использования просто своя функция, краткая и лаконичная.
Никаких dwSize в структурах требующих заполнения в клиентском коде - просто структуры с полями, никакой лишней возни с ними проводить не надо.
Вот это хорошее API, а не вот этот вот ужас из первопоста.

MrShoor
Постоялец

#29
4:33, 4 июля 2024

=A=L=X=
> Например к геймпадам теперь есть XInput API и посмотрите какая это красота и правильность кода
Как человек, пощупавший этот API могу передать им лучей поноса. XInputGetState возвращает текущее состояние на устройстве. Т.е. если между двумя опросами XInputGetState пользователь успел нажать и отжать кнопку - состояние теряется.
Из-за этого говна приходится крутить сраный XInputGetState в отдельном потоке с высокой частотой опроса, складывать состояние в очередь и в игровой логике уже разгребать это состояние