Hay varias cosas que arreglar en tu código.
De entrada, la indentación del código que aparece en la pregunta no es correcta, aunque supongo que eso es debido a un copia-pega incorrecto. Entiendo que el cuerpo de la función calificar_alumno()
llega sólo hasta el return
y que todo lo que va después, aunque en tu pregunta aparece indentado y por tanto dentro de la función, iría en realidad desindentado y afuera.
Pero una vez arreglada la indentación, al ejecutar tu código efectivamente se llega a un punto en el que aparece el error:
TypeError: '>=' not supported between instances of 'str' and 'int'
que probablemente es el que te ha hecho pensar que hay un problema por asignar una cadena en una variable que antes era entera.
En realidad no hay ningún problema por cambiar el tipo de una variable, ya que en python el tipado es dinámico. El problema viene por otra razón. Por ejemplo observa este fragmento de tu código e imagina que la nota que estás procesando tiene valor
2:
if valor>=0 and valor<3:
valor = "muy mal"
if valor>=3 and valor<5:
valor = "insuficiente"
Al ser valor
2, se entra por el primer if
, por lo que se le asigna a valor
la cadena "muy mal"
. Esa asignación no causa problemas. Pero el problema surge en el siguiente if
en el que comparas valor >= 3
. Es esta comparación la que ya no puede hacerse debido a que valor
ahora es "muy mal"
que no puede compararse con 3.
La solución es simple. Una vez se ha entrado por un if
ya no hay que mirar el resto de casos, lo que se arregla cambiando los restantes if
por elif
.
De paso vamos a arreglar un par de cosas más:
- Las funciones no deberían imprimir nada, sino retornar un resultado (en este caso retornar
notas
). Tu función tiene un return print()
pero eso solo retorna el valor devuelto por print()
que es siempre None
(y no lo que print()
imprime). Mejor dejar que sea el programa principal quien imprima el resultado.
- El cambiar
valor
no altera el diccionario. Para alterar el diccionario tienes que asignar el resultado en notas[clave]
- Si vamos a alterar directamente el diccionario, no necesitamos que la función retorne nada, ya que la propia variable
notas
del programa principal resultará alterada.
Con estas consideraciones el código queda así:
# @title modificar diccionario de notas
def calificar_alumno(notas):
for clave, valor in notas.items():
if valor>=0 and valor<3:
valor = "muy mal"
elif valor>=3 and valor<5:
valor = "insuficiente"
elif valor>=5 and valor<6:
valor = "suficiente"
elif valor>=6 and valor<7:
valor = "bien"
elif valor>=7 and valor<9:
valor = "notable"
elif valor>=9 and valor<10:
valor = "sobresaliente"
notas[clave] = valor # Modificar el diccionario
notas = {}
continuar = True
while continuar == True:
asignatura = input("Introduce la asignatura: ")
nota = int(input("Introduce la nota de la asignatura: "))
notas[asignatura] = nota
continuar = input("¿desea continuar añadiendo?(si/no): ")
if continuar == "si":
continuar = True
else:
continuar = False
calificar_alumno(notas)
print(notas)
Otra versión
En general no es muy buena idea que una función modifique el parámetro que ha recibido, para causar "efectos secundarios" en el programa que la llamó (es decir, que el programa principal obtenga su diccionario notas
modificado). Este estilo de programación puede llevar a bugs difíciles de encontrar.
Es mejor si la función en vez de modificar el diccionario que recibe como parámetro construye otro con la traducción de notas a texto.
Esto puede ser tan simple como:
def calificar_alumno(notas):
return { clave: nota_a_texto(nota) for clave, nota in notas.items()}
en donde estoy haciendo uso de la sintaxis dictionary comprehension para crear un diccionario directamente iterando por otro, y estoy suponiendo que hay una función nota_a_texto()
que recibe una nota (por ejmeplo 3) y retorna su representación como texto (por ejemplo "insuficiente"). Esta función puede ser la que aparece en la respuesta de @CandidMoe.
Por ofrecer aún más opciones, seguidamente muestro otra forma de programar una función nota_a_texto()
que en lugar de tener una serie de if
... elif
, tiene la información necesaria para la conversión en una lista e itera por ella.
def nota_a_texto(nota):
limites = ((0, "muy mal"),
(3, "insuficiente"),
(5, "suficiente"),
(6, "bien"),
(7, "notable"),
(9, "sobresaliente"))
for umbral, calificacion in limites:
if nota >= umbral:
texto = calificacion
else:
break
return texto
Ejemplo de funcionamiento:
notas = {"Mate": 2, "Lengua": 3, "TIC": 10, "Fisica": 5.5}
result = calificar_alumno(notas)
print(result)
{'Mate': 'muy mal', 'Lengua': 'insuficiente', 'TIC': 'sobresaliente', 'Fisica': 'suficiente'}