[MySQL][PHP] Prosta obiektówka, za chwilę zwariuję.. Opcje

[Dual Jack]

Witam!

Napisałem sobie bardzo prosty skrypcik połączenia z bazą danych.

Funkcja sql() tworzy obiekt łączący się z bazą danych oraz wysyła zapytanie sql.

Wszystko cacy, ale występuje jakiś nielogiczny błąd. Może Wy znajdziecie przyczynę:

[PHP] pobierz, plaintext 
class mysql_connect {
 
	public $host = "localhost";
	public $user = "root";
	public $pass = "password";
	public $db_name = "test";
 
	public function __construct($host,$user,$pass,$db_name){
		echo $db_name; // nic nie zwraca?!
		$connect = mysql_connect($host,$user,$pass);
		$database = mysql_select_db($db_name);
		mysql_query("SET NAMES utf8");
		$return_error = "<h1>Błąd połączenia z bazą danych</h1>";
		if($connect) return TRUE;[u] else die($return_error)[/u];
	}
 
	public function query($query){
		$results = mysql_query($query) or die(mysql_error());
		return $results;
	}
 
}
 
function sql($sql){
	$db = new mysql_connect;
	$db -> query($sql);
}
 
sql("SELECT * FROM blog");
[PHP] pobierz, plaintext 

Po uruchomieniu strony, uruchamia się instrukcja die() i zwraca mi ona słowa:
"No database selected".

Chodzi o to, że zmienna $db_name nie przechodzi do wnętrza metody obiektu. Dla czego? Jakim cudem?

Proszę o pomoc.

Ten post edytował Dual Jack 11.04.2013, 18:43:27

[Szymciosek]

Sprawa 1) Czemu ma przejść? Skoro ich nawet nie przekazujesz... masz na sztywno ustawione wartości zmiennych publicznych, a w konstruktorze i tak się spodziewasz podania 4 zmiennych, więc masz do wyboru:
a)

[PHP] pobierz, plaintext 
class mysql_connect {
 
	public $host = "localhost";
	public $user = "root";
	public $pass = "password";
	public $db_name = "test";
 
	public function __construct(){
		echo $db_name; // nic nie zwraca?!
		$connect = mysql_connect($this->host,$this->user,$this->pass);
		$database = mysql_select_db($this->db_name);
		mysql_query("SET NAMES utf8");
		$return_error = "<h1>Błąd połączenia z bazą danych</h1>";
		if($connect) return TRUE;[u] else die($return_error)[/u];
	}
 
	public function query($query){
		$results = mysql_query($query) or die(mysql_error());
		return $results;
	}
 
}
 
function sql($sql){
	$db = new mysql_connect;
	$db -> query($sql);
}
 
sql("SELECT * FROM blog");
[PHP] pobierz, plaintext 

b )

[PHP] pobierz, plaintext 
class mysql_connect {
 
	public function __construct($host,$user,$pass,$db_name){
		echo $db_name; // nic nie zwraca?!
		$connect = mysql_connect($host,$user,$pass);
		$database = mysql_select_db($db_name);
		mysql_query("SET NAMES utf8");
		$return_error = "<h1>Błąd połączenia z bazą danych</h1>";
		if($connect) return TRUE;[u] else die($return_error)[/u];
	}
 
	public function query($query){
		$results = mysql_query($query) or die(mysql_error());
		return $results;
	}
 
}
 
function sql($sql){
	$db = new mysql_connect('localhost', 'root', 'password', 'test');
	$db -> query($sql);
}
 
sql("SELECT * FROM blog");
[PHP] pobierz, plaintext 

2) W 18 linii używasz mysql_error... czemu nie zrobisz tego samego w 14 linii? die(mysql_error);

3) nie używa się już mysql_connect tylko PDO.

Rada od wujka Szymka - weź się za podstawy, a później pisz takie klasy czy coś. W OOP często używa się do najprostszych rzeczy $this->

Ten post edytował Szymciosek 11.04.2013, 19:06:30