Teebot\Entity\Command

Complexity

Total Complexity

12

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	2
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	20	3
A	getName()	0	4	1
A	getArgs()	0	4	1
B	getCommandFromText()	0	18	5
A	getArgsFromText()	0	12	2

<?php

namespace Teebot\Entity;

class Command extends AbstractEntity
{
    const ENTITY_TYPE             = 'Command';

    const PREFIX          = '/';

    const ARGS_SEPARATOR  = ' ';

    const PARTS_DELIMITER = '_';

    const PATTERN = '/^\/[a-zA-Z_]+.*$/';

    protected $text;

    protected $name;

    protected $args;

    public function __construct(array $data)
    {
        $command = null;

$command is not used, you could remove the assignment.

        $args    = null;

        if (isset($data['text'])) {
            $command = $this->getCommandFromText($data['text']);

            if ($command) {

The expression $command of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                $args = $this->getArgsFromText($command, $data['text']);
            }

            $data = [
                'name' => $command,
                'args' => $args
            ];
        }

        parent::__construct($data);
    }

    /**
     * @return mixed
     */
    public function getName()
    {
        return $this->name;
    }

    /**
     * @return mixed
     */
    public function getArgs()
    {
        return $this->args;
    }

    protected function getCommandFromText($text)
    {
        $command = null;

        if (!is_string($text) || !strlen($text) || $text[0] !== static::PREFIX) {
            return $command;
        }

        if (strpos($text, static::ARGS_SEPARATOR) !== false) {
            $parts = explode(static::ARGS_SEPARATOR, $text);

            $text = trim($parts[0]);
        }

        $command = trim(substr($text, 1, strlen($text) - 1));

        return $command;
    }

    protected function getArgsFromText($command, $text)
    {
        $length = strlen(static::PREFIX . $command);

        $argString = trim(substr($text, $length));

        if (strlen($argString)) {
            return $argString;
        }

        return null;
    }
}