SeleniumSetup\Locker\ServerItemFactory

Complexity

Total Complexity

9

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
B	createFromObj()	0	18	5
A	createFromProperties()	0	18	4

<?php
namespace SeleniumSetup\Locker;

class ServerItemFactory
{
    public static function createFromObj(\stdClass $server)
    {
        if (
            !isset($server->name) || 
            !isset($server->pid) ||
            !isset($server->port) ||
            !isset($server->configFilePath)
        ) {
            throw new \Exception('Server instance cannot be created. Missing keys.');
        }
        
        return (new ServerItem())
            ->setName($server->name)
            ->setPid($server->pid)
            ->setPort($server->port)
            ->setConfigFilePath($server->configFilePath)
            ->setDateStarted(date(ServerItem::DATE_FORMAT));
    }

    /**
     * The PID is no longer required to be greater than zero. (when PID = zero, this usually means it's stopped).
     *
     * @param string $name
     * @param int $pid
     * @param int $port
     * @param string $configFilePath
     *
     * @return ServerItem
     * @throws \Exception
     */
    public static function createFromProperties($name, $pid, $port, $configFilePath)
    {
        if (
            empty($name) ||
            // empty($pid) ||

72% of this comment could be valid code. Did you maybe forget this after debugging?

            empty($port) ||
            empty($configFilePath)
        ) {
            throw new \Exception('Server instance cannot be created. Missing values.');
        }

        return (new ServerItem())
            ->setName($name)
            ->setPid($pid)
            ->setPort($port)
            ->setConfigFilePath($configFilePath)
            ->setDateStarted(date(ServerItem::DATE_FORMAT));
    }
}