| 01-30-2009, 04:21 PM | #2 |
I think the idea is great :D. Code-wise it doesn't look very good (it has leaks), sorry if I can't give any recommendations, I didn't learn GUI very much. |
| 01-30-2009, 04:26 PM | #3 |
The only thing I would say to change is to make a unit group variable for the Heroes. Whenever a Hero is summoned or whatever, just add them to the group and instead of using "Pick all Units in Playable Map Area" you just have "Pick all Units in SomeUnitGroupVariable". That's just a personal preference of mine and what you have now is clearly fine. Or maybe it leaks a group. Not sure... So maybe it is best to use that Unit Group variable. |
| 01-30-2009, 04:45 PM | #5 |
I was thinking that, too. I don't know if that's really necessary because a periodic timer every 5 seconds is not taxing or anything, especially doing what you have. Using "Unit comes within Range" event is more accurate especially if heroes are moving in and out of the radius frequently. Although, you could probably get away with decreasing the periodic time down to 1 or 2 seconds. This would minimize the amount of time a hero doesn't get bonus xp when it should. I mean, I'm not too sure, though. You'd have to see if changing this particular periodic timer has any effect to your gameplay. If you change it down to 1 second and you notice some lag while using some spells when it didn't lag when using 5 seconds, maybe you should just increase it to 2 seconds. |
| 01-30-2009, 04:56 PM | #6 |
Alright, well hopefully this works fine, now I just gotta get the new respawn system working. The previous one worked but it went off spawning the unit where the previous one died, wanted to fix that but I can't seem to get it to work >< |
