cześć zrobiłem wcześniej skrypt PHP logowania do ssytemu chcę go teraz przerobić na obiektowy i zrobiłem połączenie z bazą danych:

[PHP] pobierz, plaintext 
<?php
//DANE DO LOGOWANIA DO BD
define("DB_HOST", 'localhost');
define("DB_USER", 'michal');
define("DB_PASSWORD", '');
define("DB_DATABSE", 'kurs_php');
?>
 
<?php
//PLIK LOGOWANIA DO BD I SPRAWDZANIA POLACZENIA Z BD
class dbConnect {
	function __construct() {
		require_once 'config.php';
		$db_mysqli = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_DATABSE);
		//$db_mysqli->query('SET NAMES utf8');
		if (!$db_mysqli) {
			die('Błąd połączenia (' . mysqli_connect_errno() . ') '
				. mysqli_connect_error());
		} else {
			echo '<font size="2px" color="red">Info: Połączono z bazą danych.</font><br />';
		}
	}
	public function CloseDb() {
		mysqli_close();
	}
}
?>
[PHP] pobierz, plaintext 

To połączenie działa poprawnie, problem pojawia się gdy mam się odwołać do zmiennej $db_mysqli, która przechowuje "połączenie z bazą" w skrypcie logowania:

[PHP] pobierz, plaintext 
<?php
// KLASA Z FUNKCJAMI LOGOWANIE UŻYTKOWNIKA
class Functions {
 
	function __construct() {
 
		// connecting to database
		$db_mysqli = new dbConnect();
 
	}
	function __destruct() {
 
	}
 
	public function Login($login, $hasloSha1) {
		$result = $db_mysqli->prepare("SELECT haslo FROM uzytkownicy WHERE login=? AND haslo=?");
		$result->bind_param('ss', $login, $hasloSha1);
		$result->execute();
		$result->store_result();
		$row = mysqli_fetch_assoc($result);
		$kodAktywowany = $row['kod'];
 
//sprawdzenie czy taki uzytkownik istnieje
		if ($result->num_rows == 1) {
			$_SESSION['logowanie'] = $login;
			return TRUE;
		} else {
			return FALSE;
		}
	}
 
}
?>
[PHP] pobierz, plaintext 

Nie wiem jak przerobić skrypt abym mógł się dowołać do zmiennej $db_mysqli w zapytaniu:

[PHP] pobierz, plaintext 
<?php $result = $db_mysqli->prepare("SELECT haslo FROM uzytkownicy WHERE login=? AND haslo=?"); ?>
[PHP] pobierz, plaintext 

Funkcja wywołująca klasę z instancją:

[PHP] pobierz, plaintext 
<?php
$func = new Functions();
 
if (isset($_POST['log_in'])) {
 
	if ($_POST['log_in']) {
		$login = mysqli_real_escape_string($db_mysqli, $_POST['login']);
		$haslo = mysqli_real_escape_string($db_mysqli, $_POST['pass']);
		$hasloSha1 = sha1($haslo);
		$user = $func->Login($login, $hasloSha1);
		if ($user == TRUE) {
			header("Location: panel.php");
		} else {
			header("Location: index.php?error=4");
			die();
		}
 
	}
 
}
?>
[PHP] pobierz, plaintext 

Zwraca kilka błędów:


Link do obrazka: KLIK

[PHP] pobierz, plaintext 
 
class dbConnect {
public $db_mysqli;
	function __construct() {
		require_once 'config.php';
		$this->db_mysqli = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_DATABSE);
 
 
(...)
 
class Functions {
 public $db_mysqli;
	function __construct() {
 
		// connecting to database
		$this->db_mysqli = new dbConnect();
 
	}
	function __destruct() {
 
	}
 
	public function Login($login, $hasloSha1) {
		$result = $this->db_mysqli->db_mysqli->prepare("SELECT haslo FROM uzytkownicy WHERE login=? AND haslo=?");
(....)
 
 
[PHP] pobierz, plaintext 

sazian ale public? naprawdę

[PHP] pobierz, plaintext 
 
class Functions {
 private $db_mysqli;
	function __construct($db) {
 
		$this->db_mysqli = $db;
 
	}
	function __destruct() {
 
	}
 
	public function Login($login, $hasloSha1) {
		$result = $this->db_mysqli->prepare("SELECT haslo FROM uzytkownicy WHERE login=? AND haslo=?");
(....)
[PHP] pobierz, plaintext 

I wywołanie

[PHP] pobierz, plaintext 
require_once 'config.php';
// ew jakiś try catch
$db = mysqli_connect(DB_HOST, DB_USER, DB_PASSWORD, DB_DATABSE);
 
$functions = new Functions($db);
$functions->Login(...);
[PHP] pobierz, plaintext 

Zrób z dbConnection singletona dziedziczącego po PDO. Nie będzie lepiej?

Później do Functions::Login przekaż tego singletona razem z danymi do logowania. Po co dawać go w całej Functions (no chyba, że logowanie to jej jedyna jej funkcja)

piotras

Singletona? Singleton to antywzorzec, czyli generalnie zły kod, lepiej stosować IoC i DI

Przez wielu ioc jest również uważany za antywzorzec.

Owszem np kiedy stosuję się service locator

@com
no spoko, ale co dokładnie sprawia, że użycie singletona w tej konkretnej sytuacji jest złym rozwiązaniem?

Singleton sprawia, że ta klasa staję się globalna, co generalnie nigdy nie jest dobre

Pisząc kod obiektowy chodzi o przekazywanie sobie nawzajem obiektów, od których dana funkcja/klasa jest zależna. Singleton jest zły, ponieważ zamiast przekazywać go, tworzysz go. Tworzysz przez to zależności wewnątrz obiektu, co jest trudne do przetestowania automatycznego.

Na początku singleton jest fajny. Bardzo długo go kochałem. @piotras przeczytaj sobie książkę Uncle Boba - Czysty Kod. Tam jest przedstawiony świetny model programowania obiektowego, który mimo że nie mówi że singleton jest zły, to z góry wyklucza go jako dobre rozwiązanie.

Tak, konkretnie polskie wydanie. Skarbnica wiedzy.

Oczywiście, polecam zakup. Ściąganie z neta piratów to brak szacunku do kolegów po fachu.

ok, dzięki

Wpisałem w google i wyskoczyło na pierwszym miejscu. Czytać i tak będę w wersji ksiązkowej