Anyone see an issue..?
 

Just curious, its been a loooong while since ive scripted anything, and i was never like the BEST but i considered myself pretty decent. Anyway computer got fried lost all scripts trying to rebuild, anyone see why this wouldnt work? Please and thank u for responses.

Again, thanks in advance!

Start with putting a closing bracket on your 'sub EVENT_HP' procedure.

Ok great :D now hes shouting properly, but not casting the spells now it looks like
I didnt know u could do a double close bracket i thought that was for the end ^_^

Oh and its set for 1 2 and 3 sec just for testing, not sure if that will make a difference?

It's just a matter of closing what you open :) (refrigerator doors, toilet seats, etc...)


I'm not 'real' familiar with timers in script..but are the settimer names suppose to be in quotes as well in EVENT_SPAWN?

You can always put a test message in your timer checks to verify script operation as well..even if the server isn't
processing every casting call.

if i had to guess, i would say that you can't use quest::selfcast() in EVENT_TIMER because it should be looking for the client that initiated the event.

i'd suggest starting those timers only when the npc was engaged (as well as stopping them when it is disengaged instead of just dead) and then pulling the target for the spell from the npc's hate list, since there is no need for the timers to be running constantly.

How would i make the cast do it on a target rather then selfcast then if u dont mind? And ill just keep messing with it more and hopefully figure it out thx guys

oh, yeah... it could be the syntax issue uleat just pointed out as well. :)

$npc->CastSpell(spellid,targetid) should work to target an individual.

examples:
$npc->CastSpell(6646, $npc->GetHateRandom()); // DT random hated client
$npc->CastSpell(15236, $npc->GetHateDamageTop()); // slow client that has done the most damage
$npc->CastSpell(9051, $npc->GetHateTop()); // nuke most hated client

I assume that you know about this:

http://www.eqemulator.net/wiki/wikka...=QuestTutorial


You can always check the source definitions for any changes since this was made.

great thanks i appreciate it u guys!

Ah 1 last thing, if the spell is a PB aoe it shouldnt matter which GetHate blah blah i type in correct?

i'm not sure if it checks for valid range to target and whatnot, but it should hit everyone in range of the AoE, if that's what you mean.

Ok thanks again!

been working on this all day again trying stuff i assume now its sumthing wrong with how the timers r set up cuz im trying to have him shout on a timer and thats not working either >_<

Any noticeable issues?

Also I used a perl script checker and got this

I cant seem to find whats goin on tho -.-

..I put it in GeorgeS Tools and it says im missng a right bracket } I just CANNOT figure out where

I didn't go through much, but just got some obvious things for you:

Count your brackets... 4 starting brackets, 2 ending brackets.

Jolig that is working thanks a lot! What was wrong with it?

bad_captain, I see I had 4 starting brackets and 2 ending, but where would the other 2 brackets have gone?

There's a reason people format code, because it is much more obvious when things are wrong. Compare your code to this:

Much more obvious where the problem is isn't it? I don't know what you edit with, but there are a ton of editors out there that do things like brace highlighting and matching that help to avoid these simple mistakes.

This part:


You're original posting was correct..the second one is wrong.


There could still be other issues, but this is how procedures should be wrapped.

I don't know if perl let's you use single-line conditional statements. If it does, and that's what you're attempting to do,
then the con checks shouldn't have the opening bracket after it.

IF perl does let you, it should look like this. Otherwise you need both open and close brackets around any procedure or
sub-procedure.

EDIT: omg..did it really take almost 3 hours to post this?? I'm so dense that I must be warping the very fabric of space-time...

Here try this.
Also look at this:
So you basically set it to check DT (death touch?) every 1 second, slow every 2 and nuke every 3 or am I just way tired and reading that wrong.

If you need additional help you can contact me at akaishigpg@gmail.com

as for your timer checks, when you are using a string, you should enclose it in quotations and use the eq operator (if the name of your timer is an integer, you don't need the quotations and you use == instead). it's also better practice to use elsif for when you only expect a single condition to be met, otherwise the script will continue to evaluate the rest of the conditions instead of stopping at the first one it sees. here's an example:

additionally, i don't think it's necessary to explicitly stop timers on NPC death.

You generally do not have to. there have been a few rare oddities (MB in PoI) on PEQ where timers and such seem to not stop correctly on zone shutdown, but never noticed anything hang up past NPC death.

combatstate == 0 covers both avenues (raid wipe/agro drop and death)