WIP

stevehjohn · Apr 8, 2024 · bb7551f · bb7551f
1 parent 1f9803d
commit bb7551f
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 17 deletions.
diff --git a/src/Sudoku/Extensions/IntArrayExtensions.cs b/src/Sudoku/Extensions/IntArrayExtensions.cs
@@ -27,13 +27,15 @@ public static void DumpToConsole(this int[] array, int left = -1, int top = -1)
             }
 
             Console.WriteLine(" │");
-            
-            if (y is 2 or 5)
+
+            if (y is not (2 or 5))
             {
-                SetPosition(left, top, line++);
-
-                Console.WriteLine("├───────┼───────┼───────┤");
+                continue;
             }
+
+            SetPosition(left, top, line++);
+
+            Console.WriteLine("├───────┼───────┼───────┤");
         }
 
         SetPosition(left, top, line);

diff --git a/src/Sudoku/Extensions/SpanIntExtensions.cs b/src/Sudoku/Extensions/SpanIntExtensions.cs
@@ -27,12 +27,14 @@ public static bool IsValidSudoku(this Span<int> puzzle)
                     countRow++;
                 }
 
-                if (puzzle[y + x * 9] != 0)
+                if (puzzle[y + x * 9] == 0)
                 {
-                    uniqueColumn.Add(puzzle[y + x * 9]);
-
-                    countColumn++;
+                    continue;
                 }
+
+                uniqueColumn.Add(puzzle[y + x * 9]);
+
+                countColumn++;
             }
 
             if (uniqueRow.Count < countRow || uniqueColumn.Count < countColumn)
@@ -55,12 +57,14 @@ public static bool IsValidSudoku(this Span<int> puzzle)
                 {
                     for (var y = 0; y < 3; y++)
                     {
-                        if (puzzle[(yO + y) * 9 + xO + x] != 0)
+                        if (puzzle[(yO + y) * 9 + xO + x] == 0)
                         {
-                            uniqueBox.Add(puzzle[(yO + y) * 9 + xO + x]);
-
-                            countBox++;
+                            continue;
                         }
+
+                        uniqueBox.Add(puzzle[(yO + y) * 9 + xO + x]);
+
+                        countBox++;
                     }
                 }
 

diff --git a/src/Sudoku/Generator.cs b/src/Sudoku/Generator.cs
@@ -57,12 +57,14 @@ private void RemoveCells(int[] puzzle, int cellsToRemove)
 
             for (var i = 0; i < 81; i++)
             {
-                if (puzzle[i] == 0)
+                if (puzzle[i] != 0)
                 {
-                    filledCells.Add(i);
-
-                    puzzle[i] = copy[i];
+                    continue;
                 }
+
+                filledCells.Add(i);
+
+                puzzle[i] = copy[i];
             }
         }
     }