Create::setdata() - Code Metrics - Inspection of "[UPD] Add news methods" - Guilherme-fagundes/database - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 1156a5...423021 )

by guilherme

created 2020-06-26 19:59 UTC

Create::setdata() A

↳ Parent: Create

Complexity

Conditions	2
Paths	4

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	10
c	0
b	0
f	0
dl	0
loc	15
rs	9.9332
cc	2
nc	4
nop	0

<?php


namespace database;


class Create
{

    private $table;
    private $create;
    private $result;
    private $error;
    private $data;
    private $statements;


    public function __construct()
    {

    }

    /**
     * @return mixed
     */
    public function getResult()
    {
        return $this->result;
    }



    public function create(string $table, array $data)
    {
        $this->data = $data;
        $this->table = $table;
        $this->data = array_map('strip_tags', $this->data);
        $this->data = array_map('trim', $this->data);

        if ($this->data) {

            $this->setdata();
        }
    }


    private function setdata()
    {
        $fields = implode(', ', array_keys($this->data));
        $places = ':'.implode(', :', array_keys($this->data));
        $this->create = "INSERT INTO {$this->table} ({$fields}) VALUES ({$places})";

        try{

            $this->statements = DB::connect()->prepare($this->create);
            $this->statements->execute($this->data);
            $this->result = DB::connect()->lastInsertId();
            $this->error = true;

        }catch (\PDOException $e){
            echo $e->getMessage();

        }

    }

}

1			<?php
2
3
4			namespace database;
5
6
7			class Create
8			{
9
10			private $table;
11			private $create;
12			private $result;
13			private $error;
14			private $data;
15			private $statements;
16
17
18			public function __construct()
19			{
20
21			}
22
23			/**
24			* @return mixed
25			*/
26			public function getResult()
27			{
28			return $this->result;
29			}
30
31
32
33			public function create(string $table, array $data)
34			{
35			$this->data = $data;
36			$this->table = $table;
37			$this->data = array_map('strip_tags', $this->data);
38			$this->data = array_map('trim', $this->data);
39
40			if ($this->data) {
			0 ignored issues – show Bug Best Practice introduced 2020-06-21 05:19 UTC by Report Bug Copy Issue Report The expression `$this->data` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
41			$this->setdata();
42			}
43			}
44
45
46			private function setdata()
47			{
48			$fields = implode(', ', array_keys($this->data));
49			$places = ':'.implode(', :', array_keys($this->data));
50			$this->create = "INSERT INTO {$this->table} ({$fields}) VALUES ({$places})";
51
52			try{
53
54			$this->statements = DB::connect()->prepare($this->create);
55			$this->statements->execute($this->data);
56			$this->result = DB::connect()->lastInsertId();
57			$this->error = true;
58
59			}catch (\PDOException $e){
60			echo $e->getMessage();
61
62			}
63
64			}
65
66			}

Guilherme-fagundes / database

Push — master ( 1156a5...423021 )

Create::setdata() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like