database\Create

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	getResult()	0	3	1
A	create()	0	9	2
A	setdata()	0	15	2

<?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) {

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->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();

        }

    }

}