Практика использование ООП

Сделал небольшой класс лайков и дизлайков, всё хорошо работает, нормальный ли этот код? Наверно всё таки плохо, то, что я сделал добавление ников, в поле пользователей, которые уже проголосовали, а не айдишников?

PHP:

1. <?php
2.  
3. //проверка на авторизацию.
4.     public function loginVerification()
5.     {
6.         $user = $this->auth->isLoggedIn();
7.         if(!$user) {
8.           echo 'Авторизируйтесь';
9.           die;
10.        }
11.     }
12.  
13.  
14.     //проверяет, какие пользователи, уже голосовали.
15.     public function getVoted($id)
16.     {
17.       $login = $this->auth->getUsername(); //достаёт логин из сессии
18.       $res = $this->database->find('comments', $id); //достаёт все поля строки (коммента)
19.       $allUsers = $res['logins']; //достаёт проголосовавших пользователей
20.  
21.       preg_match_all('#([a-zA-Z0-9а-яА-Я]+)#u', $allUsers, $matches);
22.  
23.       if (in_array($login, $matches[0])) {
24.         return false;
25.       }
26.  
27.       else {
28.           return $matches;
29.       }
30.     }
31.  
32.  
33.     //обновляет счётчик комментария, для выбранной строки.
34.     public function updateComment($id, $currentCount, $login, $row, $allUsers)
35.     {
36.  
37.         $oldUsers = implode(",", $allUsers[0]); //для того, что бы обновить строку с юзерами, а не заменить, на одного
38.  
39.        if($row == 'like_comment') { //для лайка
40.             $this->database->update('comments', $id, ['like_comment' => $currentCount]);
41.        }
42.        else {//для дизлайка
43.            $this->database->update('comments', $id, ['dislike_comment' => $currentCount]);
44.        }
45.  
46.       $this->database->update('comments', $id, ['logins' =>  $oldUsers.','.$login.',']);
47.      
48.     }
49.  
50.     public function like()
51.     {
52.        $id = $_POST['id'];
53.    
54.       $this->loginVerification();
55.    
56.       $login = $this->auth->getUsername();
57.    
58.       if(!$this->getVoted($id)) {
59.           echo 'Вы уже голосовали';
60.           die;
61.       }
62.  
63.       $res = $this->database->find('comments', $id);
64.  
65.       $currentCount = $res['like_comment'] + 1;
66.  
67.       $this->updateComment($id, $currentCount, $login, $row = 'like_comment', $allUsers = $this->getVoted($id));
68.  
69.  
70.       echo $currentCount;
71.  
72.     }
73.  
74.     public function dislike()
75.     {
76.       $id = $_POST['id'];
77.  
78.        $this->loginVerification();
79.    
80.        $login = $this->auth->getUsername();
81.    
82.       if(!$this->getVoted($id)) {
83.           echo 'Вы уже голосовали';
84.           die;
85.       }
86.    
87.       $res = $this->database->find('comments', $id);
88.  
89.       $currentCount = $res['dislike_comment'] + 1;
90.  
91.       $this->updateComment($id, $currentCount, $login, $row = 'dislike_comment', $allUsers = $this->getVoted($id));
92.  
93.       echo $currentCount;
94.  
95.     }
96.    

 

@Алексей Новиков без, ведь 2018 год уже

 

Кому интересно, можете потестить http://pdd-vmg.ru/article/14

 

@nospiou я взял готовый макет и он какой-то косячный

 

@nospiou да это он

 

@nospiou @mkramer @MouseZver @askanim @romach @voral @Zuldek @igordata

Хочу понять, что такое полиморфизм и для примера, решил сделать типа библиотеку, которая может работать с разными драйверами подключениями баз.

Это полиморфизм?

Можно ли как-то это сократить

PHP:

1. $pdo = new Db($host, $db, $user, $pass, (new DbPDO($host, $db, $user, '')));

, что бы не дублировать?

PHP:

1. <?php
2.  
3. error_reporting(E_ALL);
4.  
5. interface DbInterface
6. {
7.     public function show($table);
8.    
9. }
10.  
11.  
12. class Db
13. {
14.     public $host;
15.     public $dbname;
16.     public $user;
17.     public $password;
18.     public $dbdriver;
19.    
20.     public function __construct($host, $dbname, $user, $password, DbInterface $dbd)
21.     {
22.         $this->host = $host;
23.         $this->dbname = $dbname;
24.         $this->user = $user;
25.         $this->password = $password;
26.         $this->dbdriver = $dbd;
27.     }
28.    
29.     public function query()
30.     {
31.         $this->dbdriver->show($table);
32.     }
33. }
34.  
35. class DbPDO implements DbInterface
36. {
37.     private $host;
38.     private $dbname;
39.     private $user;
40.     private $password;
41.     private $dsn;
42.     private $opt = [
43.         PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,
44.         PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
45.         PDO::ATTR_EMULATE_PREPARES   => false,
46.     ];
47.    
48.     public function __construct($host, $dbname, $user, $password)
49.     {
50.         $this->host = $host;
51.         $this->dbname = $dbname;
52.         $this->user = $user;
53.         $this->password = $password;
54.         $this->dsn = "mysql:host=$this->host;dbname=$this->dbname;charset=utf8";
55.     }
56.    
57.     public function show($table)
58.     {
59.         $pdo = new PDO($this->dsn, $this->user, $this->password, $this->opt);
60.         $sql = "SELECT login FROM $table";
61.         $stmt = $pdo->query($sql);
62.         $res = $stmt->fetchAll();
63.         return $res;
64.     }
65. }
66.  
67. class DbMySqli implements DbInterface
68. {
69.     private $host;
70.     private $dbname;
71.     private $user;
72.     private $password;
73.    
74.     public function __construct($host, $dbname, $user, $password)
75.     {
76.         $this->host = $host;
77.         $this->dbname = $dbname;
78.         $this->user = $user;
79.         $this->password = $password;
80.     }
81.    
82.     public function show($table)
83.     {
84.         $mysqli = new mysqli($this->host, $this->user, $this->password, $this->dbname);
85.         $sql = "SELECT login FROM $table";
86.         $result = $mysqli->query($sql);
87.         $result = mysqli_fetch_all($result);
88.         return $result;
89.     }
90. }
91.  
92. $host = 'localhost';
93. $db   = 'oop';
94. $user = 'root';
95. $pass = '';
96.  
97. $pdo = new Db($host, $db, $user, $pass, (new DbPDO($host, $db, $user, '')));
98.  
99. $mySqli = new Db($host, $db, $user, $pass, (new DbMySqli($host, $db, $user, '')));
100.  
101. echo '<pre>';
102.     print_r($pdo->dbdriver->show('users'));
103. echo '</pre>';
104.  
105. echo '<pre>';
106.     print_r($mySqli->dbdriver->show('users'));
107. echo '</pre>';