Making new commands

Help and Feedback Scripting Support
#22

Use elseifs preferably, it would make no sense to use if statements. Use elseifs as it will be repetable, you can just stack commands.

#23

Ok, question why canā€™t I just keep using else statements would that work?

#24

Else works for only one time use usually and it would determine if the command is not it, elseifs are the modern way. However I will once remind that this is not a proper way of an administrator system.

Also your Rename command wonā€™t work as it needs to be Humanoid.DisplayName, do not rename the rootpart.

#25

I keep telling you,you cannot use else statements multiple times!! elseif statements are infinite, where as else statements are only one!

#26

Ok, here my script. Tell me if u like it.

game.Players.PlayerAdded:Connect(function(Player)
	if Player:GetRankInGroup(15423675) >= 245 then
		Player.Chatted:Connect(function(Command)
			local SplitCommand = Command:split(" ")
			--Kill Player
			if SplitCommand[1] == ":kill" or SplitCommand[1] == ":Kill" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 0
				--Kill Player
				
				--Infinite Health
			elseif SplitCommand[1] == ":god" or SplitCommand[1] == ":God" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 10000000000
				--Infinite Health
				
			end
#27

Is this script good?

game.Players.PlayerAdded:Connect(function(Player)
	if Player:GetRankInGroup(15423675) >= 245 then
		Player.Chatted:Connect(function(Command)
			local SplitCommand = Command:split(" ")
			--Kill Player
			if SplitCommand[1] == ":kill" or SplitCommand[1] == ":Kill" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 0
				--Kill Player
				
				--Infinite Health
			elseif SplitCommand[1] == ":god" or SplitCommand[1] == ":God" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 10000000000
				--Infinite Health
				
			end
#28

Overly used elseifs are not always good, it is good to use a table of commands and match of the chat message and perform an action after that, and thats why modulescripts are good to be organized.

#29 
game.Players.PlayerAdded:Connect(function(Player)
	if Player:GetRankInGroup(15423675) >= 245 then
		Player.Chatted:Connect(function(Command)
			local SplitCommand = Command:split(" ")
			--Kill Player
			if SplitCommand[1] == ":kill" or SplitCommand[1] == ":Kill" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 0
				--Kill Player

				--Infinite Health
			elseif SplitCommand[1] == ":god" or SplitCommand[1] == ":God" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 10000000000
				--Infinite Health

			end
		end)
	end
end)

This will work, your previous missed ends, also control the MaxHealth of the Humanoid.

#30

What do you mean by max health??

#31 
				Name.Character.Humanoid.Health = 10000000000
				Name.Character.Humanoid.MaxHealth = 10000000000
#32 
game.Players.PlayerAdded:Connect(function(Player)
	if Player:GetRankInGroup(15423675) >= 245 then
		Player.Chatted:Connect(function(Command)
			local SplitCommand = Command:split(" ")
			--Kill Player
			if SplitCommand[1] == ":kill" or SplitCommand[1] == ":Kill" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 0
				--Kill Player
				
				--Godmode
			elseif SplitCommand[1] == ":god" or SplitCommand[1] == ":God" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.MaxHealth = 10000000000
				--Godmode
				
				--Ungodmode
			elseif SplitCommand[1] == ":ungod" or SplitCommand[1] == ":Ungod" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 100
				--Ungodmode
				
				
			end
		end)
	end
end)

Like that?

#33

Looks good! Nice work! Admin commands are hard.

#34

No. Keep the Health Statement, and add also the Health

#35

Ye I know, I watching a video and itā€™s pretty hard. The video donā€™t cover that much so I really got to use my head. Plus Iā€™m a bad scripter lol

#36 

game.Players.PlayerAdded:Connect(function(Player)
	if Player:GetRankInGroup(15423675) >= 245 then
		Player.Chatted:Connect(function(Command)
			local SplitCommand = Command:split(" ")
			--Kill Player
			if SplitCommand[1] == ":kill" or SplitCommand[1] == ":Kill" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.Health = 0
				--Kill Player

				--Godmode
			elseif SplitCommand[1] == ":god" or SplitCommand[1] == ":God" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.MaxHealth = 10000000000
				Name.Character.Humanoid.Health = 10000000000
				--Godmode

				--Ungodmode
			elseif SplitCommand[1] == ":ungod" or SplitCommand[1] == ":Ungod" then
				local TargetedPlayer = SplitCommand[2]
				local Name = game.Players:FindFirstChild(TargetedPlayer)
				Name.Character.Humanoid.MaxHealth = 100
				Name.Character.Humanoid.Health = 100


			end
		end)
	end
end)
#37

Ok, got it! Let me try that out

#38

I have joined your server, and I am indeed willing to help you.

#39

What do you mean joined your server?

#40

Your countyā€™s server was on your profile.

#41

I thouā€¦ say wa. That ainā€™t suppose to be publicā€¦ I set it to friends onlyā€¦ Itā€™s suppose to me privateā€¦ say wa