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

How can I control "CurrentCamara" if the player's Character is nil?

Asked 281 views1 answers

I am trying to make a spectate system where when a player presses a the Shift button when he/she is dead, it will focus the camera on a player that is alive. When a player is dead in my game, his/her character is set to nil. The player's character doesn't reload until the next round.

--[[
 	NOTE: This is in a LocalScript in StarterPack.
	It makes sure if the player does not have a Character to continue.
]]--

local player = game.Players.LocalPlayer
local mouse = player:GetMouse()
local cam = game.Workspace.CurrentCamera

mouse.KeyDown:connect(function(key)
        if key:byte() == 48 then --If left shift is pressed
		if not player.Character then --If the player is dead
			local randomPlayer = nil
			for i, v in pairs (game.Players:GetPlayers()) do
				if player:FindFirstChild("Character") then
					randomPlayer = v --Finds a random player
					print("Spectating: "..randomPlayer.Name)
					cam.CameraType = Enum.CameraType.Follow
					cam.CameraSubject = randomPlayer.Character.Humanoid
					break
				end
			end
		end
	end
end)

It prints the name of the person whom you are supposed to spectate in the output, but after that, I get an error saying attempting to index a nil value for the line where I am setting the CameraSubject to the randomPlayer's humanoid. I have tried to set the CameraSubject to randomPlayer's head as well, but it gave me the same error message.

Any suggestions on how I can change the CameraSubject of a player's camera when his/her character is in a nil state?

AnonymousEdited

1 answer

✓ Accepted answer

I'm not sure how your code is even running.

Even if the player has a .Character, player:FindFirstChild("Character") will return nil -- the character is a property, not a child.

Your if on line 15 is probably supposed to be using v, not player (and again, not using :FindFirstChild).

Right now, there's no point in randomPlayer as a variable -- you might as well just use v (though I would give it a better name, like other or otherPlayer).

If you actually want to pick a random player with a Character, you should make a list of all of those with Characters (with Humanoids)

mouse.KeyDown:connect(function(key)
	if key:byte() == 48 then --If left shift is pressed
		if not player.Character then --If the player is dead
			local candidates = {}
			for _, other in pairs(game.Players:GetPlayers()) do
				if other.Character and other.Character:FindFirstChild("Humanoid") then
					table.insert(candidates, other.Character.Humanoid)
				end
			end
			
			if #candidates >= 1 then
				-- If there is at least one candidate...
				local randomHumanoid = candidates[math.random(#candidates)]
				-- Search on SH for picking random things for more
				-- information about how this works
				
				cam.CameraType = Enum.CameraType.Follow
				cam.CameraSubject = randomHumanoid
			end
		end
	end
end

Even if the Character is set to nil, the CurrentCamera will remain; that was not the cause of any issue.

AnonymousEdited Permalink

1 comment

0

Oh, okay. Thanks. That "player" in line 15 was just a mistake of transferring. I didn't want to copy and paste my exact code, so I changed it up a bit. It's not like that in the real thing.

Anonymous#

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