Proxy\db

Complexity

Total Complexity

6

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	random_proxy()	0	2	1
A	load_proxies_db()	0	9	1
A	load_proxies()	0	3	1
A	__construct()	0	7	2
A	inactive_proxies()	0	3	1

<?php

namespace Proxy;

$GLOBALS['proxy_loaded'] = null;

class db
{
  /**
   * PDO.
   *
   * @var \DB\pdo
   */
  private $pdo = null;

  public function __construct(\DB\pdo $pdo)
  {
    global $GLOBALS;
    if (!$GLOBALS['proxy_loaded']) {
      $this->load_proxies_db();
    }
    $this->pdo = $pdo;
  }

  public function random_proxy(int $n = 1)

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

  {
  }

  public function inactive_proxies()
  {
    return $this->pdo->query("SELECT * FROM `proxies` WHERE `status` = 'inactive'")->row_array();
  }

  public function load_proxies_db()
  {
    $proxies = $this->pdo
      ->select('proxies')
      ->where(['country' => null, 'status' => 'active'])
      ->row_array();
    \Filemanager\file::file(__DIR__ . '/proxies/db.json', $proxies, true);

It seems like $proxies can also be of type array; however, parameter $create of Filemanager\file::file() does only seem to accept boolean, maybe add an additional type check?

    return $proxies;
  }

  public function load_proxies()
  {
    return json_decode(\Filemanager\file::get(__DIR__ . '/proxies/db.json'));

It seems like Filemanager\file::get(__DIR__ . '/proxies/db.json') can also be of type null; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

  }
}