Skip to main content
adicionou 127 caracteres ao conteúdo
Fonte Link
Daniel Omine
  • 20,1mil
  • 1
  • 35
  • 67

O problema está na fórmula de porcentagem.

($recipientAmount * $totalAmount) / 100

Deveria ser

($recipientAmount * 100) / $totalAmount

FalandoIsso é o suficiente para resolver o problema. Caso queira saber mais, continue a leitura.





Falando um pouco mais do assunto e apresentando algumas sugestões e observações:

#A fórmula que calcula porcentagem

A * 100 / T

A -> Valor do qual quer obter a porcentagem em relação a T.

Exemplo:

30 * 100 / 118.7 resulta em 25.273799494524

#Um exemplo genérico em PHP:

/*
O array original da pergunta não possui essa estrutura, mas fiz assim apenas para simplificar. é algo irrelevante.
*/
$arr = array(
    3000,
    1000,
    7870
);

// Transforma em centavos
$arr = array_map(function($v){return $v / 100;}, $arr);

$total = array_sum($arr);

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    echo $v * 100 / $total.'<br>'.PHP_EOL;
}

#Alternativa otimizada

Note que a conversão de centavos é dividir por 100. Mas a expressão matemática fará a multiplicação por 100. Portanto, é um processo redundante, dentro do seu caso específico. Poderia simplificar sem converter os centavos.

Uma nova versão otimizada:

$arr = array(
    3000,
    1000,
    7870
);

//Pula essa bagaceira
//$arr = array_map(function($v){return $v / 100;}, $arr);

//Converte apenas o total
$total = array_sum($arr) / 100;

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    // Aqui não foi necessário a multiplicação pois seria redundante
    echo $v / $total.'<br>'.PHP_EOL;
}

#Observação sobre o round()

No código da pergunta está aplicando arredondamento com a função round(). Avalie bem se realmente quer aplicar o arredondamento, pois o resultado será 25, 8 e 66, cuja soma perfaz 99%.

Está comendo 1%. Mantenha a integridade fracionária. A menos que o modelo de negócios queira mesmo "comer" essas pequenas frações. Nesse ponto não posso opinar.

O problema está na fórmula de porcentagem.

($recipientAmount * $totalAmount) / 100

Deveria ser

($recipientAmount * 100) / $totalAmount

Falando um pouco mais do assunto e apresentando algumas sugestões:

#A fórmula que calcula porcentagem

A * 100 / T

A -> Valor do qual quer obter a porcentagem em relação a T.

Exemplo:

30 * 100 / 118.7 resulta em 25.273799494524

#Um exemplo genérico em PHP:

/*
O array original da pergunta não possui essa estrutura, mas fiz assim apenas para simplificar. é algo irrelevante.
*/
$arr = array(
    3000,
    1000,
    7870
);

// Transforma em centavos
$arr = array_map(function($v){return $v / 100;}, $arr);

$total = array_sum($arr);

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    echo $v * 100 / $total.'<br>'.PHP_EOL;
}

#Alternativa otimizada

Note que a conversão de centavos é dividir por 100. Mas a expressão matemática fará a multiplicação por 100. Portanto, é um processo redundante, dentro do seu caso específico. Poderia simplificar sem converter os centavos.

Uma nova versão otimizada:

$arr = array(
    3000,
    1000,
    7870
);

//Pula essa bagaceira
//$arr = array_map(function($v){return $v / 100;}, $arr);

//Converte apenas o total
$total = array_sum($arr) / 100;

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    // Aqui não foi necessário a multiplicação pois seria redundante
    echo $v / $total.'<br>'.PHP_EOL;
}

#Observação sobre o round()

No código da pergunta está aplicando arredondamento com a função round(). Avalie bem se realmente quer aplicar o arredondamento, pois o resultado será 25, 8 e 66, cuja soma perfaz 99%.

Está comendo 1%. Mantenha a integridade fracionária. A menos que o modelo de negócios queira mesmo "comer" essas pequenas frações. Nesse ponto não posso opinar.

O problema está na fórmula de porcentagem.

($recipientAmount * $totalAmount) / 100

Deveria ser

($recipientAmount * 100) / $totalAmount

Isso é o suficiente para resolver o problema. Caso queira saber mais, continue a leitura.





Falando um pouco mais do assunto e apresentando algumas sugestões e observações:

#A fórmula que calcula porcentagem

A * 100 / T

A -> Valor do qual quer obter a porcentagem em relação a T.

Exemplo:

30 * 100 / 118.7 resulta em 25.273799494524

#Um exemplo genérico em PHP:

/*
O array original da pergunta não possui essa estrutura, mas fiz assim apenas para simplificar. é algo irrelevante.
*/
$arr = array(
    3000,
    1000,
    7870
);

// Transforma em centavos
$arr = array_map(function($v){return $v / 100;}, $arr);

$total = array_sum($arr);

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    echo $v * 100 / $total.'<br>'.PHP_EOL;
}

#Alternativa otimizada

Note que a conversão de centavos é dividir por 100. Mas a expressão matemática fará a multiplicação por 100. Portanto, é um processo redundante, dentro do seu caso específico. Poderia simplificar sem converter os centavos.

Uma nova versão otimizada:

$arr = array(
    3000,
    1000,
    7870
);

//Pula essa bagaceira
//$arr = array_map(function($v){return $v / 100;}, $arr);

//Converte apenas o total
$total = array_sum($arr) / 100;

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    // Aqui não foi necessário a multiplicação pois seria redundante
    echo $v / $total.'<br>'.PHP_EOL;
}

#Observação sobre o round()

No código da pergunta está aplicando arredondamento com a função round(). Avalie bem se realmente quer aplicar o arredondamento, pois o resultado será 25, 8 e 66, cuja soma perfaz 99%.

Está comendo 1%. Mantenha a integridade fracionária. A menos que o modelo de negócios queira mesmo "comer" essas pequenas frações. Nesse ponto não posso opinar.

Fonte Link
Daniel Omine
  • 20,1mil
  • 1
  • 35
  • 67

O problema está na fórmula de porcentagem.

($recipientAmount * $totalAmount) / 100

Deveria ser

($recipientAmount * 100) / $totalAmount

Falando um pouco mais do assunto e apresentando algumas sugestões:

#A fórmula que calcula porcentagem

A * 100 / T

A -> Valor do qual quer obter a porcentagem em relação a T.

Exemplo:

30 * 100 / 118.7 resulta em 25.273799494524

#Um exemplo genérico em PHP:

/*
O array original da pergunta não possui essa estrutura, mas fiz assim apenas para simplificar. é algo irrelevante.
*/
$arr = array(
    3000,
    1000,
    7870
);

// Transforma em centavos
$arr = array_map(function($v){return $v / 100;}, $arr);

$total = array_sum($arr);

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    echo $v * 100 / $total.'<br>'.PHP_EOL;
}

#Alternativa otimizada

Note que a conversão de centavos é dividir por 100. Mas a expressão matemática fará a multiplicação por 100. Portanto, é um processo redundante, dentro do seu caso específico. Poderia simplificar sem converter os centavos.

Uma nova versão otimizada:

$arr = array(
    3000,
    1000,
    7870
);

//Pula essa bagaceira
//$arr = array_map(function($v){return $v / 100;}, $arr);

//Converte apenas o total
$total = array_sum($arr) / 100;

echo 'total: '.$total.'<br>';

foreach ($arr as $v) {
    // Aqui não foi necessário a multiplicação pois seria redundante
    echo $v / $total.'<br>'.PHP_EOL;
}

#Observação sobre o round()

No código da pergunta está aplicando arredondamento com a função round(). Avalie bem se realmente quer aplicar o arredondamento, pois o resultado será 25, 8 e 66, cuja soma perfaz 99%.

Está comendo 1%. Mantenha a integridade fracionária. A menos que o modelo de negócios queira mesmo "comer" essas pequenas frações. Nesse ponto não posso opinar.