for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
declare(strict_types=1);
/**
* @author Ne-Lexa
* @license MIT
*
* @see https://github.com/Ne-Lexa/google-play-scraper
*/
namespace Nelexa\GPlay\Scraper;
use Nelexa\GPlay\GPlayApps;
use Nelexa\GPlay\Model\AppId;
use Nelexa\GPlay\Util\ScraperUtil;
use Nelexa\HttpClient\ResponseHandlerInterface;
use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;
* @internal
class FindSimilarAppsUrlScraper implements ResponseHandlerInterface
{
/** @var AppId */
private $appId;
* SimilarScraper constructor.
* @param AppId $appId
public function __construct(AppId $appId)
$this->appId = $appId;
}
* @param RequestInterface $request
* @param ResponseInterface $response
* @return string|null
public function __invoke(RequestInterface $request, ResponseInterface $response): ?string
$scriptData = ScraperUtil::extractScriptData($response->getBody()->getContents());
foreach ($scriptData as $key => $scriptValue) {
if (isset($scriptValue[1][1][0][0][3][4][2])) {
return GPlayApps::GOOGLE_PLAY_URL . $scriptValue[1][1][0][0][3][4][2] .
'&' . GPlayApps::REQ_PARAM_LOCALE . '=' . urlencode($this->appId->getLocale()) .
'&' . GPlayApps::REQ_PARAM_COUNTRY . '=' . urlencode($this->appId->getCountry());
break;
break
The break statement is not necessary if it is preceded for example by a return statement:
return
switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. }
If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive.
case
return null;
Ne-Lexa /
google-play-scraper
The
breakstatement is not necessary if it is preceded for example by areturnstatement:If you would like to keep this construct to be consistent with other
casestatements, you can safely mark this issue as a false-positive.