search

nvs / gem

Gem TD+: A Warcraft III Tower Defense
https://discord.gg/PxNNp77
26 stars 2 forks source link

Fix creep attack detection #373

Closed nvs closed 5 years ago

nvs commented 5 years ago

Gem is supposed to detect when a creep attacks and then teleport said creep. However, this often fails. Need to resolve this bug as #132 seems a bit far off.

nvs commented 5 years ago

There is a possibility for abuse here. People could knowingly block in order to teleport the monsters to the next checkpoint and save time. Need to think on this potential situation. However, I would consider that abuse and invalidate any replay utilizing such behavior.

nvs commented 5 years ago

May be better to detect the attack order rather than a unit being the target of an attack. This would prevent abuse of hiding towers behind a row of rocks and no actual attack being started.

nvs commented 5 years ago

Another interesting issue: If a creep becomes confused before reaching the first checkpoint, it will then teleport to the middle!

nvs commented 5 years ago

Still not a fan of this, but it will probably have to work for now. As mentioned earlier, #132 would solve everything. But that is a bit more time intensive to setup and then test. This works now.

But it can be better. It was suggested to me to teleport the blocking units back to the start, reset their HP, and begin the trip through the maze all over again. I like that idea.

nvs commented 5 years ago

A few other ideas were explored (i.e. teleporting units to the start and pausing units upon detection of a block). But to iron those out fully in a manner that would make me happy would be a bit time consuming. More with testing than anything else.

As it is now, blocking is rare for most players. If it happens, they learn pretty quickly from their mistakes.