Alive2212\LaravelStringHelper\StringHelper

Complexity

Total Complexity

9

Size/Duplication

Total Lines	75
Duplicated Lines	21.33 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

91.3%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	plural()	0	4	1
A	toTag()	0	8	2
A	upperFirstLetter()	0	5	1
A	lowerFirstLetter()	0	5	1
A	toCamel()	8	8	2
A	toPascal()	8	8	2

<?php
/**
 * Created by PhpStorm.
 * User: alive
 * Date: 9/28/17
 * Time: 12:35 PM
 */

namespace Alive2212\LaravelStringHelper;

class StringHelper
{
    /**
     * upper letters
     *
     * @var string
     */
    protected $upperLetters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ';


    public function plural($value)

The parameter $value is not used and could be removed.

    {
        // TODO must be implementing
    }

    /**
     * @param $string
     * @param string $separator
     * @return mixed
     */
    public function toTag($string, $separator = '_')
    {
        $string[0] = strtolower($string[0]);
        foreach (str_split($this->upperLetters) as $upperLetter) {
            $string = str_replace($upperLetter, $separator . strtolower($upperLetter), $string);
        }
        return $string;
    }

    /**
     * @param $string
     * @return mixed
     */
    public function upperFirstLetter($string)
    {
        $string[0] = strtoupper($string[0]);
        return $string;
    }

    /**
     * @param $string
     * @return mixed
     */
    public function lowerFirstLetter($string)
    {
        $string[0] = strtolower($string[0]);
        return $string;
    }

    /**
     * @param $string
     * @return mixed
     */
    public function toCamel($string)

This method seems to be duplicated in your project.

    {
        $stringArray = explode('_', $string);
        for ($i = 1; $i < count($stringArray);$i++){

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $stringArray[$i] = $this->upperFirstLetter($stringArray[$i]);
        }
        return implode('', $stringArray);
    }

    /**
     * @param $string
     * @return mixed
     */
    public function toPascal($string)

This method seems to be duplicated in your project.

    {
        $stringArray = explode('_', $string);
        for ($i = 0; $i < count($stringArray);$i++){

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $stringArray[$i] = $this->upperFirstLetter($stringArray[$i]);
        }
        return implode('', $stringArray);
    }
}