• Listen to a special audio message from Bill Roper to the Hive Workshop community (Bill is a former Vice President of Blizzard Entertainment, Producer, Designer, Musician, Voice Actor) 🔗Click here to hear his message!
  • Read Evilhog's interview with Gregory Alper, the original composer of the music for WarCraft: Orcs & Humans 🔗Click here to read the full interview.

Force Attack

Level 14
Joined
Nov 18, 2007
Messages
816
I do not want to use this because:
- dynamic triggers (do i have to explain why?)
- inefficiency through KeyTimers. Plain timers work better for this.
- that AIDS textmacro looks like overkill to me (besides the horrible horrible name; it sounds as if your script was infected). For this application at least.
- What happens to the member overwritten before you abort because the attacker is already in a group?
 
Level 15
Joined
Jul 6, 2009
Messages
889
That is the most feedback I've ever gotten, I've posted this on TheHelper, no comments on the coding.

1. No you don't. FIXED - Now using 1 trigger.
2. I might try at that. Done. Using Hashtables?
3. LOL. AIDS. The best system name ever :) Although the system called SEX is better :p
4. Shrugs. Roger that to fix. DONE.

UPDATED!
 
Last edited:
Level 8
Joined
Oct 3, 2008
Messages
367
Needs fixing.

1. Dynamic timers are bad. Baaad.

2. Don't try to combine what should be done with multiple triggers and conditions into one big block. Less efficient, and more of a mess.

3. This shouldn't even compile: library ForceAttack initializer OnInit requires //, KT

4. The documentation is false.

5. Conditions are better than actions.
 
Top