Add line-of-sight restriction on ranged combat!

README.md

@@ -51,6 +51,7 @@ image to appear.  The parameters are:
 # TODO AI 
 
 - macro-move sampling: avoid duplicate macro-moves in a node's children
+- macro-move sampling: make units not moving an option
 - macro-move sampling: tune the explore/exploit ratio
 - open loop instead of chance nodes
 - implement evasion and line-of-sight rules for ranged combat, to allow the realistic game strategy to emerge
@@ -60,7 +61,6 @@ image to appear.  The parameters are:
 # TODO RULES
 
 - evasion
-- ranged combat line of sight restriction
 - more cards
 - advance after combat
 - cavalry, chariot battle again after advance

src/lib/hexlib.js

@@ -194,7 +194,7 @@ export function hasLineOfSight(toHex, fromHex, isBlocked = () => false) {
     const distance = fromHex.distance(toHex);
     const step = hex_subtract(toHex, fromHex);
     const stepSize = 1.0 / distance;
-    for (let i = 0; i < distance; i++) {
+    for (let i = 1; i < distance; i++) {
         const stepHex = hex_round(fromHex.q + (step.q * stepSize * i),
             fromHex.r + (step.r * stepSize * i),
             fromHex.s + (step.s * stepSize * i));

test/lib/hexlib_test.js

@@ -61,8 +61,8 @@ describe('line of sight', () => {
         {from: hexOf(0,0), to: hexOf(-1,2), expected: true },
     ].forEach(({from, to, expected}) => {
         test(`Line of sight from ${from} to ${to}: ${expected}`, () => {
-            console.log(`Line of sight from ${from} to ${to}: ${expected}`)
-           expect(hasLineOfSight(to, from)).toBe(expected);
+            const blocked = hex => hex === from || hex === to;
+            expect(hasLineOfSight(to, from)).toBe(expected);
         });
     });