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

Arguments and Parameters, I don't understand it?

Asked 502 views2 answers
GroupId = 290188
MinimumRank = 25

function onChatted(msg,player)
	if player:GetRankInGroup(GroupId) >= MinimumRank then

if msg:sub(1,5) == "!Host" then
	player.TeamColor = BrickColor.new("Eggplant")
	player:LoadCharacter()

	end
end

How does the script know that the variable/argument "player" is the LocalPlayer?

Anonymous

2 answers

✓ Accepted answer

##Answer

It doesn't.

##Problem

  1. Chatted scripts and scripts dealing with PlayerAdded of this nature should be handled in a Server Script. You can not access LocalPlayer through server side scripts normally as mentioned in the blog article Common Mistakes. Since you mentioned LocalPlayer, I thought I should clear that up.
  2. You have the function onChatted which is not called anywhere else in the script.
  3. You also have two arguments that are never involved in the function as well.
  4. Fourth, you're missing an end for the second if then statement.

##Solution

  1. Use a server script and use the PlayerAdded event. Inside the function for the PlayerAdded event, you can connect a function to the Chatted event.
  2. Within the chatted event function, call the function that handles the command.
  3. Within the function call you're using, have the variables for the player and their message in the parenthesis.
  4. Add that third end necessary for the if-then statement to finish.

##Recommendation

Since Roblox, at the time of this post, broke the PlayerAdded event in Play Solo mode. You will need to add a for loop at the bottom of the script to loop through all players and connect them to the PlayerAdded event. This is so you may test the script properly in play solo mode.

Also, when dealing with commands, make the chat and command the same case (as in upper or lower case). Otherwise, it might annoy the heck out of the user for not having the correct capitalization of a command.

##Final Script

GroupId = 290188
MinimumRank = 25

function onChatted(msg,player)
    if player:GetRankInGroup(GroupId) >= MinimumRank then
        if msg:sub(1,5):lower() == "!host" then --Just add :lower() to the end of the string to make whatever they say lowercase to the script. This way commands are not case sensitive.
              player.TeamColor = BrickColor.new("Eggplant")
              player:LoadCharacter()
          end --That third end you forgot.
    end
end

function PlayerAdded(player) --Player is given to us by the PlayerAdded event. It's the physical player object that will be used.
    player.Chatted:connect(function(Chat) --.Chatted is an event of a player. Every time they chat, it goes through this event. This is an anonymous function. It's not exactly a function we can call, more of the function we are using. Chat is our string that the player has said.
        onChatted(Chat, player) --Now we are calling the function Chat being msg to the OnChatted function, and player being player to the function.
    end) --Annonymous functions have an end with a ) to show the end of the connect function.
end

game.Players.PlayerAdded:connect(PlayerAdded)
for _,player in pairs(game.Players:GetPlayers()) do --For every player in Players service, do something.
    PlayerAdded(player) --Alright, we'll add them to the function we created.
end

######Hopefully, this answered your question. If it did, do not forget to hit the accept answer button. If you have any questions, feel free to post them in the comments below.

AnonymousEdited Permalink

7 comments

0

I already did that before, I just didn't post the whole script because it's really long due to other commands I put in. Unfortunatly this didn't answer my question, I don't mean "LocalPlayer" I meant like the Player under the "Players" folder. For example, if I changed my argument player to something like "random" it'd still work. How does the script do that? I haven't defined the argument (I read

Anonymous#
0

Well, I am not fully understanding your question then. I would not know how the onChatted command gets a hold of the player or message through what you have shown me.

Anonymous#
0

I posted the full script below

Anonymous#
0

Why not just edit your question? Plus, I basically went over it in my answer. The PlayerAdded event will provide the physical player that just joined the game, it was part of my recommendation to add a loop to go through all players to test in Play Solo properly. I also mentioned that the Chatted event is an event for player objects which will provide a string to your function.

Anonymous#
0

Whenever most events are fired, they will provide an object, string, boolean, or some type of value that the function can use as an argument. When you're calling your function on line 101, you're providing the player and message values the events on line 99 and 100 provide.

Anonymous#
0

You just needed to say that! Thank you, I get it now

Anonymous#
0

"--Player is given to us by the PlayerAdded event. It's the physical player object that will be used.", "--.Chatted is an event of a player. Every time they chat, it goes through this event. This is an anonymous function. It's not exactly a function we can call, more of the function we are using. Chat is our string that the player has said." I did...

Anonymous#
GroupId = 290188
MinimumRank = 25
----------------------------------------------------

------------------------------------------------------

function onChatted(msg,player)
	if player:GetRankInGroup(GroupId) >= MinimumRank then
	if msg:sub(1, 8):lower() == "!poweron" then
		game.Lighting.FogEnd = 1000000
		wait(2)
	game.Lighting.Brightness = 100
	wait(3)
	game.Workspace.Stairs.Stair.BrickColor = BrickColor.Yellow()
	wait(0.3)
	------------------------------------------------------------------------
	game.Workspace.Stairs2.Stair.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs2.Model2.Stair2.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs2.Model2.Stair25.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs2.Model.Stair3.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs2.Model.Stair35.BrickColor = BrickColor.Yellow() 
	wait(0.3)
	-------------------------------------------------------------------------
	game.Workspace.Stairs5.Model2.Stair2.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs5.Model2.Stair25.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs5.Stair.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs5.Model.Stair3.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs5.Model.Stair35.BrickColor = BrickColor.Yellow()
	wait(0.3)
	------------------------------------------------------------------------
	game.Workspace.Stairs3.Stair.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs3.Model2.Stair2.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs3.Model2.Stair25.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs3.Model.Stair3.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Stairs3.Model.Stair35.BrickColor = BrickColor.Yellow()
	wait(0.3)
	------------------------------------------------------------------------
	-----------------------------END OF STAIRS------------------------------
	game.Workspace.Line.Line1.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Line.Line2.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Line.Line3.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.Line.Line4.BrickColor = BrickColor.Yellow()
	wait(0.3)
	game.Workspace.SideLines.Part1.BrickColor = BrickColor.Yellow()
	wait(0.3)
		game.Workspace.SideLines.Part2.BrickColor = BrickColor.Yellow()
		wait(0.3)
			game.Workspace.SideLines.Part3.BrickColor = BrickColor.Yellow()
			wait(0.3)
				game.Workspace.SideLines.Part4.BrickColor = BrickColor.Yellow()
				wait(0.3)
					game.Workspace.SideLines.Part5.BrickColor = BrickColor.Yellow()
					wait(0.3)
						game.Workspace.SideLines.Part6.BrickColor = BrickColor.Yellow()
						wait(0.3)
							game.Workspace.SideLines.Part7.BrickColor = BrickColor.Yellow()
							wait(0.3)
								game.Workspace.SideLines.Part8.BrickColor = BrickColor.Yellow()
								wait(0.3)
--------------------------------------END OF LINES------------------------------------------------
--------------------------------------END OF POWER ON---------------------------------------------

			
	end
	end	
	
	if msg:sub(1,5):lower() == "!host" then	
	player.TeamColor = BrickColor.new("Eggplant")
	player:LoadCharacter()
	game.Lighting.Host:Clone().Parent = player.PlayerGui
	wait(5)
	player.PlayerGui.Host:Remove()
	
	
	

	end
	end
	




game.Players.PlayerAdded:connect(function(player)
    player.Chatted:connect(function(message) 
	onChatted(message, player) 
	end)
end)
AnonymousPermalink

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