Cześć.

Jako, że zacząłem uczyć się OOP i postanowiłem sobie napisać CMS (tylko w celu nauki) z wykorzystaniem OOP PHP.

Jednak natrafiłem na problem.

Błędy:



Struktura projektu (* - folder):
* CMS
-- index.php
** class
--- User.php
** config
--- database.php

index.php

[PHP] pobierz, plaintext 
<?php
 
include('config/database.php');
include('class/User.php');
 
$nowy = new User();
 
$nowy->addNew('siema', 'na5tyk@o2.pl', 'xxx', 'xxx');
 
?>
[PHP] pobierz, plaintext 

User.php

[PHP] pobierz, plaintext 
<?php
 
class User {
 
	//Metoda wysylajaca emaila o utworzeniu konta i prosba o aktywacja konta
	function sendActivationEmail($email) {
 
		mail($email, 'Aktywacja konta', 'Aby aktywować konto kliknij w link: xxx');
	}
 
	//Metoda tworzaca nowego uzytkownika
	function addNew($login, $email, $password, $repeatPassword) {
		$register = $db->prepare('INSERT INTO `users` (login, email, password, activation) VALUES (:login, :email. :password, :activation)');
 
		$register->bindValue(':login', $login);
		$register->bindValue(':email', $email);
		$register->bindValue(':password', $password);
		$register->bindValue(':activation', rand(1000, 9999));
 
		$register->execute();
 
		sendActivationEmail($email);
	}
 
	//Metoda ustawiajaca bana dla uzytkownika
	function setBan($id, $ban) {
		$db->execute("UPDATE users SET ban = '" . $ban . "' WHERE id = '" . $id . "'");
	}
 
	//Metoda ustawiajaca czy uzytkownik jest aktywowany
	function setActivation($id, $code) {
		$db->execute("UPDATE users SET activation = '" . $code . "' WHERE id = '" . $id . "'");
	}
 
	//Metoda ustawiajaca wartosc w okreslonej kolumnie
	function setParams($id, $param, $value) {
		$db->execute("UPDATE users SET '" . $param . "' = '" . $value . "' WHERE id = '" . $id . "'");
	}
}
 
?>
[PHP] pobierz, plaintext 

database.php

[PHP] pobierz, plaintext 
<?php
 
$host = 'localhost';
$username = 'root';
$password = '';
$database = 'cms';
 
	try {
 
		$db = new PDO('mysql:host=' . $host . ';dbname=' . $database . '', $username, $password, 
		array(
			// wyłączenie zbędnego emulate prepares
			PDO::ATTR_EMULATE_PREPARES => false,
			// ustalenie sposobu raportowania błędów
			PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION
		));
 
	} catch (PDOException $err) {
 
		print "Error: " . $err->getMessage() . "<br/>";
   		die();
	}
 
?>
[PHP] pobierz, plaintext 

$repeatPassword jest dodane przyszłościowo gdy będę sprawdzał poprawność danych.

I teraz pytanie. Czy przesłanie zmiennej $db w parametrze metody będzie dobre czy jest lepszy sposób?

Proponuję zastosować DI

I w linijce, użyć parametru $db dla klasy User

[PHP] pobierz, plaintext 
$nowy = new User($db);
[PHP] pobierz, plaintext 

a w klasie stworzyć dodatkową metodę, __construct

[PHP] pobierz, plaintext 
 
public function __construct($db) {
    $this->db = $db;
}
[PHP] pobierz, plaintext 

Dzięki temu będzie działać Ci połączenie z bazą danych np. w linijce metody addUser, gdzie oczywiście potrzeba dopisać $this

[PHP] pobierz, plaintext 
$register = $this->db->prepare();
[PHP] pobierz, plaintext 

--
Korekta nazwy metody magicznej na poprawną.

Po zastosowaniu twoich porad teraz mam błąd:

To może pokaż jak zaimplementowałeś te porady w klasie User.

Chociaż zgodnie z tym co masz w komunikacie to coś nie tak masz w linii 19.

index.php

[PHP] pobierz, plaintext 
<?php
 
include('config/database.php');
include('class/User.php');
 
$nowy = new User($db);
 
$nowy->addNew('siema', 'na5tyk@o2.pl', 'xxx', 'xxx');
 
?>
[PHP] pobierz, plaintext 

User.php

[PHP] pobierz, plaintext 
<?php
 
class User {
 
	function __constructor($db) {
 
	    $this->db = $db;
	}
 
	//Metoda wysylajaca emaila o utworzeniu konta i prosba o aktywacja konta
	function sendActivationEmail($email) {
 
		mail($email, 'Aktywacja konta', 'Aby aktywować konto kliknij w link: hihi');
	}
 
	//Metoda tworzaca nowego uzytkownika
	function addNew($login, $email, $password, $repeatPassword) {
 
		$register = $this->db->prepare('INSERT INTO `users` (login, email, password, activation) VALUES (:login, :email, :password, :activation)');
 
		$register->bindValue(':login', $login);
		$register->bindValue(':email', $email);
		$register->bindValue(':password', $password);
		$register->bindValue(':activation', rand(1000, 9999));
 
		$register->execute();
 
		sendActivationEmail($email);
	}
 
	//Metoda ustawiajaca bana dla uzytkownika
	function setBan($id, $ban) {
 
		$db->execute("UPDATE users SET ban = '" . $ban . "' WHERE id = '" . $id . "'");
	}
 
	//Metoda ustawiajaca czy uzytkownik jest aktywowany
	function setActivation($id, $code) {
 
		$db->execute("UPDATE users SET activation = '" . $code . "' WHERE id = '" . $id . "'");
	}
 
	//Metoda ustawiajaca wartosc w okreslonej kolumnie
	function setParams($id, $param, $value) {
 
		$db->execute("UPDATE users SET '" . $param . "' = '" . $value . "' WHERE id = '" . $id . "'");
	}
}
 
?>
[PHP] pobierz, plaintext 

Połączenie PDO masz poprawnie zainicjowane?

[PHP] pobierz, plaintext 
$db = new PDO('mysql:host=localhost;dbname=test', $user, $pass);
[PHP] pobierz, plaintext 

Nie $db tylko $this->db w metodach klasy do której wstrzyknąłeś $db.

Myślę, że połączenie jest dobrze zrobione:

[PHP] pobierz, plaintext 
<?php
 
$host = 'localhost';
$username = 'root';
$password = '';
$database = 'cms';
 
	try {
 
		$db = new PDO('mysql:host=' . $host . ';dbname=' . $database . '', $username, $password, 
		array(
			// wyłączenie zbędnego emulate prepares
			PDO::ATTR_EMULATE_PREPARES => false,
			// ustalenie sposobu raportowania błędów
			PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION
		));
 
	} catch (PDOException $err) {
 
		print "Error: " . $err->getMessage() . "<br/>";
   		die();
	}
 
?>
[PHP] pobierz, plaintext 

Mówimy tylko o funkcji addNew().

Nie
function __constructor($db) {

a:
function __construct($db) {

Dzięki wielkie <3