FindWindow Debug/Release и ошибка "Невозможно преобразовать const char[12] в LPСWSTR"

@Helixanon · Регистрация: 19.04.2015

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

Здравствуйте. Есть кусок кода.

C++

1	HWND hWindow = FindWindow(0, "WindowName");

В Debug компилится, в Release выбивает ошибку "невозможно преобразовать const char[12] в LPСWSTR".

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

ValeryS · 19.04.2015, 06:01

можно использовать макрос _T

C++

1	_T("text")

тогда компилятор сам выберет как кодировать строку, обыкновенными символами или широкими
для выделения памяти использовать TCHAR

C++

1	TCHAR str[]=_T("text");

Убежденный · 19.04.2015, 11:08

C++

1	FindWindowW(NULL, L"WindowName");

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

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

Проблема решилась включением поддержки многобайтовой кодировки и в релизе тоже.

Она не решилась, а замаскировалась.
Многобайтовые кодировки - прошлый век, давно пора их выкинуть на
свалку и использовать только Unicode.

@Helixanon · 19.04.2015, 11:21 **[ТС]**

Я перепробовал много вариантов решения до включения многобайтовой кодировки в релизе, и макрос _T в том числе (подключив tchar.h). Программа выдаёт неверные результаты.
Насчёт юникода, вы предлагаете вместо многобайтовой кодировки использовать L перед названием окна, я такой вариант уже пробовал. Неверные результаты.

Может быть, проблема в следующих строках кода?

C++

1
2
3

HWND hWindow = FindWindow(0, "WindowName");
DWORD procId;
GetWindowThreadProcessId(hWindow, &procId);

Убежденный · 19.04.2015, 11:30

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

Насчёт юникода, вы предлагаете вместо многобайтовой кодировки использовать L перед названием окна, я такой вариант уже пробовал. Неверные результаты.

Вы для начала проверьте, что возвращает FindWindow и какой GetLastError в
случае ее ошибки.

@Helixanon · 19.04.2015, 11:42 **[ТС]**

IDE - Visual C++ 2013 Express.
Момент, сейчас чекну.

GetLastError() выдал 0, т.е. дело уже не в поиске окна. Виноват GetWindowThreadProcessId?

Убежденный · 19.04.2015, 11:44

Так FindWindow возвращает ненулевое значение (т.е. успех), я правильно понял ?

@Helixanon · 19.04.2015, 11:47 **[ТС]**

Да, верно, что-то не так именно с этими функциями, ибо ReadProcessMemory несколькими строками ниже уже выдаёт фигню (но работает правильно при многобайтовой кодировке).
Да, правильно. Значит, результат неправильно обрабатывается в следующих строках:

C++

HWND hWindow = FindWindow(0, L"WindowName");
DWORD procId;
GetWindowThreadProcessId(hWindow, &procId);
HANDLE procHandle = OpenProcess(PROCESS_ALL_ACCESS, 0, procId);

Убежденный · 19.04.2015, 11:50

А это потому, что надо всегда проверять, что возвращают функции, а
не писать портянку вызовов без всяких проверок... Вот и здесь тоже:
где гарантия, что OpenProcess вообще вернет успех ?

@Helixanon · 19.04.2015, 11:54 **[ТС]**

Ну, если я правильно пишу, то этот код возвращает мне 0.

C++

int error = 0;
HWND hWindow = FindWindow(0, L"WindowName");
DWORD procId;
GetWindowThreadProcessId(hWindow, &procId);
HANDLE procHandle = OpenProcess(PROCESS_ALL_ACCESS, 0, procId);
error = GetLastError();
std::cout << error;
system("pause");

Убежденный · 19.04.2015, 12:00

Это тоже неправильно. GetLastError, если только в документации не
сказано обратное, имеет смысл звать только в том случае, если
функция вернула ошибку. В остальных случаях его состояние
не определено.

@Helixanon · 19.04.2015, 12:03 **[ТС]**

Ну т.е. если возвращает 0, то всё нормально и функция получает хендл. Тогда в чём может быть проблема? Ведь если использовать многобайтовую кодировку, то всё работает правильно.
P.S. Эта программа - мой первый опыт с WinAPI, так что я ещё много чего не знаю, но стараюсь.

Убежденный · 19.04.2015, 12:06

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

Тогда в чём может быть проблема ?

А что именно не работает ? В первом сообщении вообще писалось
про ошибки компиляции... Да, и показывайте уже весь проблемный код,
без отрыва от контекста, так сказать.

@Helixanon · 19.04.2015, 12:12 **[ТС]**

Ну, просто это та часть кода, которая вызывает проблемы. Именно эти строки. Я уверен, ведь следующие строки выдают не то только в случае использования Юникода. Ну вот, например. Ошибка уже на строке ReadProcessMemory (т.е. дальше неё смысла идти нет, ошибка либо в ней, либо до неё):

C++

HWND hWindow = FindWindow(0, L"WindowName");
DWORD procId;
GetWindowThreadProcessId(hWindow, &procId);
HANDLE procHandle = OpenProcess(PROCESS_ALL_ACCESS, 0, procId);
ReadProcessMemory(procHandle, (LPVOID)(0x1234, &value, sizeof(value), 0);

Я пытаюсь заменить всё на общий случай, т.е. там, конечно, не 0x1234. При том, что в многобайтовой кодировке всё работает. Проблема ДО ReadProcessMemory.

И верно, не компилируется, если убрать L до "WindowName". Если приписать, то компилируется, но неправильные результаты. Если врубить многобайтовую кодировку, то компилится и правильные результаты. Но, по вашим словам, многобайтовая кодировка - прошлый век, вот я и пытаюсь найти решение проблемы с юникодом.

Убежденный · 19.04.2015, 12:18

C++

HWND hWindow = FindWindow(0, L"WindowName");
DWORD procId;
GetWindowThreadProcessId(hWindow, &procId);
HANDLE procHandle = OpenProcess(PROCESS_ALL_ACCESS, 0, procId);
ReadProcessMemory(procHandle, (LPVOID)(0x1234, &value, sizeof(value), 0);

За код, написанный в подобном стиле, надо карать. Вилами.
Вот более контролируемый вариант:

C++

HWND hWindow = FindWindowW(NULL, L"WindowName");
 
if (NULL == hWindow)
{
    // Обрабатываем ошибку (+GetLastError).
    return false;
}
 
DWORD ProcessId;
DWORD ThreadId = GetWindowThreadProcessId(hWindow, &ProcessId);
 
if (0 == ThreadId)
{
    // Обрабатываем ошибку.
    return false;
}
 
HANDLE hProcess = OpenProcess(PROCESS_ALL_ACCESS, NULL, ProcessId);
 
if (NULL == hProcess)
{
    // Обрабатываем ошибку (+GetLastError).
    return false;
}
 
if (FALSE == ReadProcessMemory(hProcess, (LPVOID)0x1234, &value, sizeof (value), 0))
{
    // Обрабатываем ошибку (+GetLastError).
    CloseHandle(hProcess);
    return false;
}
 
CloseHandle(hProcess).
 
//
// Тут работаем с полученным буфером.
//
 
return true;

@Helixanon · 19.04.2015, 12:34 **[ТС]**

В каждом if поставил по std::cout и system("pause"), программа не остановилась ни на одном. Программа всё получает правильно, да только вот ReadProcessMemory уже выдаёт не те результаты, что нужно. Неверный хендл, значит.
Я в упор не понимаю, почему в юникоде не работает, а с многобайтовой работает (верные значения). 80% что ошибка в FindWindow, но...

Убежденный · 19.04.2015, 16:25

Чудес не бывает.
Если FindWindow вернула ненулевое значение, значит, хэндл окна получен.
То же самое и с OpenProcess. И с ReadProcessMemory. Ищите ошибку где-то в
логике работы программы, а не в multibyte/unicode.

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.04.2015, 16:25	17
	Чудес не бывает. Если FindWindow вернула ненулевое значение, значит, хэндл окна получен. То же самое и с OpenProcess. И с ReadProcessMemory. Ищите ошибку где-то в логике работы программы, а не в multibyte/unicode. 0

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.04.2015, 11:30	5
	Сообщение от Helixanon Насчёт юникода, вы предлагаете вместо многобайтовой кодировки использовать L перед названием окна, я такой вариант уже пробовал. Неверные результаты. Вы для начала проверьте, что возвращает FindWindow и какой GetLastError в случае ее ошибки. 0

@Helixanon 0 / 0 / 0 Регистрация: 19.04.2015 Сообщений: 12
	19.04.2015, 11:42 [ТС]	6
	IDE - Visual C++ 2013 Express. Момент, сейчас чекну. GetLastError() выдал 0, т.е. дело уже не в поиске окна. Виноват GetWindowThreadProcessId? 0

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.04.2015, 11:44	7
	Так FindWindow возвращает ненулевое значение (т.е. успех), я правильно понял ? 0

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.04.2015, 11:50	9
	А это потому, что надо всегда проверять, что возвращают функции, а не писать портянку вызовов без всяких проверок... Вот и здесь тоже: где гарантия, что OpenProcess вообще вернет успех ? 0

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.04.2015, 12:00	11
	Это тоже неправильно. GetLastError, если только в документации не сказано обратное, имеет смысл звать только в том случае, если функция вернула ошибку. В остальных случаях его состояние не определено. 0

@Helixanon 0 / 0 / 0 Регистрация: 19.04.2015 Сообщений: 12
	19.04.2015, 12:03 [ТС]	12
	Ну т.е. если возвращает 0, то всё нормально и функция получает хендл. Тогда в чём может быть проблема? Ведь если использовать многобайтовую кодировку, то всё работает правильно. P.S. Эта программа - мой первый опыт с WinAPI, так что я ещё много чего не знаю, но стараюсь. 0

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.04.2015, 12:06	13
	Сообщение от Helixanon Тогда в чём может быть проблема ? А что именно не работает ? В первом сообщении вообще писалось про ошибки компиляции... Да, и показывайте уже весь проблемный код, без отрыва от контекста, так сказать. 0

@Helixanon 0 / 0 / 0 Регистрация: 19.04.2015 Сообщений: 12
	19.04.2015, 12:34 [ТС]	16
	В каждом if поставил по std::cout и system("pause"), программа не остановилась ни на одном. Программа всё получает правильно, да только вот ReadProcessMemory уже выдаёт не те результаты, что нужно. Неверный хендл, значит. Я в упор не понимаю, почему в юникоде не работает, а с многобайтовой работает (верные значения). 80% что ошибка в FindWindow, но... 0