0

Estou tendo problemas em alterar um código que fiz em else if para switch case. Não estou a conseguir que o código funcione, ele recebe valores e operação pelo $_GET e faz a operação selecionada:

Deixo o código aqui:

if((!is_numeric($num1) || $num1 == '' || $num1 == '0') && $operator == '/') 
    echo "ERRO: Numero 1 inserido e' zero";
else if((!is_numeric($num2) || $num2 == '' || $num2 == '0') && $operator == '/') 
    echo "ERRO: Numero 2 inserido e' zero";
else {
   //se a divisao for feita por zero, da mensagem de erro informando que o valor inserido foi zero
    if( !empty($operator) ) {
        if($operator == '+')
            $result = $num1 + $num2;
        else if($operator == '-')
            $result = $num1 - $num2;
        else if($operator == '*')
            $result = $num1*$num2;
        else
            $result = $num1/$num2;
        echo "O resultado é: ".$result;
   }
}
4
  • 1
    Mostre o que fez com switch. E diga qual é o problema que encontrou.
    – Maniero
    Commented 7/07/2016 às 13:24
  • não fiz muito ainda, nao sei como passar para switch case a primeira parte do !is_numeric Commented 7/07/2016 às 13:29
  • vc precisa 'separar' a validação dos números de entrada com a escolha da operação, o switch provavelmente será aplicado somente na escolha da operação.
    – rray
    Commented 7/07/2016 às 13:33
  • Vou tentar responder o que dá mas só com este trecho de código não poderei ajudar muita coisa
    – Maniero
    Commented 7/07/2016 às 13:34

2 Respostas 2

4

Precisaria melhorar muita coisa, mas basicamente seria isto:

if (!is_numeric($num1) || !is_numeric($num2) {
    echo "ERRO: as entradas precisam ser numéricas";
    //encerra de alguma forma, dpende do código.
}
$num1 = intval($num1); //pode  ser um floatval
$num2 = intval($num2);
switch ($operator) {
    case '+':
        $result = $num1 + $num2;
        break;
    case '-':
        $result = $num1 - $num2;
        break;
    case '*':
        $result = $num1 * $num2;
        break;
    case '/':
        if ($num2 == 0) {
            echo "ERRO: Não pode fazer divisão por zero";
            break;
        }
        $result = $num1 / $num2;
        break;
    default:
        //talvez mereça alguma mensagem de erro se não for nenhum operador válido
}
echo "O resultado é: " . $result;

Coloquei no GitHub para referência futura.

Não converta para o switch o que o if funciona melhor, faça isso apenas com a escolha dos operadores.

Não esqueça de converter os valores que estão como texto para numérico para fazer a conta apropriadamente.

O teste se é numérico é um pouco ingênuo.

O teste se o valor do divisor é 0 só precisa ser feito se o operador for de divisão, então coloca isso lá. E o dividendo pode ser 0.

Tem várias outras coisas que poderiam ser feitas de forma melhor.

Talvez a dificuldade seja porque não estava separando cada responsabilidade. Uma coisa é verificar se os dados estão dentro dos padrões mínimos para fazer algo útil, outra coisa e selecionar o que fazer. Não acho que compense verificar o valor condicional separadamente (caso da divisão por 0).

Precisa ver se quer só inteiros ou também valores decimais. Tem uma série de outras decisões para fazer um código bom para uso. Se era um exercício básico então teria sido melhor pelo menos tentar fazer o que deseja.

2
  • Obrigado pela ajuda! Ainda sou novato no php! abraço Commented 7/07/2016 às 13:46
  • 3
    +1, especialmente pelo aviso "Não converta para o switch o que o if funciona melhor, faça isto apenas com a escolha dos operadores."
    – Largato
    Commented 7/07/2016 às 16:07
2

Você pode tentar assim:

    $num1 = $_GET['num1']; 
    $operator = $_GET['operator']; 
    $num2 = $_GET['num2']; 

    switch (true) {
    case !is_numeric($num1) && $operator == '/':
    case $num1 == '' && $operator == '/':
    case $num1 == '0' && $operator == '/':
        echo "ERRO: Numero 1 inserido e' zero";
        break;

    case !is_numeric($num2) && $operator == '/':
    case $num2 == '' && $operator == '/':
    case $num2 == '0' && $operator == '/':
        echo "ERRO: Numero 2 inserido e' zero";
        break;
    default:
        switch ($operator) {
            case '+':
                $result = $num1 + $num2;
                break;
            case '-':
                $result = $num1 - $num2;
                break;
            case '*':
                $result = $num1 * $num2;
                break;
            case '/':
                $result = $num1 / $num2;
                break;
        }
        echo "O resultado é: ".$result;
    }
6
  • Olá Ricardo, vou tentar, mas parece-me a forma mais correcta! obrigado :) Commented 7/07/2016 às 13:36
  • Eu testei e funcionou aqui Mario. Caso não consiga poste o código pra nós. Commented 7/07/2016 às 13:37
  • 5
    switch true foi a primeira vez que eu vi :)
    – Maniero
    Commented 7/07/2016 às 13:46
  • 2
    @bigown o pior vai ser quando ele for tentar usar switch invertido numa linguagem mais séria e perceber que só funciona em PHP pq nao é otimizado .
    – Largato
    Commented 7/07/2016 às 16:06
  • 1
    @RicardoMota não precisa colocar "Espero ter ajudado" em todas suas respostas. A comunidade reconhece isso pelas respostas que você tem dado. Acompanha a discussão no meta Commented 14/11/2016 às 11:12

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .