BitPrepared\Bundle\FormazioneBundle\Domain\Entity\SessioneCampo

Complexity

Total Complexity

5

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getId()	0	4	1
A	getSessione()	0	4	1
A	getTipologiaCampo()	0	4	1
A	getDocumentiCorrelati()	0	4	1

<?php

namespace BitPrepared\Bundle\FormazioneBundle\Domain\Entity;

use BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\Sessione;
use BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\TipologiaCampo;

/**
 * No comment found on ddd model
 */
final class SessioneCampo
{
    use SessioneCampoTrait;

    /**
     * identificativo univoco della sessione
     *
     * @var int
     */
    private $id;

    /**
     * Sessione generica utilizzabile in un campo
     *
     * @var BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\Sessione
     */
    private $sessione;

    /**
     * Tipologia di campo CFM/CFT/CCG/CAM
     *
     * @var BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\TipologiaCampo
     */
    private $tipologiaCampo;

    /**
     * Documenti correlati alla sessione al campo
     *
     * @var array
     */
    private $documentiCorrelati;

    /**
     * costruttore
     */
    final public function __construct($id, Sessione $sessione = null, TipologiaCampo $tipologiaCampo = null, array $documentiCorrelati = null)

Unnecessary FINAL modifier in FINAL class

    {
        $this->id = $id;
        $this->sessione = $sessione;
        $this->tipologiaCampo = $tipologiaCampo;

It seems like $tipologiaCampo can also be of type object<BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\TipologiaCampo>. However, the property $tipologiaCampo is declared as type object<BitPrepared\Bundle\FormazioneBundle\Domain\Entity\BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\TipologiaCampo>. Maybe add an additional type check?

        $this->documentiCorrelati = $documentiCorrelati;

It seems like $documentiCorrelati can be null. However, the property $documentiCorrelati is declared as array. Maybe change the type of the property to array|null or add a type check?

    }

    /**
     * @return int
     */
    final public function getId()

Unnecessary FINAL modifier in FINAL class

    {
        return $this->id;
    }

    /**
     * @return BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\Sessione
     */
    final public function getSessione()

Unnecessary FINAL modifier in FINAL class

    {
        return $this->sessione;
    }

    /**
     * @return BitPrepared\Bundle\FormazioneBundle\Domain\ValueObject\TipologiaCampo
     */
    final public function getTipologiaCampo()

Unnecessary FINAL modifier in FINAL class

    {
        return $this->tipologiaCampo;
    }

    /**
     * @return array
     */
    final public function getDocumentiCorrelati()

Unnecessary FINAL modifier in FINAL class

    {
        return $this->documentiCorrelati;
    }
}