That should work. Though, the scripts won’t run if they’re directly in the player, but that’s probably not what you meant. Putting those sorts of scripts in the backpack is a common practice.
For your purposes ContextActionService will most likely be better. I like UserInputService better if keys will correspond to more than 1 action in different situations (such as predefined combos), but I’m assuming your game wouldn’t need to be that complicated.