Skip to main content
Correciones ortográficas
Origen Enlace
FJSevilla
  • 62.1k
  • 7
  • 43
  • 68

va a fallar si la calveclave no está en el diccionario, si esto ocurre get retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be betwenbetween 1 and 5 (both included)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   player2 = Robot(6, 1)
   
if __name__ == "__main__":
   main()

He añadido un par de properties para validar x e y en la instanciación y asegurarse que están entre 1 y 5.

(2,1)
(3,1)
(4,1)
(5,1)
You are in the rigth limit
(5,2)
(5,1)
You are in the up limit
(4,1)
Invalid Argument
Traceback (most recent call last):
  File "test.py", line 84, in <module>
    main()
  File "test.py", line 81, in main
    player2 = Robot(6, 1)
  File "test.py", line 4, in __init__
    self.x = x
  File "test.py", line 20, in x
    raise ValueError("x must be betwenbetween 1 and 5 (both included)")
ValueError: x must be betwen 1 and 5 (both included)

va a fallar si la calve no está en el diccionario, si esto ocurre get retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be betwen 1 and 5 (both included)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   player2 = Robot(6, 1)
   
if __name__ == "__main__":
   main()

He añadido un properties para validar x e y en la instanciación y asegurarse que están entre 1 y 5.

(2,1)
(3,1)
(4,1)
(5,1)
You are in the rigth limit
(5,2)
(5,1)
You are in the up limit
(4,1)
Invalid Argument
Traceback (most recent call last):
  File "test.py", line 84, in <module>
    main()
  File "test.py", line 81, in main
    player2 = Robot(6, 1)
  File "test.py", line 4, in __init__
    self.x = x
  File "test.py", line 20, in x
    raise ValueError("x must be betwen 1 and 5 (included)")
ValueError: x must be betwen 1 and 5 (both included)

va a fallar si la clave no está en el diccionario, si esto ocurre get retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be between 1 and 5 (both included)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   player2 = Robot(6, 1)
   
if __name__ == "__main__":
   main()

He añadido un par de properties para validar x e y en la instanciación y asegurarse que están entre 1 y 5.

(2,1)
(3,1)
(4,1)
(5,1)
You are in the rigth limit
(5,2)
(5,1)
You are in the up limit
(4,1)
Invalid Argument
Traceback (most recent call last):
  File "test.py", line 84, in <module>
    main()
  File "test.py", line 81, in main
    player2 = Robot(6, 1)
  File "test.py", line 4, in __init__
    self.x = x
  File "test.py", line 20, in x
    raise ValueError("x must be between 1 and 5 (both included)")
ValueError: x must be betwen 1 and 5 (both included)
se añadieron 674 caracteres en el cuerpo
Origen Enlace
FJSevilla
  • 62.1k
  • 7
  • 43
  • 68

movemove es definido como un método de instancia (definido dentro de la definición de la clase), como todo método de instancia recibe un primer parámetro obligatorio que es la instancia de la propia clase, que por convención se nombra por self, este parámetro es pasado de forma automática cuando se llama al método mediante una instancia (instancia.method()). En tu caso move se define como:

Aparte de esto swither debes definirlo o dentro de movemove o mejor en el __init__ como atributo de instancia, aparte los valores del diccionario son incorrectos, deben ser key: self.metodo.

va a fallar si la calve no está eneen el diccionario, si esto ocurre getget retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be betwen 1 and 5 (includesboth included)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   player2 = Robot(6, 1)
   
if __name__ == "__main__":
   main()

La salida para ese main es:

(2,1)
(3,1)
(4,1)
(5,1)
You are in the rigth limit
(5,2)
(5,1)
You are in the up limit
(4,1)
Invalid Argument
Traceback (most recent call last):
  File "test.py", line 84, in <module>
    main()
  File "test.py", line 81, in main
    player2 = Robot(6, 1)
  File "test.py", line 4, in __init__
    self.x = x
  File "test.py", line 20, in x
    raise ValueError("x must be betwen 1 and 5 (included)")
ValueError: x must be betwen 1 and 5 (both included)

move es definido como un método de instancia (definido dentro de la definición de la clase), como todo método de instancia recibe un primer parámetro obligatorio que es la instancia de la propia clase, que por convención se nombra por self, este parámetro es pasado de forma automática cuando se llama al método mediante una instancia (instancia.method()). En tu caso move se define como:

Aparte de esto swither debes definirlo o dentro de move o mejor en el __init__ como atributo de instancia, aparte los valores del diccionario son incorrectos, deben ser key: self.metodo.

va a fallar si la calve no está ene el diccionario, si esto ocurre get retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be betwen 1 and 5 (includes)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   
if __name__ == "__main__":
   main()

move es definido como un método de instancia (definido dentro de la definición de la clase), como todo método de instancia recibe un primer parámetro obligatorio que es la instancia de la propia clase, que por convención se nombra por self, este parámetro es pasado de forma automática cuando se llama al método mediante una instancia (instancia.method()). En tu caso move se define como:

Aparte de esto swither debes definirlo o dentro de move o mejor en el __init__ como atributo de instancia, aparte los valores del diccionario son incorrectos, deben ser key: self.metodo.

va a fallar si la calve no está en el diccionario, si esto ocurre get retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be betwen 1 and 5 (both included)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   player2 = Robot(6, 1)
   
if __name__ == "__main__":
   main()

La salida para ese main es:

(2,1)
(3,1)
(4,1)
(5,1)
You are in the rigth limit
(5,2)
(5,1)
You are in the up limit
(4,1)
Invalid Argument
Traceback (most recent call last):
  File "test.py", line 84, in <module>
    main()
  File "test.py", line 81, in main
    player2 = Robot(6, 1)
  File "test.py", line 4, in __init__
    self.x = x
  File "test.py", line 20, in x
    raise ValueError("x must be betwen 1 and 5 (included)")
ValueError: x must be betwen 1 and 5 (both included)
Origen Enlace
FJSevilla
  • 62.1k
  • 7
  • 43
  • 68

move es definido como un método de instancia (definido dentro de la definición de la clase), como todo método de instancia recibe un primer parámetro obligatorio que es la instancia de la propia clase, que por convención se nombra por self, este parámetro es pasado de forma automática cuando se llama al método mediante una instancia (instancia.method()). En tu caso move se define como:

def move(argument)

Al no definir el parámetro self, cuando haces player.move("ri") se le pasa la instancia de la clase de forma automática al primer argumento que es argument, además está el parámetro que tu le pasas "ri", por lo que la función recibe dos parámetros cuando solo tiene uno definido, la referencia a "ri" no tiene a dónde ir porque argument ya fue usado por la referencia a la instancia player.

Aparte de esto swither debes definirlo o dentro de move o mejor en el __init__ como atributo de instancia, aparte los valores del diccionario son incorrectos, deben ser key: self.metodo.

Por último;

func = switcher.get(argumen, "Invalid Argument")
return func()

va a fallar si la calve no está ene el diccionario, si esto ocurre get retorna "Invalid Argument", por lo que en la siguiente línea intentas llamar a una cadena:

return "Invalid Argument"()

lo que puedes hacer es algo así:

func = self._switcher.get(argumen)
if func is None:    
    return "Invalid Argument"
return func()

class Robot:

    def __init__(self, x, y):
        self.x = x
        self.y = y
        self._switcher = {
                         "le": self.move_left,
                         "ri": self.move_right,
                         "up": self.move_up,
                         "do": self.move_down,
                         }

    @property 
    def x(self):
        return self._x
    
    @x.setter
    def x(self, x_value):
        if not  1 <= x_value <= 5:
            raise ValueError("x must be betwen 1 and 5 (includes)")
        self._x = x_value
            
    @property 
    def y(self):
        return self._y
    
    @y.setter
    def y(self, y_value):
        if not  1 <= y_value <= 5:
            raise ValueError("y must be between 1 and 5 (both included)")
        self._y = y_value

    def move_left(self):
        if (self._x == 1):
            return "You are in the left limit"
        else:
            self._x -= 1
            return "({},{})".format(self._x, self._y)

    def move_right(self):
        if (self._x == 5):
            return "You are in the rigth limit"
        else:
            self._x += 1
            return "({},{})".format(self._x, self._y)

    def move_up(self):
        if (self._y == 1):
            return "You are in the up limit"
        else:
            self._y -= 1
            return "({},{})".format(self._x, self._y)

    def move_down(self):
        if (self._y == 5):
            return "You are in the down limit"
        else:
            self._y += 1
            return "({},{})".format(self._x, self._y)

    def move(self, argumen):
        func = self._switcher.get(argumen)
        if func is None:
            return "Invalid Argument"
        return func()


def main():
   player = Robot(1, 1)
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("ri"))
   print(player.move("do"))
   print(player.move("up"))
   print(player.move("up"))
   print(player.move("le"))
   print(player.move("fo"))
   
   
if __name__ == "__main__":
   main()

He añadido un properties para validar x e y en la instanciación y asegurarse que están entre 1 y 5.