bluetree-service / symfony-console-style

src/Animation.php

<?php

namespace BlueConsole;

class Animation
{
    /**
     * @var \DateInterval
     */
    protected $interval;

    public function __construct(int $steps = 6, string $cursor = '*')
    {
        $counter = 0;
        $forward = true;
        $before = 0;
        $after = $steps - \strlen($cursor);

        echo "\n";

        while (true) {
            echo '[';

            for ($i = 1; $i <= $before; $i++) {
                echo ' ';
            }

            echo $cursor;

            for ($i = 0; $i <= $after; $i++) {
                echo ' ';
            }

            echo ']';
            echo "\r";

            $counter++;

            if ($forward) {
                $before++;
                $after--;
            } else {
                $before--;
                $after++;
            }

            if ($counter === ($steps - (\strlen($cursor) -1))) {
                $counter = 0;
                $forward = !$forward;

The condition $forward is always true.

            }

            sleep(1);
        }
    }

    public function setWaitTime(\DateInterval $dateInterval)
    {
        //        $interval = new \DateInterval('PT10S');
        $dateInterval->invert = true;

The property $invert was declared of type integer, but true is of type true. Maybe add a type cast?

        //dump($interval);

        $d1 = new \DateTime();
        while (true) {
            $d2 = new \DateTime();
            $d2->add($dateInterval);

            $iv = $d2->diff($d1);
            dump($iv->s);
            if ($iv->s === 0) {
                echo "end\n";
                exit(0);

Using exit here is not recommended.

            }
            sleep(1);
        }
    }

    public function setMessage(string $message)

The parameter $message is not used and could be removed.

    {
        
    }

    public function run($process)

The parameter $process is not used and could be removed.

    {
        //display message with animation
        //execute external process
        //wait and display animation
        //process ended -> display message and go fodder
    }

    protected function displayAnimation()
    {
        $counter = 0;
        $forward = true;
        $before = 0;
        $after = $steps - \strlen($cursor);

The variable $steps seems to be never defined.

The variable $cursor seems to be never defined.

        while (true) {
            echo '[';

            for ($i = 1; $i <= $before; $i++) {
                echo ' ';
            }

            echo $cursor;

            for ($i = 0; $i <= $after; $i++) {
                echo ' ';
            }

            echo ']';
            echo "\r";

            $counter++;

            if ($forward) {
                $before++;
                $after--;
            } else {
                $before--;
                $after++;
            }

            if ($counter === ($steps - (\strlen($cursor) -1))) {
                $counter = 0;
                $forward = !$forward;

The condition $forward is always true.

            }

            sleep(1);
        }
    }

    protected function executeProcess()
    {
        
    }
}