moja klasa z zapytaniami SQL Opcje

[damianooo]

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

[damianooo]

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