[HELP] onClientClick [Solved]

[UnG//Macaus]

I have a problem with my code, everything is working, but the function is executed 2 times when I click on a button. Can someone help me?

    for id,element_r in ipairs(getElementChildren(dxRootElement)) do 
        for id, element in ipairs(getElementChildren(element_r)) do 
            if getElementData(element,"visible") == true then 
                local g_height = getElementData(element,"height") or 0 
                local g_width = getElementData(element,"width") or 0 
                local px, py = dxGetPosition(element) 
                if (button == "left" and state == "up") then 
                    if x >= px and x <= (px+g_width) and y >= py and y <= (py + g_height) then 
                        triggerEvent("onClientDXClick",element) 
                    end 
                end 
            end 
        end 
    end 

Edited December 31, 2014 by Guest

[Anubhav]

Use getTickCount method. When you click just make a variable setting to getTickCount( ) then when you loop before that, check: if getTickCount() - yourVariables > 1000 then

--

easier method is a setTimer

[Dealman]

Is that the entire function? What if you move this part to the very top of the function?

if (button == "left" and state == "up") then 

[UnG//Macaus]

Yes, this is the full function

I put this line at the top and still have the same problem

[0xCiBeR]

That's because your for-loop executes more than one time before you release/press your key. Just break the loop using:

break 

Example: ( Tho i don't know how your script works as you are not showing it entierly ):

triggerEvent("onClientDXClick",element) 
break 

That will prevent your for-loop to continue looping after you triggered the event

[UnG//Macaus]

now is not repeating more, except for triggerEvents

Example:

addEventHandler("onClientDXClick",getRootElement(), 
function() 
    if (source == button) then 
        triggerServerEvent("test",localPlayer) 
    end 
end) 
  
--server 
addEvent("test",true) 
addEventHandler("test",getRootElement(), 
function() 
    outputChatBox ( "TEST", source, 0, 255, 0) 
end) 
  

[UnG//Macaus]

anyone?

[Castillo]

What is the problem?

[UnG//Macaus]

Only triggerServerEvent is executed 2 times

[Castillo]

I guess "onClientDXClick" is being triggered twice then.

[Anubhav]

Use getTickCount method. When you click just make a variable setting to getTickCount( ) then when you loop before that, check: if getTickCount() - yourVariables > 1000 then

--

easier method is a setTimer

This is it, then.

[UnG//Macaus]

I think you have misunderstood the problem I have now

addEventHandler("onClientDXClick",getRootElement(), 
function() 
    if (source == button) then 
        triggerServerEvent("test",localPlayer) --run 2 times (2 outputChatBox) 
        outputChatBox("test") --run 1 time (1 outputChatBox) 
    end 
end) 

[Castillo]

Something else is triggering that event then.

[3B00DG4MER]

You can use that function :

function isMouseWithinRangeOf(posX, posY, sizeX, sizeY) 
  if isCursorShowing() == false then 
    return false 
  end 
  local cx,cy = getCursorPosition() 
  cx,cy = cx*sW,cy*sH 
  if cx >= posX and cx <= posX+sizeX and cy >= posY and cy <= posY+sizeY then 
    return true,cx,cy 
  else 
    return false 
  end 
end 
  

[UnG//Macaus]

As i Know , there is nothing that is triggering the event, does someone have a idea or solution how to solve this problem ?