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

Using ObjectValue as string?

Asked 187 views1 answers
OwnerName = script.Parent.Parent.Parent.Owner.Value
while true do
	if script.Parent.Parent.Parent.Injuries.Value > 35 then
		wait (1)
	 script.Parent.Parent.Parent.Parent.Parent.Parent.Parent.Players[OwnerName].PlayerGui.VisitorPopup.TextButton.Visible = true
		script.Parent.Head.Transparency = 0
		script.Parent.Head.CanCollide = true
		print ("Call Visitor to office appeared!")
		return false
	else
		wait (1)
	end
end

I receive an error trying to use the value of Owner as OwnName when referring to the players. "bad arguement #2 to '?' (string expected, got nil)

Ideally, the script is supposed to take the value of owner, set that as "OwnerName" and use it when referencing Players so the GUI would only appear on that specific player's screen.

I think the error is coming from the fact that Owner.Value is an ObjectValue instead of a StringValue, but for the purposes of what I'm trying to do, I cannot change it to a string. Is there any other way to make this work?

Anonymous

1 answer

✓ Accepted answer

What's wrong is that you're setting OwnerName to Owner's value. You should instead set it to simply Owner, like below.

Also, I'm not sure if you can implicitly convert an object to a string, but I assume not, so you should use OwnerName.Value.Name

Finally, I noticed that you have a lot of indexing (i.e. script.Parent.Parent.Parent etc.). Since it's running on a loop, this can cause performance issues. Not to mention, it's also very long and unnecessary.

Here is the final script with the improvements and a few extra comments:

local Owner = script.Parent.Parent.Parent.Owner -- Global variables are evil.  Use local variables instead.
while true do
    if script.Parent.Parent.Parent.Injuries.Value > 35 then -- You may wish to optimise this using variables.
        wait (1)
	-- Here I assume the variable is the Player object.  If not, use game.Players[Owner.Value.Name] instead of Owner.Value
     Owner.Value.PlayerGui.VisitorPopup.TextButton.Visible = true
        script.Parent.Head.Transparency = 0
        script.Parent.Head.CanCollide = true
        print ("Call Visitor to office appeared!")
        return false
    else
        wait (1)
    end
end
AnonymousEdited Permalink

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