C++ Builder KillTimer()

[coNsept]

Доброго времени суток уважаемые experts :D
Возникла проблема с уничтожением таймера, неизвестно по каким причинам он напрочь отказывается умирать... "Или может я не правильно что-то делаю или эт глюки в среде, вообщем без понятий но по идее код правильный".

Здесь по заголовку окна, я включаю Timer и начинаю нервно click'ать.

void __fastcall TForm2::Button1Click(TObject *Sender)
{
	HWND hWnd;
	hWnd = FindWindow("[WINDOWNAME]", NULL);
	SetTimer(hWnd, IDT_TIMER1, 20, (TIMERPROC)NULL);
	SendMessageA(hWnd, WM_LBUTTONDOWN, 0, 0);
	SendMessageA(hWnd, WM_LBUTTONUP, 0, 0);
}

Здесь, при попытке его уничтожения он все равно продолжает нервно click'ать...

void __fastcall TForm2::Button3Click(TObject *Sender)
{
	HWND hWnd;
	hWnd = FindWindow("[WINDOWNAME]", NULL);
	KillTimer(hWnd, IDT_TIMER1);
}

[dexterua]

Насколько я помню, то SetTimer возвращает идентификатор таймера, а не забивается вручную.

Потому предложил бы попробовать

void __fastcall TForm2::Button1Click(TObject *Sender)
{
	HWND hWnd;
	hWnd = FindWindow("[WINDOWNAME]", NULL);
	IDT_TIMER1=SetTimer(hWnd, 0, 20, (TIMERPROC)NULL);
	SendMessageA(hWnd, WM_LBUTTONDOWN, 0, 0);
	SendMessageA(hWnd, WM_LBUTTONUP, 0, 0);
}

Соответственно объявив IDT_TIMER1 где-то, чтоб в зоне видимости была для обеих.

[coNsept]

Безрезультатно, таймер не останавливается.

p.s. Его можно остановить только таким методом, полностью выключив таймер: Timer1->Enabled = false; Ну а мне желательно чтобы KillTimer остановил!

[sashonk]

идентификатор таймера задается в ручную. Может вы не храните на него глобальную ссылку или она теряет свое значение? Возможно, вы ошиблись в каком-то другом месте

[coNsept]

Объявил int IDT_TIMER1 = 1; глобально, по кнопкам раскидал SetTimer и KillTimer, я собственно уже приводил пример. Первоначально я устанавливаю компонент Timer как disabled, далее уже на кнопки->Button1 включаю. А вариант уважаемого dexterua не работает.

void __fastcall TForm2::Button1Click(TObject *Sender)
{
	HWND hWnd; hWnd = FindWindow("[WINDOWNAME]", NULL);
        Timer1->Enabled = true;
	SetTimer(hWnd, IDT_TIMER1, 1000, (TIMERPROC)NULL);
	SendMessageA(hWnd, WM_LBUTTONDOWN, 0, 0);
	SendMessageA(hWnd, WM_LBUTTONUP, 0, 0);
}

void __fastcall TForm2::Button3Click(TObject *Sender)
{
	HWND hWnd;
	hWnd = FindWindow("[WINDOWNAME]", NULL);
	KillTimer(hWnd, IDT_TIMER1);
}

[dexterua]

Да был не прав, забыл, что возвращается идентификатор только, когда первый параметр NULL.
А так по тексту - если не сложно выложите весь код, пожалуйста.
Потому как синтаксис вроде верен, возможно ошибка немного в другом месте.
А также было бы интересно узнать какое значение возвращает KillTimer.

[coNsept]

Это весь код собственно, одна форма и две кнопки. Запуск эмуляции нажатия левой кнопки мыши по таймеру и соответственно остановка эмуляции.

# Всем спасибо кто помогал, проблему решил

[Assemblerru]

А зачем такое, как мы все поняли ты работаеш под C++ Builder. Не изобретай велосипед перенеси на форму таймер со странице Win32.

тогда твой код можно модефецировать, но задав параметры пересчета таймера.
Свойства таймера
Timer1 ->Enabled = true/false доступность таймера
Timer1 ->Interval = 20;//интервал в милисекундах
Если Enabled = false или Interval = 0 таймер должен остановиться
попробуй оба свойства изменять

Да и по достижении тайм аута возникает событие OnTimer попробуйте использовать его.

и пожалуста прокоментируйте

SetTimer(hWnd, IDT_TIMER1, 1000, (TIMERPROC)NULL);

[coNsept]

А зачем такое, как мы все поняли ты работаеш под C++ Builder. Не изобретай велосипед перенеси на форму таймер со странице Win32.

Немного не понял с изобретением велосипеда, вроде нормально все.
OnTimer у меня стоит на Button1

void __fastcall TForm2::Button1Click(TObject *Sender)
{
	HWND hWnd;
	UnicodeString String1 = Edit1->Text;
	Edit1->Text = String1;
	UnicodeString String2 = Edit1->Text;

	hWnd = FindWindow(String2.t_str(), NULL);

	Timer1->Enabled = TRUE;

	/*
	+ ComboxBox будет задавать интервал времени
	*/

	SetTimer(hWnd, IDT_TIMER1, 1000, (TIMERPROC)NULL);

	if (Button5->Enabled == FALSE)
	{ SendMessageA(hWnd, WM_LBUTTONDOWN, 0, 0); SendMessageA(hWnd, WM_LBUTTONUP, 0, 0); }
	if (Button6->Enabled == FALSE)
	{ SendMessageA(hWnd, WM_RBUTTONDOWN, 0, 0); SendMessageA(hWnd, WM_RBUTTONUP, 0, 0); }
}

[Assemblerru]

Ну согласен
А зачем ты пишиш

SetTimer(hWnd, IDT_TIMER1, 1000, (TIMERPROC)NULL);

ЗАЧЕМ тебе еше раз создавать таймер, насколько я понимаю он у тебя уже есть. Нельзя ли ввести пересчет для имеющегося проше...

Timer1 ->Interval = 1000;

А в обработчик события OnTimer, происходящее по достижения таймера 0, написать код..