La fórmula del cálculo de la cuota que pones al principio no se lee bien, pues entiendo que estaba en formato matemático y al pegarlo se ha perdido el formato. Creo que debería ser así:
No hay mucho margen de mejora en el código ya que consiste directamente en traducir esa fórmula a la sintaxis Python, cosa que has hecho correctamente.
Las únicas mejoras que se me ocurren son de legibilidad. Es decir, que quien lea el código pueda comprender fácilmente qué se está haciendo. Esto suele hacerse mediante comentarios, documentación y mediante una elección adecuada del nombre de las variables. También el uso de espacios alrededor de los operadores ayuda a leerlos mejor.
Usando estas ideas, la función quedaría así:
def calcular_mensualidad(hipoteca, numero_mesduracion, interes):
"""Esta función calcula el importe de una mensualidad
de una hipoteca, a partir de los datos de entrada:
hipoteca: cantidad total prestada
numero_mesduracion: número del mes para el cual se desea obtener
duración elen importeaños dedel lapréstamo mensualidad
interes: interés anual en tanto por ciento
"""
# Calculo del interés mensual, en tanto por 1
r = interes/(100*12)
# Cálculo de la mensualidad, según la fórmula
mensualidad = (hipoteca * r) / (1 - (1 + r)**(-12 * numero_mesduracion))
return mensualidad
Observa el uso del docstring que es la cadena entre triple comillas que aparece justo debajo de la cabecera de la función. Esta cadena no es un mero comentario, sino que pasa a formar parte de la documentación de la función, y sería mostrada en un intérprete en el que pusieras help(calcula_mensualidad)
.
Por lo demás, la funcionalidad es exactamente la misma (he quitado la última línea en la que llamabas a calcular()
pues supongo que se te coló por error; esa línea nunca llegaría a ejecutarse por estar después del return
, pero si se ejecutara sería peor, pues se llamaba a sí misma y causaría una recursión infinita).