Waca\Providers\CachedRDnsLookupProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	23.08 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	getReverseDNS()	9	29	3

<?php
namespace Waca\Providers;

use Waca\DataObjects\RDnsCache;
use Waca\PdoDatabase;
use Waca\Providers\Interfaces\IRDnsProvider;

/**
 * Cached rDNS Lookup Provider
 *
 * Provides a service to look up the reverse DNS of an IP address, and caches
 * the result in the database.
 */
class CachedRDnsLookupProvider implements IRDnsProvider
{
	private $database;

	public function __construct(PdoDatabase $database)
	{
		$this->database = $database;
	}

	public function getReverseDNS($address)
	{
		$address = trim($address);

		// lets look in our cache database first.
		$rDns = RDnsCache::getByAddress($address, $this->database);

		if ($rDns instanceof RDnsCache) {
			// touch cache timer
			$rDns->save();

			return $rDns->getData();
		}

		// OK, it's not there, let's do an rDNS lookup.
		$result = @ gethostbyaddr($address);

		if ($result !== false) {

This code seems to be duplicated across your project.

			$rDns = new RDnsCache();
			$rDns->setDatabase($this->database);
			$rDns->setAddress($address);
			$rDns->setData($result);
			$rDns->save();

			return $result;
		}

		return null;
	}
}