El problema es que asumes que la matriz es cuadrada cuando no lo es. El rango de índices del segundo for
debe ser el número de columnas, es decir, en vez de:
for cx in range(len(self.map))
debe ser:
for cx in range(len(self.map[0]))
Al hacer for cx in range(len(self.map))
te dejas sin recorrer la mitad de la matriz, las últimas 5 columnas, justamente donde está tu 1 perdido...
No obstante, te aconsejo que no uses range
+ indizado y uses enumerate
para esto, es más legible y considerablemente más eficiente. Dado que asumimos un solo 1 en la matriz, es importante que rompas el ciclo cuando lo encuentres, así te evitas recorrer toda la matriz siempre, aún habiendo encontrado ya el número.
class Foo:
def __init__(self):
self.map=[[2,2,2,2,2,2,2,2,2,2],
[2,4,2,4,4,4,2,4,2,2],
[2,4,4,4,0,4,4,1,4,2],
[2,4,4,2,4,4,4,4,3,2],
[2,2,2,2,2,2,2,2,2,2]
]
self.position_cx = None
self.position_cy = None
def boxesposition(self):
for cy, row in enumerate(self.map):
for cx, n in enumerate(row):
if n == 1:
self.position_cx = cx
self.position_cy = cy
return
inst = Foo()
print(inst.position_cx, inst.position_cy)
inst.boxesposition()
print(inst.position_cx, inst.position_cy)
None None
7 2
Solo por curiosidad, un pequeño test de tiempo de ejecución para esa misma lista con los diferentes métodos:
range
+ indizado sin romper el ciclo:
8.31 µs ± 158 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)
range
+ indizado rompiendo el ciclo:
5.1 µs ± 80.9 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)
enumerate
rompiendo el ciclo:
2.4 µs ± 98.8 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)
self.map
?