Forum PHP.pl > [mysql] Przejscie na PDO, najlepszy sposób?

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [mysql] Przejscie na PDO, najlepszy sposób?

starterrrrr

3.08.2016, 18:23:05

Witam.

Zarządzam aplikacją, która korzysta z przestażałych już funkcji mysql w php.
Chciałem przejśc na PDO, prosże o podpowiedź jak najszybciej to można zrobić.

Do każdego pliku dołączany jest plik connect.php, który zawiera funkcje połączenia z bażą danych:

[PHP] pobierz, plaintext 
//PLIK: CONNECT.PHP
 
function connection($wybrana_baza = '') {
 
  global $dir_functions;
  //$wybrana_baza = '_public';
 
  include($dir_functions.'/config'.$wybrana_baza.'.php'); 
	global $conn;
 
    if($conn) {
        return $conn;
    } else {
        $conn = mysql_connect($dbhost,$dbuser,$dbpass);
        if (!$conn || !mysql_select_db($db,$conn)) {
            return 0;
        } else {
		mysql_query('SET NAMES utf8');
		mysql_query('SET CHARACTER SET utf8');
		mysql_query("SET collation_connection = utf8_polish_ci");
		return $conn;
        }
    }
}
 
function sql($query) {
	global $zap; 
    if(!($conn = connection($wybrana_baza))) {
        return 0;
 
    } else {
        $result = mysql_query($query,$conn);
 
        if(!$result) {
            print('<DIV>Wystąpił błąd:<BR>['.$query.']<BR>['.mysql_error().']</DIV>');
        } else {
            return $result;
        }
    }
}
 
//PLIK: CONFIG.PHP:
 
 
$dbhost = '....';
$dbuser = '....';
$dbpass = '...';
$db = '....'; 
$dbprefix = '....';
 
[PHP] pobierz, plaintext

Następnie w kodzie korzystam z funkcji sql(). Przykłady:

[PHP] pobierz, plaintext 
//update
sql("UPDATE ".$dbprefix."news SET zablokuj  = '1' WHERE id = '".del($zmienna)."';");
 
//insert
$max_id = oblicz_max('news', 'id');
sql("INSERT INTO ".$dbprefix."news (id, tytul, rozwiniecie) VALUES ('".$max_id."','".del($tytul)."','".del($rozwiniecie)."')");
 
 
//select
$result = sql("SELECT * FROM ".$dbprefix."news WHERE id='".del($id)."' ORDER BY $sort $jak LIMIT $start,$limit");
while($row = mysql_fetch_array($result)) {
 
	print(' - '. $row['id'].' '.$row['rozwiniecie']);
 
}
[PHP] pobierz, plaintext

Jak to można najlepiej zrobić? Warto modyfikowac funkcje sql(), czy po prostu, po kolei kolejne moduły przerabiac na PDO i chwilowo korzystac z obu mechanizmów?

Prosze o jakaś podpowiedź, szukałem na forach, ale nie znalazłem konkretnych przypadków. Troche sie obawiam, że bede musiałz dwa tygodnie siedziec nad tym :/

nospor

3.08.2016, 18:26:32

Jesli wszedzie korzystasz z funkcji connection() i sql() to na razie wystarczy ze tylko tam zmienisz, czyli 5 minut roboty

A z czasem mozesz sobie to rozbudowac o pelne wykorzystanie PDO jak bindowanie

Lion

3.08.2016, 19:08:12

Oczyść obecny kod z zbędnych instrukcji, a następnie opakuj swoją funkcją connection PDO. Takie rozwiązanie szybko pozwoli Ci uruchomić aplikację, jednak stracisz takie funkcjonalności jak ochrona przez SQL injection (poprzez brak ww. bindowania) oraz prepared statements które mogłoby przyśpieszyć wykonywanie zapytań. Możesz coś znaleźć w Internecie wyszukując "PDO wrapper".

viking

3.08.2016, 19:27:50

Albo najłatwiej na szybko zastąpić ten kod strukturalnym mysqli a w przyszłości pomyśleć jak to przepisać poprawnie.

nospor

3.08.2016, 19:57:27

@viking ale po co? Przeciez on musi to zmienic tylko w dwoch funkcjach to niech zmieni od razu na PDO.

viking

3.08.2016, 20:12:31

Bo choćby struktura funkcji sql sugeruje że to nie jest cały kod. Pisanie tego dalej w taki sposób to robienie jeszcze większego śmietnika. IMO lepiej teraz szybko wymienić zło na trochę mniejsze zło niż brnąć dalej w taki bajzel.

starterrrrr

3.08.2016, 20:33:17

Cytat(Lion @ 3.08.2016, 20:08:12 )

tak zrobie, z czasem bede poszczególne elementy zastępował zapytaniami z bindowaniem i prepared statements.

Funkcje sql() przerobiłem tak:

[PHP] pobierz, plaintext 
function sql($query) {
 
    global $db;
    global $dbuser;
    global $dbpass;
    global $dbhost;
 
    $db = new PDO('mysql:host='.$dbhost.';dbname='.$db, $dbuser, $dbpass, array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'") );
    try {
        $stmt = $db->prepare( $query );
        $stmt->execute();
 
    }
    catch (PDOException $e)
    {
        echo $e->getMessage();
        ///die();
    }
 
}
[PHP] pobierz, plaintext

Wydaje mi się, że coś jest nie tak, funkcja powinna zwracać chyhba jakas wartość.

Musze zamienić $row = mysql_fetch_array($result) na $row = $stmt->fetch(PDO::FETCH_ASSOC);

------------ EDIT -----

WERSJA 2

[PHP] pobierz, plaintext 
function sql($query) {
 
    global $db;
    global $dbuser;
    global $dbpass;
    global $dbhost;
 
    $db = new PDO('mysql:host='.$dbhost.';dbname='.$db, $dbuser, $dbpass, array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'") );
    try {
 
        $result = $db->query( $query ) ;
        return $result; 
    }
    catch (PDOException $e)
    {
        echo $e->getMessage();
        ///die();
    }
 
}
 
 
//wcześniej dla SELECT
$result = sql("SELECT ... ");
while($row = mysql_fetch_array($result)) { ... }
 
//teraz
$result = sql("SELECT ... ");
while($row = $result->fetch ()) { ... }
 
 
//dla UPDATE, DELETE trzeba bedzie zmodyfikowac kod w całej aplikacji.
$result = sql(" UPDATE ...");
$result->execute(); //tutaj nie jestem pewien
[PHP] pobierz, plaintext

viking

4.08.2016, 05:37:22

Ja bym użył metodę 1 z drobną modyfikacją. Do funkcji przekazuj ($query, array $params = null) i $params może być przekazane do execute(). Dzięki temu miałbyś bindowanie. Funkcja może zwracać $stmt.

nospor

4.08.2016, 09:59:11

Cytat

Tak, masz racje. Zupelnie nie zauwazylem ze pozniej po sql() jest zwykle mysql_fetch_array.

starterrrrr

4.08.2016, 22:31:20

Cytat(nospor @ 4.08.2016, 10:59:11 )

Tak, masz racje. Zupelnie nie zauwazylem ze pozniej po sql() jest zwykle mysql_fetch_array.

Czyli jak byście proponowali przerobić całość?

nospor

5.08.2016, 07:41:51

Tak jak napisal viking: zastapic to libem mysqli w wersji strukturalnej- zmiany beda male i szybko powinienies to zmienic. A pozniej na spokojnie stopniowo przerzucic sie na PDO

starterrrrr

23.08.2016, 00:43:04

Witam.
Dziekuje za podpowiedzi, zrobiłem tak jak viking kazał, przeszedłem najpierw na MySQLi a z czasem bede migrował na PDO.

Jest jednak pewien problem, przestał działac mechanizm SQL_CALC_FOUND_ROWS przy wykorzystaniu funkcji MySQLi w wersji proceduralnej.
Szukałem na forach zagranicznych, jest pełno podpowiedzi, ale nie dziąłają żadne.

[PHP] pobierz, plaintext 
 
function sql($query) {
    global $dbhost; 
    global $dbuser; 
    global $dbpass; 
    global $db; 
 
    $link = mysqli_connect($dbhost, $dbuser, $dbpass, $db);
    mysqli_set_charset($link,"utf8");
 
    if (mysqli_connect_errno()) {
        printf("Błąd połączenia z bazą: %s\n", mysqli_connect_error());
        exit();
    }
 
    $result = mysqli_query($link, $query);
 
    if (!$result) {
        printf("Błąd bazy danych: %s\n", mysqli_error($link));
    }
 
    return $result;
    mysqli_close($link);
}
 
//zapytanie 
$result = sql(" SELECT SQL_CALC_FOUND_ROWS, * FROM tabela WHERE ... LIMIT 20-60;");
while ($row = mysql_fetch_assoc($result)) {
 ...
}
 
$result_ilosc = sql("SELECT FOUND_ROWS() as ilosc_pozycji;"); //obliczamy ilosc pozycji
$total        = mysqli_fetch_assoc($result_ilosc);
$l_odp        = $total['ilosc_pozycji'];
 
///$l_odp - nie zwraca wartosci :/
[PHP] pobierz, plaintext

Powyżej przedstawiłem jak wygląda mój kod, nie stety nie zwraca mi ilości wszystkich wpisów.

trueblue

23.08.2016, 07:07:01

Każde zapytanie otwiera nowe połączenie z bazą.

mmmmmmm

28.08.2016, 21:50:18

Nie podaje ci ilości, bo wcześniej się wywala errorem.
W tej linii:
while ($row = mysql_fetch_assoc($result)) {

LIMIT 20-60 też jest fajne...

starterrrrr

25.10.2016, 13:17:43

Witam Ponownie. PDO jest juz w podstawowym wymiarze opanowane, jednak daje się odczuć brak doświadczenia w tym mechaniźmnie.

Mam zapytanie SELECT, które posiada warunki w klauzuli WHERE.
Przed tym zapytaniem SQL w kodzie php mam kilka waruinków, które w zależności od zmiennej, wstawiaja dodatkowe warunki do zapytania, jak takie coś bindować?

[PHP] pobierz, plaintext 
 
$dodatek_sql = '';
if ($dodaj_kod == 1) {
    $dodatek_sql = ' AND sprawdzony=$sprawdzony ';
}
 
$stmt = $db->prepare(  
    "SELECT * FROM tabela WHERE login=:login AND haslo=:haslo  $dodatek_sql "  
);  
$stmt->bindValue(':login', $login, PDO::PARAM_STR);  
$stmt->bindValue(':haslo', $haslo, PDO::PARAM_STR);  
$stmt->execute();  
$uzytkownicy = $stmt->fetchAll(PDO::FETCH_ASSOC);  
[PHP] pobierz, plaintext

Jak przebindować zmienną $dodatek_sql, albo $sprawdzony, czy cala czesc warunku.

Jeszczejedno pytanie. Czy za kazdym razem musze takie zapytanie dawac w TRY {} CATCH zeby przejmowac bledy, czy zrobic jakas funkcje, do ktorej bede przekazywal parametyry laczenia?

Tomplus

25.10.2016, 18:18:20

Wykonaj prepare() dużo wcześniej, a pod prepare dodaj do warunku kolejny bind.

[PHP] pobierz, plaintext 
if ($dodaj_kod == 1) {
    $dodatek_sql = ' AND sprawdzony = :sprawdz ';
    $stmt->bindValue(':sprawdz', $sprawdzony, PDO::PARAM_BOOL); //_INT
}
[PHP] pobierz, plaintext

starterrrrr

26.10.2016, 10:29:31

Cytat(Tomplus @ 25.10.2016, 19:18:20 )

Wykonaj prepare() dużo wcześniej, a pod prepare dodaj do warunku kolejny bind.

[PHP] pobierz, plaintext 
if ($dodaj_kod == 1) {
    $dodatek_sql = ' AND sprawdzony = :sprawdz ';
    $stmt->bindValue(':sprawdz', $sprawdzony, PDO::PARAM_BOOL); //_INT
}
[PHP] pobierz, plaintext

To bedzie problemem, w skrypcie jest ze 40 zapytań w jednym pliku php, narobi sie balagan :/

Myślałem, żeby cały ten kawalek kodu bindowac:

[PHP] pobierz, plaintext 
 
$dodatek_sql = '';
if ($dodaj_kod == 1) {
    $dodatek_sql = ' AND sprawdzony="'.$sprawdzony.'" ';
}
 
$stmt = $db->prepare(  
    "SELECT * FROM tabela WHERE login=:login AND haslo=:haslo  :dodatek_sql "  
);  
$stmt->bindValue(':login', $login, PDO::PARAM_STR);  
$stmt->bindValue(':haslo', $haslo, PDO::PARAM_STR);  
$stmt->bindValue(':dodatek_sql', $dodatek_sql, PDO::PARAM_STR);  //czy takie coś przejdzie?
 
 
$stmt->execute();  
$uzytkownicy = $stmt->fetchAll(PDO::FETCH_ASSOC);  
 
[PHP] pobierz, plaintext

trueblue

26.10.2016, 11:26:00

To możesz tak zrobić:

[PHP] pobierz, plaintext 
$stmt = $db->prepare(  
    "SELECT * FROM tabela WHERE login=:login AND haslo=:haslo AND IF(:dodaj_kod,sprawdzony=:sprawdz,TRUE)"
);  
$stmt->bindValue(':login', $login, PDO::PARAM_STR);  
$stmt->bindValue(':haslo', $haslo, PDO::PARAM_STR);  
$stmt->bindValue(':sprawdz', $sprawdzony, PDO::PARAM_STR);  
$stmt->bindValue(':dodaj_kod', $dodaj_kod, PDO::PARAM_INT);
[PHP] pobierz, plaintext

mar1aczi

26.10.2016, 11:58:43

Cytat(starterrrrr @ 23.08.2016, 01:43:04 )

[PHP] pobierz, plaintext 
...
    return $result;
    mysqli_close($link);
...
[PHP] pobierz, plaintext

Po "trafieniu" na return już chyba mysqli_close() nie zaskoczy.

Tomplus

26.10.2016, 12:34:53

Ja korzystam z rozwiązania z tej klasy:
https://github.com/indieteq/indieteq-php-my...-database-class

Dzięki temu parametry do bindValue możesz przekazywać jako parametr funkcji tudzież metody.

[PHP] pobierz, plaintext 
$bind = ["firstname"=>"John","id"=>"1"];
$person   =  $db->query("SELECT * FROM Persons WHERE firstname = :firstname AND id = :id",$bind);
$person   =  $db->query("SELECT * FROM Persons WHERE firstname = :firstname AND id = :id",array("firstname"=>"John","id"=>"1"));
[PHP] pobierz, plaintext

luis2luis

28.10.2016, 10:45:12

Cytat(Tomplus @ 26.10.2016, 13:34:53 )

Ja korzystam z rozwiązania z tej klasy:
https://github.com/indieteq/indieteq-php-my...-database-class

Dzięki temu parametry do bindValue możesz przekazywać jako parametr funkcji tudzież metody.

[PHP] pobierz, plaintext 
$bind = ["firstname"=>"John","id"=>"1"];
$person   =  $db->query("SELECT * FROM Persons WHERE firstname = :firstname AND id = :id",$bind);
$person   =  $db->query("SELECT * FROM Persons WHERE firstname = :firstname AND id = :id",array("firstname"=>"John","id"=>"1"));
[PHP] pobierz, plaintext

Fajnie ta klasa wyglada, widze, ze nie aktualizowana 2 lata, ale raczej php 7 nie powinno niczym zaskoczyc nowym w tej kwestii.

Tomplus

28.10.2016, 17:40:49

Na PHP7 działa bez problemu.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.