Ошибка в программе (forcing value to bool 'true' or 'false' (performance warning) )

@rampage4k · Регистрация: 28.02.2013

Студворк — интернет-сервис помощи студентам

.\DX9Window.cpp(86) : warning C4800: 'WORD' : forcing value to bool 'true' or 'false' (performance warning)

Участок на который студия ругается.

C++
1
2
3
4
5
6
7
8
9
10
LRESULT WINAPI MsgProc( HWND hWnd, UINT msg, WPARAM wParam, LPARAM lParam )
{
    switch( msg ) {
    case WM_PAINT:
        if (pRender != NULL) pRender(0);
        break;
 
    case WM_ACTIVATE:
        IsAppActive =(bool) (LOWORD(wParam)); // тут она ругается
        break;

Как можно исправить ошибку?

Добавлено через 1 час 14 минут
Вопрос все еще не решен (

@vbloodv · 28.02.2013, 20:12

Неявное преобразование к типу bool типа int (DWORD переопределенный), ошибки тут нет, но может быть ! Поэтому и выскакивает предупреждение. В принципе можешь так и оставить, хотя сейчас модно, правильно и объектно-ориентораванно писать static_cast<bool> вместо (bool)

Добавлено через 2 минуты
А вообще оставляй функцию обратного вызова пустой, потому что рендерить через нее это вроде плохой тон, к тому же если сообщения в стек не поступит, то MsgProc не будет вызываться, лучше в игровом цикле это сделай, но за обработкой сообщения...

Добавлено через 54 секунды

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
while(msg.message!=WM_QUIT) // Возможно имеет смысл подлючение игровых состояний
        { 
            if(PeekMessage(&msg,NULL,0,0,PM_REMOVE)) 
            {
                MainWindow.Logic(); // Клавиотурная логика, т.к. пока нет отдельного класса для клавиотуры
                Mouse.Input(); // Класс мыши который может быть не использован для дальнейшей градации
                TranslateMessage(&msg);
                DispatchMessage(&msg); //timeGetTime
            }
        
        Render.SetBackground(iBackgorund);
        Render.Visuality(); 
        
        }

Добавлено через 30 секунд
Для примера, это всевдокод

@rampage4k · 28.02.2013, 20:59 **[ТС]**

Спасибо. А так сойдет?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
//DX9Window.cpp
void RunEngine()
{
    if ( IsDX9Inited == false ) return;
 
    MSG msg;
    ZeroMemory( &msg, sizeof(msg) );
    
    while( msg.message!=WM_QUIT )
    {
        if( PeekMessage( &msg, NULL, 0U, 0U, PM_REMOVE ) )
        {
            TranslateMessage( &msg );
            DispatchMessage( &msg );
        }
        else
            if (IsAppActive) 
            {
                if (pRender != NULL) pRender(0); 
            }
    }
}
 
// main.cpp
#include "Common.h"
#include "DX9Window.h"
 
void Render(float fTime) 
{
    D3DDevice->Clear( 0, NULL, D3DCLEAR_TARGET, D3DCOLOR_XRGB(0,0,255), 1.0f, 0 );
    
    if( SUCCEEDED( D3DDevice->BeginScene() ) ) 
    {
 
 
 
 
        D3DDevice->EndScene();
    }
    
    D3DDevice->Present( NULL, NULL, NULL, NULL );
}
 
void Loading() 
{
    printf("loading game resources\n");
 
}
 
void Free() 
{
    printf("unloading game resources\n");
 
    //system("pause");
    //Sleep(2000);
};
 
int main() 
{
    InitEngine();
    Loading();
    SetRenderer(Render);
    SetFree(Free);
    RunEngine();
    return 0;
}

Добавлено через 2 минуты

C++
1
IsAppActive = static_cast<bool>(LOWORD(wParam));

Все равно ругается, может сначала к char привести и потом только к bool ?

Добавлено через 27 минут
Что за глупый варнинг( IsAppActive = static_cast<bool>(static_cast<char>(LOWO RD(wParam)));
Разве bool = char = 1 байт?

Добавлено через 3 минуты
Как вы думаете это правильно? Компилятор перестал ругаться.

C++
1
IsAppActive = static_cast<bool>(LOWORD(wParam) > 0);

@vbloodv · 01.03.2013, 13:29

Сообщение от rampage4k

Как вы думаете это правильно? Компилятор перестал ругаться.

Думаю не то чтобы это не правильно, это ВНИМАНИЕ !(как ты сказал варнинг), а не ошибка. Поэтому имеет место быть много замечаний.

Сообщение от rampage4k

while( msg.message!=WM_QUIT )
* * {
* * * * if( PeekMessage( &msg, NULL, 0U, 0U, PM_REMOVE ) )
* * * * {
* * * * * * TranslateMessage( &msg );
* * * * * * DispatchMessage( &msg );
* * * * }
* * * * else
* * * * * if (IsAppActive)
* * * * * * {
* * * * * * * * if (pRender != NULL) pRender(0);
* * * * * * }
* * }

Не совсем,

C++
1
 if( PeekMessage( &msg, NULL, 0U, 0U, PM_REMOVE ) ) // Если получено сообщение, то работает логика

А рендеринг происходит если сообщений от клава+мышь не получено, следовательно пока ты мышкой двигаешь, экран не рисуется... Задумка правильная рендерить только при фокусе, но если я правильно понимаю то зачем тогда и сообщения ввода получать.
Не уверен, но мой совет это избавься от winProc(или как она у тебя), обрабатывай через DirectXInput8 раз уж ты действительно вдумчиво делаешь. Для winProc только одну обработку сделай, это выход из программы.

C++
1
2
void Render(float fTime)
void Loading()

Функции красиво называются, но фактически ты пишешь на С, если это только для того чтобы разобраться с окном то наверное все верно, ну а вообще глянь что-нибудь про "Паттерны проектирования".Имеет смысл порыться на джитхабе и украсть чей нибудь код в целях разобраться со структурой. WinAPI моветон...

Добавлено через 12 минут

C++
1
[quote=rampage4k;4194691]0U, 0U[/quote]

Что это значит, не встречал, преобразовании к unsigned ?

@ITcrusader · 01.03.2013, 13:33

Сообщение от vbloodv

WinAPI моветон...

Ты чего такое говоришь))) кто знает чистый WinAPI - может поработить мир, используя любой другой, более высокоуровневый, framework.

@vbloodv · 01.03.2013, 13:52

Сообщение от ITcrusader

Ты чего такое говоришь))) кто знает чистый WinAPI - может поработить мир, используя любой другой, более высокоуровневый, framework.

Не хочу спорить, но в данному случае, DirectInput быстрее, а если бы это был не DirectX а GL то из-за winApi терялась бы кросплатформенностью. К тому же WinAPI это набор функций, что приводит к функциональному программированию следовательно не соответствует духу C++

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .