Problem z klasą mysql, Porszę o pomoc! Opcje

[virtualman]

Witam zacząłem pisać własną klase, ale mam błędy:

[PHP] pobierz, plaintext 
 ! ) Warning: mysql_query() expects parameter 2 to be resource, null given in C:\wamp\www\gra\class.php on line 47
Call Stack
#	Time	Memory	Function	Location
1	0.0006	670512	{main}( )	..\index.php:0
2	0.0052	725288	user->add( )	..\index.php:6
3	0.0054	726152	mysql->query( )	..\class.php:97
4	0.0054	726200	mysql_query ( )	..\class.php:47
 
 
( ! ) Warning: mysql_close() expects parameter 1 to be resource, null given in C:\wamp\www\gra\class.php on line 62
Call Stack
#	Time	Memory	Function	Location
1	0.0057	716504	mysql->__destruct( )	..\class.php:0
2	0.0057	716552	mysql_close ( )	..\cl
[PHP] pobierz, plaintext 

Kod PHP to:

[PHP] pobierz, plaintext 
class mysql
{
	private $host='localhost';
	private $name='root';
	private $pas='';
	private $db='game';
	private $handler;
 
	public function connect(){
		$this->handler=mysql_connect($this->host,$this->name,$this->pas)
			or die('Błąd podczas łączenia z bazą danych.');
		mysql_select_db($this->db,$this->handler)
			or die('Błąd podczas łączenia z bazą danych.');
	}
 
	public function set($host,$name,$pas,$db){
		$this->host = $host;
		$this->name = $name;
		$this->pas = $pas;
		$this->db = $db;
	}
 
	public function query($query){
		$return=mysql_query($query,$this->handler);
		return $return;
	}
 
	public function query_num($query){
		$result = mysq_query($query,$this->handler);
		return mysql_num_rows($result);
	}
 
	public function query_fetch($query){
		$result = mysql_query($query,$this->handler);
		return mysql_fetch_array($result);
	}
 
	public function __destruct(){
	mysql_close($this->handler);
	}
}
 
class user extends mysql
{
 
	private $id;
 
	public function setid($id){
		$this->id=$id;
	}
 
	public function getid(){
		return $this->id;
	}
 
	public function add($username,$password,$mail){
		$time=time();
		$data=date("d.m.Y");
		$code="";
		for($l=1; $l<=24; $l++){
			$a=rand(0,2);
				switch($a){
				    case 0:
						$code .= chr(rand(30,39));
						break;
				    case 1:
						$code .= chr(rand(65,90));
						break;
					case 2:
						$code .= chr(rand(97,122));
						break;
			}
		}
		$this->query("INSERT INTO user ('user','password','mail','date','last','activate') VALUES ('$username','$password','$mail','$data','$time',$code)");
	}
}
[PHP] pobierz, plaintext 

Proszę o pomoc!

[crocodillo]

Po pierwsze w destruktorze powinieneś sprawdzać, czy jest zdefiniowana zmienna $handler np:

[PHP] pobierz, plaintext 
public function __destruct(){
	if ($this->handler) mysql_close($this->handler);
[PHP] pobierz, plaintext 

Nie przyglądałem się dokładnie kodowi klasy, więc nie wiem czy są jakieś błędy, ale sprawdź w kodzie, czy wywołujesz funkcje set() i connect()

[PHP] pobierz, plaintext 
 
$u = new user();
$u->set('abc','abc','abc','abc');
$u->connect();
[PHP] pobierz, plaintext 

Jeśli łączysz się tylko z jedną bazą, to ja bym te funkcje (tzn set() i connect() )wrzucił do konstruktora klasy mysql.

[tehaha]

jeżeli piszesz sobie klasę w celach edukacyjnych to ok, ale w praktyce lepiej będzie jeśli będziesz używał PDO, a tak na marginesie to nie logicznie to zrobiłeś, np. w metodach query_num i query_fetch nie powinno być wysyłania zapytania bo wysłaniem zapytania zajmuje się metoda query(), po drugie lepiej jakbyś zmienne do łączenia przesyłał w konstruktorze a trzymał je w pliku konfiguracyjnym

[virtualman]

[PHP] pobierz, plaintext 
class mysql
{
	private $handler;
 
	public function	__construct($host='localhost',$name='root',$pas='',$db='game'){
		$this->host = $host;
		$this->name = $name;
		$this->pas = $pas;
		$this->db = $db;
		$this->handler=mysql_connect($host,$name,$pas)
			or die('Błąd podczas łączenia z bazą danych.');
		mysql_select_db($db,$this->handler)
			or die('Błąd podczas łączenia z bazą danych.');
	}
 
	public function query($query){
		$return=mysql_query($query,$this->handler);
		return $return;
	}
 
	public function query_num($query){
		$result = mysq_query($query,$this->handler);
		return mysql_num_rows($result);
	}
 
	public function query_fetch($query){
		$result = mysql_query($query,$this->handler);
		return mysql_fetch_array($result);
	}
 
	public function __destruct(){
	if($this->handler) mysql_close($this->handler);
	}
}
[PHP] pobierz, plaintext 

Przerobiłem troszkę i dalej jest jeden błąd:

[PHP] pobierz, plaintext 
Warning: mysql_close(): 4 is not a valid MySQL-Link resource in C:\wamp\www\gra\class.php on line 55
Call Stack
#	Time	Memory	Function	Location
1	0.0944	706072	mysql->__destruct( )	..\class.php:0
2	0.0944	706120	mysql_close ( )	..\class.php:55
[PHP] pobierz, plaintext 

żadnych pomysłów?

[crocodillo]

O ile mi wiadomo to mysql_close() nie jest konieczne, gdyż połączenia są automatycznie zamykane na końcu skryptu. Być może to automatyczne zamknięcie następuje przed wywołaniem destruktora? Ale to tylko przypuszczenie, bo tak naprawdę to nie mam pojęcia w jakich kolejnościach co jest wywoływane.

[virtualman]

Ja też nie orientuje się w kolejności wykonywania, ale wiem, że jak przeglądałem inną klase to było mysql_connect w destruktorze... Jakie mogą być skutki "uboczne" nie użycia funkcji mysql_connect()
?

[crocodillo]

Takie, że zasoby zostaną zwolnione dopiero po zakończeniu wykonywania się skryptu. PHP używam od kilku lat, co prawda nigdy nie pisałem nic tak wielkiego, że trzeba było bardzo dbać o pamięć, ale nigdy nie używałem mysql_close(). O ile się nie mylę to na php.net też chyba jest napisane, że używanie tej funkcji nie jest konieczne. Oczywiście to jest tylko moje zdanie, być może inni uważają inaczej.

Ten post edytował crocodillo 13.10.2011, 17:36:59

[virtualman]

czyli jeśli nie korzystam z kilku baz danych na raz to nie muszę tego używać? to w sumie chyba sobie odpuszczę, choć wolałbym wiedzieć co nie tak, bo lepiej rozwiązać problem niż pominąć funkcję...

[greycoffey]

Czy klasa "user" rozszerza klasę "mysql"? Nie, ona ją wykorzystuję. Poczytaj o różnicy między kompozycją a dziedziczeniem.
Natomiast co do błędu, sprawdziłbym czy resource jest zwracany z mysql_connecta.
Masz włączone E_NOTICE? Moim pomysłem jest to, że dziedziczysz klasę mysql na user, w związku z czym wywołujesz zmienną PRYWATNĄ $this->handler.
Tymczasowym uleczeniem tego będzie zmienienie modyfikatora $this->handler na protected. To tylko zaślepka, ponieważ kod jest do przepisania