fix: Ensure is_safe() returns boolean instead of int

akiels · akiels · commit d052b88ebe33 · 2025-04-28T18:41:32.000+02:00
Previously, is_safe() could return the value of a cell (int 1 or 0), which caused test failures.

This fix ensures that is_safe() returns True if the cell is safe to visit and False otherwise
diff --git a/matrix/count_islands_in_matrix.py b/matrix/count_islands_in_matrix.py
@@ -87,7 +87,7 @@ def is_safe(self, i: int, j: int, visited: list[list[bool]]) -> bool:
             0 <= i < self.ROW
             and 0 <= j < self.COL
             and not visited[i][j]
-            and self.graph[i][j]
+            and self.graph[i][j] == 1
         )
 
     def diffs(self, i: int, j: int, visited: list[list[bool]]) -> None:

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ def is_safe(self, i: int, j: int, visited: list[list[bool]]) -> bool:`
`87`	`87`	`0 <= i < self.ROW`
`88`	`88`	`and 0 <= j < self.COL`
`89`	`89`	`and not visited[i][j]`
`90`		`- and self.graph[i][j]`
	`90`	`+ and self.graph[i][j] == 1`
`91`	`91`	`)`
`92`	`92`
`93`	`93`	`def diffs(self, i: int, j: int, visited: list[list[bool]]) -> None:`