Why does ContextActionService fire UserInputState.Begin twice on keypad input?
Long Title for such a small thing.
Why does ContextActionService fire UserInputState.Begin twice on keypad keys and once on normal keys?
It should only be firing once with Enum.UserInputState.Begin and once with Enum.UserInputState.End when you release a key.
< It does this when you connect a 'normal' key from your keyboard.
Here's my small experimental chunk of local code:
local ContextActionService = game:GetService("ContextActionService")
ContextActionService:BindAction("keypad", print, false, Enum.KeyCode.KeypadFive)
ContextActionService:BindAction("normal", print, false, Enum.KeyCode.H)
Some months ago when I was testing this, it didn't fire UserInputState.End on keypad key release at all, now it fires twice Begin and one End ._.
thanks
2 answers
(Strange... it only fires one Begin and one End for me. I've found the UserInputService to sometimes fire events twice in other scenarios, however.)
To fix when Roblox is sending out duplicate events, use 'debounce'. ex, here is a modified print that will only print its argument if it's not been printed in this time frame before:
do
local args
local lastTime = -1
function printOnce(arg)
if lastTime ~= workspace.DistributedGameTime then
args = {}
lastTime = workspace.DistributedGameTime
end
if args[arg] then return end --already printed this
args[arg] = true
print(arg)
end
end
printOnce(1) --Prints 1
printOnce(2) --Prints 2
printOnce(1) --Duplicate, so doesn't print anything
wait()
printOnce(1) --Time has passed, so prints 1
To prevent having to write this code repeatedly, you might use a wrapper function like these:
function CallFuncOncePerTime(func) --pass in the function
local lastTime = -1 --this local variable is created for each function you pass in
return function(...) --this is the function to return; it accepts any number of arguments
if lastTime == workspace.DistributedGameTime then return end --we want the function to do nothing in this condition
lastTime = workspace.DistributedGameTime --update lastTime
func(...) --call the original function with any arguments passed in
end
end
function CallFuncOncePerTimeArg(func) --func will run only if its first argument is unique for this time frame
local lastTime = -1
local args
return function(...)
if lastTime ~= workspace.DistributedGameTime then --reset args
args = {}
lastTime = workspace.DistributedGameTime
end
local arg = {...} --get first arg
arg = arg[1]
if args[arg] then return end --this is a duplicate; return
args[arg] = true
func(...)
end
end
--Usage example
function test()
print("Test")
end
test = CallFuncOncePerTime(test)
--or
test = CallFuncOncePerTime(function()
print("Test")
end
--example 2:
function test(arg)
print("Test", arg)
end
test = CallFuncOncePerTimeArg(test)
--or
test = CallFuncOncePerTimeArg(function(arg)
print("Test", arg)
end)
You can thereby modify as many functions as you need so that they all exhibit this behaviour. The idea of a wrapper function is to "wrap" a function in another function. The Roblox article on debounce gives another example of this idea (under "Advanced Notation").
0
0
I recomend ContextActionService, its a eaiser version of UserInputServer
local createMobileButton = true -- if u want mobile buttons
local actionName = "CallFuncOncePerTimeArg"
local function onMouseDown(actionName,state,info)
if state == Enum.UserInputState.Begin then
function CallFuncOncePerTime()
local lastTime = -1
return function(...)
if lastTime == workspace.DistributedGameTime then return end
lastTime = workspace.DistributedGameTime
func(...)
end
end
elseif state == Enum.UserInputState.End then
print("Stopped Using Func")
end
end
contextAction:BindAction(actionName,onMouseDown,createMobileButton,Enum.UserInputType.MouseButton1)