Witam,

mam klasę MySQL, składa się z konstruktora i dekonstruktora, który zamyka połączenie z mysql - mysql_close()

tworzę klasę Users która dziedziczy metody z klasy MySQL

[PHP] pobierz, plaintext 
class Users extends MySQL
{
    function ok()
    {
        return 'ok';
    }
}
[PHP] pobierz, plaintext 

po uruchomieniu skryptu wyskakuje błąd:


Warning: mysql_close(): no MySQL-Link resource supplied in class\MySQL\class.mysql.php on line 242

gdzie 242 linijka to

[PHP] pobierz, plaintext 
return mysql_close();
[PHP] pobierz, plaintext 

cała metoda:

[PHP] pobierz, plaintext 
public function __destruct()
	{
		if($this->connection)
		{
			$this->connection = false;
			return mysql_close();
		}
	}
[PHP] pobierz, plaintext 

Dlaczego taki błąd występuje?

Zobacz pierwszy z brzegu komentarz do mysql_close, PHP 5.3? A w ogóle trochę nie widzę sensu takiego tworzenia klasy. Dlaczego klasa użytkownika jest zwiazana bezpośrednio z mysql?

próbowałem tego też, ale wtedy pokazuje się błąd:

Warning: mysql_close(): 24 is not a valid MySQL-Link resource in


Dlaczego błędem jest wiązanie klasy użytkownika z mysql, jeżeli ta klasa ma np dodawać / edytować i usuwać użytkowników?

Nie lepiej Ci się zainteresować PDO niż męczyć się dalej z mysql? Nauki niedużo, a efekty widoczne.

[PHP] pobierz, plaintext 
public function __destruct()
	{
		if($this->connection)
		{
			var_dump($this->connection);
			mysql_close($this->connection);
$this->connection = false;
		}
	}
[PHP] pobierz, plaintext 

To są podstawy dziedziczenia.... poszukaj w necie o dziedziczeniu, pełno tego po polsku jest.

var_dump zwrócił mi resource(24) of type (mysql link) resource(24) of type (Unknown), ale nic mi to nie mówi


co do dziedziczenia klas - zapoznam się z książką która leży mi w szafce i w razie wątpliwości odezwę się

Widać w takim razie, że dwukrotnie wywołujesz ten destruktor. Totalnie bezsensu. Destruktor połączenia z bazą powinien być wywoływany raz, tak samo jak raz powinno być w ogóle tworzone połączenie z bazą.

rzeczywiście... przeczytałem gdzieś na tym forum w jakimś temacie więc mały update:

[PHP] pobierz, plaintext 
class Users
{
    private $db;
    function __construct($db = '')
    {
        $this->db = $db;
    }
 
    function ok()
    {
        print_r($this->db->fetchAssoc("SELECT * FROM `all_cms_settings`"));
    }
}
[PHP] pobierz, plaintext 

i w głównym pliku

[PHP] pobierz, plaintext 
require DOCUMENT_ROOT.'/class/MySQL/class.mysql.php';
$sql = new MySQL;
 
require DOCUMENT_ROOT.'/class/MySQL/class.users.php';
$users = new Users($sql);
$users->ok();
[PHP] pobierz, plaintext 

może być takie rozwiązanie?

Zdecydowanie lepiej

ps:
require DOCUMENT_ROOT.'/class/MySQL/class.users.php';
Tylko wywal usera z katalogu mysql. Jedno z drugim nie ma żadnego związku