[7.6] Niedziałający skrypt

[Cwelf]

Witam,
jestem początkującym skrypterem i mam nadzieję, że ktoś mi pomoże. To tylko początkowy skrypt, aby chociaż troszkę ogarnąć to wszystko. Proszę o wyrozumiałość. ; )
A więc nie działa mi ten skrypt:

-- Skrypt by Cwelf --
function onUse(cid, item, fromPosition, itemEx, toPosition)
if item.itemid ==2547 then
doRemoveItem(item.uid,2547)
doPlayerAddHealth (cid, 100)
doSendMagicEffect(playerpos, 12)
doPlayerSay (cid, "Aaaaach...", 16)
else
return 1
end

[Grim90]

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Witam,
jestem początkującym skrypterem i mam nadzieję, że ktoś mi pomoże. To tylko początkowy skrypt, aby chociaż troszkę ogarnąć to wszystko. Proszę o wyrozumiałość. ; )
A więc nie działa mi ten skrypt:

-- Skrypt by Cwelf --
function onUse(cid, item, fromPosition, itemEx, toPosition)
if item.itemid ==2547 then
doRemoveItem(item.uid,2547)
doPlayerAddHealth (cid, 100)
doSendMagicEffect(playerpos, 12)
doPlayerSay (cid, "Aaaaach...", 16)
else
return 1
end

Po pierwsze stosuj tabulator jako wcięcia.
Spróbuj zrozumieć swój kod:

Jeśli ID itemu jest takie i takie, to usuń go, dodaj 100 hp, wyświetl efekt magiczny, oraz powiedz "aaach". W przeciwnym razie zwróć 1, koniec.

Podstawowe pytanie, czy dodałeś linijkę w actions.xml? Każdy skrypt jest czytany z XML, który następnie odnosi do skryptu LUA.

[Cro]

-- Skrypt by Cwelf --
function onUse(cid, item, frompos, item2, topos)
    if item.itemid == 2547 then
        doRemoveItem(item.uid, 1)
        doPlayerAddHealth(cid, 100)
        doSendMagicEffect(getPlayerPosition(cid), 12)
        doPlayerSay (cid, "Aaaaach...", 16)
    else
        return 1
    end
end

No więc tak.

function onUse(cid, item, fromPosition, itemEx, toPosition)

Tak chyba jest na Avestach czy coś, bo na YurOTSach i innych dziadkach jest:

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

Jak masz Avestę to zostaw tak jak było.

Ok, dalej:

doRemoveItem(item.uid, 2547)

ID itemu nie musisz tutaj podawać, bo skrypt wie, który dokładnie item to jest (czyt. dokładnie ten, który użyłeś, a nie jakiśtam na mapie). To działa tak:

doRemoveItem(item.uid, ILOŚĆ!!!)

Jedziemy dalej:

doSendMagicEffect(playerpos, 12)

playerpos mógłbyś użyć gdybyś wcześniej utworzył np. zmienną o takiej nazwe. A że taka nie istnieje to co biedny skrypt ma zrobić? Wywali błąd i tyle.

doSendMagicEffect(getPlayerPosition(cid), 12)

Dlatego możesz użyć wbudowaną w silnik funkcję getPlayerPosition(cid)

doPlayerSay(cid, "Aaaaach...", 16)

Nie wiem jak to było na 7.6, ale jakby Cię zdebugowało, to spróbuj

doPlayerSay(cid, 16, "Aaaaach...")

Do tego musiałbyś ogarnąć stawianie spacji i tabulatorów.
Przyjrzyj się skryptowi poprawionego przeze mnie.

[Grim90]

Cro, w argumentach funkcji można podać co się chce, nawet:

function onUse(ja, przedmiot, zPoz, przedmiotNa, doPoz)

W związku z tym twoje pierwsze wskazówki są całkiem niepotrzebne. :P

Natomiast funkcja doPlayerSay nie powinna być błędna, chyba, że ten, kto modyfikował silnik, walnął taki śmieszny błąd.

[Cro]

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Cro, w argumentach funkcji można podać co się chce, nawet:

function onUse(ja, przedmiot, zPoz, przedmiotNa, doPoz)

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Natomiast funkcja doPlayerSay nie powinna być błędna, chyba, że ten, kto modyfikował silnik, walnął taki śmieszny błąd.

No nie no ale pamiętam, że jak z Evolutions przeszedłem na TFS'a (8.1) to właśnie przez to mnie debugowało przy jedzeniu fooda - typ wiadomości i sama wiadomość były na Evo w odwrotnej kolejności niż na TFSie

[zakius]

musisz zwrócić true(a w starszych 1), inaczej skrypt się wykona ale klient dostanie "sorry not possible"
i jak przypiszesz tylko 1 id do danego skryptu nie ma najmniejszego sensu sprawdzać tego id wewnątrz