M1\Env\Traits\ValueTypeCheckable

Complexity

Total Complexity

8

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	isString()	0	4	2
A	isBool()	0	4	1
A	isNumber()	0	4	1
A	isNull()	0	4	1
A	isVariableClone()	0	4	3

<?php

/**
 * This file is part of the m1\env library
 *
 * (c) m1 <hello@milescroxford.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @package     m1/env
 * @version     0.2.0
 * @author      Miles Croxford <hello@milescroxford.com>
 * @copyright   Copyright (c) Miles Croxford <hello@milescroxford.com>
 * @license     http://github.com/m1/env/blob/master/LICENSE.md
 * @link        http://github.com/m1/env/blob/master/README.md Documentation
 */

namespace M1\Env\Traits;

/**
 * The trait for checking types
 *
 * @since 0.2.0
 */
trait ValueTypeCheckable
{

    /**
     * Returns if value is a string
     *
     * @param string $value The value to test
     *
     * @return bool Is a value a string
     */
    private function isString($value)

This method is not used, and could be removed.

    {
        return $this->startsWith('\'', $value) || $this->startsWith('"', $value);

It seems like startsWith() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Returns if value is a bool
     *
     * @param string $value The value to test
     *
     * @return bool Is a value a bool
     */
    private function isBool($value)

This method is not used, and could be removed.

    {
        return in_array(strtolower($value), self::$bool_variants);
    }

    /**
     * Returns if value is number
     *
     * @param string $value The value to test
     *
     * @return bool Is a value a number
     */
    private function isNumber($value)

This method is not used, and could be removed.

    {
        return is_numeric($value);
    }

    /**
     * Returns if value is null
     *
     * @param string $value The value to test
     *
     * @return bool Is a value null
     */
    private function isNull($value)

This method is not used, and could be removed.

    {
        return $value === 'null';
    }

    /**
     * Returns if variable value is a clone, e.g. BOOL = $(BOOL_1)
     *
     * @param string $value         The value to test
     * @param array  $matches       The matches of the variables
     * @param bool   $quoted_string If the value is in a quoted string
     *
     * @return bool Is a value null
     */
    private function isVariableClone($value, $matches, $quoted_string)
    {
        return count($matches[0] === 1) && $value == $matches[0][0] && !$quoted_string;
    }
}