onLook

[GarQet]

Siemanko!
Mam mały problem...
Skrypt ogólnie działa, ale ładnie mi spamuje konsole TFSka błędem z którym nie mogę się uporać :<

Skrypt:

function onLook(cid, thing, position, lookDistance)

	local DB_points = db.getResult("SELECT `points` FROM `players` WHERE `id` = '"..getPlayerGUID(thing.uid).."';")
        points = DB_points:getDataInt("points")
        DB_points:free()

	if(isPlayer(thing.uid) == true) then
		doPlayerSetSpecialDescription(thing.uid, (getPlayerSex(thing.uid) == 0 and ".\nShe" or ".\nHe") .. " have "..points.." points")
	return true
	end
end

"..getPlayerGUID(thing.uid).."

Dałem thing.uid bo jak daje cid to pokazuje mi ilość danego gracza który daje look na innego.

Jak już mówiłem skrypt działa poprawnie, ale wywala błąd:

[Error - CreatureScript Interface]
data/creaturescripts/scripts/points.lua:onLook
Description:
data/creaturescripts/scripts/points.lua:3: attempt to concatenate global 'getPlayerGUID' (a function value)
stack traceback:
　　　　data/creaturescripts/scripts/points.lua:3: in function <data/creaturescripts/scripts/points.lua:1>

Nie wiem czemu wywala błąd, uważam że wszystko jest dobrze.
Proszę o pomoc, z góry dzięki +rep

Wy mnie nie lubicie, że nie chcecie mi pomóc?

 

[koob]

if isPlayer(thing) then ?

 

[Paxton]

Co ty, za kazdym razem bedziesz robic DB query?

 

[Deaven]

Ustawże najpierw za pomoca onLogin opis gracza..

 

[Boski Roman]

Spróbuj jak pisze pan @up

#Czaku
Czekam na ODP

 

[Azi]

function onLook(cid, thing, position, lookDistance) 
	thing.stackpos = STACKPOS_TOP_CREATURE
	player = getThingFromPos(thing).uid
	if not(isPlayer(thing)) then return TRUE end
	local DB_points = db.getResult("SELECT `points` FROM `players` WHERE `id` = '"..getPlayerGUID(thing).."';") 
	if(DB_points:getID() == -1)then return TRUE end
	points = DB_points:getDataInt("points") 
    DB_points:free() 
	doPlayerSetSpecialDescription(thing.uid, (getPlayerSex(thing.uid) == 0 and ".\nShe" or ".\nHe") .. " have "..points.." points") 
	return TRUE
end

skutki ciągłego zapytania do bazy mogą mieć nieprzewidziane skutki, jeżeli na bocie 10 osób zrobi sobie co 100ms. zapytanie (looknie na kogoś) 10*10 = 100 zapytań w ciągu sekundy. w ciągu minuty jest to 6,000 zapytań.
Więc odradzam tą metodę...

 

[koob]

@up
lepiej to zrobic na storage + exhaustion
onLogin
jakistamstorage = points

onLogout
database.points = jakistamstorage.

onLook
description += jakistamstorage


@topic
zaimplementuj sobie ten piekny pseudokod.

 

[I16]

function onLook(cid, thing, position, lookDistance)
thing.stackpos = STACKPOS_TOP_CREATURE
player = getThingFromPos(player).uid
if not(isPlayer(thing)) then return TRUE end
local DB_points = db.getResult("SELECT `points` FROM `players` WHERE `id` = "2391"..getPlayerGUID(thing).."';")
if(DB_points:getID() == -1)then return TRUE end
points = DB_points:getDataInt("points")
DB_points:free()
doPlayerSetSpecialDescription(thing.uid, (getPlayerSex(thing.uid) == 5 and ".\eShe" or ".\oHe") .. " have "..points.." points")
return TRUE
end

he, ja mam tak, pewnie zle ale dałem
Liczą się chęci ;s

 

[Azi]

@up
lepiej to zrobic na storage + exhaustion
onLogin
jakistamstorage = points

onLogout
database.points = jakistamstorage.

onLook
description += jakistamstorage

ale po co mi to piszesz? ja tylko przerobiłem na działający jego skrypt, oraz go ostrzegłem o skutkach. Chciał, żeby działał - działa.

he, ja mam tak, pewnie zle ale dałem
Liczą się chęci ;s

wypierdalaj na tnet śmieciu zajebany.

 

[koob]

ale po co mi to piszesz? ja tylko przerobiłem na działający jego skrypt, oraz go ostrzegłem o skutkach. Chciał, żeby działał - działa.

pseudokod byl dla autora tematu.

 

[GarQet]

function onLook(cid, thing, position, lookDistance) 
	thing.stackpos = STACKPOS_TOP_CREATURE
	player = getThingFromPos(thing).uid
	if not(isPlayer(thing)) then return TRUE end
	local DB_points = db.getResult("SELECT `points` FROM `players` WHERE `id` = '"..getPlayerGUID(thing).."';") 
	if(DB_points:getID() == -1)then return TRUE end
	points = DB_points:getDataInt("points") 
    DB_points:free() 
	doPlayerSetSpecialDescription(thing.uid, (getPlayerSex(thing.uid) == 0 and ".\nShe" or ".\nHe") .. " have "..points.." points") 
	return TRUE
end

skutki ciągłego zapytania do bazy mogą mieć nieprzewidziane skutki, jeżeli na bocie 10 osób zrobi sobie co 100ms. zapytanie (looknie na kogoś) 10*10 = 100 zapytań w ciągu sekundy. w ciągu minuty jest to 6,000 zapytań.
Więc odradzam tą metodę...

[Error - CreatureScript Interface]
data/creaturescripts/scripts/points.lua:onLook
Description:
(luaGetThingFromPos) Tile not found

 

[chojrak]

function onLook(cid, thing, position, lookDistance)  
	if(isPlayer(thing.uid)) then
		local res = db.getResult("SELECT `points` FROM `players` WHERE `name` = '".. getCreatureName(thing.uid) ..";")
		if(res:getID() ~= -1) then
			local points = res:getDataInt("points")
			-- tu radz sobie sam.
		end
	end
	return true
end

Btw. nie radze trzymac pola w players, ktore i tak przy login/logout sie zapisuje, lepiej stworz oddzielna tabele lub uzywaj storage.