[SOLUCIONADO] Mi calculadora no resuelve

[valenfontana11]

Hola amigos, para terminar de sacar informatica me pidieron una calculadora simple, que solo resuelva la operacion, sin nada mas complejo. Bien, el problema es que en el lugar donde tendria que aparecer el resultado aparece la operacion que el usuario hizo. Si no me explico bien les dejo una foto



En vez de mostrar 10, muestra 7+3. Bueno eso es todo, espero que puedan solucionar mi problema. Aca dejo el codigo :) . Muchas gracias

Código PHP:

 <form name="formulario" method="post" action="<?php echo $_SERVER['PHP_SELF'];?>"> 
Operacion: <input type="text" name="calculadora" value=""> <p>
<input type="submit" /><p>
</form>
</div>
<?php
$resultado=$_POST['calculadora'];  
echo "Resultado: $resultado";
?>   

[informacionsys]

hola..

puedes usar la funcion nativa eval de php, aunque no es recomendable ...
[/HIGHLIGHT]

[valenfontana11]

Gracias, excelente, dejo el codigo por si alguien tiene el mismo problema que yo. Aunque me pueden explicar porque es peligroso usar eval() ?

Aca el codigo:

Código PHP:

 <form name="formulario" method="post" action="<?php echo $_SERVER['PHP_SELF'];?>"> 
Operacion: <input type="text" name="calculadora" value=""> <p>
<input type="submit" /><p>
</form>
</div>
<?php
$resultado=$_POST['calculadora'];  
$resultadofinal="echo $resultado;"; 
eval($resultadofinal); 
?>   

[pateketrueke]

Iniciado por valenfontana11

Aunque me pueden explicar porque es peligroso usar eval() ?

Eso se explica en el manual: http://php.net/manual/es/function.eval.php

Extracto:

El constructor de lenguaje eval() es muy peligroso porque permite la ejecución de código de PHP arbitrario. Su uso está totalmente desaconsejado. Si se ha verificado cuidadosamente que no existe otra opción que usar este constructor, se ha de poner especial atención en no pasar ninguna información proporcionada por el usuario a esta función sin haberla validado apropiadamente con anterioridad.

[NSD]

Una calculadora que sume, reste, multiplique y divida puede ser implementada de forma simple y segura así:

Código PHP:

<?php
    // La operación a resolver.
    $op = "7 + 5 - 4 / 2 + 3 * 2";
 
    // Eliminar espacios y caracteres @ existentes, luego agregar un @ antes de cada signo y al final de la formula.
    $op = str_replace(["@", " ", "+", "-", "*", "/"], ["", "", "@+", "@-", "@*", "@/"], $op)."@";
 
    // Variables a usar.
    $final = 0; // Acumulador, almacena el resultado final.
    $parcial = 0; // Acumulador, almacena el resultado parcial de cada termino.
    $signo = "+"; // Indicador, almacena el ultimo signo empleado, por defecto suma.
 
    // Recorrer cada numero y operación de la formula de forma implicita.
    // Se usa la variable original como iteradora para ahorrar memoria.
    foreach(explode("@", $op) as $op)
    {
        // Si es una operación de alta prioridad se procesa dentro del termino parcial.
        if($op[0] == "*" || $op[0] == "/") // El primer carácter es la operación.
            $parcial *= ($op[0] == "*" ? substr($op, 1) : 1/substr($op, 1)); // Multiplicar o dividir.
        else
        {
            // Si es una operacion de baja prioridad se procesa dentro del resultado final.
            $final += $parcial * ($signo == "+" ? 1 : -1); // Sumar o restar.
 
            // Si el numero tiene un signo.
            if($op[0] == "+" || $op[0] == "-")
            {
                $parcial = substr($op, 1); // Guardar el numero sin signo.
                $signo = $op[0]; // Guardar el signo.
            }
            else // Es un numero sin signo, Ej: el primer numero de la formula.
                $parcial = $op;
        }
    }
    unset($parcial, $signo, $op); // Liberar la memoria.
    echo $final; // Mostrar el resultado.