Forum PHP.pl

Witam,

Proszę o sprawdzenie czy zapis mojej klasy z zapytaniami SQL jest poprawny pod względem OOP, co można by zmienić, uproscić itd. Potrzebuję takiej klasy aby zrobić sobie podstawowe zapytania CRUD dla obsługi jednej tabeli bazy danych. Dobrze się do tego zabralem ?

[PHP] pobierz, plaintext 
class MyQueries extends PDO
{
 
   public function __construct()
   {
      parent::__construct('mysql:host=localhost;dbname=mydatabase', 'login', 'password');
   }
 
   public static function getCities()
   {
      try {
         $conn = new MyQueries();
         $stmt = $conn->prepare('SELECT * FROM city');
         $stmt->execute();
      } catch (PDOException $e) {
         echo 'ERROR: ' . $e->getMessage();
      }
 
      return $stmt;
   }
 
   public static function create(){
      // code
   }
 
   public static function delete($id){
      // code
   }
 
}
[PHP] pobierz, plaintext 

dzięki

Ten post edytował damianooo 26.06.2013, 18:11:58

Źle. Wydziel model odpowiedzialny za łączenie z bazą. Wywal static. Obecnie jest to zbiór funkcji zamkniętych w klasie. Nie ma to nic wspólnego z oop.

Czy tak będzie lepiej ? ... nie wiem jak z tym rozdzieleniem połączenia z bazą danych. Można prosić o podpowiedź. Bo teraz w tej postaci to mi nie działa.

Teraz mam tak:

[PHP] pobierz, plaintext 
class DBObject extends PDO
{
 
   public function __construct()
   {
      parent::__construct('mysql:host=localhost;dbname=database', 'login', 'password');
   }
 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
class City extends DBObject
{
   public $name;
   protected $country;
 
   public function __construct($name, $country)
   {
      $this->setName($name);
      $this->setCountry($country);
   }
 
   public function setName($name)
   {
      $this->name = $name;
      return $this;
   }
 
   public function getName()
   {
      return $this->name;
   }
 
   public function setCountry(Country $country)
   {
      $this->country = $country;
      return $this;
   }
 
   public function getCountry()
   {
      return $this->country;
   }
 
   public function save()
   {
      $stmt = $this->prepare('INSERT INTO city VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => $this->country_id));
   }
 
   public function load()
   {
      $stmt = $this->prepare('SELECT c.city_id,c.name,cn.name as country FROM city c INNER JOIN country cn ON c.country_id = cn.country_id');
      $stmt->execute();
   }
 
}
[PHP] pobierz, plaintext 

Bardzo proszę o znaleznienie błędów i poprawienie mnie.
Dzięki

Ten post edytował damianooo 27.06.2013, 22:16:43

zrób nową klasę od łączenia z bazą danych i przekaż jej obiekt jako parametr do konstruktora np:

[PHP] pobierz, plaintext 
$db = new database(new connection());
[PHP] pobierz, plaintext 

czy tak? :

[PHP] pobierz, plaintext 
class DBObject extends PDO
{
   public function __construct()
   {
      parent::__construct('mysql:host=localhost;dbname=database', 'login', 'password');
   }
}
[PHP] pobierz, plaintext 

i wtedy tak:

[PHP] pobierz, plaintext 
class City extends DBObject
{
   public $name;
   protected $country;
   public $db;
 
   public function __construct($name, $country)
   {
      $this->db = new DBObject();
      $this->setName($name);
      $this->setCountry($country);
   }
 
   public function setName($name)
   {
      $this->name = $name;
      return $this;
   }
 
   public function getName()
   {
      return $this->name;
   }
 
   public function setCountry(Country $country)
   {
      $this->country = $country;
      return $this;
   }
 
   public function getCountry()
   {
      return $this->country;
   }
 
   public function save()
   {
      $stmt = $this->db->prepare('INSERT INTO city VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => $this->country_id));
   }
 
   public function load()
   {
      $stmt = $this->d->prepare('SELECT c.city_id,c.name,cn.name as country FROM city c INNER JOIN country cn ON c.country_id = cn.country_id');
      $stmt->execute();
   }
 
   public function update()
   {
      // code here
   }
 
   public function delete()
   {
      // code here
   }
 
}
[PHP] pobierz, plaintext 

Ten post edytował damianooo 27.06.2013, 22:31:30

W twoim przypadku jeśli klasa dziedzicząca PDO nie robi nic, żadnego query buildera itd. to można by ją sobie darować.
Do konstruktora przekazuj $pdo i tyle. Poczytaj też o dziedziczeniu, kiedy się je stosuje i po co. Bo klasa City nie powinna dziedziczyć klasy do db (IMG:style_emoticons/default/wink.gif)

zrobiłem tak ale nie wiem czemu nie zapisuje mi rekordu do bazy danych. Co mogę jeszcze mieć źle ?

[PHP] pobierz, plaintext 
class City
{
   public $name;
   public $db;
 
   public function __construct($name)
   {
      $this->db = new PDO('mysql:host=localhost;dbname=database', 'login', 'password');
      $this->setName($name);
   }
 
   public function setName($name)
   {
      $this->name = $name;
      return $this;
   }
 
   public function save()
   {
      $stmt = $this->db->prepare('INSERT INTO city VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => 1));
   }
 
}
[PHP] pobierz, plaintext 

i wywołanie w innym pliku:

[PHP] pobierz, plaintext 
require_once 'Model/City.php';
$city = new City('Madrid');
$city->save();
[PHP] pobierz, plaintext 

Ten post edytował damianooo 28.06.2013, 08:24:15

Zrobiłem tak , ale nadal coś nie tak z tym moim połączeniem, nadal w bazie po odświeżeniu strony nie ma żadnej zmiany, coś nadal robię źle ... ehh .

[PHP] pobierz, plaintext 
class City
{
   public $name;
   public $db;
 
   public function __construct(PDO $pdo)
   {
      $this->db = $pdo;
   }
 
   public function setName($name)
   {
      $this->name = $name;
      return $this;
   }
 
   public function add()
   {
      $stmt = $this->db->prepare('INSERT INTO city VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => 1));
   }
 
}
[PHP] pobierz, plaintext 

i wywołanie:

[PHP] pobierz, plaintext 
require_once 'Model/City.php';
 
$pdo = new PDO('mysql:host=localhost;dbname=database', 'login', 'password');
$city = new City($pdo);
$city->add();
[PHP] pobierz, plaintext 

Ten post edytował damianooo 28.06.2013, 08:35:29

No to wyswietl blad PDO a bedziesz wiedzial co jest nie tak :/

ps: nie masz juz przypadkiem miasta o ID = 1?

W wywołaniu zapomniałeś o ustawieniu name.

Daj private lub protected:

[PHP] pobierz, plaintext 
 public $name;
public $db
[PHP] pobierz, plaintext 

http://pl.wikipedia.org/wiki/Hermetyzacja_%28informatyka%29

ehh faktycznie (IMG:style_emoticons/default/smile.gif) ... jednak nadal nie działa ... postanowiłem wyświetlić co PDO pluje i tak:

Mam tak:

[PHP] pobierz, plaintext 
class City
{
   private $name;
   private $db;
 
   public function __construct(PDO $pdo)
   {
      $this->db = $pdo;
   }
 
   public function setName($name)
   {
      $this->name = $name;
      return $this;
   }
 
   public function add()
   {
      $stmt = $this->db->prepare('INSERT INTO city VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => 1));
   }
 
}
[PHP] pobierz, plaintext 

i wywołanie:

[PHP] pobierz, plaintext 
require_once 'Model/City.php';
 
$pdo = new PDO('mysql:host=localhost;dbname=database', 'login', 'password');
$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
try {
   $city = new City($pdo);
   $city->setName('Katowice');
   $city->add();
} catch (PDOException $e) {
   print_r($e->getTrace());
}
[PHP] pobierz, plaintext 

a PDO krzyczy to:

Array ( [0] => Array ( [file] => /var/www/test/Model/City.php [line] => 43 [function] => execute [class] => PDOStatement [type] => -> [args] => Array ( [0] => Array ( [:name] => Katowice [:country_id] => 1 ) ) ) [1] => Array ( [file] => /var/www/test/test.php [line] => 12 [function] => add [class] => City [type] => -> [args] => Array ( ) ) )

Ten post edytował damianooo 28.06.2013, 08:45:58

rety..... ty nie wyswietlaj sciezki bledu, tylko wyswietlaj blad....

a no tak , sorry (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
SQLSTATE[21S01]: Insert value list does not match column list: 1136 Column count doesn't match value count at row 1
[PHP] pobierz, plaintext 

No to chyba masz juz teraz napisane.... liczba kolumn jakie wstawiasz do tabeli jest mniejsza od wymaganej liczby kolumn jakie ustawiles dla tej tabeli

OK, mam , miało być tak:

[PHP] pobierz, plaintext 
   public function add()
   {
      $stmt = $this->db->prepare('INSERT INTO city (name,country_id) VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => 1));
   }
[PHP] pobierz, plaintext 

Jeszcze chciałem prosić o podpowiedź jak pobrać ID zapisanego obiektu w bazie i przekazać go jako parametr przy zapisywaniu pola innego obiektu w sytuacji gdy encje połączone są w relacji "jeden do wielu" .

i potrzebuję zapisać to coś w ten deseń (country będzie pobierane na stronie z pola rozwijanego):

W klasie "Country" mam tak:

[PHP] pobierz, plaintext 
   public function findById($id)
   {
      $stmt = $this->db->prepare('SELECT * FROM Country WHERE country_id :id');
      $stmt->execute(array('id' => $id));
      $row = $stmt->fetch();
      return $row;
   }
[PHP] pobierz, plaintext 

i zapisuję to tak:

[PHP] pobierz, plaintext 
require_once 'Model/Country.php';
require_once 'Model/City.php';
 
$pdo = new PDO('mysql:host=localhost;dbname=database', 'login', 'password');
$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
try {
   $country = new Country($pdo);
   $country->findById(2);
   $city = new City($pdo);
   $city->setName('Liverpool');
   $city->setCountry($country);
   $city->add();
} catch (PDOException $e) {
   print_r($e->getMessage());
}
[PHP] pobierz, plaintext 

PDO krzyczy:

[HTML] pobierz, plaintext 
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''2'' at line 1
[HTML] pobierz, plaintext 

Ten post edytował damianooo 28.06.2013, 11:15:18

findById ci zwraca rekord, ale ty go w ogole nie przechwytujesz nigdzie

nie przychwytuję tzn. ?

zmieniłem tak:

[PHP] pobierz, plaintext 
   $country = new Country($pdo);
   $countryObject = $country->findById(2);
   $city = new City($pdo);
   $city->setName('Liverpool');
   $city->setCountry($countryObject);
   $city->add();
[PHP] pobierz, plaintext 

ale nadal źle

$countryObject = $country->findById(2);
var_dump($countryObject);

No juz tak banalna analize to moglbys robic sam.... :/

poza tym pokaz jak wyglda teraz kod metody ADD

poza tym, skoro city potrzebuje ID. a ty ID przeciez masz, to po grzyba bawisz sie w budowanie obiektu. Chyba ze twoim celem jest sprawdzenie czy ten obiekt istnieje, w co akurat watpie (bez urazy)

obiekt nie istnieje masz rację, zrobiłem tak:

[PHP] pobierz, plaintext 
   $country = new Country($pdo);
   $countryObject = $country->findById(2);
   var_dump($countryObject);
[PHP] pobierz, plaintext 

i mam pusty ekran. Poza tym z obsługą wyjątków krzyczał mi:

[HTML] pobierz, plaintext 
    SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''2'' at line 1
[HTML] pobierz, plaintext 

metoda add City to:

[PHP] pobierz, plaintext 
   public function add()
   {
      $stmt = $this->db->prepare('INSERT INTO city (name,country_id)VALUES(:name,:country_id)');
      $stmt->execute(array(':name' => $this->name, ':country_id' => $this->country));
   }
[PHP] pobierz, plaintext 

Ten post edytował damianooo 28.06.2013, 11:48:49