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

How to make message only pop up one time?

Asked by

5 years ago

I am trying to make this message script only pop up one time. It works when the player touches the block and a message pops up. When the message is over and the player touches the block again it shows pops up again which is not what I want. Any help?

01function message(m)
02 
03 
04 
05---------------------------------------------------------------\\
06 
07 
08m.Text = "How do I make you pop up once?"
09wait(5)
10 
11 
12 
13 
14---------------------------------------------------------------//
15 
16-- Color codes:
17-- (1,0,0) is Red
18-- (0,1,0) is Green
19-- (0,0,1) is Blue
20-- (1,1,1) is White
21-- And you can also mix colors. (1,0,1) is Purple for example
22 
23-- For more settings, read the green text around the section below
24 
25 
26 
27 
28 
29 
30 
31end
32function onTouch(hit)
33 
34    if hit.Parent:FindFirstChild("Humanoid") ~= nil then
35 
36        if game.Players:FindFirstChild(hit.Parent.Name) ~= nil then
37        local player = game.Players:FindFirstChild(hit.Parent.Name)
38 
39            if player:FindFirstChild("PlayerGui"):FindFirstChild("ScreenGui") == nil then
40            sg = Instance.new("ScreenGui")
41            sg.Parent = player:FindFirstChild("PlayerGui")
42            end
43 
44        if player.PlayerGui.ScreenGui:FindFirstChild("MessageBox") == nil then
45 
46                    -- In this section below, you can edit how the text box looks like
47        local f = Instance.new("Frame")
48        f.Name = "MessageBox"
49        f.Position = UDim2.new(0.3, 0, 0.1, 0)
50        f.Size = UDim2.new(0.4, 0, 0, 0)            -- If Text box needs to be bigger, simply change "0.4"
51        f.Style = "RobloxRound"                         -- Styles: "ChatBlue", "Chat Red", "Chat Green"
52        f.Parent = player.PlayerGui:FindFirstChild("ScreenGui")
53 
54        local m = Instance.new("TextLabel")
55        m.Position = UDim2.new(0.5, 0, 0.5, 500)
56        m.FontSize = "Size14"                           -- Font Sizes: 10, 11, 12, 14, 18, 24
57        m.TextColor3 = Color3.new(1,1,1)                -- Colors: (1,0,0) = Red, (0,1,0) = Green, (0,0,1) = Blue
58        m.Parent = f
59 
60        message(m)
61 
62 
63        f:Destroy()
64        end
65        end
66    end
67end
68 
69script.Parent.Touched:connect(onTouch)
 View all 69 lines...

I mean, you could remove the part you're using to trigger this once the message is finished and you're done displaying it. Though that may not be the best way. shipleydog2 -3 — 5y

3 answers

Answered by

Gavinboy3000 98

5 years ago

Edited 5 years ago

Put a variable at the top like this:

1local activated = false
2 
3function message(m)
4    if activate == false then
5        activated = true
6        m.Text = "How do I make you pop up once?"
7        wait(5)
8    end
9end

Answered by

Elyzzia 1294

5 years ago

Edited 5 years ago

since you're connecting to an event, you can disconnect the function once you're done with RBXScriptConnection:Disconnect()

in your code, that would look something like this:

01local connection -- define the variable before the function, because the function needs to be able to access it
02local activated = false -- if you need to know if the part was touched for whatever reason
03 
04local function onTouch(hit)
05    if whatever then
06        if blahBlahBlah then
07            -- stuff goes here
08            activated = true
09            connection:Disconnect()
10        end
11    end
12end
13connection = script.Parent.Touched:Connect(onTouch) -- when you Connect to an event, it returns an RBXScriptConnection which you can use to disconnect the function later

also, make sure you put local whenever you define a variable or function

why? it's good practice

why is it good practice? it's faster when you do that, also there's global environment blah blah blah whatever It's Not Good stuff but i'm not advanced enough to tell you why global environment is bad

Answered by

haba_nero 386

5 years ago

Hello! I can see your issue! I'd suggest using tables to store players that have already touched the brick, like so:

01function message(m)
02 
03 
04 
05---------------------------------------------------------------\\
06 
07 
08m.Text = "How do I make you pop up once?"
09wait(5)
10 
11 
12 
13 
14---------------------------------------------------------------//
15 
16-- Color codes:
17-- (1,0,0) is Red
18-- (0,1,0) is Green
19-- (0,0,1) is Blue
20-- (1,1,1) is White
21-- And you can also mix colors. (1,0,1) is Purple for example
22 
23-- For more settings, read the green text around the section below
24 
25local AlreadyTouched = {}
26 
27 
28 
29 
30 
31end
32function onTouch(hit)
33local AlreadyTouched = false--Automatically determines player has not touched
34    for i, player in pairs(AlreadyTouched) do--Checks for player that already touched
35        if hit.Parent.Name == player then
36            AlreadyTouched = true--Player has already touched
37        end
38 
39    end
40   if AlreadyTouched == false then
41 if hit.Parent:FindFirstChild("Humanoid") ~= nil then
42table.insert(AlreadyTouched,hit.Parent.Name)--Adds player to table
43 
44        if game.Players:FindFirstChild(hit.Parent.Name) ~= nil then
45        local player = game.Players:FindFirstChild(hit.Parent.Name)
46 
47            if player:FindFirstChild("PlayerGui"):FindFirstChild("ScreenGui") == nil then
48            sg = Instance.new("ScreenGui")
49            sg.Parent = player:FindFirstChild("PlayerGui")
50            end
51 
52        if player.PlayerGui.ScreenGui:FindFirstChild("MessageBox") == nil then
53 
54                    -- In this section below, you can edit how the text box looks like
55        local f = Instance.new("Frame")
56        f.Name = "MessageBox"
57        f.Position = UDim2.new(0.3, 0, 0.1, 0)
58        f.Size = UDim2.new(0.4, 0, 0, 0)            -- If Text box needs to be bigger, simply change "0.4"
59        f.Style = "RobloxRound"                         -- Styles: "ChatBlue", "Chat Red", "Chat Green"
60        f.Parent = player.PlayerGui:FindFirstChild("ScreenGui")
61 
62        local m = Instance.new("TextLabel")
63        m.Position = UDim2.new(0.5, 0, 0.5, 500)
64        m.FontSize = "Size14"                           -- Font Sizes: 10, 11, 12, 14, 18, 24
65        m.TextColor3 = Color3.new(1,1,1)                -- Colors: (1,0,0) = Red, (0,1,0) = Green, (0,0,1) = Blue
66        m.Parent = f
67 
68        message(m)
69 
70 
71        f:Destroy()
72        end
73        end
74    end
75end
76end
77 
78script.Parent.Touched:connect(onTouch)
 View all 78 lines...

If you have any questions, comment below!

Best,

TheLastHabanero

How to make message only pop up one time?

3 answers

Answer this question