ajuda Error Script Outfit

[Xiolones 8]

Olá galera do TK!

 

Gostaria de uma ajuda no seguinte script

function onUse(cid, item, frompos, item2, topos)

        if item.uid == 4113 then
          if getPlayerItemCount(cid,2476) >= 5 then
      doPlayerRemoveItem(cid,2476,5)
      doPlayerRemoveItem(cid,2477,5)
      doPlayerRemoveItem(cid,2396,5)
        queststatus = getPlayerStorageValue(cid,100004)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full knight addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,139,3)
                doPlayerAddOutfit(cid,131,3)
        setPlayerStorageValue(cid,100004,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"You do not have the items needed to make the addon!")
        end
        else
        return 0
        end

        return 1
        end

ps: (não sei colocar spoiler nesse editor)

 

Como funciona:

Coloquei o UniqueID em uma alavanca, que ao clicar nesta alavanca, ela removerá os items (knight armor, knight legs e ice rapiers (5x cada), e lhe dará o addon de Knight !

Porém, está com um bug, que só confere a knight armor, se o cara estiver com apenas 5 knights armor, ele consegue concluir o addon !

Provavelmente o error está entre as linhas 3 à 6 .. e não consigo arrumar!

 

Me ajudem a resolver este problema por favor, e disponibilizar este script pra galera do TK !
Sistema testado TFS 0.3.6 - Tibia 8.6

 

Vale REP +

 

[kbelin 51]

Como você pode ver na quarta linha>           if getPlayerItemCount(cid,2476) >= 5 then         <  O count só está configurada para a knight armor, adicione count para os outros.

[Xiolones 8]

Sempre que eu adiciona da error !
Pode dar um exemplo de como faz sem dar error ?

 

////edit

function onUse(cid, item, frompos, item2, topos)

        if item.uid == 4113 then
          if getPlayerItemCount(cid,2476) >= 5 then
      doPlayerRemoveItem(cid,2476,5)
          if getPlayerItemCount(cid,2477) >= 5 then
      doPlayerRemoveItem(cid,2477,5)
          if getPlayerItemCount(cid,2396) >= 5 then
      doPlayerRemoveItem(cid,2396,5)
        queststatus = getPlayerStorageValue(cid,100004)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full knight addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,139,3)
                doPlayerAddOutfit(cid,131,3)
        setPlayerStorageValue(cid,100004,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"You do not have the items needed to make the addon!")
        end
        else
        return 0
        end

        return 1
        end

faço assim e da error no script !
daquele outro jeito o script dá certo..mas buga !

Editado Outubro 31, 2014 por Xiolones (veja o histórico de edições)

[D i M i T r E s C u 127]

Nuss fiz esse teste

 

script

function onUse(cid, item, frompos, item2, topos)
local itens = {2476, 2477, 2396}

if item.uid == 4113 then
  if getPlayerItemCount(cid, itens) >= 5 then
      doPlayerRemoveItem(cid, itens, 5)
        queststatus = getPlayerStorageValue(cid,100004)
if queststatus == -1 then
    doPlayerSendTextMessage(cid,22,"Você ganhou um addon cavaleiro completo!")
      doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
      doPlayerAddOutfit(cid, 139,3)
       doPlayerAddOutfit(cid, 131,3)
        setPlayerStorageValue(cid,100004,1)
else
    doPlayerSendTextMessage(cid,22,"Você já possui este addon!")
end


else
    doPlayerSendTextMessage(cid,22,"Você não tem os itens necessários para fazer o addon!")
end
else
return 0
end


return 1
end

Editado Outubro 31, 2014 por alanmtd1 (veja o histórico de edições)

[Xiolones 8]

Vou testar aqui alan, e já dou a resposta !

 

/////edit

Alan, deu certo! Porém, tipo estou fazendo isso com todos os addons, dai na opção de quantidade tem algumas falhas..

function onUse(cid, item, frompos, item2, topos)

        if item.uid == 4112 then
          if getPlayerItemCount(cid,2160) >= 25 then
      doPlayerRemoveItem(cid,2160,25)
      doPlayerRemoveItem(cid,2189,1)
      doPlayerRemoveItem(cid,2191,1)
      doPlayerRemoveItem(cid,2187,1)
      doPlayerRemoveItem(cid,2190,1)
      doPlayerRemoveItem(cid,2181,1)
      doPlayerRemoveItem(cid,2188,1)
      doPlayerRemoveItem(cid,8921,1)
        queststatus = getPlayerStorageValue(cid,100003)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full mage addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,138,3)
                doPlayerAddOutfit(cid,120,3)
        setPlayerStorageValue(cid,100003,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"I need 1 wand of cosmic energy, 1 wand of draconia, 1 wand of dragonbreath, 1 wanf of inferno, 1 wand of vortex, 1 terra rod, 1 common akilles wand and more 250.000 cash for this addon!")
        end
        else
        return 0
        end

        return 1
        end

Por exemplo esse, que remove 1 wand de cada, e 25 crystal coins .. como eu faria para funcionar pelo 'local items' ?

 

 

//////edit

 

 

acho que vou fazer um local itens2 .. será que da certo ?

Editado Outubro 31, 2014 por Xiolones (veja o histórico de edições)

[D i M i T r E s C u 127]

Nao entendi oque voce disse ,  e o script anterior tem falhas?

se tiver acho que ja sei oque é!

[Xiolones 8]

  Em 31/10/2014 em 23:19, alanmtd1 disse:

Nao entendi oque voce disse ,  e o script anterior tem falhas?

se tiver acho que ja sei oque é!

Mostrar mais  

function onUse(cid, item, frompos, item2, topos)
local itens = {2160}
local itens2 = {2189, 2191, 2187, 2190, 2181, 2188, 8921}

        if item.uid == 4112 then
        if getPlayerItemCount(cid, itens) >= 25 then
      doPlayerRemoveItem(cid, itens, 25)
        if getPlayerItemCount(cid, itens2) >= 1 then
      doPlayerRemoveItem(cid, itens2, 1)
        queststatus = getPlayerStorageValue(cid,100003)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full mage addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,138,3)
                doPlayerAddOutfit(cid,120,3)
        setPlayerStorageValue(cid,100003,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"I need 1 wand of cosmic energy, 1 wand of draconia, 1 wand of dragonbreath, 1 wanf of inferno, 1 wand of vortex, 1 terra rod, 1 common akilles wand and more 250.000 cash for this addon!")
        end
        else
        return 0
        end

        return 1        
end

Fiz isso não deu certo .. acho que tem alguma a coisa haver cm os 2 'ifs'

 

O seu script não tem falhas! O problema é que só serve se for pegar items com a mesma quantidade.. exemplo 5 wand of inferno, 5 wands of decay ..

Se fosse pra pegar 4 wands of inferno e 5 wands of decay, ele não encaixaria mais,, entende ?

Editado Outubro 31, 2014 por Xiolones (veja o histórico de edições)

[D i M i T r E s C u 127]

voce fez quase certo 

testa esse

function onUse(cid, item, frompos, item2, topos)
local dinheiro = {2160}
local grana = {2189, 2191, 2187, 2190, 2181, 2188, 8921}


if item.uid == 4112 then
  if getPlayerItemCount(cid, grana) = 25 then
      doPlayerRemoveItem(cid, grana, 25)
end     
if getPlayerItemCount(cid, itens) = 1 then
      doPlayerRemoveItem(cid, itens, 1)
        queststatus = getPlayerStorageValue(cid,100003)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full mage addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,138,3)
                doPlayerAddOutfit(cid,120,3)
        setPlayerStorageValue(cid,100003,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"I need 1 wand of cosmic energy, 1 wand of draconia, 1 wand of dragonbreath, 1 wanf of inferno, 1 wand of vortex, 1 terra rod, 1 common akilles wand and more 250.000 cash for this addon!")
        end
        else
        return 0
        end


        return 1        
end

[Xiolones 8]

 

 

 

dá o mesmo error !

'--'

[D i M i T r E s C u 127]

EU ja sei oque foi pera to vendo uma coisa nessa script jaja te mando

acho que agora vai e.e

 

function onUse(cid, item, frompos, item2, topos)
local dinheiro = {2160}
local grana = {2189, 2191, 2187, 2190, 2181, 2188, 8921}


if item.uid == 4112 then
  if getPlayerItemCount(cid, grana) == 25 then
      doPlayerRemoveItem(cid, grana, 25)
end     
if getPlayerItemCount(cid, itens) = 1 then
      doPlayerRemoveItem(cid, itens, 1)
        queststatus = getPlayerStorageValue(cid,100003)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full mage addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,138,3)
                doPlayerAddOutfit(cid,120,3)
        setPlayerStorageValue(cid,100003,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"I need 1 wand of cosmic energy, 1 wand of draconia, 1 wand of dragonbreath, 1 wanf of inferno, 1 wand of vortex, 1 terra rod, 1 common akilles wand and more 250.000 cash for this addon!")
        end
        else
        return 0
        end


        return 1        
end

[Xiolones 8]

:'(

 

Deu certo também não, o erro foi solucionado, porém mesmo com todos os items necessários, continua falando que está faltando os items!

 

Já fiz todos os scripts, o único problema é colocar 2 items diferentes com quantidades diferentes .. '---'

Editado Novembro 1, 2014 por Xiolones (veja o histórico de edições)

[Danves 45]

O erro do distro é porque tava if getPlayerItemCount(cid, itens) = 1 then
onde no caso deveria ser "==", pois o "=" serve pra definir algo, enquanto == é pra conferir se ta igual a x.

[D i M i T r E s C u 127]

amigo teste mais 1 vez

function onUse(cid, item, frompos, item2, topos)
local dinheiro = {2160}
local grana = {2189, 2191, 2187, 2190, 2181, 2188, 8921}


if item.uid == 4112 then
  if getPlayerItemCount(cid, grana) == 25 then
      doPlayerRemoveItem(cid, grana, 25)
end     
if getPlayerItemCount(cid, itens) == 1 then
      doPlayerRemoveItem(cid, itens, 1)
        queststatus = getPlayerStorageValue(cid,100003)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full mage addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,138,3)
                doPlayerAddOutfit(cid,120,3)
        setPlayerStorageValue(cid,100003,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"I need 1 wand of cosmic energy, 1 wand of draconia, 1 wand of dragonbreath, 1 wanf of inferno, 1 wand of vortex, 1 terra rod, 1 common akilles wand and more 250.000 cash for this addon!")
        end
        else
        return 0
        end


        return 1        
end

Danvez era isso mesmo eu só fui ver agora , vlw por me alerta!

[Xiolones 8]

  Em 31/10/2014 em 23:59, Danves disse:

O erro do distro é porque tava if getPlayerItemCount(cid, itens) = 1 then

onde no caso deveria ser "==", pois o "=" serve pra definir algo, enquanto == é pra conferir se ta igual a x.

Mostrar mais  

 

arrumei o erro danves, vlw por esclarecer !

Porém, não sei porque, eu com todos os items na backpack, ainda fala que é necessários tais items pra poder concluir a quest..

Pela minha vista, o script não tem erros..mas sou leigo!

  Em 01/11/2014 em 00:07, alanmtd1 disse:

amigo teste mais 1 vez

function onUse(cid, item, frompos, item2, topos)
local dinheiro = {2160}
local grana = {2189, 2191, 2187, 2190, 2181, 2188, 8921}


if item.uid == 4112 then
  if getPlayerItemCount(cid, grana) == 25 then
      doPlayerRemoveItem(cid, grana, 25)
end     
if getPlayerItemCount(cid, itens) == 1 then
      doPlayerRemoveItem(cid, itens, 1)
        queststatus = getPlayerStorageValue(cid,100003)
        if queststatus == -1 then
        doPlayerSendTextMessage(cid,22,"You won a full mage addon!")
                doSendAnimatedText(getPlayerPosition(cid), "", TEXTCOLOR_ORANGE)
                doPlayerAddOutfit(cid,138,3)
                doPlayerAddOutfit(cid,120,3)
        setPlayerStorageValue(cid,100003,1)
        else
        doPlayerSendTextMessage(cid,22,"You own this addon!")
        end
        else
        doPlayerSendTextMessage(cid,22,"I need 1 wand of cosmic energy, 1 wand of draconia, 1 wand of dragonbreath, 1 wanf of inferno, 1 wand of vortex, 1 terra rod, 1 common akilles wand and more 250.000 cash for this addon!")
        end
        else
        return 0
        end


        return 1        
end

Danvez era isso mesmo eu só fui ver agora , vlw por me alerta!

Mostrar mais  

 

Alan, tipo, o script deu certo, não tem erros na distro.. porém, mesmo eu com todos os items necessários, a quest me diz que ainda faltam items, que eu não os tenho! '---'

Não to entendendo porque, o script parece estar certo!

 

Editado Novembro 1, 2014 por Xiolones (veja o histórico de edições)

[Orochi Elf 499]

Bom amigo, eu tentei desenvolver algo mais fácil de edição.

Eu fiz de um modo que você pode adicionar todas as outfits em um script só.

Basta ir editando a tabela de outfits, do mesmo modo que eu fiz

 

Não testei.

local outfits = {
    [4113] = {
        items = {
            [1] = {itemid = 2476, ammount = 5},
            [2] = {itemid = 2477, ammount = 5},
            [3] = {itemid = 2396, ammount = 5},
        },
        out = {
            [1] = 131, -- Female
            [2] = 139, -- Male
        },
        storageId = 10101,
    },
}

function onUse(cid, item)    
    local t = outfits[item.uid]
    
    if t then
        if getPlayerStorageValue(cid, t.storageId) <= 0 then
            for r = 1, #t.items do
                if(getPlayerItemCount(cid, t.items[r].itemid) < t.items[r].ammount) then
                
                    doPlayerSendTextMessage(cid, 22, "You need "..t.items[r].ammount.." "..getItemNameById(t.items[r].itemid))
                    return true
                end
                
                doPlayerRemoveItem(cid, t.items[r].itemid, t.items[r].ammount)
            end
            
            doPlayerAddOutfit(cid, t.out[getPlayerSex(cid) + 1], 3)
            doPlayerSendTextMessage(cid, 22, "You won a new outfit.")
            setPlayerStorageValue(cid, t.storageId, 1)
        end
    end
    return true
end

[Xiolones 8]

  Em 01/11/2014 em 00:43, Orochi Elf disse:

 

Bom amigo, eu tentei desenvolver algo mais fácil de edição.

Eu fiz de um modo que você pode adicionar todas as outfits em um script só.

Basta ir editando a tabela de outfits, do mesmo modo que eu fiz

 

Não testei.

local outfits = {
    [4113] = {
        items = {
            [1] = {itemid = 2476, ammount = 5},
            [2] = {itemid = 2477, ammount = 5},
            [3] = {itemid = 2396, ammount = 5},
        },
        out = {
            [1] = 131, -- Female
            [2] = 139, -- Male
        },
        storageId = 10101,
    },
}

function onUse(cid, item)    
    local t = outfits[item.uid]
    
    if t then
        if getPlayerStorageValue(cid, t.storageId) <= 0 then
            for r = 1, #t.items do
                if(getPlayerItemCount(cid, t.items[r].itemid) < t.items[r].ammount) then
                
                    doPlayerSendTextMessage(cid, 22, "You need "..t.items[r].ammount.." "..getItemNameById(t.items[r].itemid))
                    return true
                end
                
                doPlayerRemoveItem(cid, t.items[r].itemid, t.items[r].ammount)
            end
            
            doPlayerAddOutfit(cid, t.out[getPlayerSex(cid) + 1], 3)
            doPlayerSendTextMessage(cid, 22, "You won a new outfit.")
            setPlayerStorageValue(cid, t.storageId, 1)
        end
    end
    return true
end

Expand   Mostrar mais  

Nossa Orochi, belo script '-' me sinto humilhado! hehe

 

Não deu error na distro, porém algo não deu certo..

Eu puxo a alavanca, e além dela não virar, ela não me manda nenhuma mensagem que eu preciso dos items ou que ganhei o addon ..

Alguma coisa não funcionou, talvez a forma que coloquei.. olha como coloquei o script..pode ver se fiz algo errado ?

Sò editei os items,storage(seguindo padrão), e os addons ..

local outfits = {
    [4112] = {
        items = {
            [1] = {itemid = 2189, ammount = 1},
            [2] = {itemid = 2191, ammount = 1},
            [3] = {itemid = 2187, ammount = 1},
            [4] = {itemid = 2190, ammount = 1},
            [5] = {itemid = 2181, ammount = 1},
            [6] = {itemid = 2188, ammount = 1},
            [7] = {itemid = 8921, ammount = 1},
            [8] = {itemid = 2160, ammount = 25},
        },
        out = {
            [1] = 130, -- Female
            [2] = 138, -- Male
        },
        storageId = 100003,
    },
}

function onUse(cid, item)    
    local t = outfits[item.uid]
    
    if t then
        if getPlayerStorageValue(cid, t.storageId) <= 0 then
            for r = 1, #t.items do
                if(getPlayerItemCount(cid, t.items[r].itemid) < t.items[r].ammount) then
                
                    doPlayerSendTextMessage(cid, 22, "You need "..t.items[r].ammount.." "..getItemNameById(t.items[r].itemid))
                    return true
                end
                
                doPlayerRemoveItem(cid, t.items[r].itemid, t.items[r].ammount)
            end
            
            doPlayerAddOutfit(cid, t.out[getPlayerSex(cid) + 1], 3)
            doPlayerSendTextMessage(cid, 22, "You won a new outfit.")
            setPlayerStorageValue(cid, t.storageId, 1)
        end
    end
    return true
end

[Orochi Elf 499]

Bom.. voce sabe que ali na tabela onde voce colocou [4112], é o UniqueID, certo?

Confere direitinho ai.

 

Se não funcionar, a gente começa uma bateria de testes kk

[Xiolones 8]

Sei sim, o uniqueID está certo conforme a alavanca .. tô tentando aqui, se conseguir me ajudar, agente vai testando!

////up