2x2 Windows App Aiming and Shooting Bug about raic-2019

[kuviman]

Looks like that one is bug in your code

[alvinalvord]

Okay, i found the problem. Apparently I shouldn't reuse the unit action object. The problem fixes itself when I reinitialize a new unit action every time get action is called. I've inserted the code I've been testing for a while :)) thanks anyway.

import model.*;

public class MyStrategy {
	UnitAction action;
	
	MyStrategy () {
		action = new UnitAction ();
	}
	
	public UnitAction getAction(Unit unit, Game game, Debug debug) {
		Unit ally = null;
		Unit enemy = null;
		for (Unit u: game.getUnits ()) {
			if (u.getPlayerId () == unit.getPlayerId () && u.getId () != unit.getId ()) {
				ally = u;
			} else if (u.getPlayerId () != unit.getPlayerId ()) {
				enemy = u;
			}
		}

		// uncomment to fix bug lol
		// action = new UnitAction();
		if (ally != null && unit.getId () < ally.getId ()) {
			action.setAim (new Vec2Double (-40, 0));
			action.setShoot (true);
		} else {
			action.setAim (new Vec2Double (40, 0));
			action.setShoot (false);
		}
		
		action.setVelocity (enemy.getPosition ().getX () - unit.getPosition ().getX ());
		action.setJump (enemy.getPosition ().getY () > unit.getPosition ().getY ());
		action.setJumpDown (false);
		action.setReload (false);
		action.setSwapWeapon(false);
		action.setPlantMine(false);
		return action;
	}
}