Skip to main content
se añadieron 486 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Imagino que el propósito del ejercicio es aprender el uso de if ... elseif ... else y el error de sintaxis es porque no usas bien esa estructura, tienes dos if seguidos y terminas con un else.

La sintaxis correcta en esos casos es:

if (expresion) {

} elseif (expresion) {

} else {

}

Así debería funcionar:

if(isset($btn_multiplicar)) {
   echo $resultado_multiplicacion;
} elseif (isset($btn_sumar)) {
    echo $resultado_suma;
} else {
    echo "nada";
}

Ahora bien, tu código queda expuesto a posibles casos de Undefined index y deberías usar una especie de variable final.

Quizá algo así es más coherente:

$outPut="Nada posteado";
if ( isset($_GET['numero1']) && isset($_GET['numero2']) ) {
    $numero1 = $_GET['numero1'];
    $numero2 = $_GET['numero2'];
    if ( isset($_GET['multiplicar']) ) {
        $outPut = $numero1 * $numero2;
    } elseif ( isset($_GET['sumar']) ) {
        $outPut = $numero1 + $numero2;
    } else {
        $outPut = "Ninguna operación posible";
    }
}
echo $outPut;

El código verifica primero que hay materia (los dos números fueron enviados) y luego decide la operación a realizar con esos números. Y, por otra parte, no declara variables que quizá no vayas a usar. Controla todo en una única variable de salida.

Cabe decir que en programas reales, estas verificaciones son más simples con el operador de fusión de null o con operadores ternarios, pero entiendo que lo tuyo es un ejercicio y no conviene pasarse de listo usando cosas que no hayas visto en clases.

Imagino que el propósito del ejercicio es aprender el uso de if ... elseif y el error de sintaxis es porque no usas bien esa estructura.

Así debería funcionar:

if(isset($btn_multiplicar)) {
   echo $resultado_multiplicacion;
} elseif (isset($btn_sumar)) {
    echo $resultado_suma;
} else {
    echo "nada";
}

Ahora bien, tu código queda expuesto a posibles casos de Undefined index y deberías usar una especie de variable final.

Quizá algo así es más coherente:

$outPut="Nada posteado";
if ( isset($_GET['numero1']) && isset($_GET['numero2']) ) {
    $numero1 = $_GET['numero1'];
    $numero2 = $_GET['numero2'];
    if ( isset($_GET['multiplicar']) ) {
        $outPut = $numero1 * $numero2;
    } elseif ( isset($_GET['sumar']) ) {
        $outPut = $numero1 + $numero2;
    } else {
        $outPut = "Ninguna operación posible";
    }
}
echo $outPut;

Cabe decir que en programas reales, estas verificaciones son más simples con el operador de fusión de null o con operadores ternarios, pero entiendo que lo tuyo es un ejercicio y no conviene pasarse de listo.

Imagino que el propósito del ejercicio es aprender el uso de if ... elseif ... else y el error de sintaxis es porque no usas bien esa estructura, tienes dos if seguidos y terminas con un else.

La sintaxis correcta en esos casos es:

if (expresion) {

} elseif (expresion) {

} else {

}

Así debería funcionar:

if(isset($btn_multiplicar)) {
   echo $resultado_multiplicacion;
} elseif (isset($btn_sumar)) {
    echo $resultado_suma;
} else {
    echo "nada";
}

Ahora bien, tu código queda expuesto a posibles casos de Undefined index y deberías usar una especie de variable final.

Quizá algo así es más coherente:

$outPut="Nada posteado";
if ( isset($_GET['numero1']) && isset($_GET['numero2']) ) {
    $numero1 = $_GET['numero1'];
    $numero2 = $_GET['numero2'];
    if ( isset($_GET['multiplicar']) ) {
        $outPut = $numero1 * $numero2;
    } elseif ( isset($_GET['sumar']) ) {
        $outPut = $numero1 + $numero2;
    } else {
        $outPut = "Ninguna operación posible";
    }
}
echo $outPut;

El código verifica primero que hay materia (los dos números fueron enviados) y luego decide la operación a realizar con esos números. Y, por otra parte, no declara variables que quizá no vayas a usar. Controla todo en una única variable de salida.

Cabe decir que en programas reales, estas verificaciones son más simples con el operador de fusión de null o con operadores ternarios, pero entiendo que lo tuyo es un ejercicio y no conviene pasarse de listo usando cosas que no hayas visto en clases.

Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Imagino que el propósito del ejercicio es aprender el uso de if ... elseif y el error de sintaxis es porque no usas bien esa estructura.

Así debería funcionar:

if(isset($btn_multiplicar)) {
   echo $resultado_multiplicacion;
} elseif (isset($btn_sumar)) {
    echo $resultado_suma;
} else {
    echo "nada";
}

Ahora bien, tu código queda expuesto a posibles casos de Undefined index y deberías usar una especie de variable final.

Quizá algo así es más coherente:

$outPut="Nada posteado";
if ( isset($_GET['numero1']) && isset($_GET['numero2']) ) {
    $numero1 = $_GET['numero1'];
    $numero2 = $_GET['numero2'];
    if ( isset($_GET['multiplicar']) ) {
        $outPut = $numero1 * $numero2;
    } elseif ( isset($_GET['sumar']) ) {
        $outPut = $numero1 + $numero2;
    } else {
        $outPut = "Ninguna operación posible";
    }
}
echo $outPut;

Cabe decir que en programas reales, estas verificaciones son más simples con el operador de fusión de null o con operadores ternarios, pero entiendo que lo tuyo es un ejercicio y no conviene pasarse de listo.