Forcing a return on End() seems undesirable. Make Summary() call required to obtain summary.

All times should be stored in a long decimal and not rounded until summary (and even then the rounded number shouldn't override the original number stored in the class/object).

It's not compatible with PHP8 and will prompt an error message Deprecated: Implicit conversion from float 1681103373.812089 to int loses precision in /var/www/6005/others/missions/vendor/jsanc623/phpbenchtime/src/Timer.php on line 90.

Pls, fix composer.json

 "psr-0": {"": "src/"}

to

 "psr-0": {"PHPBenchTime\\": "src/"}

Docs: https://getcomposer.org/doc/04-schema.md#psr-0

You don't allow multiple pauses (it will give a faulty output). In your unpause methoed:

$this->totalPauseTime = $this->getCurrentTime() - $this->pauseTime;

should probably be

$this->totalPauseTime += $this->getCurrentTime() - $this->pauseTime;

You decieded to not end a lap if you pause. this is correct. But, it gives some peculiar output:

$t->lap('test');
sleep(1);
$t->pause();
sleep(1);
$t->unpause();
sleep(1);
$t->endlap();
The lap has no knowledge of a pause.

In the endlap, you calculate the current position of the array pointer. There is afunction for that: key So:

$lapCount = key($this->laps);

another option here would be to use a pointer reference instead of accessing the laps array multiple time. But this should only be done if it reads easier:

$lastLap = &current($this->laps);
$lastLap['end'] = $this->getCurrentTime();
$lastLap['total'] = $lastLap['end'] - $lastLap['start'];

private function setRunningPaused( $running, $paused ) {
$this->isRunning = !!$running ;
$this->isPaused = !!$paused ;
}

-or- use constants and a state:

const RUNNING = 1;
const PAUSED  = 0;
const STOPPED = -1;
protected $state

$this->state = Timer::RUNNING

You are creating a $lapTime variable, but you aren't really using it. Only to pass it to the array. This will do the job aswell:

$this->laps[] = array(
"start" => $this->getCurrentTime()
);

$t = new Timer();
$t->start();
//I need to call lap now because I can't pass in a name in the start method
$t->lap('database');
//do some DB stuff
$t->lap();
//do some random stuff
$t->lap('database');
//do some more db stuff
$t->end();

We now have lap called start that added some execution time, but nothing happened. It is only there because we wanted to pass in a different name then start.

I get the following error when trying to use this class as per the instructions:

  [ErrorException]                                                                                                     
  Missing argument 1 for PHPBenchTime\Timer::Start(), called in /opt/bitnami/apache2/htdocs/vendor/jsanc623/phpbencht  
  ime/src/PHPBenchTime.php on line 71 and defined  

Need unit tests on all methods

You return a totaltime in your summary. It is even a property of the class. but it is only used inside summary() method. So simply remove itfrom the class and make it a variable of the method.

Ideally end() should end the timer and return a summary() method.

And summary() should return the following:

Array
(
    [running]   => false
    [start]     => 1353195346.6762
    [end]       => 1353195349.6765
    [total]     => 3.0003
    [laps] =>
        [0] => 
            [name]  => Lap 1
            [start] => 1353195346.6762
            [end]   =>1353195347.6763
            [total] => 1.0001
        [1] => 
            [name]  => Some query, or something
            [start] => 1353195347.6763
            [end]   =>1353195349.6764
            [total] => 1.0001
        [2] => 1353195349.6764
            [name]  => Last lap!
            [start] => 1353195349.6764
            [end]   =>1353195349.6765
            [total] => 1.0001
)

$timer->summary(); could be called before the time is done as well.

Array
(
    [running]   => true
    [start]     => 1353195346.6762
    [laps] =>
        [0] => 
            [name]  => Lap 1
            [start] => 1353195346.6762
            [end]   =>1353195347.6763
            [total] => 1.0001
        [1] => 
            [name]  => Some query, or something
            [start] => 1353195347.6763
)