ByJG\AnyDataset\Database\DBMySQLFunctions

Complexity

Total Complexity

32

Size/Duplication

Total Lines	181
Duplicated Lines	20.99 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	6
Bugs	1	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	limit()	8	8	2
A	top()	0	4	1
A	hasTop()	0	4	1
A	hasLimit()	0	4	1
C	sqlDate()	8	68	20
A	toDate()	0	4	1
A	fromDate()	0	4	1
A	executeAndGetInsertedId()	11	11	2
A	concat()	11	11	3

<?php

namespace ByJG\AnyDataset\Database;

use ByJG\AnyDataset\Repository\DBDataset;

class DBMySQLFunctions extends DBBaseFunctions
{

    private $sysTimeStamp = 'NOW()';

    function concat($s1, $s2 = null)

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        $sql = "concat(";
        for ($i = 0, $numArgs = func_num_args(); $i < $numArgs; $i++) {
            $var = func_get_arg($i);
            $sql .= ($i == 0 ? "" : ",") . $var;
        }
        $sql .= ")";

        return $sql;
    }

    /**
     * Given a SQL returns it with the proper LIMIT or equivalent method included
     * @param string $sql
     * @param int $start
     * @param int $qty
     * @return string
     */
    function limit($sql, $start, $qty)

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if (strpos($sql, ' LIMIT ') === false) {
            return $sql . " LIMIT $start, $qty ";
        } else {
            return $sql;
        }
    }

    /**
     * Given a SQL returns it with the proper TOP or equivalent method included
     * @param string $sql
     * @param int $qty
     * @return string
     */
    function top($sql, $qty)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return $this->limit($sql, 0, $qty);
    }

    /**
     * Return if the database provider have a top or similar function
     * @return bool
     */
    function hasTop()

It is generally recommended to explicitly declare the visibility for methods.

    {
        return true;
    }

    /**
     * Return if the database provider have a limit function
     * @return bool
     */
    function hasLimit()

It is generally recommended to explicitly declare the visibility for methods.

    {
        return true;
    }

    /**
     * Format date column in sql string given an input format that understands Y M D
     * @param string $fmt
     * @param string|bool $col
     * @return string
     * @example $db->getDbFunctions()->SQLDate("d/m/Y H:i", "dtcriacao")
     */
    function sqlDate($fmt, $col = false)

It is generally recommended to explicitly declare the visibility for methods.

    {
        if (!$col) $col = $this->sysTimeStamp;
        $s = 'DATE_FORMAT(' . $col . ",'";
        $concat = false;
        $len = strlen($fmt);
        for ($i = 0; $i < $len; $i++) {
            $ch = $fmt[$i];
            switch ($ch) {
                case 'Y':
                case 'y':
                    $s .= '%Y';
                    break;
                case 'Q':
                case 'q':
                    $s .= "'),Quarter($col)";

                    if ($len > $i + 1) $s .= ",DATE_FORMAT($col,'";
                    else $s .= ",('";
                    $concat = true;
                    break;
                case 'M':
                    $s .= '%b';
                    break;

                case 'm':
                    $s .= '%m';
                    break;
                case 'D':
                case 'd':
                    $s .= '%d';
                    break;

                case 'H':
                    $s .= '%H';
                    break;

                case 'h':
                    $s .= '%I';
                    break;

                case 'i':
                    $s .= '%i';
                    break;

                case 's':
                    $s .= '%s';
                    break;

                case 'a':
                case 'A':
                    $s .= '%p';
                    break;

                default:

This code seems to be duplicated across your project.

The default body in a switch statement must start on the line following the statement.

                    if ($ch == '\\') {
                        $i++;
                        $ch = substr($fmt, $i, 1);
                    }
                    $s .= $ch;
                    break;
            }
        }
        $s.="')";
        if ($concat) $s = "CONCAT($s)";
        return $s;
    }

    /**
     * Format a string date to a string database readable format.
     *
     * @param string $date
     * @param string $dateFormat
     * @return string
     */
    function toDate($date, $dateFormat)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return parent::toDate($date, $dateFormat);
    }

    /**
     * Format a string database readable format to a string date in a free format.
     *
     * @param string $date
     * @param string $dateFormat
     * @return string
     */
    function fromDate($date, $dateFormat)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return parent::fromDate($date, $dateFormat);
    }

    /**
     *
     * @param DBDataset $dbdataset
     * @param string $sql
     * @param array $param
     * @return int
     */
    function executeAndGetInsertedId($dbdataset, $sql, $param)

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        $id = parent::executeAndGetInsertedId($dbdataset, $sql, $param);
        $it = $dbdataset->getIterator("select LAST_INSERT_ID() id");
        if ($it->hasNext()) {
            $sr = $it->moveNext();
            $id = $sr->getField("id");
        }

        return $id;
    }
}