Editing Radar/getAllEvents

Robo Home | Changes | Preferences | AllPages

<pre>
public void getScannedBots(){
        Vector v=getAllEvents();
        Iterator i=v.iterator();
        while(i.hasNext()) {
                ScannedRobotEvent e = (ScannedRobotEvent) i.next();
                if(e instanceof ScannedBotEvent){ onScannedRobot(e); }
        }
}
</pre>
Kinsen noticed that the radar might 'miss' some bots that are behind another. I haven't tested this code yet, but it should work. Just call getScannedBots() right before the execute(); line. I'm not sure if there is a function called getAllScannedRobotEvents(), but if there is, feel free to change the code above--[[Starrynte]]

Try using generics. Since Robocode needs Java 5 anyways.... Vector<ScannedRobotEvent> eliminates need to use instanceof. -- [[Aziz]]

Yes, I remember reading about that in some Java book...didnt think about it. thx 4 the suggestion though, code changed above --[[Starrynte]]

Well, since the <nowiki>getAllEvents</nowiki> method returns Events and not ScannedRobotEvents, I'm not sure this line <code>Vector<ScannedRobotEvent> v=getAllEvents();</code> would work. If it had ScannedRobotEvents, you could cast them to Event, but not the other way around, right? Also, you do need the check if the object is of ScannedRobotEvent type or not, since you just want the SREs and not the other events. As written, I don't see how this is any different than what Robocode already does automatically for you with the onScannedRobotEvent method; and unless you changed the method signature of the call you're passing the SRE too, this code would end up with that method being called twice per ScannedRobotEvent. In general, I could see some usefulness in melee for collecting all the events and then dealing with them at once, of course. -- [[Voidious]]

In that case...I'll change the code back to what it was originally. I don't use generics much yet anyways...But feel free to change the code if it'll make the codesize smaller, I'm not a nano expert. --[[Starrynte]]
* Code like this does not belong in a CodeSize restricted league.  As [[Voidious]] mentioned, onScannedRobot can already recieve all the scan events.  The "scan hiding" phenomenon [[Kinsen]] noticed, I'm guessing, came from using a blocking call in the onScannedRobot method?  As soon as you use a blocking call (like turnRadarRight instead of setTurnRadarRight, or fire instead of setFire) all the rest of the events for that turn will be thrown away (since that call will block until (at least) the next turn).  Just avoid any of those and you should be all set. -- [[Simonton]]
** Ok, then that would eliminate the need for this... -- [[Starrynte]]
*** Well, as written, the code doesn't do anything new or special for you. But it serves well as sample code for fetching all the ScannedRobotEvents after the fact, which ''is'' something that could be useful in melee. Processing each SRE on the fly could pose issues in melee that would be solved by waiting until they all come through and then examining them all before taking action. -- [[Voidious]]
**** Yes (I know, this is the 4th nested bullet!), but will the SRE be called twice? --[[Starrynte]]
***** If you use the same method signature that Robocode calls, yes, it will be called twice. But you could easily just leave your <nowiki>onScannedRobot</nowiki> event blank and make your own method (that also takes a ScannedRobotEvent) to call in the loop of <nowiki>getAllEvents</nowiki>. -- [[Voidious]]
** The "scan hiding" happened when I was not using a blocking call. If you used a blocking call then it should be expected unless the blocking call was made a method with lower priority than the onScannedRobot method. -- [[Kinsen]]

Summary:
Consciousness check: 5 * 3 =
This change is a minor edit. Send email notification that Radar/getAllEvents has been changed.
(Visit Preferences to set your user name.)

View other revisions
Robo Home | Radar | Changes | Preferences | AllPages