Lua Function isPlayerOnline

[Shinmaru]

Ok I've made a function to check if a player is online, but not sure if it's correctly done.

function isPlayerOnline(cid)
	local players = {}
	for _, pid in ipairs(getPlayersOnline()) do
		table.insert(players, pid)
	end
	local online = players[cid]
	if online then
		return true
	end
return false	
end

 

[Shinmaru]

BUMPING For JUSTICE!

 

[VirrageS]

function isPlayerOnline(cid)
	local online = false
	for _, pid in ipairs(getPlayersOnline()) do
		if cid == pid then
			online = true
		end
	end
	return online
end

 

[Shinmaru]

function isPlayerOnline(cid)
	local online = false
	for _, pid in ipairs(getPlayersOnline()) do
		if cid == pid then
			online = true
		end
	end
	return online
end

Thanks Will test it!

 

[koob]

function isPlayerOnline(cid)
	for _, pid in ipairs(getPlayersOnline()) do
		if cid == pid then
			return true -- no need to iterate next ones
		end
	end
	return false
end

Should work same:

function isPlayerOnline(cid)
        return isPlayer(cid)
end

Fixed. Didnt you wanted to use isPlayerOnline(name) ? This one is kinda useless.

 

[Shinmaru]

Should work same:

function isPlayerOnline(cid)
        return isPlayer(cid)
end

Fixed. Didnt you wanted to use isPlayerOnline(name) ? This one is kinda useless.

Virrages's worked fine and so will yours.
Useless!? Maybe, but I use this on looped addEvents, just in case the player logout the loop does not keep going on an infinite cycle.

 

[Exedion]

use this, work exactly you want for on/offline players

isCreature(cid)

 

[koob]

Useless!? Maybe, but I use this on looped addEvents, just in case the player logout the loop does not keep going on an infinite cycle.

Thats why its useless since u can use just isPlayer() or isCreature(). No need to loop playersOnline to check it.

 

[Shinmaru]

Thats why its useless since u can use just isPlayer() or isCreature(). No need to loop playersOnline to check it.

will try it!

 

[Exedion]

will try it!

only use isCreature, because isPlayer send a error is the player is offline

 

[VirrageS]

only use isCreature, because isPlayer send a error is the player is offline

I don't think so.

 

[knightxd]

Virrages's worked fine and so will yours.
Useless!? Maybe, but I use this on looped addEvents, just in case the player logout the loop does not keep going on an infinite cycle.

function onLogout(cid)
stopEvent(eventid)
end

or also

while isPlayer(cid) do
..
sleep(ms)
end

...