Как правильно написать php класс для работы с mysqli?

Question

[Radoncheg]

Добрый день! Я пишу классы для работы с mysqli чтобы каждый раз ее не вызывать. Функция connect которая соединяется с базой данных получилась, а вот как оформить функцию select которая выбирает и возвращает id нужного имени пользователя и пароля через класс Database? То же самое планирую сделать с функцией create. Я дважды получается обращаюсь к классу Database.php Что не так в синтаксисе? Браузер выдает ошибку

Warning: mysqli_query() expects parameter 1 to be mysqli, null given in E:\openserver\OSPanel\domains\notebook.ru\classes\Database.php on line 8

Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result, null given in E:\openserver\OSPanel\domains\notebook.ru\classes\Database.php on line 9

Введенные данные не верны
Класс Database:

<?php
class Database {
    public function connect() {
        return mysqli_connect("localhost", "root", "", "notebook");
    }

    public function select() {
       $user = mysqli_query($sql, "SELECT `id` FROM `users` WHERE `username` = '$username' AND `password` = '$password'");
       return mysqli_fetch_array($user);
    }
}

Класс User:

<?php
require_once("Database.php");
class User {

function find($username, $password) {
$database = new Database();
$sql = $database->connect();
$database->select();
}

function create($username, $password, $email) {
$database = new Database();
$sql = $database->connect();    
mysqli_query($sql, "INSERT INTO users(username, password, email) VALUES ('$username', '$password', '$email')");
}

}

Comments

[Konata Izumi]

Подключите ORM и не мучайтесь велосипедостроением.

[edward_freedom]

зачем каждый вызов функции, создавать новый экземпляр бд?

[Сергей delphinpro]

Оформите вопрос как полагается.

Answer 1

[AUser0]

Конкретно в вашем коде первая ошибка из-за того, что переменная $sql (открытое соединение с MySQL) не попадает внутрь функции select(). Или передавайте в качестве аргумента (что неудобно из-за лишнего кода), или храните эту переменную внутри класса Database{}.

Вторая ошибка вытекает из первой, поскольку mysqli_query() не отрабатывает корректно, не возвращает результат, необходимый в mysqli_fetch_array().

Answer 2

[kafkiansky]

Все очень плохо. Изучайте PDO.

Comments

[Дмитрий]

Дополню
И подготовленные выражения

[Сергей delphinpro]

Каким боком здесь PDO?
У автора проблема с базовым пониманием использования функций, классов, передачей параметров.
Хоть ты ему PDO дай, хоть доктриной огрей...

[kafkiansky]

Сергей delphinpro, потому что сразу пачкой изучит какое-никакое ООП, подготовленные выражения и в целом современные приемы по работе с бд в пыхе.

[Сергей delphinpro]

kafkiansky, ну может быть.

[kafkiansky]

Сергей delphinpro, но при условии, что начнет копать и думать, а не брать и писать, так что да, возможно, с пдо я поспешил.

Answer 3

[Евгений]

class Database {

  private $host = "localhost";
  private $adm = "root";
  private $pass = "";
  private $db;
  private $sql;
  public $user;
  
  function __construct($db) {
  $this->db = $db;
  $this->connect($this->db);
}

public function connect($db) {
$this->sql = mysqli_connect($this->host, $this->adm, $this->pass, $db);
}

public function select($username, $password) {
$this->user = mysqli_query($this->sql, "SELECT `id` FROM `users` WHERE `username` = '$username' AND `password` = '$password'");
return mysqli_fetch_array($this->user);
}
}

Я сам только начал изучать ООП может и криво(надеюсь знающие ребята в коментах поправят, мне это будет полезно), но примерно так это должно работать...
И работа с классом отличается...

$db = "netbook";
$user = $_POST['user'];
$pass = $_POST['password'];

$bd_user = new Database ($db);
$val_uset = $bd_user -> select($user, $pass);

Comments

[kafkiansky]

Во-первых, у вас инъекции в запросе. Во-вторых, конфиги надо хранить снаружи, чтобы всегда была возможность у вас или у ваших коллег изменять их сбоку, а не лезть внутрь класса. В-третьих, используйте PDO. В-четвертых, я не понял необходимость наличия свойства $user. Еще советую почитать про psr.

[Евгений]

kafkiansky, про pdo, подготовленные запросы и кофиги отдельно - я в курсе, с телефона переписал код автора без отсебятины))
Про свойство юзера спасибо, сомневался, решил лучше пусть лишнее будет...

[Евгений]

kafkiansky, psr - учу, да, криво, но из-за телефона, так у меня поприятнее код, без таких в глаза бросающихся косяков)) спасибо за комент

[crazyhat]

Согласно solid ваш класс выполняет как минимум 2 действия:
1 - коннектиться к бд
2 - реализует методы для работы с этой бд

Вам надо, по-хорошему, разделить на 2 класса, каждый из которых отвечает за что-то одно. В вашем случае: класс, который возвращает коннект к бд (singltone) и класс, который реализует методы работы с коннектом.

[Евгений]

crazyhat, ок, понял, куча классов с одним конкретным действием) спасибо)

[crazyhat]

Евгений, Да не куча, а 2)

[Евгений]

crazyhat, ну это тут) а в крупной реализации как минимум на коннект, селект, апдейт, инсерт...

[ThunderCat]

crazyhat, что за ерунда? Вас послушать - круд надо в 4 разных объекта вынести. Принцип единой ответственности - это об атомарных сущностях, а не об атомарных действиях над ними.

[crazyhat]

ThunderCat, Так я это и не имел в виду. Я писал про то, что один класс отдает бд, а другой работает с объектом бд. Про отдельный класс для каждой операции я не писал.