Foro de elhacker.net

Hola, acabo de empezar hace un par de dias y he hecho esta calculadora:
Tiene algunas limitaciones ya que tienes que meter una secuencia de operaciones
ejemplo:
1+2+3+4 , es por eso que solo acepta numeros de 1 digito, y ademas es con parentesis
1+2*(3+4) daria un mal resultado, para que te de bien tendrias q poner 1+(2*(3+4))
Bueno aqui pongo el codigo para que lo vean, Si ven que puese hacerse mejoras o hacer un codigo mas pequeño ps genial, me lo comentan para seguir aprendiendo.

 
def calcula(a):
	total=int(a[0])
	i=1
	while (i<len(a)):
		if (a[i]=='+'):
			if (a[i+1]=='('):
				M=calcula(a[i+2:len(a)-1])
				total = total+ M[0]
				i+=M[1]
			else:
				total=total+int(a[i+1])
				i=i+1
		elif (a[i]=='-'):
			if (a[i+1]=='('):
				M=calcula(a[i+2:len(a)-1])
				total = total- M[0]
				i+=M[1]
			else:
				total=total-int(a[i+1])
				i=i+1
		elif (a[i]=='*'):
			if (a[i+1]=='('):
				M=calcula(a[i+2:len(a)-1])
				total = total*M[0]
				i+=M[1]
			else:
				total=total*int(a[i+1])
				i=i+1
		elif (a[i]=='/'):
			if (a[i+1]=='('):
				M=calcula(a[i+2:len(a)-1])
				total = total/M[0]
				i+=M[1]
			else:
				total=total/int(a[i+1])
				i=i+1
		elif (a[i]==')'):
			break
		i+=1
	return [total,i+2]
opcion=0
while opcion!=2:
	print 'Las opciones de la Calculador son: '
	print '1)Calcular la secuencia de datos'
	print '2)Salir'
	opcion=input('Escoga la opcion q desea:  ')
	if (opcion==2):
		break
	else:
		a = list (raw_input('Dame la secuencia:   '))
		print 'El resultado de la secuencia es:   ',calcula(a)[0]
		opcion=input('Si desea salir pulse 2 en caso contrario cualquier tecla:   ')
print 'Gracias por usar la Calculadora lDanny :)'
 

Mucho más corta :D

opera = raw_input('Ingrese cantidad de divisores: ')
print('Resultado: ' + str(eval(opera)))

Naaaa, es broma, el tuyo realmente revisa la operación y este otro se lo tira a python

Se me ocurre hacerlo con un diccionario en lugar de los IF :D

Saludos

Novlucker, jejeje es verdad es mas corta, la mia es mas larga xDDDDDD
Bueno lo he probado y es verdad que es buena, pero la verdad por lo que yo hice este code era por esto
5+3+4*3(2+3)+5
aunque mi code lo hace, pero para ello ahi que meter parentesis
Y lo de diccionario no lo he mirado todavía, lo mirare para que se me hagan mas cortos los programas
Gracias Novlucker

Claro, eso si no lo hace python :D

Mira: http://foro.elhacker.net/net/calculadora_por_jerarquia_c-t262744.0.html

Creo que en realidad con eso te lo estoy empeorando, además de que el code esta en C#, pero si lo entiendes ligeramente (creo que no vas a tener problemas) puedes tomar nuevas ideas ;)

Saludos

Gracias, lo voy a mirar, aunque este calculadora recien lo hice hace 2 horas, solo queria hacer mi calculadora ya que veo, q varios novatos hacen una, solo q  queria hacerla un poco mejor xDDD
Ahora estoy resolviendo los ejercicios de project euler, tratare de mejorar la calculadora (cuando aprenda mas cosas) y cuando la tengo mejorada lo volvere a poner el code para que vena q tal.
Gracias por responder y por las ayudas.

Hola, aqui pongo el codigo de la calculadora pero mejorada, aunque sigue teniendo la limitacion de que es solo para numeros de 1 digito, Gracias Novlucker, por el link que me pasaste ya que en ese link se discutia como podria mejorarse la calculadora y pusieron una idea sobre una pila, y de ahi he mejorado mi funcion.
Usando la lista como pila, ya que lo ultimo que entra es lo primero en salir.
Bueno mi codigo, si ven que se puede mejorar diganlo me ayudaran a mejorar.

 
def Calculadora(L):
	cal=0
	M=[]
	while (len(L)>0):
		total=len(L)
		if (L[total-1]==')'):
			L=L[:total-1]
			M=Calculadora(L)
			L=M[1]
			cal = cal+M[0]
		elif (L[total-1]=='+'):
			cal=cal+int(L[total-2])
			L=L[:total-2]
		elif (L[total-1]=='-'):
			cal=cal-int(L[total-2])
			L=L[:total-2]
		elif (L[total-1]=='*'):
			cal=cal*int(L[total-2])
			L=L[:total-2]
		elif (L[total-1]=='/'):
			cal=cal/int(L[total-2])
			L=L[:total-2]
		elif (L[total-1]=='('):
			L=L[:total-1]
			break
		else:
			cal=cal+int(L[total-1])
			L=L[:total-1]
	return [cal,L]
opcion=0
while opcion!=2:
	print 'Las opciones de la Calculador son: '
	print '1)Calcular la secuencia de datos'
	print '2)Salir'
	opcion=input('Escoga la opcion q desea:  ')
	if (opcion==2):
		break
	else:
		a = list (raw_input('Dame la secuencia:   '))
		print 'El resultado de la secuencia es:   ',Calculadora(a)[0]
		opcion=input('Si desea salir pulse 2 en caso contrario cualquier tecla:   ')
print 'Gracias por usar la Calculadora lDanny :)'