logowanie admin user Opcje

[login90]

Witajcie
Mam taki kod. Wiecie może jak zrobić aby było rozgraniecznie logowanie admin/user. Mam w bazie danych type admin user Dzięki

[PHP] pobierz, plaintext 
class Account {
 
    private $db;
 
    public function __construct() {
        $conn = new Db();
        $this->db = $conn->db();
    }
 
    public function login($username, $password) {
        $query = $this->db->query("SELECT count('id') as total,username,password from signup WHERE (username = '$username' OR email = '$username') LIMIT 1");
 
        while ($current_user = $query->fetch_assoc()) {
            if ($current_user['total'] == 1 && $this->varify_pass($password, $current_user['password']) === TRUE) {
                $_SESSION['username'] = $current_user['username'];
                $_SESSION['logged_in'] = TRUE;
                header("Location: dashboard.php");
            } else {
                echo "Username & Password is Incorrect";
            }
        }
    }
 
    public function varify_pass($input_pass, $pass_hash_fromdb) {
        if (password_verify($input_pass, $pass_hash_fromdb)) {
            return TRUE;
        }
    }
[PHP] pobierz, plaintext 

Ten post edytował login90 7.06.2020, 19:09:54

[gino]

najpierw odpowiedź na pytanie: można na wiele sposobów.Dodatkowe pole np. status tinyint gdzie 0 to admin, 1 to user itd albo dodatkowa tablica gdzie można zdefiniować user to admin i do czego ma dostęp itd, itp.

Co do kodu: to nie jest programowanie obiektowe, to można zrealizować na zwykłych funkcjach, metody nie są logicznie i metodycznie ze sobą powiązane, proponuję odejść od takiego sposobu uczenia się, że wrzuca się do klasy funkcje i sądzić, że jest ok. Nie oceniam samego kodu ale brak jest obsługi błędów, pod co w zapytaniu LIMIT 1, spodziewasz się wielu userów o tym samym mailu? To niedopuszczalne jak i o tym samym loginie.

gino

[StreetFighter]

Co do kodu: to nie jest programowanie obiektowe, to można zrealizować na zwykłych funkcjach

Jako że jestem początkujący, podłącze się do pytania , mając kolegi kod powyżej, piszesz że nie jest to programowanie obiektowe. Jak można byłoby to wykonać w programowaniu obiektowym?

[gitbejbe]

programowanie obiektowe to styl życia, nie da się tego od tak zrozumieć ; )

[StreetFighter]

Wydaje mi się że nie wszystko trzeba pisać obiektowo. Czasami nie ma co na siłę tego robić.
Jeżeli np. klasa ma mieć tylko jedną instancje, nie widzę sensu, może być po prostu zbiorem metod statycznych.
Chyba że dla zachowania spójności danej aplikacji.