Как пользоваться функцией EnumWindows?

Question

[Limons]

Вот код:

static BOOL CALLBACK enumWindowCallback(HWND func_hWnd, LPARAM lparam) {
  int length = GetWindowTextLength(func_hWnd);

  TCHAR* buffer;
  char* buffer = new char[length + 1];
  GetWindowText(hWnd, buffer, length + 1);
  string windowTitle(buffer);


  if (length != 0) {
    std::cout << func_hWnd << ":  " << windowTitle << std::endl;
  }
  return TRUE;
}

Что не так?

Comments

[Александр Ананьев]

Зачем два раза определена переменная buffer?

[Евгений Шатунов]

Limons , обрати внимание на П3.8 регламента работы сервиса.
Код в скриншотах запрещено выкладывать.

[Limons]

Александр Ананьев, оу, случайно так вышло, по мере теста.

[Limons]

Евгений Шатунов, хорошо, извините..

Answer 1

[none7]

#include <Windows.h>
#include <string>
#include <iostream>

using namespace std;

static BOOL CALLBACK enumWindowCallback(HWND func_hWnd, LPARAM lparam) {
    int length = GetWindowTextLengthA(func_hWnd);

    if (length != 0) {
        string windowTitle(length, '\0');
        GetWindowTextA(func_hWnd, windowTitle.data(), length + 1); // ANSI version and func_hWnd
        std::cout << func_hWnd << ":  " << windowTitle << std::endl;
    }
    return TRUE;
}

int main() {
    SetConsoleOutputCP(GetACP());
    EnumWindows((WNDENUMPROC)enumWindowCallback, 0);
    system("pause");
    return 0;
}

Comments

[User700]

Если скомпилируется, возможно тогда и у ......Length букву A, хотя если оно длину текста, а не кол-во байт возвращает, то всё равно...
И, если будет ошибка, то
const_cast<char*>(windowTitle.data())

[none7]

User700, Если компилятор поддерживает современный C++, то скомпилируется. Насчёт LengthA это правда, но для русского и английского языка разница просто незаметна. Только на системах с языками в которых очень много символов это может стать проблемой, только вот, а есть ли у них однобайтовая кодировка?

[Limons]

none7, ваш код сработал! Но, я добавил (LPSTR) перед windowTitle.data(), т.к. ругался компилятор, но вроде как это из-за компилятора.

[none7]

Limons, Лучше приучать себя себя использовать конструкции C++, потому, что конвертация типов из Си уж очень уж гибкая. Например она может преобразовать char в char* или LPCSTR в LPWSTR. При этом данные превратятся в кашу. User700 правильно написал const_cast<char*>(windowTitle.data())Сишное преобразование в C++ это анти-паттерн программирования.

Answer 2

[User700]

Переменная buffer объявлена два раза. Делаете new без delete -- утечка памяти. Я бы этот момент реализовал наподобие такого:

string title(length, '\0');
GetWindowText(hWnd, (char*)(title.data()), length+1);

И buffer тогда не надо.
В некоторых стандартах data возвращает не конст. указатель, но проще явно привести (см. также const_cast).
Но понятно ли, что это только колбэк для функции. Вы должны вызвать саму ф-ию WinAPI, передав ей свой колбэк.

Comments

[edward_freedom]

что два минуса идущих подряд означают? --

[Limons]

delete я потерял в процессе тестов, и поста сюда.

Но понятно ли, что это только колбэк для функции. Вы должны вызвать саму ф-ию WinAPI, передав ей свой колбэк.

Не совсем понял, какая именно ф-ция winapi?
EnumWindows(); (это и так сама себя вызывает)
или?
GetWindowText();

Насчёт вашего кода.
GetWindowText();
требует не char*, а LPWSTR.

Но мне не понятно, что не так, или точнее как всё таки узнать заголовок..

static BOOL CALLBACK enumWindowCallback(HWND func_hWnd, LPARAM lparam) {
	int length = GetWindowTextLength(func_hWnd);

	string title(length, '\0');
	GetWindowText(hWnd, (LPWSTR)(title.data()), length + 1);
	string windowTitle(title);


	if (length != 0) {
		cout << func_hWnd << "\t:  " << windowTitle << endl;
	}
	return TRUE;
}

[Limons]

edward_freedom, +

[jcmvbkbc]

что два минуса идущих подряд означают? --

edward_freedom, тебе уже несколько раз объясняли.

[edward_freedom]

jcmvbkbc, что то где то там кто то сказал не считается. Обозначение в правилах русского языка нужно

[User700]

Limons, А зачем теперь два string. В title и будет заголовок. Можете поменять название переменной на windowTitle.
char* и LPSTR это должно быть одно и то же. Т.е. эти все типы это псевдонимы (typedef'ы). Так, тут есть проблема с юникодом, замечу предотвращая вопрос о том, что речь шла о LPWSTR. И еще была в первоначальном коде возможна ошибка, связанная с завершением нулем строки (но в последнем нет).
По идеи я рассчитывал, что указав именно (char*) мы как-бы проверим, что LPSTR это именно char*, т.е. указатель на байты. Если нет, то думал должна появиться ошибка типа "const char* cannot be cast to LPWSTR (aka wchar_t*)".
А по существу -- вы же в своём ответе указали вызов ф-ии WinAPI с передачей собственной колбэк-функции.

[User700]

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

[Limons]

User700, Проблема с юникодом, как решить эту проблему?

[Limons]

User700, я знаю, Что такое колбэк, как минимум поверхностно,

Но понятно ли, что это только колбэк для функции. Вы должны вызвать саму ф-ию WinAPI, передав ей свой колбэк.

Мне не понятно больше то, что после предложения о колбеке.

саму ф-ию WinAPI -- EnumWindows(); ?

Просто к чему это предложение, я вот тут допустил ошибку?
EnumWindows(enumWindowCallback, NULL);

[User700]

Limons, вы могли назвать функцию-колбэк не enumWindowCallback, а mywcb_1, для компилятора это просто функция с определённой сигнатурой; кроме того, могли объявить точно так же mywcb_2, которая делает что-то другое. В функию EnumWindows первым параметром передаётся функция, или точнее, указатель на функцию, которую она будет использовать как колбэк.
Что касается юникода. Пробовали вариант из ответа выше/ниже, где с буковкой A ф-ия WinAPI и еще set...cp в main?

[User700]

Limons, вы могли назвать функцию-колбэк не enumWindowCallback, а mywcb_1, для компилятора это просто функция с определённой сигнатурой; кроме того, могли объявить точно так же mywcb_2, которая делает что-то другое. В функию EnumWindows первым параметром передаётся функция, или точнее, указатель на функцию, которую она будет использовать как колбэк.
Что касается юникода. Пробовали вариант из ответа выше/ниже, где с буковкой A ф-ия WinAPI и еще set...cp в main?

[User700]

В последнем случае тогда не с юникодом, а кодировкой. Можно просто преобразовать cp1251 в 866. Или аккуратно настроить консоль для отображения 1251

[User700]

string cptocp (string s, const char[256] tbl) {
    for (char& c : s) c = tbl[c];
    return move(s);
}
#define cp1251to866(s) cptocp(s,TBL_1251_866)

....

cout << cp1251to866(move(windowTitle)) << endl;
....

Только таблицу найти

[Limons]

User700, насчёт коллекции функции я знаю.

А вот это не понял, что за буква А

Что касается юникода. Пробовали вариант из ответа выше/ниже, где с буковкой A ф-ия WinAPI и еще set...cp в main?

[Limons]

User700, причём тут отображение в консоль? Не понял. Проблема в том, что код не компилировался.

[User700]

Limons, так нужно обозначать чётко проблему, ошибку. Сами же написали "Проблема с юникодом, как решить эту проблему?", т.е. это означает что выводятся "кракозябры", а не некомпилируется

[User700]

Limons, а зачем тогда спрашиваете про параметр у EnumWinows, если знаете

[Limons]

User700, оу. Значит я не правильно понял:

Так, тут есть проблема с юникодом, замечу предотвращая вопрос о том, что речь шла о LPWSTR..

а зачем тогда спрашиваете про параметр у EnumWinows, если знает

Я относительно нормально знаю, как работают callback ф-ции, но при этом не работало. Я не знал почему, на самом деле видимо всё было в некорректных типах данных.