Util::backupData() - Code Metrics - Inspection of "camelCase methods and properties" - aviat4ion/Query - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 111dce...b8d476 )

by Timothy

created 2016-10-14 01:55 UTC

Util::backupData() B

↳ Parent: Util

Complexity

Conditions	5
Paths	8

Size

Total Lines	56
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	24
nc	8
nop	1
dl	0
loc	56
rs	8.7592
c	0
b	0
f	0

How to fix Long Method

<?php declare(strict_types=1);
/**
 * Query
 *
 * SQL Query Builder / Database Abstraction Layer
 *
 * PHP version 7
 *
 * @package     Query
 * @author      Timothy J. Warren <[email protected]>
 * @copyright   2012 - 2016 Timothy J. Warren
 * @license     http://www.opensource.org/licenses/mit-license.html  MIT License
 * @link        https://git.timshomepage.net/aviat4ion/Query
 */
namespace Query\Drivers\Pgsql;

use Query\Drivers\AbstractUtil;

/**
 * Posgres-specific backup, import and creation methods
 */
class Util extends AbstractUtil {

	/**
	 * Create an SQL backup file for the current database's structure
	 *
	 * @return string
	 */
	public function backupStructure()
	{
		// @TODO Implement Backup function
		return '';
	}

	/**
	 * Create an SQL backup file for the current database's data
	 *
	 * @param array $exclude
	 * @return string
	 */
	public function backupData($exclude=[])
	{
		$tables = $this->getDriver()->getTables();

		// Filter out the tables you don't want
		if( ! empty($exclude))
		{
			$tables = array_diff($tables, $exclude);
		}

		$outputSql = '';

		// Get the data for each object
		foreach($tables as $t)
		{
			$sql = 'SELECT * FROM "'.trim($t).'"';
			$res = $this->getDriver()->query($sql);

			$objRes = $res->fetchAll(\PDO::FETCH_ASSOC);

			// Don't add to the file if the table is empty
			if (count($objRes) < 1)
			{
				continue;
			}

			$res = NULL;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

			// Nab the column names by getting the keys of the first row
			$columns = @array_keys($objRes[0]);

			$insertRows = [];

			// Create the insert statements
			foreach($objRes as $row)
			{
				$row = array_values($row);

				// Quote values as needed by type
				$row = array_map([$this->getDriver(), 'quote'], $row);
				$row = array_map('trim', $row);


				$rowString = 'INSERT INTO "'.trim($t).'" ("'.implode('","', $columns).'") VALUES ('.implode(',', $row).');';

				$row = NULL;

				$insertRows[] = $rowString;
			}

			$objRes = NULL;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

			$outputSql .= "\n\n".implode("\n", $insertRows)."\n";
		}

		return $outputSql;
	}
}

1			<?php declare(strict_types=1);
2			/**
3			* Query
4			*
5			* SQL Query Builder / Database Abstraction Layer
6			*
7			* PHP version 7
8			*
9			* @package Query
10			* @author Timothy J. Warren <[email protected]>
11			* @copyright 2012 - 2016 Timothy J. Warren
12			* @license http://www.opensource.org/licenses/mit-license.html MIT License
13			* @link https://git.timshomepage.net/aviat4ion/Query
14			*/
15			namespace Query\Drivers\Pgsql;
16
17			use Query\Drivers\AbstractUtil;
18
19			/**
20			* Posgres-specific backup, import and creation methods
21			*/
22			class Util extends AbstractUtil {
23
24			/**
25			* Create an SQL backup file for the current database's structure
26			*
27			* @return string
28			*/
29			public function backupStructure()
30			{
31			// @TODO Implement Backup function
32			return '';
33			}
34
35			/**
36			* Create an SQL backup file for the current database's data
37			*
38			* @param array $exclude
39			* @return string
40			*/
41			public function backupData($exclude=[])
42			{
43			$tables = $this->getDriver()->getTables();
44
45			// Filter out the tables you don't want
46			if( ! empty($exclude))
47			{
48			$tables = array_diff($tables, $exclude);
49			}
50
51			$outputSql = '';
52
53			// Get the data for each object
54			foreach($tables as $t)
55			{
56			$sql = 'SELECT * FROM "'.trim($t).'"';
57			$res = $this->getDriver()->query($sql);
			0 ignored issues – show Unused Code introduced 2016-10-13 00:43 UTC by Report Bug Copy Issue Report The call to `DriverInterface::query()` has too many arguments starting with `$sql`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
58			$objRes = $res->fetchAll(\PDO::FETCH_ASSOC);
59
60			// Don't add to the file if the table is empty
61			if (count($objRes) < 1)
62			{
63			continue;
64			}
65
66			$res = NULL;
			0 ignored issues – show Unused Code introduced 2016-07-19 21:10 UTC by Report Bug Copy Issue Report `$res` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
67
68			// Nab the column names by getting the keys of the first row
69			$columns = @array_keys($objRes[0]);
70
71			$insertRows = [];
72
73			// Create the insert statements
74			foreach($objRes as $row)
75			{
76			$row = array_values($row);
77
78			// Quote values as needed by type
79			$row = array_map([$this->getDriver(), 'quote'], $row);
80			$row = array_map('trim', $row);
81
82
83			$rowString = 'INSERT INTO "'.trim($t).'" ("'.implode('","', $columns).'") VALUES ('.implode(',', $row).');';
84
85			$row = NULL;
86
87			$insertRows[] = $rowString;
88			}
89
90			$objRes = NULL;
			0 ignored issues – show Unused Code introduced 2016-10-14 01:58 UTC by Report Bug Copy Issue Report `$objRes` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
91
92			$outputSql .= "\n\n".implode("\n", $insertRows)."\n";
93			}
94
95			return $outputSql;
96			}
97			}

aviat4ion / Query

Push — develop ( 111dce...b8d476 )

Util::backupData() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like