Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Function calling help, what can I do to circumvent this error?
Hello, I've been trying to create an inventory system where it will redraw GUI every time the descendants of the inventory folder has been changed. I created a function to do this in a LocalScript called RedrawGrids(), but when I tried to call it using a MouseEnter I receive the following error in the output:
06:02:49.260 - Players.Rheines.PlayerGui.NewInventory.MainFrame.Inventory.Padding.LocalScript:9: attempt to call global 'RedrawGrids' (a nil value)
This is my script:
01 | local PlayerService = game:GetService("Players") |
02 | local ReplicatedStorage = game:GetService("ReplicatedStorage") |
03 |
04 | local Player = PlayerService.LocalPlayer |
05 | local Equipped = Player:WaitForChild("Equipped") |
06 | local Inventory = Player:WaitForChild("Inventory") |
07 |
08 | --Currently use MouseEnter to check if it works. |
09 | script.Parent.MouseEnter:Connect(RedrawGrids()) |
10 |
11 | --Redraw inventory grids. |
12 | function RedrawGrids() |
13 | print('entered function') |
14 | --Clear all gui to redraw with updated inventory. |
15 | for _,guis in pairs(script.Parent:GetChildren()) do |
View all 26 lines...
However, when I made RedrawGrids into an anonymous function such as
01 | script.Parent.MouseEnter:Connect(function() |
02 | print('entered function') |
03 | --Clear all gui to redraw with updated inventory. |
04 | for _,guis in pairs(script.Parent:GetChildren()) do |
05 | if guis:IsA("ImageLabel") then |
06 | guis:Destroy() |
07 | end |
08 | end |
09 | --Redraw inventory contents. |
10 | for _,gear in pairs(Inventory:GetChildren()) do |
11 | local gui = ReplicatedStorage.ItemGUI:Clone() |
12 | gui.Name = gear.Type.Value |
13 | gui.Parent = script.Parent |
14 | end |
15 | end) |
it works perfectly fine. However, I want to call RedrawGrids as a variable so that I can reuse it as often as I like. Is there something that I did wrong?
0
You're calling the function before you define it on Line 9. xPolarium 1388 — 6y
0
The issue is that in your first attempt - function RedrawGrids doesn't exist when you try to use it. In your second attempt - it works because you are hooking MouseEnter directly to the events you want to happen. To circumvent this, create the RedrawGrids function before you try to hook it to an event. SummerEquinox 643 — 6y
1 answer
Log in to vote
0
The error is: in :Connect do not use :Connect(function()) you need to use :Connect(function) and you need to put it after function to detect that it is not a nil value
Here is fixed script:
01 | local PlayerService = game:GetService("Players") |
02 | local ReplicatedStorage = game:GetService("ReplicatedStorage") |
03 |
04 | local Player = PlayerService.LocalPlayer |
05 | local Equipped = Player:WaitForChild("Equipped") |
06 | local Inventory = Player:WaitForChild("Inventory") |
07 |
08 | -- Changed location of MouseEnter. |
09 |
10 | --Redraw inventory grids. |
11 | function RedrawGrids() |
12 | print('entered function') |
13 | --Clear all gui to redraw with updated inventory. |
14 | for _,guis in pairs(script.Parent:GetChildren()) do |
15 | if guis:IsA("ImageLabel") then |
View all 28 lines...
Hope it helped :D
Errors? tell-me on comments.
Solved your problems? put in title [SOLVED] or accept a answer.
0
Simple mistake, thank you so much! Learned something new. Rheines 661 — 6y