Simplario\Checker\Checker\Gateway

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	25 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	curl()	0	13	2
A	testExists()	12	12	3

<?php

namespace Simplario\Checker\Checker;

use Simplario\Checker\ResultException\FailException;
use Simplario\Checker\ResultException\SuccessException;

/**
 * Class Gateway
 *
 * @package Simplario\Checker\Checker
 */
class Gateway extends AbstractChecker
{
    /**
     * @var string
     */
    protected $target = 'url';

    /**
     * @param string $target
     * @param array  $options
     *
     * @return boolean
     */
    protected function curl($target, array $options = [])

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

    {
        $success = true;
        $ch = curl_init();
        curl_setopt($ch, CURLOPT_URL, $target);
        curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
        if (!curl_exec($ch)) {
            $success = false;
        }
        curl_close($ch);

        return $success;
    }

    /**
     * @param string  $target
     * @param boolean $expectExists
     * @param array   $task
     *
     * @throws FailException
     * @throws SuccessException
     */
    protected function testExists($target, $expectExists, array $task)

This method seems to be duplicated in your project.

    {
        $success = $this->curl($target, []);

        if ($success === $expectExists) {
            throw new SuccessException('Ok', $task);
        }

        $msg = $expectExists ? "Target '{$target}' is not available'" : "Target '{$target}' is available";

        throw new FailException($msg, $task);
    }
}