drupol\Yaroc\Examples\Pi

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

4

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	4	1
A	run()	0	21	3

<?php

namespace drupol\Yaroc\Examples;

use drupol\Yaroc\Plugin\Provider;

/**
 * Class Pi.
 */
class Pi extends BaseExample
{
    /**
     * @var float
     */
    protected $estimation;

    /**
     * @param int $throws
     *
     * @throws \Http\Client\Exception
     *
     * @return $this
     *
     * @SuppressWarnings(PHPMD.StaticAccess)
     */
    public function run($throws = 1000)
    {
        $provider = Provider::withResource('generateDecimalFractions')
            ->withParameters(['n' => $throws * 2, 'decimalPlaces' => 6]);

        $numbers = $this->getRandomOrgAPI()->getData($provider);

        $inside = 0;
        for ($i = 0; $i < $throws; $i++) {
            $x = $numbers[$i];

Avoid variables with short names like $x. Configured minimum length is 3.

            $y = $numbers[$i+1];

Avoid variables with short names like $y. Configured minimum length is 3.

            if (sqrt($x*$x + $y*$y) <= 1) {
                $inside++;
            }
        }

        $this->estimation = 4 * $inside/$throws;

The property $estimation was declared of type double, but 4 * $inside / $throws is of type integer. Maybe add a type cast?

        return $this;
    }

    /**
     * @return float
     */
    public function get()
    {
        return $this->estimation;
    }
}