IBM\Watson\ToneAnalyzer\Client

Complexity

Total Complexity

7

Size/Duplication

Total Lines	83
Duplicated Lines	10.84 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	9	1
A	tone()	0	4	1
A	toneChat()	0	4	1
A	__construct()	9	9	4

<?php

namespace IBM\Watson\ToneAnalyzer;

use Http\Client\HttpClient;
use Http\Discovery\HttpClientDiscovery;
use IBM\Watson\Common\HttpClient\Builder;
use IBM\Watson\Common\Hydrator\HydratorInterface;
use IBM\Watson\Common\Hydrator\ModelHydrator;
use IBM\Watson\Common\RequestBuilder;

final class Client
{
    /**
     * Base tone analyzer uri
     */
    const BASE_URI = 'https://gateway.watsonplatform.net/tone-analyzer';

    /**
     * @var \Http\Client\HttpClient
     */
    private $httpClient;

    /**
     * @var \IBM\Watson\Common\Hydrator\ModelHydrator
     */
    private $hydrator;

    /**
     * @var \IBM\Watson\Common\RequestBuilder
     */
    private $requestBuilder;

    /**
     * Client constructor.
     *
     * @param \Http\Client\HttpClient|null                       $httpClient
     * @param \IBM\Watson\Common\Hydrator\HydratorInterface|null $hydrator
     * @param \IBM\Watson\Common\RequestBuilder|null             $requestBuilder
     *
     * @throws \Http\Discovery\Exception\NotFoundException
     */
    public function __construct(

This method seems to be duplicated in your project.

        HttpClient $httpClient = null,
        HydratorInterface $hydrator = null,
        RequestBuilder $requestBuilder = null
    ) {
        $this->httpClient = $httpClient ?: HttpClientDiscovery::find();
        $this->hydrator = $hydrator ?: new ModelHydrator;

$hydrator ?: new \IBM\Watson\Common\Hydrator\ModelHydrator() is of type object<IBM\Watson\Common\Hydrator\HydratorInterface>, but the property $hydrator was declared to be of type object<IBM\Watson\Common\Hydrator\ModelHydrator>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->requestBuilder = $requestBuilder ?: new RequestBuilder;
    }

    /**
     * Create tone analyzer client with username and password
     *
     * @param string $username
     * @param string $password
     *
     * @return \IBM\Watson\ToneAnalyzer\Client
     *
     * @throws \Http\Discovery\Exception\NotFoundException
     * @throws \RuntimeException
     * @throws \InvalidArgumentException
     */
    public static function create($username, $password)
    {
        $httpClient = (new Builder())
            ->withEndpoint(static::BASE_URI)
            ->withCredentials($username, $password)
            ->createConfiguredClient();

        return new self($httpClient);
    }

    /**
     * Create tone api request
     *
     * @return \IBM\Watson\ToneAnalyzer\Api\Tone
     */
    public function tone()
    {
        return new Api\Tone($this->httpClient, $this->hydrator, $this->requestBuilder);
    }

    /**
     * Create tone chat api request
     *
     * @return \IBM\Watson\ToneAnalyzer\Api\ToneChat
     */
    public function toneChat()
    {
        return new Api\ToneChat($this->httpClient, $this->hydrator, $this->requestBuilder);
    }
}