How come this GUI only pops up for one item in the table?
The GUI will only pop up if the item the mouse hovers over is at the bottom of the 'items' table. Script:
01 | local plr = game.Players.LocalPlayer |
02 | repeat wait() until plr.Character |
03 | local char = plr.Character |
04 | local mouse = plr:GetMouse() |
05 | local items = { |
06 | "cup", |
07 | "metal scraps", |
08 | "bottlecap", |
09 | } |
10 |
11 | while wait() do |
12 | if mouse.Target ~= nil then |
13 | for _,v in pairs(items) do |
14 | if mouse.Target.Name:lower() == v then |
15 | script.Parent.Visible = true |
It will work if my mouse hovers over an item named "Bottlecap", but if I hover over an item named "Metal Scraps", it won't. Help?
2 answers
There's something subtle about your code. Let's step through it for "bottlecap" first:
1.a. if target is "cup" then
1.b. (it's not)
1.c. else
1.d. hide gui
2.a. if target is "metal scraps" then
2.b. (it's not)
2.c. else
2.d. hide gui
3.a. if target is "bottlecap" then
3.b. show "pick up " .. "Bottlecap"
3.c. else
3.d. (it was)
Now compare this to running through with "metal scraps"
1.a. if target is "cup" then
1.b. (it's not)
1.c. else
1.d. hide gui
2.a. if target is "metal scraps" then
2.b. show "pick up " .. "Metal Scraps"
2.c. else
2.d. (it was)
3.a. if target is "bottlecap" then
3.b. (it's not)
3.c. else
3.d. (hide gui)
Notice the problem? For each name, you take a definitive action -- either show or hide -- thus the only one that ends up mattering is the last one.
You only want to make it invisible when none match.
A simple way to accomplish that is start by hiding it, then just show it when you put text:
Also, tab your code properly!
01 | while wait() do |
02 | if mouse.Target ~= nil then |
03 | script.Parent.Visible = false |
04 | for _,v in pairs(items) do |
05 | if mouse.Target.Name:lower() == v then |
06 | script.Parent.Visible = true |
07 | script.Parent.Text = "[ Press 'E' to pick up "..mouse.Target.Name.." ]" |
08 | end |
09 | end |
10 | end |
11 | end |
Read comment
01 | local plr = game.Players.LocalPlayer |
02 | repeat wait() until plr.Character |
03 | local char = plr.Character |
04 | local mouse = plr:GetMouse() |
05 | local items = { |
06 | "cup", |
07 | "metal scraps", |
08 | "bottlecap", |
09 | } |
10 |
11 | while wait() do |
12 | if mouse.Target ~= nil then |
13 | for _,v in pairs(items[1, 2, 3]) do --I hope this works. Try to get all items in the table named "items" |
14 | if mouse.Target.Name:lower() == v then |
15 | script.Parent.Visible = true |
I hope this helps
0
0
0
0
0