Samshal\Scripd\JsynExtractor

Complexity

Total Complexity

11

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	3
Bugs	1	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setJsynFile()	0	11	2
A	setSqlSyntax()	0	10	2
A	getJsyn()	0	4	1
A	formatJsyn()	0	10	4
A	__toString()	0	4	1

<?php

/*
 * This file is part of the samshal/scripd package.
 *
 * (c) Samuel Adeshina <samueladeshina73@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Samshal\Scripd;

/**
 * Contains utility methods for parsing a jsyn file.
 *
 * @since  1.0
 *
 * @author Samuel Adeshina <samueladeshina73@gmail.com>
 */
final class JsynExtractor
{
    /**
     * @var string
     *
     * Content of the JSYN File
     */
    private $jsyn;

    /**
     * @var string
     *
     * SQL Syntax to use for script generation
     */
    private $sqlSyntax;

    /**
     * @param $jsynFile string | PathUtil
     * @param $sqlSyntax string
     *
     * @return null

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($jsynFile, $sqlSyntax)
    {
        self::setJsynFile($jsynFile);
        self::setSqlSyntax($sqlSyntax);
    }

    /**
     * @param $jsynFile string | PathUtil
     *
     * Setter function for the jsonFile global property
     *
     * @return null
     */
    public function setJsynFile($jsynFile)
    {
        $this->jsyn = json_decode(file_get_contents($jsynFile));

        if (isset($this->sqlSyntax)) {
            $sqlSyntax = $this->sqlSyntax;
            $this->jsyn = $this->jsyn->$sqlSyntax;
        }

        return;
    }

     /**
     * @param $sqlSyntax string
     *
     * Setter function for the sqlSyntax global property
     *
     * @return null
     */
    public function setSqlSyntax($sqlSyntax)
    {
        $this->sqlSyntax = $sqlSyntax;

        if (isset($this->jsyn->$sqlSyntax)) {
            $this->jsyn = $this->jsyn->$sqlSyntax;
        }

        return;
    }

     /**
     * @return Array
     */
    public function getJsyn()
    {
        return $this->jsyn;
    }

     /**
     * Performs extraction of the appropriate sql syntax
     * fromthe supplied jsyn file.
     *
     * @return null
     */
    public function formatJsyn()
    {
        for ($i = 0; $i < count($this->jsyn); ++$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.

            if (strpos($this->jsyn[$i], '[') === 0 || strpos($this->jsyn[$i], '{') === (int) 0) {
                $this->jsyn[$i] = strtolower($this->jsyn[$i]);
            } else {
                $this->jsyn[$i] = strtoupper($this->jsyn[$i]);
            }
        }
    }

    /**
     * Returns the extracted jsyn as a string
     *
     * @return null
     */
    public function __toString()
    {
        return implode(' ', $this->getJsyn());
    }
}