Keeping AdvancedRobot instance in static field

Jump to navigation Jump to search
Revision as of 21 August 2018 at 02:31.
The highlighted comment was created in this revision.

Keeping AdvancedRobot instance in static field

When keeping AdvancedRobot instance in static field, robot will show warning:

Static references to robots can cause unwanted behaviour with the robot using these.

However I've been using this pattern (the way robocode can't detect, and I cannot avoid as well) since the first day I write robocode. However, robocode did not mention what's wrong with this pattern. Do anyone know what's the exact meaning of this warning?

    Xor (talk)11:45, 19 August 2018

    I think the risk is that the static robot reference is not guaranteed to be valid after a given round, so if you don’t update the reference things may not work as expected.

      Enamel 32 (talk)16:32, 19 August 2018

      Yes, that's also what I guess, and since I'm updating the reference every round, I guess it would not be a problem. However, since robocode did not say what unwanted behavior it is, I'm worrying about more problems: race condition (since each round you have a new thread), GC problem (no longer a problem since robocode cleans static field each battle) and more

        Xor (talk)03:43, 20 August 2018

        Since you're updating it every round, why do you need it to be static?

          Skilgannon (talk)20:38, 20 August 2018

          Ease of access, probably. I was doing the same before my big inversion of control refactoring.

            Enamel 32 (talk)22:41, 20 August 2018
             

            because I’m simply making everything under a static variable, just like lazy singleton.

              Xor (talk)04:31, 21 August 2018