Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
All questions/Question #48545
✓ ANSWER ACCEPTED

should I make my paths different in a normal script than a local script?

Asked 226 views1 answers

I tried using a normal script instead of a local script but I keep getting errors like "this is not a valid member of model" my code was originally in a localscript but when i move it to a normal script I get the errors. my script is down.

local plr = game.Players.LocalPlayer
local char = plr.Character
local mouse = plr:GetMouse()
local tool = script.Parent
local event = game.ReplicatedStorage:WaitForChild("RemoteEvent")
tool.Activated:Connect(function()
    local hit = mouse.Hit
    Event:FireServer(Hit)
end)

should I get rid of the local in front of my variables?

AnonymousEdited

2 comments

0

Please explain more, like what path did you use on the normal script? Your questions might get moderated if you do not provide enough info. Providing a script also helps.

Anonymous#
0

k

Anonymous#

1 answer

✓ Accepted answer

Try this, also sorry if there are some errors this was typed on the phone.

if script.Parent.Parent.ClassName ~= "Model" then
    local plr = script.Parent.Parent
elseif script.Parent.Parent.ClassName == "Model" then
    local plr = game.Players:GetPlayerFromCharacter(script.Parent.Parent)
end
local char = plr.Character
local mouse = plr:GetMouse()
local tool = script.Parent
local event = game.ReplicatedStorage:WaitForChild("RemoteEvent")
tool.Activated:Connect(function()
    local hit = mouse.Hit
    Event:FireServer(hit)
end)

At line 1 you attempted to call LocalPlayer which can only be accessed by a local script. Now how I got to my edit of that line is simple, it checks if the tool is equipped via method of if the tool is a child of players or the character, depending on the situation the variable will be different. Also do not remove the "local" before each variable, its better on the script and is good practice. Also on your original line 8 you incorrectly notated "hit".Remember variables are case sensitive.

AnonymousPermalink

2 comments

0

ClassName is read only, you would have to use:IsA(), so technically, for line1 through 5 you would do if not script.Parent.Parent:IsA("Model") then local plr = script.Parent.Parent elseif script.Parent.Parent:IsA("Model") then local plr = game.Players:GetPlayerFromCharacter(script.Parent.Parent) end

Anonymous#
0

he's not attempting to write to ClassName^

Anonymous#

This discussion is preserved as it was. New answers, comments, and votes are closed.