Почему не работает мой класс для подключения к MySql?

Question

[Андрей Самойленко]

Пишу свой первый класс на php, выдает ошибку подключения к БД: Access denied for user ''@'localhost' (using password: NO) Из сие ошибки можно увидеть, что переменная с именем пользователя root не прочиталась при подключении (в ошибки её просто нет).

Вот класс:

<?php
   class Database {
	   private $host = "localhost";
	   private $basename = "base100500";
	   private $bd_user = "root";
	   private $pass = "";
	   
	   function sender($ip, $client, $ref, $time) {
		   $mysqli = new mysqli($host, $bd_user, $pass, $basename);
	       if(mysqli_connect_errno()) {
              echo "Подключение невозможно: ".mysqli_connect_error();
           }
		   $mysqli->query("INSERT INTO visits(ip, client, ref, time) VALUES('$ip', '$client', '$ref', '$time')");
		   $mysqli->close();
	   }
	   
	   function receiver() {
		   $mysqli = new mysqli($host, $bd_user, $pass, $basename);
		   if(mysqli_connect_errno()) {
              echo "Подключение невозможно: ".mysqli_connect_error();
           }
		   $result = $mysqli->query("SELECT * FROM visits ORDER BY id DESC LIMIT 1000");
		   
		   //Вывод записей
		   while($row = $result->fetch_assoc()) {
			   echo "<tr>";
			   echo "<td>".$row['id']."</td> <td>".$row['ip']."</td> <td>".$row['client']."</td> <td>".$row['ref']."</td> <td>".$row['time']."</td>";
			   echo "</tr>";
		   }
		   $mysqli->close();
	   }
   }




?>

Если подставить значение переменных напрямую в конструктор - то все работает, а вот так вот не работает. Почему?

Comments

[Zhainar]

зачем вам вообще этот класс, по крайней мере в таком виде? проще сделать две функции типа db_sender и db_receiver.

[Андрей Самойленко]

Zhainar: я понимаю, но на чем мне еще изучать ООП как не на простых вещах?

[Андрей Самойленко]

Zhainar: а в каком виде вы бы посоветовали? мне кажется удобно.. все что связано с выводом и записью в бд у меня в одном классе.

[Zhainar]

Андрей Самойленко: ответил

Answer 1

[D']

Это основы. Открываем php.net/manual/ru/language.oop5.basic.php и играем в игру "Найди 10 отличий".

PS
Подсказка:
Читаем про доступ к методам и свойствам классов в PHP.

Comments

[Андрей Самойленко]

Спасибо, буду учиться ;)

Answer 2

[Zhainar]

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

<?php
   class Database {
     private $host = "localhost";
     private $basename = "base100500";
     private $bd_user = "root";
     private $pass = "";
     private $mysqli;

    public function connection() 
{
    if(!$this->mysqli) { $this->mysqli= new mysqli($this->host, $this->bd_user, $this->pass, $this->basename); }
    if ($this->mysqli->connect_errno) { throw new Exception('Ошибка соединения: ' . $this->mysqli->connect_errno); }
    return $this->mysqli;
}

public function closeConnection()
{
    return $this->connection()->close();
}
     
     public function setSender($ip, $client, $ref, $time) {
       return $this->connection()->query("INSERT INTO visits(ip, client, ref, time) VALUES('$ip', '$client', '$ref', '$time')");
     }
     
     public function getReceiver() {
      return $this->connection()->query("SELECT * FROM visits ORDER BY id DESC LIMIT 1000");
    }
    
    public function showReceiver($receiver) {
       while($row = $receiver->fetch_assoc()) {
         echo "<tr>";
         echo "<td>".$row['id']."</td> <td>".$row['ip']."</td> <td>".$row['client']."</td> <td>".$row['ref']."</td> <td>".$row['time']."</td>";
         echo "</tr>";
       }
    }

   }




?>