Forum PHP.pl

Rozpocząłem starcie z PDO Postanowiłem przepisać nieco kodu z MySQL na PDO w ramach treningu. Początkowo szło dobrze, ale teraz przegrywam walkę. Połączenie to zdaje się błahostka:

[PHP] pobierz, plaintext 
try {
	$pdo = new PDO('mysql:host='.$db_host.';dbname='.$db_name.';encoding=utf8', $db_user, $db_pass);
	//echo 'Połączenie nawiązane!';
} catch(PDOException $e) {
	echo 'Error: ' . $e->getMessage(). "<br/>";
  die();
}
[PHP] pobierz, plaintext 

Zacząłem od takiej oto funkcji, zwracającej dane usera:

[PHP] pobierz, plaintext 
function getUser($id) {
$User = mysql_fetch_array(mysql_query("SELECT * FROM g_users WHERE user=$id"));
if(!empty($User))
  $User['data_product'] = mysql_fetch_array(mysql_query("SELECT * FROM g_products WHERE product=".$User['id_product']));
if(empty($User))
  $User['function'] = -1;
return $User; 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
function getUser2($id) {
  extract($GLOBALS);
  $r = $pdo->query("SELECT * FROM g_users WHERE user=$id");
  $User = $r->fetch(PDO::FETCH_ASSOC);
  if(!empty($User)) {
    $r = $pdo->query("SELECT * FROM g_products WHERE product=".$User['id_product']."");
    $User['data_product'] = $r->fetch(PDO::FETCH_ASSOC);
  }
  if(empty($User))
    $User['function'] = -1;
  return $User; 
}
[PHP] pobierz, plaintext 

Funkcja zwraca identyczne dane. Czy można to zrobić lepiej?

Jeżeli chodzi o PDO to używaj "prepared statements".

1) extract($GLOBALS); - naprawdę?

2)

[PHP] pobierz, plaintext 
  if(!empty($User)) {
     ....
  }
  if(empty($User))
[PHP] pobierz, plaintext 

To można zamienić na if else.

3) te dwa zapytania do bazy na 99% można zastąpić jednym

1. Inaczej zgłasza błąd Fatal error: Call to a member function query()
2. No można.
3. Ledwie skleciłem to Muszę wpierw załapać PDO.

Zgadza się, jest to związane z zakresem zmiennych (funkcja getUser2() nie ma dostępu do zmiennej $pdo).

Opcje:
1) wystarczy: global $pdo; (brzydko)
2) przekaż do funkcji zmienną $pdo jako argument (lepiej)
3) przepisać to na kod obiektowy np. z użyciem wzorca singleton i poczytać o dependency injection (najlepiej)

1. global $pdo nie działało
2. to ma sens
3. a świstak siedzi i zawija w sreberka

Mam kolejny problem. Taki kod:

[PHP] pobierz, plaintext 
$prods = mysql_query("SELECT * FROM tabela1 LEFT JOIN tabela2 ON prod=prod_id WHERE kat_id=".$User['id_kat']);
 
if(mysql_num_rows($prods) == 0) {
[PHP] pobierz, plaintext 

na taki:

[PHP] pobierz, plaintext 
$prods = $pdo->query("SELECT * FROM tabela1 LEFT JOIN tabela2  ON prod=prod_id WHERE kat_id=".$User['id_kat']);
 
if ($prods->fetchColumn() === 0) {
[PHP] pobierz, plaintext 

To działa, zwraca takie same wyniki, ale czy można lepiej?

Druga rzecz. Kodowanie połączenia. Miałem:

[PHP] pobierz, plaintext 
mysql_query("SET NAMES 'utf8'");
[PHP] pobierz, plaintext 

W bazie polskie znaki, pliki również UTF-8 a po zmianie na PDO wychodzą krzaki...

Łącząc się z bazą za pomocą PDO musisz ustalić kodowanie

[PHP] pobierz, plaintext 
$db = new PDO('mysql:host=localhost;dbname=xxx', 'user', 'pass', array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'"))
[PHP] pobierz, plaintext 

A w jaki sposób to wywołujesz? Dla mysql możesz bezpośrednio

[PHP] pobierz, plaintext 
$connect = new PDO(
  "mysql:host=$host;dbname=$db", 
  $user, 
  $pass, 
  array(
    PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
    PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"
  )
);
[PHP] pobierz, plaintext 

albo ....exec("set names utf8");

W pierwszym poście zamieściłem połączenie z bazą. Czyli coś takiego powinienem użyć?

[PHP] pobierz, plaintext 
$pdo = new PDO('mysql:host='.$db_host.';dbname='.$db_name.';encoding=utf8', $db_user, $db_pass, array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'"))
[PHP] pobierz, plaintext 

Zobacz czy działa a potem pytaj

No właśnie, nie działa...

Jaka wersja PHP? Poniżej 5.coś nie działało. Spróbuj jeszcze drugą metodę z exec

Nie, nie, mam powyżej 5. Być może to wina tego zapytania ostatniego. Zauważyłem teraz, że ucina mi pierwszy rekord.

[PHP] pobierz, plaintext 
$prods = $pdo->query("SELECT * FROM tabela1 LEFT JOIN tabela2  ON prod=prod_id WHERE kat_id=".$User['id_kat']);
if($prods->rowCount()>0){
    while($row=$prods->fetchAll()){
        //cos tam
    }
}
else echo "Brak wyników";
[PHP] pobierz, plaintext 

poza tym powinieneś bindować dane, inaczej stosowanie PDO nie ma sensu

Ten post edytował Turson 23.03.2014, 15:23:28

To wiem, tak robię. Tylko nigdzie nie mogę znaleźć dobrego artykułu na temat PDO. Nie wiem np., czy ma sens bindowanie zapytań z danymi niepochodzącymi z formularzy i linków...

while($row = $prods->fetchAll()) ?

Bardziej: while($row=$prods->fetch()){ .. }

Tak właśnie miałem, zmieniłem:

[PHP] pobierz, plaintext 
while($prod = mysql_fetch_array($prods)) {
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
while($prod = $prods->fetch()) {
[PHP] pobierz, plaintext 

Inaczej nie zwracało wyników.

Nadal nie mam polskich znaków. Tabele w utf8_general_ci.

Po połączeniu wykonaj trzy zapytania:
SET character_set_connection=utf8
SET character_set_client=utf8
SET character_set_results=utf8

Więc może te dane nie są w UTF-8 albo nie ustawiasz header. W stopce mam artykuł o PDO. Może Ci pomoże.

Czytałem ten artykuł. Wszystko wyświetla się dobrze, z zapytań MySQL, po zmianie na PDO krzaczy.

$dane = $prods->fetchAll();
foreach($dane as $v)

tak zazwyczaj piszę stąd mi się pomyliło.

Kschyhoo, ja zawsze robię tak:
- baza w utf8_polish_ci
- plik w utf8, dokument w utf8
- połączenie z bazą jak podałem
i nigdy nie mam problemu z pl znakami