[MySQL][PHP]Klasy i obiekty, nauka Opcje

[elldiablo]

Witam,
Utworzyłem klasę która łączy się z bazą i pobieram dane z bazy. Do utworzenie nowej klasy tj. "shop" pobieram dane z bazy i var_dump() zwraca mi dane. Jest wszystko OK.
Problem pojawia się gdy tworzę nową klasę i przez nią chcę pobrać dane z bazy, czyli klasa "shop".
Chciałbym aby klasa "shop" pobierała dane. Niestety zwraca mi "NULL".
Co robię nie tak?

[PHP] pobierz, plaintext 
define (_HOST_, 'localhost');
define (_USER_, 'root');
define (_PASSWORD_, 'admin');
define (_DATABASE_, 'test');
 
class DB {
	protected $_host;
	protected $_user;
	protected $_password;
	protected $_database;
	public $_connect;
	public $_result;
	public $_row;
 
	public function __construct() {
		$this->_host = _HOST_;
		$this->_user = _USER_;
		$this->_password = _PASSWORD_;
		$this->_database = _DATABASE_;
		$this->_connect = $_connect;
		$this->_result = $_result;
		$this->_row = $_row;
	}
 
	public function connect() {
		if($this->_connect = mysql_connect($this->_host, $this->_user, $this->_password)) {
			if(!$this->_setDB($this->_database)) die ('Nie można odnaleźć bazy danych');
		} else {
			return 'Nie można nawiązać połączenia z bazą danych';
		}
 
	}
 
	public function _setDB($db_name) {
		return mysql_select_db ($db_name, $this->_connect);
	}
 
	public function Execute($query) {
		$this->_result = false;
		if($this->_connect) {
			if($this->_result = mysql_query($query)) {
				while ($this->_row = mysql_fetch_assoc($this->_result))
					$resultArray[] = $this->_row;
				return $resultArray;
			}
		}
	}
}
$db = new DB();
$db->connect();
// Tutaj pobieram dane z bazy - tablica 
//$result = $db->Execute("SELECT * FROM shop"); 
 
class shop {
	public $_query;
 
	public function __construct() {
		$this->_query = $_query;
	}
 
	public function shops() {		
		$this->_query = DB::Execute("SELECT * FROM shop");
		return $this->_query;
	}
}
 
$shop = new shop();
$result = $shop->shops();
 
// Tutaj zwraca mi NULL.
echo '<pre>';
	var_dump($result);
echo '</pre>';
[PHP] pobierz, plaintext 

[Mephistofeles]

To co robisz nie ma sensu.
Używasz klasy jako kontenera na funkcje, a to nie o to chodzi. Poza tym Twoja klasa jest niepotrzebna, skoro chcesz użyć OOP skorzystaj z PDO.

[sazian]

ok, po kolei
konstruktor

[PHP] pobierz, plaintext 
		$this->_host = _HOST_;
		$this->_user = _USER_;
		$this->_password = _PASSWORD_;
		$this->_database = _DATABASE_;
		$this->_connect = $_connect;
		$this->_result = $_result;
		$this->_row = $_row;
[PHP] pobierz, plaintext 

1) nie chce mi się sprawdzać ale wydaje mi się że nie masz dostępu do tych stałych wewnątrz klasy
2) skąd bierzesz zmienne $_connect $_result i $_row?? to samo jest w klasie shop

[PHP] pobierz, plaintext 
while ($this->_row = mysql_fetch_assoc($this->_result))
					$resultArray[] = $this->_row;
				return $resultArray;
[PHP] pobierz, plaintext 

a nie lepiej dać mysql_fetch_array i tą tablice wyrzucić teraz przepisujesz jedną tablicę do drugiej i dopiero ją wywalasz

[PHP] pobierz, plaintext 
public function shops() {		
		$this->_query = DB::Execute("SELECT * FROM shop");
		return $this->_query;
	}
[PHP] pobierz, plaintext 

metoda Execute klasy DB nie jest statyczna

Ten post edytował sazian 17.12.2010, 20:11:54