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 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:
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 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.
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 between 1 and 5 (both included)")
ValueError: x must be betwen 1 and 5 (both included)
move()
es incorrecta, debiera ser:def move(self, argumen):
(te faltó elself
)