How do i cast a spell at each mob in the room...

[Glaves]

I realize this is probably insanely simple but i am still fairly new to coding and I have been trying to write a script that will cast a spell once at each mob in the room and i am getting nowhere and I was hoping somebody could help me out. It will cast at the first target, then exits and doesn't continue on casting at the others.

GameObj.npcs.each { |mob|

if mob.status !=~ /dead|gone/
	1.times {
		fput "prep 917"
		fput "cast ##{mob.id}"
		waitcastrt?
		waitrt?
	}
end
}

If somebody could help me out it would be most appreciated.

[Rjex]

if mob.status !~ /dead|gone/

[Rjex]

GameObj.npcs.each { |mob|
if (mob.status !~ /dead|gone/)
	Spell[917].cast("##{mob.id}") if Spell[917].affordable?
	waitcastrt?
	waitrt?
end
}

[Talonhawke]

GameObj.npcs.each { |mob|
	if mob.status !~ /dead|gone/ and mob.type =~ /\baggressive\b/
		if Spell[917].affordable?
			Spell[917].cast mob
			waitcastrt?
			waitrt?
		end
	end
}

Been away a while, so not sure if the aggressive is still needed or not to make sure you don't attack things you don't intend to.

[Glaves]

THANK YOU BOTH! I added the aggressive tag to the code Rjex provided and it seems to be working great!!!

I was so close too.

[Glaves]

Now i just need to figure out how to make it not include certain mobs and it'll be perfect

[Glaves]

I got it! Seriously, thank you! Been trying to get this to work for a couple days now and it was driving me nuts. Very much appreciated.

GameObj.npcs.each { |mob|
if (mob.status !~ /dead|gone|flying/ && !['raptor'].include?(mob.noun) && mob.type =~ /\baggressive\b/)
	Spell[917].cast("##{mob.id}") if Spell[917].affordable?
	waitcastrt?
	waitrt?
end
}

[Donquix]

set a macro to: \xtarget next\rincant 917\r
tape the key down
go afk

come back fried with full wizard juice.

You're welcome.

[Rjex]

Incidentally, wouldn't this be cleaner anyway?

GameObj.targets.each { |mob|
if (mob.status !~ /flying/)
	Spell[917].cast("##{mob.id}") if Spell[917].affordable?
	waitcastrt?
	waitrt?
end
}

targets already filters out everything that's dead, gone, or familiar type

[Glaves]

Incidentally, wouldn't this be cleaner anyway?

GameObj.targets.each { |mob|
if (mob.status !~ /flying/)
	Spell[917].cast("##{mob.id}") if Spell[917].affordable?
	waitcastrt?
	waitrt?
end
}

targets already filters out everything that's dead, gone, or familiar type

Ohhh i didn't know they auto filtered that stuff. The raptor isn't a familiar it's a mob i ignore in a hunting ground. When did that change? I remember when i was on shattered i had to add the dead/gone and my animal companion to avoid the friendly npc. It would seem the update to ruby 2.0 changed a bunch of stuff. I also have just been gone for a couple years now and am sooo rusty.