Empezando con python. Calcular ecuaciones de 2° grado

[BaByRoDrI]

Estoy empezando a programar en python gracias a un tutorial de maestros del web, y después de de haber leído el primer capitulo y leer un poco de la documentación decidi tomar un código que había escrito en C y pasarlo a Python, al ejecutarlo en Eclipse me va bien, solo quería ver que cosas podría mejorar y algunas buenas practicas que pueda seguir.

El código:

Código Python:

'''
Created on 10/02/2012
 
@author: rodrigo
'''
def calcular(a, b, c):
    import math
    disc = (math.pow(b, 2)) - (4 * a * c)
    if disc > 0:
        x1 = (-b + math.sqrt(disc)) / (2 / a)
        x2 = (-b - math.sqrt(disc)) / (2 / a)
        print "\n\nLa ecacuacion tiene dos posibles soluciones: x1 = %.2f y x2 = %.2f" % (x1, x2)
    else:
        if disc == 0:
            x1 = -b / (2 * a)
            print "\n\nLa ecuacion tiene una posible solucion: x = %.2f" % (x1)
        else:
            xr = -b / (2 * a)
            xi = (math.sqrt(-disc)) / (2 * a)
            print "\n\nLa ecuacion tiene dos posibles soluciones, una real xr = %.2f y una imaginarinaria xi = %.2fi" % (xr, xi)
 
print "Solucionador de ecuaciones de segundo grado"
print "_" * 43
a = raw_input("\tEscriba el valor de a: ")
while a == 0:
    a = raw_input("\tEl valor de a no puede ser 0, por favor corrijalo: ")
b = raw_input("\tEscriba el valor de b: ")
c = raw_input("\tEscriba el valor de c: ")
 
a = float(a)
b = float(b)
c = float(c)
 
calcular(a, b, c)
 

Muchas gracias a todos los que contesten

[razpeitia]

Si quieres hacer tu código mas legible entonces lee la guía de estilo y listo. Si quieres seguir buenas practicas entonces puedes leer libros acerca del tema, pero nada te dará más conocimiento que la practica.

Código Python:

ef calcular(a, b, c):
    discriminante  = b * b - 4 * a * c
    if discriminante >= 0:
        return ((-b + (discriminante ** .5)) / (2.0 / a),
                (-b - (discriminante ** .5)) / (2.0 / a))
    else:
        discriminante = -discriminante
        return (complex(-b, (discriminante ** .5)) / (2.0 * a),
                complex(-b, -(discriminante ** .5)) / (2.0 * a))
 
def menu():
    print "Solucionador de ecuaciones de segundo grado"
    print "_" * 43
 
def leerFlotante(texto):
    return float(raw_input(texto))
 
 
menu()
a = leerFlotante("\tEscriba el valor de a: ")
while a == 0.0:
    a = leerFlotante("\tEl valor de a no puede ser 0, por favor corrijalo: ")
b = leerFlotante("\tEscriba el valor de b: ")
c = leerFlotante("\tEscriba el valor de c: ")
 
x1, x2 = calcular(a, b, c)
 
if x1 == x2:
    print "Soluciones reales e iguales: x1 = x2 = %.2f" % x1
elif type(x1) == complex:
    print "Soluciones complejas y conjugadas x1 = %.2f+%.2fi x2 = %.2f+%.2fi" % (x1.real, x1.imag, x2.real, x2.imag)
else:
    print "Soluciones reales y diferentes x1 = %.2f, x2 = %.2f" % (x1, x2)
 

Te dejo mi código para que lo puedas comprar con el tuyo.

El tuyo no funciona bien cuando pones el 0.

Algunos puntos importantes:
1.- Quite la librería math, que por ciertos los imports solo se hacen al principio del script, no dentro de funciones o de algún ciclo, por un import por lo regular tiene un costo computacional alto.
2.- Calcula solo hace lo que debe calcular, no imprime no hace nada mas, solo calcula.
3.- Cree una función leerFlotante por que al parecer se repite mucho, ademas si mas adelante quisieras hacer una validación lo puedes hacer en esta función y no cambiar todos los raw_input a lo largo de todo tu programa.
4.- Hice una función menu en caso de que lo utilices en varias partes entonces, solo mandas a llamar a la función menu,.
5.- etc... (Algunos otros cambios creo que los descubrirás por tu cuenta.)

[BaByRoDrI]

Solo quería preguntar una cosa más acerca de tu algoritmo, veo que usas la función complex, y posteriormente veo que tiene x1.real x1.imag

Estos valores REAL e IMAG son resultado propiamente de la operación complex, cierto?

Y una observación, en la línea 4 y 5, en vez de que el 2 divida a "a", deberían estarse multiplicando.
De allí en fuera creo haber entendido todo y ya solo me queda seguir aprendiendo y checar esa guía de estilo.

Muchas gracias!

Iniciado por razpeitia

Si quieres hacer tu código mas legible entonces lee la guía de estilo y listo. Si quieres seguir buenas practicas entonces puedes leer libros acerca del tema, pero nada te dará más conocimiento que la practica.

Código Python:

ef calcular(a, b, c):
    discriminante  = b * b - 4 * a * c
    if discriminante >= 0:
        return ((-b + (discriminante ** .5)) / (2.0 / a),
                (-b - (discriminante ** .5)) / (2.0 / a))
    else:
        discriminante = -discriminante
        return (complex(-b, (discriminante ** .5)) / (2.0 * a),
                complex(-b, -(discriminante ** .5)) / (2.0 * a))
 
def menu():
    print "Solucionador de ecuaciones de segundo grado"
    print "_" * 43
 
def leerFlotante(texto):
    return float(raw_input(texto))
 
 
menu()
a = leerFlotante("\tEscriba el valor de a: ")
while a == 0.0:
    a = leerFlotante("\tEl valor de a no puede ser 0, por favor corrijalo: ")
b = leerFlotante("\tEscriba el valor de b: ")
c = leerFlotante("\tEscriba el valor de c: ")
 
x1, x2 = calcular(a, b, c)
 
if x1 == x2:
    print "Soluciones reales e iguales: x1 = x2 = %.2f" % x1
elif type(x1) == complex:
    print "Soluciones complejas y conjugadas x1 = %.2f+%.2fi x2 = %.2f+%.2fi" % (x1.real, x1.imag, x2.real, x2.imag)
else:
    print "Soluciones reales y diferentes x1 = %.2f, x2 = %.2f" % (x1, x2)
 

Te dejo mi código para que lo puedas comprar con el tuyo.

El tuyo no funciona bien cuando pones el 0.

Algunos puntos importantes:
1.- Quite la librería math, que por ciertos los imports solo se hacen al principio del script, no dentro de funciones o de algún ciclo, por un import por lo regular tiene un costo computacional alto.
2.- Calcula solo hace lo que debe calcular, no imprime no hace nada mas, solo calcula.
3.- Cree una función leerFlotante por que al parecer se repite mucho, ademas si mas adelante quisieras hacer una validación lo puedes hacer en esta función y no cambiar todos los raw_input a lo largo de todo tu programa.
4.- Hice una función menu en caso de que lo utilices en varias partes entonces, solo mandas a llamar a la función menu,.
5.- etc... (Algunos otros cambios creo que los descubrirás por tu cuenta.)

[razpeitia]

Iniciado por BaByRoDrI

Y una observación, en la línea 4 y 5, en vez de que el 2 divida a "a", deberían estarse multiplicando.

Si deberían de estarse multiplicando, creo que eso también cuenta para tu código .