wip perf

meikpiep · Oct 26, 2024 · caa4627 · caa4627
1 parent f35fcd0
commit caa4627
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/...in/org/piepmeyer/gauguin/difficulty/human/strategy/ImpossibleCombinationInLineDetector.kt b/...in/org/piepmeyer/gauguin/difficulty/human/strategy/ImpossibleCombinationInLineDetector.kt
@@ -13,7 +13,7 @@ object ImpossibleCombinationInLineDetector {
     fun fillCells(
         grid: Grid,
         cache: PossiblesCache,
-        isImpossible: (Grid, GridLine, GridCage, cache: PossiblesCache, List<Int>) -> Boolean,
+        isImpossible: (GridLine, GridCage, cache: PossiblesCache, List<Int>) -> Boolean,
     ): Boolean {
         val lines = GridLines(grid).linesWithEachPossibleValue()
 
@@ -58,7 +58,7 @@ object ImpossibleCombinationInLineDetector {
                         logger.trace { "Relevant line indexes: $lineCageCellsIndexes" }
                         logger.trace { "Single possible: $singlePossible" }
 
-                        if (singlePossible.isNotEmpty() && isImpossible.invoke(grid, line, cage, cache, singlePossible)) {
+                        if (singlePossible.isNotEmpty() && isImpossible.invoke(line, cage, cache, singlePossible)) {
                             cell.removePossible(singleCombinationPossible)
                             return true
                         }

diff --git a/...fficulty/human/strategy/RemoveImpossibleCombinationInLineBecauseOfPossiblesOfOtherCage.kt b/...fficulty/human/strategy/RemoveImpossibleCombinationInLineBecauseOfPossiblesOfOtherCage.kt
@@ -18,7 +18,6 @@ class RemoveImpossibleCombinationInLineBecauseOfPossiblesOfOtherCage : HumanSolv
     ): Boolean = ImpossibleCombinationInLineDetector.fillCells(grid, cache, this::isImpossible)
 
     private fun isImpossible(
-        grid: Grid,
         line: GridLine,
         cage: GridCage,
         cache: PossiblesCache,

diff --git a/...gauguin/difficulty/human/strategy/RemoveImpossibleCombinationInLineBecauseOfSingleCell.kt b/...gauguin/difficulty/human/strategy/RemoveImpossibleCombinationInLineBecauseOfSingleCell.kt
@@ -21,7 +21,6 @@ class RemoveImpossibleCombinationInLineBecauseOfSingleCell : HumanSolverStrategy
     ): Boolean = ImpossibleCombinationInLineDetector.fillCells(grid, cache, this::isImpossible)
 
     private fun isImpossible(
-        grid: Grid,
         line: GridLine,
         cage: GridCage,
         cache: PossiblesCache,