1

Trato de que busque el número 1 en una lista con números repetidos y trato de que me de su ubicación en el mapa, pero sólo busca el numero 0. Y si coloco por ejemplo el 1, no me da la ubicación y no se por qué.

def map1(self):#nivel 1
    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]
    ]

def boxesposition(self):
    for cy in range(len(self.map)):#busca la posicion en x de la caja
        for cx in range(len(self.map)):#busca la posicion en y de la caja
            if self.map[cy][cx] == 1:
                self.position_cx =cx
                self.position_cy =cy
                print (self.map[cx][cy])
6
  • podrías editar tu pregunta añadiendo como es self.map?
    – kirbylife
    Commented el 17 jul. 2019 a las 18:13
  • esque es para un juego y el 1 por ejemplo es para caja y el 0 es personaje y trato de buscar la ubicacion de la caja porque el del personaje si me lo da pero si coloco otro numero no lo encuentra
    – Jesus
    Commented el 17 jul. 2019 a las 18:15
  • el self.map es porque es una variable global
    – Jesus
    Commented el 17 jul. 2019 a las 18:16
  • si pero sin importar que lo cambie no me da ninguna ubicacion solo cuando declaro que sea ==0
    – Jesus
    Commented el 17 jul. 2019 a las 18:18
  • si ya vi esos detalles de la posicion en el print pero aun asi no encuentra el 1 solo funciona si pongo el 0 y no se porque
    – Jesus
    Commented el 17 jul. 2019 a las 18:21

1 respuesta 1

3

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)

1
  • muchísimas gracias ya tenia mas de 2 horas sin poder resolverlo
    – Jesus
    Commented el 17 jul. 2019 a las 18:33

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.