for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace App\Middlewares;
class AuthMiddleware extends BaseMiddleware
{
/**
* @throws \Exception
*/
public function handle()
if ($logged = false) {
$logged
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
$myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; }
Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.
$myVar
$higher
throw new \Exception('Must be logged in');
}
return $this->handleNext();
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
Both the
$myVar
assignment in line 1 and the$higher
assignment in line 2 are dead. The first because$myVar
is never used and the second because$higher
is always overwritten for every possible time line.