Question

我想使用2个计时器，一个用于读取传感器，另一个用于使用visual studio VC ++ 2010类库向机器人发出命令。

如果单击按钮1则打开第一个计时器，使用按钮3打开第二个计时器。

但是当我点击按钮2时，即使我没有按下按钮1，我的读取传感器仍在工作，反之亦然。

这是我的计划：

我的代码出了什么问题？

const int cTimer1 = 1;
const int cTimer2 = 1;

 void CENVSConfigDlg::OnBnClickedButton1()
{
SetTimer(cTimer1,1000,NULL);
}

 void CENVSConfigDlg::OnBnClickedButton2()
{
KillTimer(cTimer1);
}

 void CENVSConfigDlg::OnBnClickedButton3()
{   
SetTimer(cTimer2,10,NULL);  
}

void CENVSConfigDlg::OnBnClickedButton4()
{
    KillTimer(cTimer2);
}

void CENVSConfigDlg::OnTimer(UINT_PTR ID){
if(ID==cTimer1){

    char buffer[30],tempStr[5];
    int sensor[6];

    DWORD nbytes;

    MessageBeep(0);

    //Read Sensors

    if(!WriteFile( hnd_serial, "1", 1, &nbytes, NULL )){KillTimer(cTimer1);MessageBox(L"Write Com Port fail!");return;}
    Sleep(30);
    if(!ReadFile( hnd_serial, buffer, 30, &nbytes, NULL )){KillTimer(cTimer1);MessageBox(L"Read Com Port fail!");return;}
    Sleep(300);



if(ID==cTimer2)
{
    if(GetAsyncKeyState(0x53) != 0)
 {
 DWORD nbytes;
if(!WriteFile( hnd_serial, "s", 1, &nbytes, NULL )){MessageBox(L"Write Com Port fail!");return;}
    Sleep(30);
 tampil.SetWindowText( (LPCTSTR)"s" ); //backward
 }

else if(GetAsyncKeyState(0x57) != 0)
{
    DWORD nbytes;
if(!WriteFile( hnd_serial, "w", 1, &nbytes, NULL )){MessageBox(L"Write Com Port fail!");return;}
    Sleep(30);
    tampil.SetWindowText( (LPCTSTR)"w" );//forward
}


}

}


}

Answer 1

你发现自己的错误是件好事。但是你的代码仍有一些问题。

您正在计时器（WM_TIMER消息）中执行读，写，休眠。这将导致UI线程冻结。您最好使用另一个线程进行处理，这涉及UI更新以外的其他内容。
您的计时器ID的名称类似于Timer1，Timer2等，这对程序员不友好。你应该使用更好的名字来反映实际意义。
多个计时器不能同时运行。再次，你最好使用多线程！

使用OnTimer（UINT_PTR ID）和vc ++的2计时器

1 个答案: