DockerCompose\ComposeFileCollection

Complexity

Total Complexity

9

Size/Duplication

Total Lines	85
Duplicated Lines	11.76 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	10	25	5
A	getAll()	0	4	1
A	add()	0	5	1
A	setProjectName()	0	5	1
A	getProjectName()	0	4	1

<?php

namespace DockerCompose;

use Exception;

/**
 * DockerCompose\ComposeFileCollection
 */
class ComposeFileCollection
{
    /**
     * @var ComposeFiles
     */
    private $composeFiles = [];

    /**
     * @var projectName
     */
    private $projectName;

    /**
     * @param array of string or ComposeFile
     *
     * @throws \Exception When a composeFile definition is invalid
     */
    public function __construct()
    {
        $args = func_get_arg(0);

        if (!is_array($args)) {
            throw new \Exception('Invalid parameter "(' . gettype($args) . ')');
        }

        foreach ($args as $composeFile) {
            if ($composeFile instanceof ComposeFile) {
                $this->add($composeFile);
            } elseif (is_string($composeFile)) {
                $this->add(new ComposeFile($composeFile));
            } else {

This code seems to be duplicated across your project.

                throw new \Exception(
                    'Invalid composeFile definition "(' . gettype(
                        $composeFile
                    ) . ') ' . var_export(
                        $composeFile,
                        true
                    ) . '"'
                );
            }
        }
    }

    /**
     * Return all composeFiles
     *
     * @return array

Should the return type not be ComposeFiles?

     */
    public function getAll()
    {
        return $this->composeFiles;
    }

    /**
     * Add a compose file
     */
    public function add(ComposeFile $composeFile)
    {
        $this->composeFiles[] = $composeFile;
        return $this;
    }

    /**
     * Set Project Name
     *
     * @param string $projectName
     *
     * @return ComposeFileCollection
     */
    public function setProjectName($projectName)
    {
        $this->projectName = $projectName;

It seems like $projectName of type string is incompatible with the declared type object<DockerCompose\projectName> of property $projectName.

        return $this;
    }

    /**
     * Get Project Name
     *
     * @return string

Should the return type not be projectName?

     */
    public function getProjectName()
    {
        return $this->projectName;
    }
}