3

Tenho a seguinte função em PHP:

public function teste() {
    $conta = "0/(0+0+0)";
    eval('$result = (' . $conta . ');');
    echo $result;
}

No caso, a fórmula (no exemplo 0/(0+0+0) obviamente volta erro por causa da divisão por zero. O problema é que esta fórmula é construída dinamicamente e pode ocorrer de acontecer divisão por zero. Neste caso gostaria que retornasse apenas zero e não o erro.

8
  • Se esses dados são dinamicos voce pode fazer uma verificação neles pra saber se são iguais a zero ou não, antes de fazer esse calculo.
    – Marcelo
    Commented 21/02/2018 às 12:48
  • 1
    poderia usar de exemplo: a/b+c+d, ficaria mais legivel.... if (b+c+d = 0) echo '0' else ....calcula... !? Commented 21/02/2018 às 12:49
  • 1
    O grande problema é que a formula pode ter divisão ou não ter. O usuário consegue montar diversas fórmulas matemáticas. Commented 21/02/2018 às 12:55
  • 1
    @FelipeSaraiva se você valida você está já compilando (as constantes eu pus só de exemplo). Se você está compilando, você pode montar a árvore de expressão e tratar ela internamente, de acordo com a ordem da precedência. Commented 21/02/2018 às 13:03
  • 1
    @FelipeSaraiva escreva uma resposta oficialmente. Estou com particular preguiça/falta de tempo para fazer uma resposta formal Commented 21/02/2018 às 13:08

2 Respostas 2

8

Primeiro, não deveria usar eval(). Tem que ter um domínio muito forte em programação para usá-lo sem problemas. E quem tem esse domínio sempre acha uma solução melhor.

Divisão por zero é considerado um erro de programação, é a mesma coisa que dar um erro de ponteiro (PHP não dá isso, acho, mas ele trata de forma que considero prejudicial), ele nunca deveria ocorrer, a solução é garantir que não ocorra a divisão por zero, então deve garantir que o divisor não seja zero antes de usá-lo. Um simples if antes resolve. Mas neste caso nem precisa já que você sabe que é 0, não tem porque fazer isto em código real.

Sempre tratar a divisão por 0 como resultando em 0 é um erro matemático, se isto fosse correto a matemática faria isto. E quando fizer sentido por algum outro motivo que não o matemático está usando o número para uma função que ele não deve ter. Novamente, erros de programação não devem ser tratados como se fossem normais, ou como se fossem exceções.

Quando for capturar exceção, e em códigos bons isto raramente ocorre, nunca capture Exception, esta exceção deveria até ser abstrata. Talvez tudo que serve para herdar deveria.

Veja também:

Coloquei no GitHub para referência futura.

1

Colocar um try / catch não seria a solução?

public function teste() {
    $conta = "0/(0+0+0)";
    try {
      eval('$result = (' . $conta . ');');
      echo $result;
    }
    catch(Exception $e) {
      echo 0;
    }    
}
9
  • 4
    funciona... mas se você pode checar se o valor é zero antes de calcular, porque disparar uma exception ?! ps. O -1 não fui eu =] Commented 21/02/2018 às 12:53
  • 1
    @RovannLinhalis, só uma compilaçãozinha básica ;-) Commented 21/02/2018 às 12:58
  • 1
    Uma baita complicação, rsss. Tentei esta solução mas ainda apresenta erro. Commented 21/02/2018 às 13:00
  • 1
    @RovannLinhalis correto, respondi sobre isso
    – Maniero
    Commented 21/02/2018 às 14:30
  • 3
    @RovannLinhalis a implmentação com eval() está, é inseguro e dá uma experiência péssima onde na prática que for cadastrar a fórmula precisa ser praticamente programador. Se tiver que fazer um recurso de cadastrar fórmula é necessário fazer uma mecanismo bem complexo e deve permitir indicar restrições de dados que esta fórmula deve ter, incluindo não dividir por 0. Claro, a pessoa pode fazer o que quiser, mas é uma enorme gambiarra simplificar o que é complexo por natureza. Tem uma solução bem mais simples.Ñ deixa a pessoa cadastrar fórmulas. Pode ser a melhor solução e melhor do que imaginam
    – Maniero
    Commented 21/02/2018 às 15:43

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 .