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

Why isn't my PlayerGui working as intended?

Asked by

BreckleShrimp 23

4 years ago

Edited 4 years ago

So I wrote this script that allows a player to find/collect special coins, and when they do, the coins become semi-transparent, signifying to the player collecting that they already collected this coin and they cannot collect it again. It then adds the found coin to a coin count in a gui at the bottom left corner (It allows them to go from 1/7, 2/7...7/7 kind of like in slender when you need to collect all 7 pages). I have it set under the PlayerGui, however, everyone's gui changes with the person who collects the coins. Instead, I'd like everyone's Coin count in the gui to remain at 0 and the coins to remain opaque until the players collect the coins themselves.

Also, once a player resets, the coins collected remain transparent for all players, but the coin counter resets and goes back to 0, which is not acceptable.

the script is in StarterGui but it is being called in PlayerGui
the coins are meshes in the workspace

If anyone knows my problem and knows how to fix it, please do explain. Thank you.

001local coinTrack1 = true
002local coinTrack2 = true
003local coinTrack3 = true
004local coinTrack4 = true
005local coinTrack5 = true
006local coinTrack6 = true
007local coinTrack7 = true
008local coinNum = 0
009 
010player.PlayerGui.CoinTracker.Frame.TextLabel.Text = "Coins "..coinNum.."/7"
011game.Workspace.RedCoin1.Touched:Connect(function(hit)
012    if coinTrack1 == true then
013        if hit.Parent:WaitForChild("Humanoid") then
014            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
015            coinNum = coinNum + 1
016            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
017            game.Workspace.RedCoin1.Transparency = .7
018 
019            coinTrack1 = false
020            return coinNum
021 
022 
023        end
024    end
025 
026end)
027game.Workspace.RedCoin2.Touched:Connect(function(hit)
028    if coinTrack2 == true then
029        if hit.Parent:WaitForChild("Humanoid") then
030            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
031            coinNum = coinNum + 1
032            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
033            game.Workspace.RedCoin2.Transparency = .7
034 
035            coinTrack2 = false
036            return coinNum
037 
038 
039        end
040    end
041 
042end)
043game.Workspace.RedCoin3.Touched:Connect(function(hit)
044    if coinTrack3 == true then
045        if hit.Parent:WaitForChild("Humanoid") then
046            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
047            coinNum = coinNum + 1
048            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
049            game.Workspace.RedCoin3.Transparency = .7
050 
051            coinTrack3 = false
052            return coinNum
053 
054 
055        end
056    end
057 
058end)
059game.Workspace.RedCoin4.Touched:Connect(function(hit)
060    if coinTrack4 == true then
061        if hit.Parent:WaitForChild("Humanoid") then
062            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
063            coinNum = coinNum + 1
064            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
065            game.Workspace.RedCoin4.Transparency = .7
066 
067            coinTrack4 = false
068            return coinNum
069 
070 
071        end
072    end
073 
074end)
075game.Workspace.RedCoin5.Touched:Connect(function(hit)
076    if coinTrack5 == true then
077        if hit.Parent:WaitForChild("Humanoid") then
078            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
079            coinNum = coinNum + 1
080            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
081            game.Workspace.RedCoin5.Transparency = .7
082 
083            coinTrack5 = false
084            return coinNum
085 
086 
087        end
088    end
089 
090end)
091game.Workspace.RedCoin6.Touched:Connect(function(hit)
092    if coinTrack6 == true then
093        if hit.Parent:WaitForChild("Humanoid") then
094            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
095            coinNum = coinNum + 1
096            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
097            game.Workspace.RedCoin6.Transparency = .7
098 
099            coinTrack6 = false
100            return coinNum
101 
102 
103        end
104    end
105 
106end)
107game.Workspace.RedCoin7.Touched:Connect(function(hit)
108    if coinTrack7 == true then
109        if hit.Parent:WaitForChild("Humanoid") then
110            local player = game.Players:GetPlayerFromCharacter(hit.Parent)
111            coinNum = coinNum + 1
112            player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
113            game.Workspace.RedCoin7.Transparency = .7
114 
115            coinTrack7 = false
116            return coinNum
117 
118 
119        end
120    end
121 
122end)
 View all 122 lines...

1 answer

Answered by

Longjohnnn 28

4 years ago

Edited 4 years ago

I think it's because you are detecting if someone touches the coin and making it so that it changes for everyone so instead you could get the player name that touched the coin and make it specifically so that it changes only for them. You did

1local player = game.Players.LocalPlayer
2player.PlayerGui.CoinTracker.Frame.TextLabel.Text = "Coins "..coinNum.."/7"
3game.Workspace.RedCoin1.Touched:Connect(function(hit)
4 if coinTrack1 == true then
5if hit.Parent:WaitForChild("Humanoid") then
6 coinNum = coinNum + 1
7player.PlayerGui.CoinTracker.Frame.TextLabel.Text =  "Coins "..coinNum.."/7"
8game.Workspace.RedCoin1.Transparency = .7
9return coinNum

where as you need to take hit and get the name of the player that hit it from there and change their Gui

I might be wrong but that's all I've got

Good idea, but I don't think it will change the transparency problem of the coins only changing for a player who collected it BreckleShrimp 23 — 4y

hmm, you could make it transparent for all players using "player" but change the coin amount using hit.Parent Longjohnnn 28 — 4y

I'm not sure what you mean, though i am going to accept this answer because I think in order to do this I'll have to create a LocalMesh which I think is a whole other question. BreckleShrimp 23 — 4y

Sorry for not explaining it properly. I mean find the name of the player that touched the coin by doing hit.parent.parent then comparing it to the localPlayers name and if it is the same then change the Gui but if it is different then don't. Then for making it transparent you could detect if it has been hit and just make it transparent for everyone for example Longjohnnn 28 — 4y

View all comments (4 more)

Coin.touchedConnect(function(hit) Coin.Transparency = 0.7 Longjohnnn 28 — 4y

Also I recommend doing something like local Coin1 = game.Workspace.RedCoin1 to make the script shorter Longjohnnn 28 — 4y

Thank you! BreckleShrimp 23 — 4y

You're welcome :D Longjohnnn 28 — 4y

Why isn't my PlayerGui working as intended?

1 answer

Answer this question