Commitd468888

committed

found a bug when I test it

at line 46, if you do not times 3, some positions would never be checked while some positions were checked many times.

1 parent2d279dc commitd468888Copy full SHA for d468888

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public static boolean isValidSudoku(char[][] board) {`
`43`	`43`	`// clear the set at every block.`
`44`	`44`	`set.clear();`
`45`	`45`	`for (intk =0;k <9;k++) {`
`46`		`-if (!isValidChar(board[i +k /3][j +k %3],set)) {`
	`46`	`+if (!isValidChar(board[i3 +k /3][j3 +k %3],set)) {// here is a bug`
`47`	`47`	`returnfalse;`
`48`	`48`	`}`
`49`	`49`	`}`
`@@ -71,4 +71,4 @@ public static boolean isValidChar(char c, HashSet<Character> set) {`
`71`	`71`
`72`	`72`	`returntrue;`
`73`	`73`	`}`
`74`		`-}`
	`74`	`+}`

Comments

(0)