[MySQL][PHP]PDO po raz kolejny Opcje

[Marcinekk]

Witam, w config.php mam funkcje, która pobiera dane użytkownika:

[PHP] pobierz, plaintext 
function get_user_data($user_id = -1) {
 
 
	if($user_id == -1) {
		$user_id = $_SESSION['user_id'];
	}
	$result = $pdo->query("SELECT * FROM `panel_users` WHERE `user_id` = '{$user_id}' LIMIT 1");
	if(mysql_num_rows($result) == 0) {
		return false;
	}
	return mysql_fetch_assoc($result);
}
[PHP] pobierz, plaintext 

Trochę tutaj pozamieniałem na PDO lecz wywala bląd:

Kod

Fatal error: Call to a member function query() on a non-object in /home/xxx/public_html/config.php on line 36

Kilka linijek wyżej jest połączenie wykorzystujące PDO, mimo to wywala ten błąd, proszę o pomoc.

[Marcinekk]

[PHP] pobierz, plaintext 
try {
		$pdo = new PDO('mysql:host='.$mysql_host.';dbname='.$database.';port='.$port, $username, $password );
		echo 'Połączenie nawiązane!';
		} catch(PDOException $e){
	echo 'Połączenie nie mogło zostać utworzone.<br />';	
	}
 
 
function get_user_data($user_id = -1) {
 
 
	if($user_id == -1) {
		$user_id = $_SESSION['user_id'];
	}
	$result = $pdo-> exec("SELECT * FROM `panel_users` WHERE `user_id` = '{$user_id}' LIMIT 1");
	if(mysql_num_rows($result) == 0) {
		return false;
	}
	return mysql_fetch_assoc($result);
}
[PHP] pobierz, plaintext 

[Felan]

[PHP] pobierz, plaintext 
function get_user_data($user_id = -1) {
 
 
	if($user_id == -1) {
		$user_id = $_SESSION['user_id'];
	}
	$result = $pdo-> exec("SELECT * FROM `panel_users` WHERE `user_id` = '{$user_id}' LIMIT 1");
	if(mysql_num_rows($result) == 0) {
		return false;
	}
	return mysql_fetch_assoc($result);
}
[PHP] pobierz, plaintext 

W PDO metoda exec() to nie to samo co query(), czy połączenie prepare() i execute() - polecam doczytać, bo tak napisane nie zadziała - exec() nie służy do odczytywania rekordów z bazy.
Polecam również zapomnieć o mysql_num_rows i innych tego typu starociach, poczytać konkretnie o PDO i tego się trzymać.

Żeby być pomocnym, a nie po prostu opowiadać pierdoły, dwa fragmenty:

[PHP] pobierz, plaintext 
//tak wyglądał kiedyś kod zapytania MySQL:
 
mysql_connect('host', 'user', 'pass');
mysql_select_db('baza');
$result = mysql_query("SELECT id, email FROM user WHERE username = '".$_GET['username']."'");
$user = mysql_fetch_assoc($result);
 
echo $user['email']." (".$user['id'].")";
 
mysql_close();
 
//I to jest do kitu. Teraz robi się to tak (zapomnieć o wszystkim, co ma w nazwie "mysql_" !)
 
$pdo = new PDO('mysql:host=host;dbname=baza', 'user', 'pass');
 
$stmt = $pdo->prepare("SELECT id, email FROM user WHERE username = :username");
 
$stmt->execute(array(':username' => $_GET['username']));
 
$user = $stmt->fetch(PDO::FETCH_ASSOC);
 
echo $user['email']." (".$user['id'].")";
[PHP] pobierz, plaintext 

Autor dostaje błąd tylko i wyłącznie dla tego, że jego funkcja nie ma dostępu do zmiennej $pdo, ponieważ ta ma inny scope. A żeby miała ten sam, to w jego przypadku należy ją przekazać jako parametr. Innego rozwiązania nie ma. I wtedy nie będzie miał błędu "Call to a member function query() on a non-object ".

Nie ma innego rozwiązania? Zawsze jest inne rozwiązanie! (IMG:style_emoticons/default/smile.gif) Obiekt PDO można stworzyć wewnątrz funkcji, a jako parametry podać wartości z $_POST; będzie przejrzyściej - chyba że mamy w planach dużo innych zapytań, ale nie wygląda na to. Faktem jest, że trzeba coś tej funkcji dać (albo zrobić wszystko "na miejscu").

[SQL] pobierz, plaintext 
SELECT `user_id` FROM `panel_users` WHERE `user_name` = '{$_POST['name']}' AND `user_password` = '{$_POST['password']}' LIMIT 1
[SQL] pobierz, plaintext 

Nie piszemy zapytań w ten sposób! Poczytaj o SQL injection, a następnie o prepared statements (czyli metodach PDO: prepare() i execute() ).

I jeszcze:

[PHP] pobierz, plaintext 
$find = $pdo->query("SELECT `user_id` FROM `panel_users` WHERE `user_name` = '{$_POST['name']}' AND `user_password` = '{$_POST['password']}' LIMIT 1");
$find->execute();
[PHP] pobierz, plaintext 

- nie zadziała. Jeszcze raz polecam lekturę w temacie PDO: query(), prepare(), execute(), exec() - szczególnie o tym, co zwracają te metody.