Pixelpeter\Genderize\GenderizeClient

Complexity

Total Complexity

8

Size/Duplication

Total Lines	127
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	country()	0	5	1
A	buildQuery()	0	10	1
A	names()	0	5	1
A	get()	0	7	1
A	__construct()	0	4	1
A	name()	0	9	2
A	lang()	0	5	1

<?php

namespace Pixelpeter\Genderize;

use Pixelpeter\Genderize\Models\GenderizeResponse;
use Unirest\Request;

class GenderizeClient
{
    /**
     * @var array $names Holds one or more names to check
     */
    protected $names;

    /**
     * @var string $lang The language used for the check
     */
    protected $lang;

    /**
     * @var string $country The country used for the check
     */
    protected $country;

    /**
     * @var string $apikey The api key used for the request
     */
    protected $apikey;

    /**
     * @var \Unirest\Request
     */
    protected $request;

    /**
     * @const API_URL The URL of the api endpoint
     */
    const API_URL = 'https://api.genderize.io';


    /**
     * Create new instance of Pixelpeter\Genderize\GenderizeClient
     */
    public function __construct(Request $request)
    {
        $this->apikey = config('genderize.apikey');
        $this->request = $request;
    }

    /**
     * Fluent setter for names given as string
     *
     * @param string $name
     * @return $this|GenderizeClient
     */
    public function name($name = '')
    {
        if (is_array($name)) {

The condition is_array($name) is always false.

            return $this->names($name);
        }

        $this->names = [$name];

        return $this;
    }

    /**
     * Fluent setter for names given as array
     *
     * @param array $names
     * @return $this
     */
    public function names(array $names)
    {
        $this->names = $names;

        return $this;
    }

    /**
     * Fluent setter for language
     *
     * @param null $lang

Are you sure the doc-type for parameter $lang is correct as it would always require null to be passed?

     * @return $this
     */
    public function lang($lang = null)
    {
        $this->lang = $lang;

        return $this;
    }

    /**
     * Fluent setter for country
     *
     * @param null $country

Are you sure the doc-type for parameter $country is correct as it would always require null to be passed?

     * @return $this
     */
    public function country($country = null)
    {
        $this->country = $country;

        return $this;
    }

    /**
     * Send the request
     *
     * @return GenderizeResponse
     */
    public function get()
    {
        $headers = array('Accept' => 'application/json');

        $response = $this->request->get(self::API_URL, $headers, $this->buildQuery());

        return new GenderizeResponse($response);
    }

    /**
     * Build the query
     *
     * @return array
     */
    protected function buildQuery()
    {
        $query = [
            'name' => $this->names,
            'country_id' => $this->country,
            'language_id' => $this->lang,
            'apikey' => $this->apikey
        ];

        return array_filter($query);
    }
}