Also the way your defining Players is quite odd to me. (I highly HIGHLY suggest you use this () unless your trying to obfuscate it, which still isn’t a good idea and you use it for instance.new() aswell)
I think task.spawn is better than spawn (not sure, but it’s that way for wait())
I’d suggest changing the “Value” variable name to something better so it’s easier to read aswell