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

Whats wrong with my script?

Asked by

docrobloxman52 407

9 years ago

Hi I was scripting this game and I came across an error in the output that said "ClassicSword is not a valid member of StarterGear." Can you help?

01price = 550
02plr = game.Players.LocalPlayer
03debounce = false
04if plr.StarterGear.ClassicSword.Configurations.LungeDamage == 50 then
05    script.Parent.Active = false
06    script.Parent.Visible = false
07elseif plr.StarterGear.ClassicSword.Configurations.LungeDamage > 50 or plr.StarterGear.ClassicSword.Configurations.LungeDamage < 50 then
08    script.Parent.MouseButton1Click:connect(function()
09   if debounce then return end
10    local v = plr.leaderstats.Credits
11    debounce = true
12    if v.Value >= price then
13        plr.StarterGear.ClassicSword.Configurations.LungeDamage.Value = 50
14        v.Value = v.Value - price
15        script.Parent.Parent.Buy.Visible = true
16        wait(0.2) 
17 
18        for i = 0.1,1,0.1 do
19            wait(0.1)
20            script.Parent.Parent.Buy.TextTransparency = i
21        end
22 
23        script.Parent.Active = false
24        script.Parent.Visible = false
25        script.Parent.Parent.upgrade2.Active = true
26        script.Parent.Parent.upgrade2.Visible = true
27        script.Parent.Parent.error.Position = UDim2.new(0, 200, 0, 100)
28        script.Parent.Parent.Buy.Position = UDim2.new(0, 200, 0, 100)
29 
30        print('You bought the item!')
31    else
32 
33    print ('Cant buy item!')
34        script.Parent.Parent.error.Visible = true
35script.Parent.Parent.Parent.Frame.error2.Text = "You need "..550 - plr.leaderstats.Credits.Value.." more Credits to buy this!"
36            script.Parent.Parent.error.TextTransparency = 0
37            script.Parent.Parent.error2.TextTransparency = 0
38            wait(2)
39                    for i = .1,1,.1 do
40wait(0.1)
41            script.Parent.Parent.error.TextTransparency = i
42            script.Parent.Parent.error2.TextTransparency = i            
43        end
44    end
45debounce = false
46end)
47end
 View all 47 lines...

1 answer

Answered by

DevChris 235

9 years ago

If your sword is in ServerStorage, then you'd need to clone it in to the backpack like this: game.ServerStorage.ClassicSword:Clone().Parent = plr.Backpack This will not work with FilteringEnabled on if you have it on. Therefore, you should really move it to ReplicatedStorage so it becomes this: game.ServerStorage.ReplicatedStorage.ClassicSword:Clone().Parent = plr.Backpack Also, it's really good practice to make a variable for the sword, like this:

1local sword = plr.Backpack:FindFirstChild("ClassicSword")
2--Now I can use "sword" without having to use the whole path all the time!
3if sword then
4    print("Yay! This worked!")
5end

You can also do this for other objects too, for example the error object you seem to use a lot.

backpack didn't work docrobloxman52 407 — 9y

"Backpack", not "backpack" DevChris 235 — 9y

Yes I know it still didn't work :( docrobloxman52 407 — 9y

It still says sword is not a valid member of Backpack docrobloxman52 407 — 9y

View all comments (3 more)

Is ClassicSword a member of StarterPack? DevChris 235 — 9y

Use WaitForChild instead of indexing it directly if you're expecting it to be there and it isn't yet. 1waffle1 2908 — 9y

^ DevChris 235 — 9y

Whats wrong with my script?

1 answer

Answer this question