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

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

Ja też nie miałem nigdy problemu, a na pewno do czasu PDO ;P Nie mogę dać utf8_polish_ci...

utf8_general_ci też na pewno działa. Jaki masz?

utf8_general_ci

Czy dobrze zrozumiałem. Muszę zamykać każde połączenie (fachowo chyba kursora do obiektu klasy PDOStatement)?

[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->fetchl()) {
    // cos tam
  }
} else
  echo "Brak wyników";
 
$prods->closeCursor();
[PHP] pobierz, plaintext 

Chodzi mi o odpowiednik mysql_close();

I jeszcze jedno. Jak mam wyświetlać błędy? tak?

[PHP] pobierz, plaintext 
} catch(PDOException $e) {
	echo 'Error: ' . $e->getMessage(). "<br/>";
	$pdo->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION );
  //die();
}
[PHP] pobierz, plaintext 

Jak nie zamkniesz połączenia to nic się stanie. Co do błędów, to można też tak:

[PHP] pobierz, plaintext 
$stmt = $pdo->prepare("SELECT....");
if(!$stmt->execute()) print_r($stmt->errorInfo());
[PHP] pobierz, plaintext 

działa tylko w przypadku prepare-execute

A nie można tego jakoś w konfiguracji wklepać?

Można. Uwzględnieniem ci to specjalnie w poście 7. błędy przechwytuj przez wyjątki. Bo i możesz logowanie ich dodać, i też od razu będziesz w stanie obsłużyć transakcje. Rollback i commit.

A kodowanie nie ma opcji żeby nie działało. Set names + utf8 w edytorze + nagłówki. Jeśli jest inaczej to znaczy ze znaki były np w latin1 błędnie zapisane i działało przez sumę błędów :-)

Trochę się tu pogubiłem, z czym masz problem teraz?
I btw. jeśli chcesz mieć polskie znaki zamiast krzaków - spójrz na mój kod. U mnie zawsze są polskie znaki.

Jest to kawałek kodu mojej klasy.

[PHP] pobierz, plaintext 
Class Itemshop
{
	private $host_db = "localhost"; 
	private $name_db = "test";
	private $port_db = "3306";
	private $user_db = "";
	private $pswd_db = "";
	public $conn;
	public $error;
	public function __construct()
	{
		$this->Connect();
	}
	public function Connect()
	{
		try
		{
			$pdo = new PDO('mysql:host=localhost;port=3306;dbname=test', '', '', array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'UTF8'"));
			$pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			if($pdo)
			{
				$this->conn = $pdo;
				return true;
			}
		}
		catch(PDOException $e)
		{
			echo "Błąd: " . $e->getMessage();
		}
}
	}
[PHP] pobierz, plaintext 

[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 

skąd tam Ci się bierze encoding=utf8 ?

Stąd: wikibooks.org/wiki/PHP/Biblioteka_PDO

Dobra, czas na INSERT. Przykład z MySQL:

[PHP] pobierz, plaintext 
mysql_query("INSERT INTO g_a_topics (a_id, title) VALUE (".$User['id_a'].", '".$_POST['title']."')");
$id = mysql_insert_id();
[PHP] pobierz, plaintext 

i PDO:

[PHP] pobierz, plaintext 
$b = $pdo->prepare("INSERT INTO g_a_topics (`id_a`,`title`) VALUES (:id_a, :title)");
$b->bindValue(":id_a", $User['id_a']);
$b->bindValue(":title", $_POST['title']);
$b->execute();
 
$id = $pdo->lastInsertId();
[PHP] pobierz, plaintext 

Lub inaczej:

[PHP] pobierz, plaintext 
$b = $pdo->prepare("INSERT INTO g_a_topics SET id_a=?, title=?");
 
$b->bindValue(1, $User['id_a']);
$b->bindValue(2, $_POST['title']);
$b->execute();
 
$id = $pdo->lastInsertId();
[PHP] pobierz, plaintext 

Nie działają obydwa przykłady.

Brakuje trzeciego parametru w bindowaniu: PDO::PARAM_STR / PDO::PARAM_INT itd.


INSERT INTO g_a_topics SET id_a=?, title=?
od kiedy tak insert wygląda?

Ale nie działają w sensie, że zapytanie się wcale nie wykonuje, czy tylko nie zwraca żadnego ID?

[PHP] pobierz, plaintext 
var_dump($b->errorInfo());
[PHP] pobierz, plaintext 

coś konkretnego pokazuje?

@Turson


bindValue przyjmuje domyślnie PDO::PARAM_STR jako trzeci parametr



http://dev.mysql.com/doc/refman/5.6/en/insert.html

Właśnie nic nie pokazuje. Strona się przeładowuje. tak ma być?:

[PHP] pobierz, plaintext 
$b->bindValue(":id_a", $User['id_a'], PDO::PARAM_INT);
$b->bindValue(":title", $_POST['title'], PDO::PARAM_STR);
 
$b->bindValue(1, $User['id_association'], PDO::PARAM_INT);
$b->bindValue(2, $_POST['title'], PDO::PARAM_STR);
[PHP] pobierz, plaintext 

Przykłady znalazłem: PHP_MYSQL_PDO_-_kompendium www.mpcforum.pl/topic/876926-tut-poradnik-pdo/

A masz AUTO_INCREMENT ustawione na kolumnie `id_a`? Bo lastInsertId() zadziała tylko dla kolumn AUTO_INCEREMENT.

[edit]
To zapewne literówka, ale tak gwoli ścisłości...

W przykładzie MYSQL wstawiasz do kolumny o nazwie `a_id`, a w PDO do kolumny `id_a`

Z tym że jeśli kolumna by nie istniała to errorInfo() musiałby jakiś konkretny błąd pokazać...

Tak mam, problem, że w ogóle nie dodaje nic do bazy.

OK, znalazłem babola. id_a != a_id

I to:

[PHP] pobierz, plaintext 
var_dump($b->errorInfo());
[PHP] pobierz, plaintext 

nie pokazuje żadnego błędu? Tzn. pierwszy element tablicy ma same zera? Z tym że to ma być wywołane zaraz PO execute() i oczywiście chodzi mi o PDOStatement::errorInfo(), nie o PDO::errorInfo().

[edit]
a jednak

Właśnie nie wiem dlaczego nie zwraca mi żadnych błędów. Musiałem wyciąć kawałek kody i przenieść do innego pliku i dopiero wtedy wywaliło błąd.

Jak mam dodać datę w bindowaniu?

[PHP] pobierz, plaintext 
$p->bindValue(":data", now(), PDO::PARAM_INT);
[PHP] pobierz, plaintext 

Binduje się wartości przekazywane do PDO a nie funkcje, już wiem

data np.strtotime i robisz PARAM_STR nie int, int-em bedzie timestamp

A gdybyś się zastanawiał, tak jak ja kiedyś, co właściwie robi PDO::PARAM_* masz odpowiedź http://stackoverflow.com/questions/833510/...ow-does-it-work Jak widać niewiele robi.

NOW() traktuj jako STR a nie INT no i możesz bezpośrednio w zapytaniu NOW dać bez bindowania

On nie moze.... on MUSI. Funkcji mysql nie da sie zbindowac.

Mam problem z takim kodem:

[PHP] pobierz, plaintext 
$atts = mysql_query("SELECT * 
                                FROM g_atts 
				INNER JOIN g_ts
				ON target=t 
				INNER JOIN g_events e 
				ON e.number=att
				WHERE (e.type = 10 OR e.type = 11)
				AND e.t_id=".$User['id_t']);
if(mysql_num_rows($atts) > 0) {
  // ...
  $ref = $atts[0]['finish'];
  while($att = mysql_fetch_array($atts)) {
[PHP] pobierz, plaintext 

Rozumiem, że dostawałem tablicę. A tera jak mam to sprawdzić? Jak robię tak:

[PHP] pobierz, plaintext 
$r = $pdo->query("SELECT * 
                                 FROM g_att 
                                 INNER JOIN g_ts 
                                 ON target=t
                                 INNER JOIN g_events e 
                                 ON e.number=att
                                 WHERE (e.type = 10 OR e.type = 11)
                                 AND e.town_id=".$User['id_t']);
$atts = $r->fetch(PDO::FETCH_ASSOC);
if($atts) {
  //
  $ref = $atts[0]['finish'];
  while ($att = $atts->fetch()) {
[PHP] pobierz, plaintext 

To var_dump($attacks); zwraca bool(false).

O co w zasadzie pytasz? Odpowiednikiem mysql_num_rows jest $r->rowCount()
var_dump($attacks); zwraca bool(false). - nie widzę $attacks

Pisz proszę jak na opiekuna przystało, a nie "mam problem, pomóżcie"

Źle wkleiłem, poprawione
Chodzi o to, że w 1 zapytaniu zwraca np:
resource(3) of type (mysql result) Resource id #3
a w PDO zwraca zapytanie.

Napisałem, że dostawałem tablicę, to znaczyło by, że teraz jej nie otrzymuję ;p

[PHP] pobierz, plaintext 
$r = $pdo->query("SELECT * 
                                 FROM g_att 
                                 INNER JOIN g_ts 
                                 ON target=t
                                 INNER JOIN g_events e 
                                 ON e.number=att
                                 WHERE (e.type = 10 OR e.type = 11)
                                 AND e.town_id=".$User['id_t']);
 
if($r->rowCount()>0) {
  $tablica = $r->fetchAll(PDO::FETCH_ASSOC);
  var_dump($tablica);
  foreach($tablica as $v){
    //
  }
}
else echo "Brak";
[PHP] pobierz, plaintext 

Czyli to:

[PHP] pobierz, plaintext 
$tt = mysql_fetch_array(mysql_query("SELECT name FROM g_ts WHERE t=".$_GET['att']." LIMIT 1"));
$tt = $tt[0];
[PHP] pobierz, plaintext 

Mam zapisać tak:

[PHP] pobierz, plaintext 
$r = $pdo->query("SELECT name FROM g_ts WHERE t=".$_GET['att']." LIMIT 1"));
$tt = $r->fetchAll(PDO::FETCH_ASSOC);
[PHP] pobierz, plaintext 

Tak, w $tt będziesz miał wszystkie wyniki, choć w przypadku "LIMIT 1" samo fetch(). Nawias za dużo skopiowałeś w zapytaniu

Chyba musisz trochę do dokumentacji przysiąść, w szczególności do zwracanych typów
http://www.php.net/manual/en/class.pdo.php
Zwróć uwagę że masz metody ogólne obiektu PDO które w obsługują adapter db czyli transakcje, błędy i przygotowanie zapytań. Oraz PDOStatement które to ma za zadanie obsługę już samego zapytania (kursora - co za tym idzie wyświetlania wyników, bindowanie parametrów). I np za dokumentacją:

[PHP] pobierz, plaintext 
public [b]PDOStatement[/b] prepare ( string $statement [, array $driver_options = array() ] )
public [b]PDOStatement[/b] query ( string $statement )
[PHP] pobierz, plaintext 

więc jeśli gdzieś masz query to musisz stosować metody z http://www.php.net/manual/en/class.pdostatement.php

2 uwagi. Spróbuj używać wyjątków albo dla całej aplikacji, albo chociaż samego zapytania bo wcześniejsze problemy wynikały w dużej mierze z ich braku. PDOException nie było nigdzie łapane, nie wiedziałeś co się dzieje. Nie wiem czy podajesz tylko do wglądu ale nie używasz prepare więc te zapytania nie są ani bezpieczne, ani nie ma żadnych zalet nad zwykłym mysql_.
I w sumie trzecia jeszcze. http://www.php.net/manual/en/pdostatement.rowcount.php

[PHP] pobierz, plaintext 
$r = $pdo->query("SELECT * 
                                 FROM g_att 
                                 INNER JOIN g_ts 
                                 ON target=t
                                 INNER JOIN g_events e 
                                 ON e.number=att
                                 WHERE (e.type = 10 OR e.type = 11)
                                 AND e.town_id=".$User['id_t']);
 
if($r->rowCount()>0) {
  $tablica = $r->fetchAll(PDO::FETCH_ASSOC);
  var_dump($tablica);
  foreach($tablica as $v){
    //
  }
}
else echo "Brak";
[PHP] pobierz, plaintext 

Jak takie cudo miałbym zapisać w pętli while? Do czego to porównać?


Nie kumam tego przykładu, konkretnie:

[PHP] pobierz, plaintext 
// przygotowanie zapytania
$zapytanie = $baza->prepare("SELECT login, email FROM gracze WHERE level = :poziom AND klasa= :klasa");
// filtracja wskazanych danych
$zapytanie->bindValue(':poziom', $_POST['poziom'], PDO::PARAM_INT);
$zapytanie->bindValue(':klasa', trim($_GET['klasa']), PDO::PARAM_STR);
// właściwe wykonanie
$zapytanie->execute();
 
if ($gracze->rowCount() > 0) {
	while ($gracz = $zapytanie->fetch()) {
		echo 'Gracz: '.$gracz['login'].' ma adres emaill: '.$gracz['email'].'<br/>';
	}
} else echo 'Brak graczy o podanych parametrach..';
[PHP] pobierz, plaintext 

if ($gracze - skąd wytrzepał $gracze?

Przecież to da:
Notice: Undefined variable: gracze in
A w efekcie:
Fatal error: Call to a member function rowCount() on a non-object in

Zamiast $gracze powinno być $zapytanie, widocznie pomyłka