Parece que en lugar de la condición i=="-"
has escrito i="-"
.
No obstante voy a aprovechar la respuesta para refactorizar un poco el código y hacerlo más legible (y por tanto más mantenible). Ya que Python admite retornos de carro dentro de una expresión entre paréntesis, podemos encerrar toda la condición tras el if
entre paréntesis para así poder partirla en varias líneas y leerla mejor.
Primera refactorización, sólo reformatear:
operador=["+","-","","/","^"]
for i in operador:
if ((i in entrada)
and not (i=="-" and entrada.find("-")==0
or (entrada[entrada.find("-")-1] in ["+","-","","/","(","|","^"])
)
):
recursivo=True
Entrando a ver qué se intenta evaluar en el if
, veo entrada.find("-")==0
que aparentemente trata de decidir si el primer carácter de entrada
es un "-"
. Podemos usar en su lugar entrada.startswith("-")
, que parece más claro en cuanto a su intención.
¿Qué está evaluendo el and not ( ... and ... or ...)
? Aparentemente, que no suceda una de las siguientes:
- Que
i
sea "-"
(si lo es, ya no se cumple el if
)
- Que
entrada
no comience por "-"
(si comienza, ya no se cumple el if
)
- Que, si se cumplen las dos anteriores, el carácter anterior al primer
"-"
sea uno de estos: ["+","-","","/","(","|","^"]
. Si no lo es, ya no se cumple el if
.
Por tanto ese conjunto de caracteres podría denominarse "operadores válidos antes del menos".
Pero aún así, la lógica interna del if
me resulta difícil de leer. Propongo deshacer ese único if
en varios, testeando en cada uno la condición que me haría el if
falso y saltando a la siguiente iteración (continue
) cuando así ocurra.
Segunda refactorización. Cambiar condición compleja por varios if
:
operador=["+","-","","/","^"]
operadores_validos_ante_menos = ["+","-","","/","(","|","^"]
for i in operador:
if i not in entrada: # Saltarse operadores que no estén en la entrada
continue
if i != "-": # Saltarse operadores que no sean el "-"
continue
if entrada.startswith("-"): # Saltarse el - si está al inicio
continue
if entrada[entrada.find("-")-1] in operadores_validos_ante_menos:
continue
recursivo=True
Creo que, si no me he equivocado, esta implementación captura la misma condición que implementaste en tu compleja expresión booleana (he hecho algunos tests y devuelve los mismos resultados). No tengo claro no obstante si hace lo que esperabas. Quizás el reescribirlo de esta otra forma te haga más fácil ver algún posible fallo en su lógica.
not(i="-"
debería sernot(i=="-"
, supongo"if"
y el":"