Method and Description |
---|
net.sf.colossus.game.Battle.countBrambleHexes(BattleHex, BattleHex)
another function with explicit reference to Bramble
that should be fixed.
|
net.sf.colossus.game.Battle.countBrambleHexesDir(BattleHex, BattleHex, boolean, int)
another function with explicit reference to Bramble
that should be fixed.
|
net.sf.colossus.variant.HazardHexside.getHexsideByCode(char) |
net.sf.colossus.game.Battle.isObstacle(char)
This is the realm of HazardEdge, not direct use of hexside
|
net.sf.colossus.client.BattleClientSide.isRangestrikePossible(BattleCritter, BattleCritter)
Should use an extension of Battle instead of Strike, with
extension of Creature instead of BattleCritter and extra BattleHex
|
net.sf.colossus.client.BattleClientSide.minRangeToEnemy(BattleCritter)
Should use an extension of Battle instead of Strike
// END OLD COMMENT
Now this is moved from Strike to BattleClientSide.
IMHO this is not a total duplicate of a method in Battle: Battle
does not have a minRangeToEnemy, just minRange between concrete hexes,
which IS actually called here.
TODO can they be unified? Or move to e.g. some class in ai.helper package?
|