Issue 6271: Offboard units to the North and West will be placed prope…

…rly; use one method for determining artillery travel time
MegaMek · Dec 23, 2024 · 837ceda · 837ceda
1 parent 76a6c50
commit 837ceda
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 17 deletions.
diff --git a/megamek/src/megamek/common/Compute.java b/megamek/src/megamek/common/Compute.java
@@ -7636,7 +7636,7 @@ public static boolean isFlakAttack(Entity attacker, Entity target) {
 
     public static int turnsTilHit(int distance) {
         final int turnsTilHit;
-        // See indirect flight times table, TO p181
+        // See indirect flight times table, TO:AR p149
         if (distance <= Board.DEFAULT_BOARD_HEIGHT) {
             turnsTilHit = 0;
         } else if (distance <= (8 * Board.DEFAULT_BOARD_HEIGHT)) {

diff --git a/megamek/src/megamek/common/Entity.java b/megamek/src/megamek/common/Entity.java
@@ -10272,7 +10272,7 @@ public void deployOffBoard(int round) {
             case NORTH:
                 setPosition(new Coords((game.getBoard().getWidth() / 2)
                         + (game.getBoard().getWidth() % 2),
-                        -getOffBoardDistance()));
+                        -getOffBoardDistance() - 1));
                 setFacing(3);
                 break;
             case SOUTH:
@@ -10290,7 +10290,7 @@ public void deployOffBoard(int round) {
                 setFacing(5);
                 break;
             case WEST:
-                setPosition(new Coords(-getOffBoardDistance(), (game.getBoard()
+                setPosition(new Coords(-getOffBoardDistance() - 1, (game.getBoard()
                         .getHeight() / 2) + (game.getBoard().getHeight() % 2)));
                 setFacing(1);
                 break;

diff --git a/megamek/src/megamek/common/actions/ArtilleryAttackAction.java b/megamek/src/megamek/common/actions/ArtilleryAttackAction.java
@@ -60,20 +60,7 @@ public ArtilleryAttackAction(int entityId, int targetType, int targetId,
                         turnsTilHit = 2;
                     }
                 } else {
-                    // See indirect flight times table, TO p181
-                    if (distance <= Board.DEFAULT_BOARD_HEIGHT) {
-                        turnsTilHit = 0;
-                    } else if (distance <= (8 * Board.DEFAULT_BOARD_HEIGHT)) {
-                        turnsTilHit = 1;
-                    } else if (distance <= (15 * Board.DEFAULT_BOARD_HEIGHT)) {
-                        turnsTilHit = 2;
-                    } else if (distance <= (21 * Board.DEFAULT_BOARD_HEIGHT)) {
-                        turnsTilHit = 3;
-                    } else if (distance <= (26 * Board.DEFAULT_BOARD_HEIGHT)) {
-                        turnsTilHit = 4;
-                    } else {
-                        turnsTilHit = 5;
-                    }
+                    turnsTilHit = Compute.turnsTilHit(distance);
                 }
             }
             return;