Primero aclarar que no es una excepción lanzada por el intérprete, es solo una advertencia del analizador de código del IDE que nos avisa que se puede dar la situación en la que la variable opcion
no esté definida y por tanto, ocasionar una excepción en tiempo de ejecución:
Si la función recibe "Automatica", "automatica", "Manual" o "manual":
>>> print(opcion1("Automatica"))
"Usted eligio carga automatica"
Nada ocurre, la función retorna correctamente porque se cumple el primer condicional y se define opción opcion = "Usted eligio carga automatica"
.
Si la función recibe cualquier cosa que no sea las cuatro cadenas anteriores:
>>> print(opcion1("AUTOMATICA"))
Tenemos el problema sobre el que el IDE nos avisaba:
Exception has occurred: UnboundLocalError
local variable 'opcion' referenced before assignment
Esto ocurre porque no se cumple ningún condicional y dado que opcion
solo se define dentro de los dos condicionales,queda sin definir, por lo que en return opción
la variable no existe.
Aún planteándote que que tu función solo va a recibir valores válidos, no debes dar nunca pie a que esta situación pueda ocurrir. opcion
debe ser definida antes de llegar al return
o impedir que se llegue al return
si no va a estar definida.
Las soluciones son muchas, pero primero dos apuntes:
- Tu
else
con el if
dentro no tiene sentido, usa un elif
simplemente.
- Puedes (si no es una tarea y no te dejan) usar
str.lower()
para pasar la cadena recibida a minúscula, lo cual facilita la comparación.
Dicho esto tienes muchas opciones para resolver la situación, entre ellas:
Puedes plantearte definir la variable con un valor por defecto si no se entra en ningún condicional, tal como comenta @JackNavaRow:
def opcion1(carga):
opcion = "Usted no ha elegido una opcion valida"
carga = carga.lower()
if carga == "automatica":
opcion = "Usted eligio carga automatica"
elif carga == "manual":
opcion = "Usted eligio carga manual"
return opcion
También puedes usar una construcción if-elif-else
:
def opcion1(carga):
carga = carga.lower()
if carga == "automatica":
opcion = "Usted eligio carga automatica"
elif carga == "manual":
opcion = "Usted eligio carga manual"
else:
carga = "Usted no ha elegido una opcion valida"
return opcion
Prescindir de la variable y usar múltiples return
:
def opcion1(carga):
carga = carga.lower()
if carga == "automatica":
return "Usted eligio carga automatica"
elif carga == "manual":
return "Usted eligio carga manual"
else:
return "Usted no ha elegido una opcion valida"
Otra opción, aunque creo que no se ajusta al uso que le das a tu función, sería lanzar una excepción adecuada (ValueError
) si el valor del argumento no se ajusta a lo esperado:
def opcion1(carga):
carga = carga.lower()
if carga == "automatica":
return "Usted eligio carga automatica"
elif carga == "manual":
return "Usted eligio carga manual"
else:
raise ValueError("El argumento carga debe ser 'automatica' o 'manual'")
opción
no será definida... ¿Si ninguno de los condicionales se cumpliera que debería retornar la función? Es decir, ¿si carga no es "automatica", "Automatica", "manual" o "Manual" que debería pasar?