local CashOnClick = 1 clickdetector.MouseClick:Connect(function() clickdetector.Parent:Destroy() for i,v in pairs(game.Players:GetPlayers()) do if v:FindFirstChild("leaderstats") then v.leaderstats['Apples Clicked'].Value = v.leaderstats['Apples Clicked'].Value + 1 v.leaderstats['Cash'].Value = v.leaderstats['Cash'].Value + 1 if v.leaderstats['Apples Clicked'].Value == 10 then v.leaderstats['Cash'].Value = CashOnClick.Value + 1 end end end end)

I have not included the whole script but im attempting to make it so every time "Apples Clicked"'s value goes up by 10, the CashOnClick Value goes up by 1, therefor increasing the amount of cash gained per click. Yet It gives me the error "attempt to index global 'CashOnClick' (a number value)"

That is because CashOnClick is literally a number. Not an IntValue/NumberValue. Remove the .Value and it will fix the problem. Now for the second question if you want to make the cash go up every 10 clicks use the modulus operator %. It divides two numbers and instead of returning the quotient, it will return the remainder instead.

Examples:

print(12 % 6) -- 0 (12/6 no remainder)
print(300 % 200) -- 100 (200 goes into 300 only once, it going twice results in a remainder of 100

Final product

local CashOnClick = 1 

clickdetector.MouseClick:Connect(function() 
    clickdetector.Parent:Destroy() 
    for i,v in pairs(game.Players:GetPlayers()) do 
        if v:FindFirstChild('leaderstats') then 
            v.leaderstats['Apples Clicked'].Value = v.leaderstats['Apples Clicked'].Value + 1
            if v.leaderstats['Apples Clicked'].Value % 10 == 0 then
                CashOnClick = CashOnClick + 1
                v.leaderstats['Cash'].Value = v.leaderstats['Cash'].Value + (CashOnClick + 1)
            end 
        end 
    end 
end)

So if the number modulated by 10 is 0 that means it is divisible by 10.