Help on stamina bar script

[DasKairo (Cairo)]

There are a couple of Issues with your Script, some are minor, while some are bigger and completely unnecessary to have at all.

To Start off, I see no purpose to Have a sprintingPlayer table, you can just use this RemoteEvent to fire when the State Changes, Plus, the Player is the first argument within a Server Event, This will make it much easier for you to keep track of the Player.

For your IntValue creation, There is a reason Developers never use the Second Argument to Parent:

PSA: Don't use Instance.new() with parent argument

I’ve discovered a pretty bad performance issue in one of top games that has to do with Instance.new and wanted to write about this, since this is not obvious unless you know the system inside out. Tthere are several ways to create a ROBLOX object in Lua: local obj = Instance.new(‘type’); fill obj fields local obj = Instance.new(‘type’, parent); fill obj fields local obj = util.Create(‘type’, { field1 = value1, … }) If you care at all about performance, please only use the first option - I wi…

It causes Preformace issues.
You should Also first (before anything else) Apply the Name, then the Properties, this should be the “proper” way to Create an Instance

local stamina = Instance.new("IntValue") -- Create Instance
stamina.Name = "Stamina" -- Prioritize this FIRST (Always)
stamina.Value = maxStamina -- Apply Properties
stamina.Parent = player -- Prioritize this LAST (Always)

Your Server script doesn’t require a while loop to function, if anything its a waste of Server Resources to be using it for this purpose, this is what makes an inefficient system, Instead it would be best Practice to use math.clamp for this case, math.clamp will limit the Value between the minimum value and maximum value, this is useful because because if a number is over or below a limit, it will automatically be set to this value, Here is an Example:

isStamina.Value = math.clamp(isStamina.Value - 1, 0, 100)
-- 0 is the minimum
-- 100 is the maximum

or, if you only plan to set one limit for a certain part, you can use math.min() or math.max() to handle this:

isStamina.Value = math.max(0, isStamina.Value - 1) -- returns the Largest Value

isStamina.Value = math.min(100, isStamina.Value - 1) -- returns the Smallest Value

---

@BRUHjaime

print("cum")

Why?