mcorten87\rabbitmq_api\jobs\JobBindingToQueueCreate

Complexity

Total Complexity

6

Size/Duplication

Total Lines	83
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getVirtualHost()	4	4	1
A	getQueueName()	4	4	1
A	getExchangeName()	4	4	1
A	getDestinationType()	4	4	1
A	getRoutingKey()	4	4	1
A	__construct()	9	9	1

<?php

namespace mcorten87\rabbitmq_api\jobs;

use mcorten87\rabbitmq_api\objects\DestinationType;
use mcorten87\rabbitmq_api\objects\Exchange;
use mcorten87\rabbitmq_api\objects\ExchangeName;
use mcorten87\rabbitmq_api\objects\QueueName;
use mcorten87\rabbitmq_api\objects\RoutingKey;
use mcorten87\rabbitmq_api\objects\VirtualHost;

class JobBindingToQueueCreate extends JobBase

This class seems to be duplicated in your project.

{
    /**
     * @var VirtualHost
     */
    private $virtualHost;


    /**
     * @var QueueName
     */
    private $queueName;

    /**
     * @var ExchangeName
     */
    private $exchangeName;

    /**
     * @var DestinationType
     */
    private $destinationType;

    /**
     * @var RoutingKey
     */
    private $routingKey;

    /**
     * @return VirtualHost
     */
    public function getVirtualHost() : VirtualHost
    {
        return $this->virtualHost;
    }

    /**
     * @return QueueName
     */
    public function getQueueName(): QueueName
    {
        return $this->queueName;
    }

    /**
     * @return ExchangeName
     */
    public function getExchangeName()
    {
        return $this->exchangeName;
    }

    /**
     * @return DestinationType
     */
    public function getDestinationType(): string
    {
        return $this->destinationType;
    }

    /**
     * @return RoutingKey
     */
    public function getRoutingKey()
    {
        return $this->routingKey;
    }

    /**
     * JobExchangeCreate constructor.
     * @param VirtualHost $virtualHost
     * @param ExchangeName $exchangeName
     */
    public function __construct(VirtualHost $virtualHost, QueueName $queueName, ExchangeName $exchangeName, RoutingKey $routingKey = null)

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

    {
        $this->virtualHost = $virtualHost;
        $this->queueName = $queueName;
        $this->exchangeName = $exchangeName;

        $this->destinationType = new DestinationType(DestinationType::QUEUE);
        $this->routingKey = new RoutingKey('');
    }
}