Sudoku review #1

Open
BordedDev wants to merge 19 commits from BordedDev/sudoku:main into main
pull from: BordedDev/sudoku:main
merge into: retoor:main
Conversation 3 Commits 19 Files Changed 8 +16
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ce5dead2cb - Show all commits

16
reviews/BordedDev/sudoku.rewrite.js
View File

@ -32,6 +32,10 @@ class SudokuPuzzle extends EventTarget {
*/
BordedDev commented 2025-01-16 23:51:24 +00:00
Outdated
Review
Copy Link

This contains a stack of states, essentially the initial state + all steps

This contains a stack of states, essentially the initial state + all steps
#activeState = new Array(9 * 9)
retoor commented 2025-01-16 18:39:29 +00:00
Outdated
Review
Copy Link

Ooooh, magic numbers :P Did i have that too?

Ooooh, magic numbers :P Did i have that too?
BordedDev commented 2025-01-16 21:26:53 +00:00
Outdated
Review
Copy Link

Dang it I missed it, it's meant to be SUDOKU_GRID_SIZE. You did with the new Puzzle(9)

Dang it I missed it, it's meant to be `SUDOKU_GRID_SIZE`. You did with the `new Puzzle(9)`
/**
*
* @returns {(number|null|undefined)[]}
*/
get grid() {
const gridValue = [...this.#activeState]
Object.freeze(gridValue)
@ -216,6 +220,10 @@ class SudokuPuzzle extends EventTarget {
return newState
}
/**
*
* @returns {(number|null|undefined)[]}
*/
#generateRandomState() {
const newState = Array.from({ length: SUDOKU_GRID_SIZE })
for (let i = 0; i < 17; i++) {
@ -225,6 +233,10 @@ class SudokuPuzzle extends EventTarget {
return newState
}
/**
*
* @returns {(number|null|undefined)[]}
*/
get baseState() {
return this.#state[0]
}
@ -239,6 +251,10 @@ class SudokuPuzzle extends EventTarget {
)
}
/**
*
* @param newState {(number|null|undefined)[]}
*/
applyState(newState) {
this.#state.push(newState)
this.#activeState = SudokuPuzzle.collapseStates(...this.#state)