Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP][PDO] Zabezpieczanie zapytań
Forum PHP.pl > Forum > Przedszkole
Kerth
Witajcie smile.gif

Podobno, każde zapytania powinno się zabezpieczać za pomocą bindparam() czy jakoś tak. Może mi podać ktoś przykład takiego zapytania? Np na moim kodzie:

  1. $register_zadania = $db->exec("INSERT INTO `zadania` (`nick`, `ip`, `data`, `nazwa` ) VALUES ('$nick', '$ip', '$data', '$nazwa' )") or die(mysql_error());
Crozin
Wiesz, że dane należy podpinać przez bindValue/bindParam, więc dlaczego nie sprawdzisz bezpośrednio w dokumentacji: http://www.php.net/manual/en/pdostatement.bindvalue.php
Kerth
  1. $register = $db->exec("INSERT INTO `gracze` (`nick`, `ip`, `email`, `haslo`, `haslo2`, `data`, `nazwa` ) VALUES (?,?,?,?,?,?,?)") or die(mysql_error());
  2.  
  3. $register->bindValue(1, $nick, PDO::PARAM_STR);
  4. $register->bindValue(2, $ip, PDO::PARAM_INT);
  5. $register->bindValue(3, $email, PDO::PARAM_STR);
  6. $register->bindValue(4, $haslo, PDO::PARAM_STR);
  7. $register->bindValue(5, $haslo2, PDO::PARAM_STR);
  8. $register->bindValue(6, $data, PDO::PARAM_INT);
  9. $register->bindValue(7, $nazwa, PDO::PARAM_STR);


Skorzystałem z innego zapytania. Może mi ktoś powiedzieć co jest nie tak? Bo niestety nie działa mi ten kod.
Turson
Spójrz jeszcze raz na przykład w manualu i porównaj ze swoim...
PawelC
  1. $result = $this->db->prepare("insert into category values(NULL, :name)");
  2. $result->bindParam(':name', $name, PDO::PARAM_STR);

U mnie np tak to wygląda smile.gif W manualu masz bardzo dobre przykłady co i jak zrobić smile.gif To co zrobiłeś to masło maślane.
Kerth
Mam teraz tak:

  1. $register = $db->prepare("INSERT INTO `gracze` (`nick`, `ip`, `email`, `haslo`, `haslo2`, `data`, `nazwa` ) VALUES(nick, :ip, :email, :haslo, :haslo2, :data, :nazwa )") or die(mysql_error());
  2. $register->bindValue(":nick", $nick, PDO::PARAM_STR);
  3. $register->bindValue(":ip", $ip, PDO::PARAM_STR);
  4. $register->bindValue(":email", $email, PDO::PARAM_STR);
  5. $register->bindValue(":haslo", $haslo, PDO::PARAM_STR);
  6. $register->bindValue(":haslo2", $haslo2, PDO::PARAM_STR);
  7. $register->bindValue(":data", $data, PDO::PARAM_INT);
  8. $register->bindValue(":nazwa", $nazwa, PDO::PARAM_STR);
  9. $register->execute();



Powoduje to:
  1. Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in C:\WebServ\httpd\strona4\register.php on line 205


Korzystam z tego: http://turson.pl/blog/php-mysql-prosty-skr...do/#comment-766

Oraz możecie mi powiedzieć czy dobrze przepisałem 3 argument bindValue dla każdego z pól? Czy data ma mieć PDO::PARAM_INT itd?
PawelC
  1. VALUES(nick,

Błąd

Do podglądu zobacz w pełni działajacy kod:
  1. public function add($name)
  2. {
  3. $result = $this->db->prepare("insert into category values(NULL, :name)");
  4. $result->bindParam(':name', $name, PDO::PARAM_STR);
  5.  
  6. if(!$result->execute()){
  7. throw new Exeption("Wystąpił błąd podczas dodawania kategorii");
  8. }
  9. }
Turson
Datę jako date/datetime traktuj jako string, dla timestamp to będzie int
Kerth
A to NULL? to jest z pola np. `id` AUTO_INCREMENT?
Turson
Jeżeli w INSERT gracze(kolumny) nie pomijasz id, to null będzie dla auto increment
Kerth
Aha to rozumiem. A jak jest z zapytaniami typu UPDATE oraz SELECT?

UPDATE:

  1. $kasa_start = 1000;
  2. $register_kasa_start = $db->prepare("UPDATE `kasa` FROM `gracze` = :kasa_start WHERE `Nick` = :nick") or die(mysql_error());
  3. $register_kasa_start->bindValue(":nick", $nick, PDO::PARAM_STR);
  4. $register_kasa_start->bindValue(":kasa_start", $kasa_start, PDO::PARAM_INT);
  5. $register_kasa_start->execute();



Czy robi się to tak czy jakoś inaczej?
PawelC
Jeżeli jest update to wpisujesz tylko te kolumny które aktualizujesz + warunek where.

Masz złe zapytanie, jest update nazwa tabeli set kolumny, czyli
  1. UPDATE `gracze` SET `kasa` = :kasa_start WHERE `Nick` = :nick") or die(mysql_error());
Kerth
Ale gdy chcę tak jak mam powyżej, że chcę dodać po 1000 dla każdego zarejestrowanego?
PawelC
Wtedy wpisujesz bez warunku where smile.gif
Czyli:
  1. UPDATE `gracze` SET `kasa`= :kasa
Kerth
Wiem, że może trochę te pytania nudne jak flaki z olejem ale dlaczego to mi nic nie dodaje?

  1. $kasa_start = 1000;
  2. $register_kasa_start = $db->prepare("UPDATE `gracze` SET `kasa`= :kasa_start") or die(mysql_error());
  3. $register_kasa_start->bindValue(":kasa_start", $kasa_start, PDO::PARAM_INT);
  4. $register_kasa_start->execute();
PawelC
Do wglądu masz kolejny kod, zobacz różnice smile.gif
  1. $results=$this->db->prepare("update users set pass=:pass where mail=:mail");
  2. $results->bindParam(":pass", $pass, PDO::PARAM_STR);
  3. $results->bindParam(":mail", $mail, PDO::PARAM_STR);
  4.  
  5. if(!$results->execute()){
  6. throw new Exception("Wystąpił błąd podczas resetowania hasła.");
  7. }
ZaXaZ
Pierwszy raz widzę żeby ktoś robił or die w pdo do tego mieszając z spl ;d, zajrzyj może do wiki tam dość dobrze wytłumaczone pdo...
  1. $kasa_start = 1000;
  2. $register_kasa_start = $db->prepare('UPDATE `gracze` SET `kasa`= :kasa_start');
  3. $register_kasa_start->bindValue(':kasa_start', $kasa_start, PDO::PARAM_INT);
  4. $register_kasa_start->execute();


poczytaj np. o metodzie errorInfo() w manualu.
ber32
Tu masz link do książki, która opisuje te i inne działania na pdo i jeszcze parę innych żeczy. Książka tania 12,50zł "pdf" polecam.

http://ebookpoint.pl/ksiazki/sto-pytan-i-o...00.htm#format/e
Turson
Jest też darmowa "książka" - http://pl.wikibooks.org/wiki/PHP/Biblioteka_PDO
Nie wiem jak można by płacić za książkę do PDO, gdzie nawet najbardziej nieogarnięta osoba, która miała doświadczenie z mysql_ ogarnie to do godziny
ber32
Turson sprawdź o której Kerth napisał pierwszego posta, a o której jeszcze pytał dalej, to trochę więcej jak godzina i przypuszczam że już wcześniej miał doczynienia z mysql_
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.