Scripting Helpers is winding down operations and is now read-only. More info→

how to make a clicking cool down when you click the sky ?

Asked by

Random_Haxer5 10

4 years ago

Edited 4 years ago

A few hours ago i've made a pistol script (keep in mind that for the pistol script i used mouse.Target) but i met a error along the way in which i can't figure out a solution, so basically a pistol has a cool down, right ? it won't just shoot out bullets like a rifle so i made a cool down IF the mouse.Target is either terrain or a player, but what about the sky, the sky is equal to nil, how do i add a cool down if my mouse is pointing at the sky ? (debounce just messes up the script)

01local handle = script.Parent.Handle
02local tool = script.Parent
03local player = game.Players.LocalPlayer
04local gui = player:WaitForChild("PlayerGui")
05local screenGui = gui:WaitForChild("ScreenGui")
06local TextLabel = screenGui:WaitForChild("Ammo")
07local mouse = player:GetMouse()
08local mxAm = 10
09local Ammo1 = mxAm
10local serivce = game:GetService("UserInputService")
11local oldIcon = mouse.Icon
12 
13local function ammo2()
14    Ammo1 = mxAm
15end
16 
17tool.Equipped:Connect(function()
18    mouse.Icon = "http://www.roblox.com/asset/?id=79658449"
19    TextLabel.Visible = true   
20end)
21 
22tool.Activated:Connect(function()
23    if tool.Enabled == true then
24        if Ammo1 > 0 then
25            Ammo1 = Ammo1 - 1
26            if mouse.Target.Parent:FindFirstChild("Humanoid") then
27                script.Parent.Damage:FireServer(mouse.Target)
28                tool.Enabled = false
29                print("target hit")
30                wait(0.2)
31                tool.Enabled = true
32            else
33                tool.Enabled = false
34                print("miss")
35                wait(0.2)
36                tool.Enabled = true
37            end
38        end    
39    end
40end)    
41 
42while wait() do
43    TextLabel.Text = Ammo1.."/"..mxAm
44 
45tool.Unequipped:Connect(function()
46    mouse.Icon = oldIcon
47    TextLabel.Visible = false
48end)
49 
50serivce.InputBegan:Connect(function(key)
51    if key.KeyCode == Enum.KeyCode.R and Ammo1 ~= mxAm then
52        wait(1)
53        ammo2()
54        end
55    end)
56end
 View all 56 lines...

1 answer

Answered by

TheLuminent 1204

4 years ago

Here's a script with the cooldown applied, I also rewrote it for you.

01if not game:IsLoaded() then
02    repeat
03        wait()
04    until game:IsLoaded()
05end
06 
07local Players = game:GetService("Players")
08local UserInputService = game:GetService("UserInputService")
09local RunService = game:GetService("RunService")
10 
11local LocalPlayer = Players.LocalPlayer
12local Character = LocalPlayer.Character
13local PlayerGui = LocalPlayer:FindFirstChildOfClass("PlayerGui")
14local Mouse = LocalPlayer:GetMouse()
15 
16local ScreenGui = PlayerGui:FindFirstChildOfClass("ScreenGui")
17local TextLabel = ScreenGui:FindFirstChild("Ammo")
18 
19 
20local Tool = (script.Parent:IsA("Tool") and script.Parent) or script:FindFirstAncestorOfClass("Tool")
21local Handle = Tool:FindFirstChild("Handle")
22 
23local MaxAmmo = 10
24local OldMouseIcon = Mouse.Icon
25local Bind = nil
26local CooldownProgression = false
27 
28Tool.Equipped:Connect(function()
29    Mouse.Icon = ("http://www.roblox.com/asset/?id=79658449");
30    TextLabel.Visible = true
31    Bind = Tool.Activated:Connect(function()
32        if not Tool.Enabled or Ammo1 <= 0 or CooldownProgression == true then
33            return
34        end
35        CooldownProgression = true
36        Ammo1 = Ammo1 - 1
37        local S = pcall(function()
38            local TargetModel = (Mouse.Target:IsA("Model") and Mouse.Target) or (Mouse.Target.Parent:IsA("Model") and Mouse.Target.Parent) or Mouse.Target:FindFirstAncestorOfClass("Model")
39            local TargetHumanoid = TargetModel:FindFirstChildOfClass("Humanoid")
40            Tool.Enabled = false
41            print("Target hit");
42            wait(.2)
43            Tool.Enabled = true
44        end)
45        if not S then
46            Tool.Enabled = true
47            print("Missed");
48            wait(.2)
49            Tool.Enabled = false
50        end
51        coroutine.wrap(function()
52            wait(.25) -- Cooldown time.
53            CooldownProgression = false
54        end)()
55    end)
56    Tool.Unequipped:Wait(function()
57        Bind:Disconnect()
58    end)
59end)
60 
61Tool.Unequipped:Connect(function()
62    if Bind ~= nil and typeof(Bind) == "RBXScriptConnection" then
63        Bind:Disconnect()
64    end
65    Mouse.Icon = OldMouseIcon
66    TextLabel.Visible = false
67    return
68end)
69 
70UserInputService.InputBegan:Connect(function(InputObject,GameProcessedEvent)
71    if GameProcessedEvent then
72        return
73    end
74    if InputObject.KeyCode == Enum.KeyCode.R and Ammo1 ~= MaxAmmo then
75        CooldownProgression = true -- Make this true so you can't fire while reloading...
76        print("Reloading...")
77        wait(1)
78        Ammo1 = MaxAmmo
79        print("Reloaded!")
80        CooldownProgression = false -- Reloading complete, allow the user to fire.
81    end
82end)
83 
84coroutine.resume(coroutine.create(function()
85    while true do
86        RunService.Stepped:Wait()
87        TextLabel.text = tostring(Ammo1) .. (" / ") .. tostring(MaxAmmo)
88    end
89end))
 View all 89 lines...

thanks but i am getting errors saying: attempt to index nil with 'Text' and attempt to index nil with 'Visible' Random_Haxer5 10 — 4y

i respect what you did but the gun breaks when you shoot at the sky, it just stops working :/ Random_Haxer5 10 — 4y

how to make a clicking cool down when you click the sky ?

1 answer

Answer this question