N
N
NastyaG2017-01-02 12:49:35
PHP
NastyaG, 2017-01-02 12:49:35

How to properly use databases in php page?

Hello. I can not figure out how to work with the database.
I have a database class:

class DataBase
{
    private $mysqli;
    private $dbConfig;

    public function __construct()
    {

        $this->dbConfig = require "db/database_config.php";
        $this->mysqli = mysqli_connect($this->dbConfig['host'], $this->dbConfig['username'], $this->dbConfig['password'], $this->dbConfig['db_name']);
        if (mysqli_connect_errno($this->mysqli)) {
            echo "Не удалось подключиться к MySQL: " . mysqli_connect_error();
        }
    }

  public function getEmployees($where='1',$start, $perPage){
        $sql="SELECT e.name,e.birthday,d.title_dep,p.title_pos,t.title_type,e.salary FROM `employees` AS e INNER JOIN departments AS d ON e.id_dep=d.id
INNER JOIN positions AS p ON e.id_pos=p.id
INNER JOIN payment_types AS t ON e.id_type=t.id  where $where LIMIT $start,$perPage";

        $res = $this->mysqli->query($sql);
        $row=$res->fetch_all(MYSQLI_ASSOC);
        return $row;
    }

....
}

And on the php page, the methods of this class are called for me:
$db=new DataBase();
$empl=$db->getEmployees($where,$start,$perPage);

Question here in what: in what place it is necessary to close connection with a DB?
In every method? But if I have many methods on the page that are called from the 1st connection?
Or is it better to just call the $db->closeConnection() method at the end of the page; ?

Answer the question

In order to leave comments, you need to log in

1 answer(s)
I
index0h, 2017-01-02
@index0h

Where should the connection to the database be closed?

before the end of the process. Even though it will close.
No, permanent reconnects will be expensive.
--- Some Code Review ---
// Не давайте общие имена конкретным реализациям
// Почитайте, проникнитесь и используйте PSR-2 и PSR-4
class DataBase
{
    private $mysqli;
    private $dbConfig;

    public function __construct()
    {
// Класс по работе с БД не должен знать даже о существовании неких файлов, где-то там. Это не его забота.
// Передавайте в конструктор готовое подключение к БД, если нужно.
        $this->dbConfig = require "db/database_config.php";
// Вот никак понять не могу, за что так любят этот mysqli, ну что в нем прям такого раз такого, по сравнению с PDO?
        $this->mysqli = mysqli_connect($this->dbConfig['host'], $this->dbConfig['username'], $this->dbConfig['password'], $this->dbConfig['db_name']);
// Почему вдруг класс по работе с БД занимается операцией вывода?
// Если что-то не так - бросайте исключение, ни каких echo, die, exit, trigger_error
        if (mysqli_connect_errno($this->mysqli)) {
            echo "Не удалось подключиться к MySQL: " . mysqli_connect_error();
        }
    }
// Вы не проверяете аргументы, это плохо, очень.
// Что бы нагнуть ваш проект достаточно передать в любой из аргументов: '1; DROP TABLE employees;'
  public function getEmployees($where='1',$start, $perPage){
// ЗАБУДЬТЕ про подстановку данных через конкатенацию, используйте плейсхолдеры
// http://php.net/manual/ru/pdo.prepared-statements.php
        $sql="SELECT e.name,e.birthday,d.title_dep,p.title_pos,t.title_type,e.salary FROM `employees` AS e INNER JOIN departments AS d ON e.id_dep=d.id
INNER JOIN positions AS p ON e.id_pos=p.id
INNER JOIN payment_types AS t ON e.id_type=t.id  where $where LIMIT $start,$perPage";

// Вам ни переменная $res, ни $row не нужны
        $res = $this->mysqli->query($sql);
        $row=$res->fetch_all(MYSQLI_ASSOC);
        return $row;
    }
}

Didn't find what you were looking for?

Ask your question

Ask a Question

731 491 924 answers to any question