создание движка на php и mysql 2.0

скоро дойду до этого и тогда смогу ответить. Сейчас его один курс изучаю.

 

Не советую. Дешёвый недопрограммист, который пытается заработать на свои недокурсах. Пару лет назад, он задавал тупейшие вопросы по PHP на одном достаточно популярном форуме и имел неосторожность представиться и дать ссылку на свой сайт. Критики, которая посыпалась в его адрес, чувак не выдержал и удалил свой аккаунт. Хотя, я практически уверен, что он до сих пор на том форуме (и не только), но под другой учеткой. Уж больно стилистика некоторых вопросов схожа. Я не буду его сравнивать с Женечкой Поповым, но всё это зараза из одной бочки.

 

PHP:

1. <?php
2. class User {
3.    
4.     private $db;
5.    
6.         private static $user = null ;
7.     public function __construct(){
8.         $this->db = new mysqli("localhost", "root", "", "mybase");
9.         $this->db->query("GET NAMES 'utf8'");
10.    
11.         }
12.    
13.             public static function getObject(){
14.                 if (self::$user === null) self::$user = new User();
15.                 return self::$user;
16.                
17.             }
18.             public function regUser($login, $password){
19.                 if ($login == "") return false;
20.                 if ($password == "") return false;
21.                 $password = md5($password);
22.                 return $this->db->query("INSERT INTO  'users' (`login`, `password`, `regdate`) VALUES ('$login', '$password', '".time()."')");
23.             }
24.            
25.             private function checkUser($login, $password){
26.                 $result_set = $this->db->query("SELECT  `password` FROM `users` WHERE `login` = '$login'");
27.                 $user = $result_set->fetch_assoc();
28.                 $result_set->close;
29.                 if (!$user) return false;
30.                 return $user["password"] === $password;
31.             }
32.             public function isAuth($login, $password){
33.                 session_start();
34.                 $login = $_SESSION["login"];
35.                 $password = $_SESSION["password"];
36.                 return $this->checkUser($login, $password);
37.             }
38.             public function login($login, $password){
39.             $password = md5($password);
40.             if ($this->checkUser($login, $password)){
41.                     session_start();
42.                     $_SESSION["login"] = $login;
43.                     $_SESSION["password"] = $password;
44.                     return true;
45.             }
46.             else return false;
47.             }
48.            
49.            
50.                 public function __destruct(){
51.                     if ($this->db) $this->db->close;
52.                 }
53.            
54.     }
55. ?>

 

у которого логика абсурдна.

1) в md5 запечатывать пароль - прошлый век. Сейчас все юзают password_hash без соли
2) нету проверки веденых данных юзером, тобишь в логик могу записать делете юзер табле и капец вашему цирку.
3) что - то маловато юзаем session_start() хотяб для приличия дубовым способом условие поставить - юзался старт / нет ?
4) public static function getObject() - рукаПукан.
5) в сессию никто не записывает пароль, кроме говнокодеров
6) public function isAuth - насильственное деяние бд, уже писал в начальных строках 3 - 6:
https://github.com/MouseZver/Mouse_Project/blob/master/inc/Application/Controller/User/AuthMe.php
7) Устал перечислять.