Drukowana wersja tematu

Napisany przez: major697 9.04.2016, 11:30:01

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
http://www.php.net/define("DB_HOST", 'localhost');
http://www.php.net/define("DB_USER", 'michal');
http://www.php.net/define("DB_PASSWORD", '');
http://www.php.net/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) {
			http://www.php.net/die('Błąd połączenia (' . mysqli_connect_errno() . ') '
				. mysqli_connect_error());
		} else {
			http://www.php.net/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 (http://www.php.net/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) {
			http://www.php.net/header("Location: panel.php");
		} else {
			http://www.php.net/header("Location: index.php?error=4");
			http://www.php.net/die();
		}
 
	}
 
}
?>
[PHP] pobierz, plaintext 

Zwraca kilka błędów:


Link do obrazka: http://i.imgur.com/euep4qz.png

Napisany przez: sazian 9.04.2016, 15:38:59

[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 

Napisany przez: com 10.04.2016, 13:05:47

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 

Napisany przez: piotras 11.04.2016, 09:05:51

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)

Napisany przez: com 12.04.2016, 21:55:49

piotras

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

Napisany przez: viking 13.04.2016, 05:05:32

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

Napisany przez: com 13.04.2016, 18:57:24

Owszem np kiedy stosuję się service locator

Napisany przez: piotras 14.04.2016, 09:25:06

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

Napisany przez: com 14.04.2016, 16:53:23

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

Napisany przez: mrc 15.04.2016, 07:44:06

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.

Napisany przez: mrc 15.04.2016, 20:50:02

Tak, konkretnie polskie wydanie. Skarbnica wiedzy.

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

Napisany przez: piotras 15.04.2016, 21:04:22

ok, dzięki

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