Answer the question
In order to leave comments, you need to log in
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;
}
....
}
$db=new DataBase();
$empl=$db->getEmployees($where,$start,$perPage);
Answer the question
In order to leave comments, you need to log in
Where should the connection to the database be closed?
// Не давайте общие имена конкретным реализациям
// Почитайте, проникнитесь и используйте 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 questionAsk a Question
731 491 924 answers to any question