[RESOLVED] the timer isn't created :(

[Cambalinho]

see these code:

case WM_LBUTTONUP:
                case WM_RBUTTONUP:
                case WM_MBUTTONUP:
                case WM_XBUTTONUP:
                {
                    int a= SetTimer(inst ->hWnd(),JoystickTimer,120,NULL);
                    SetWindowText(inst->hwnd,to_string(a).c_str());
//...............

the inst is the form instance pointer.
seems a normal code.
the set window text works fine, so the problem isn't the 'inst' pointer. but the timer isn't created.. my question: why the timer isn't created?

[Cambalinho]

see the GetlastError():

 case WM_LBUTTONUP:
                case WM_RBUTTONUP:
                case WM_MBUTTONUP:
                case WM_XBUTTONUP:
                {
                    SetTimer(inst ->hWnd(),JoystickTimer,120,NULL);
                    int a=GetLastError();
                    SetWindowText(inst->hwnd,to_string(a).c_str());

the SetTimer() give me '1', and the GetLastError() give 0(zero). so what isn't right?

[OReubens]

inst->hWnd() ???
wut ?

[Cambalinho]

inst->hWnd() ???
wut ?

sorry about that. but it's the same of inst->hwnd

[2kaud]

but the timer isn't created
..
the SetTimer() give me '1', and the GetLastError() give 0(zero)

How do you know the timer isn't created as SetTimer() returns a non-zero integer to indicate success.

[Cambalinho]

How do you know the timer isn't created as SetTimer() returns a non-zero integer to indicate success.

in form window procedure:

 case WM_TIMER:
                {
                    if (wParam == timerid)//for mouse stoped and it's working normaly
                    {
                        PreviousLocation = Location;
                        GetCursorPos(&Location);
                        if ((Location.x = PreviousLocation.x) && (Location.y = PreviousLocation.y))
                        {
                            KillTimer(inst->hWnd(), timerid);
                            inst->MouseStoped();
                        }
                    }
                    else if(wParam==JoystickTimer)//for joystick, but the timer isn't created :(
                    {
                        MessageBox(NULL,"heello","hi",MB_OK);
                        inst->setText("joystick");
                        joySetCapture(inst->hWnd(),JOYSTICKID1,120,true);
                        JOYINFOEX  b;
                        b.dwSize=sizeof(JOYINFOEX );
                        b.dwFlags=JOY_RETURNALL;
                        joyGetPosEx(JOYSTICKID1,&b);
                        inst->setText(to_string(b.dwButtonNumber));
                        joyReleaseCapture(JOYSTICKID1);
                    }
                }
                break;

the messagebox isn't showed

[VladimirF]

Is this
case WM_TIMER:
in the WindowProc of the inst->hWnd?

[Cambalinho]

Is this
case WM_TIMER:
in the WindowProc of the inst->hWnd?

yes..

static LRESULT CALLBACK WndProcForm(HWND HandleWindow, UINT msg, WPARAM wParam, LPARAM lParam)
        {
            static POINT PreviousLocation, Location;
            static bool Tracking = false;
            static MouseButtons MBButtons;
            static bool blControl = false;
            static bool blShift = false;
            static bool blResize = false;
            static int xPos = 0;
            static int yPos = 0;
            static UINT_PTR timerid=0;
            static UINT_PTR JoystickTimer=0;
            static bool blnDrag = false;
            static bool KeyPressed=false;
            static int KeyDownCount=0;
            UINT JoystickCount =0;

            form *inst = (form *)GetWindowLongPtr(HandleWindow, GWLP_USERDATA);//the inst is created here



            if(inst!=NULL && inst->Create!=NULL )
            {
                static bool i=true;
                 if(i==true)
                 {
                    i=false;

                    inst -> Create(inst->getLeft(), inst->getTop());
                 }
            }

            HBRUSH g_hbrBackground = NULL;

            //Working with messages
            switch(msg)
            {
                case WM_NCCREATE:
                {
                    CREATESTRUCT *p = (CREATESTRUCT *)lParam;
                    inst = (form *)p->lpCreateParams;
                    SetWindowLongPtr(HandleWindow, GWLP_USERDATA, (LONG_PTR)inst);
                    inst->hwnd = HandleWindow;


                }
                break;
                case WM_TIMER:
                {
                    if (wParam == timerid)//for mouse stoped and it's working normaly
                    {
                        PreviousLocation = Location;
                        GetCursorPos(&Location);
                        if ((Location.x = PreviousLocation.x) && (Location.y = PreviousLocation.y))
                        {
                            KillTimer(inst->hWnd(), timerid);
                            inst->MouseStoped();
                        }
                    }
                    else if(wParam==JoystickTimer)//for joystick, but the timer isn't created :(
                    {
                        MessageBox(NULL,"heello","hi",MB_OK);
                        inst->setText("joystick");
                        joySetCapture(inst->hWnd(),JOYSTICKID1,120,true);
                        JOYINFOEX  b;
                        b.dwSize=sizeof(JOYINFOEX );
                        b.dwFlags=JOY_RETURNALL;
                        joyGetPosEx(JOYSTICKID1,&b);
                        inst->setText(to_string(b.dwButtonNumber));
                        joyReleaseCapture(JOYSTICKID1);
                    }
                }
                break;

[2kaud]

static UINT_PTR timerid=0;
static UINT_PTR JoystickTimer=0;

Are you changing these values to distinct ones somewhere - otherwise your timeid for mouse and joystick timer have the same ID!!!!!!!!

Also shouldn't

if ((Location.x = PreviousLocation.x) && (Location.y = PreviousLocation.y))

be

if ((Location.x == PreviousLocation.x) && (Location.y == PreviousLocation.y))

[Cambalinho]

static UINT_PTR timerid=0;
static UINT_PTR JoystickTimer=0;

Are you changing these values to distinct ones somewhere - otherwise your timeid for mouse and joystick timer have the same ID!!!!!!!!

Also shouldn't

if ((Location.x = PreviousLocation.x) && (Location.y = PreviousLocation.y))

be

if ((Location.x == PreviousLocation.x) && (Location.y == PreviousLocation.y))

you have right in both situations.. thanks for all
now it's working.
thanks for all.. thank you