07-02-2012, 09:27 PM
Rev 639:
Removed the cWorld:GetPlayer() function both from the API and from the C++ code altogether, it was unsafe. Use cWorld:ForEachPlayer() instead
Added the cWorld:DoWithPlayer() function to both C++ code and the Lua API. It takes two arguments, a player name and a callback to call. It works similar to ForEachPlayer(), but calls the callback only up to once, with the player with the specified name. Returns true if the player was found, false if not. Callback return value is ignored.
Here's an actual code snippet that works with DoWithPlayer():
Removed the cWorld:GetPlayer() function both from the API and from the C++ code altogether, it was unsafe. Use cWorld:ForEachPlayer() instead
Added the cWorld:DoWithPlayer() function to both C++ code and the Lua API. It takes two arguments, a player name and a callback to call. It works similar to ForEachPlayer(), but calls the callback only up to once, with the player with the specified name. Returns true if the player was found, false if not. Callback return value is ignored.
Here's an actual code snippet that works with DoWithPlayer():
Code:
function HandleKillPlayer(Split, InitiatorPlayer)
-- InitiatorPlayer is valid throughout this function, even within the local function KillPlayer()
local World = cRoot:Get():GetDefaultWorld()
local PlayerName = Split[2] or ""
if (PlayerName == "") then
InitiatorPlayer:SendMessage(cChatColor.Green .. "Expected the player name to kill as the parameter")
return true
end
local KillPlayer = function(Player)
-- Player is valid only within this function, it cannot be stord and used later!
Player:SendMessage("Incoming gift from " .. InitiatorPlayer:GetName() .. " to " .. PlayerName .. ": " .. (Split[3] or ""))
Player:TakeDamage(100, InitiatorPlayer)
HasFound = true
end
if (not(World:DoWithPlayer(PlayerName, KillPlayer))) then
InitiatorPlayer:SendMessage(cChatColor.Green .. "Could not find player " .. PlayerName)
end
return true
end