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

[nospor]

Rozumiem, że w tej postaci jak mam teraz jest dobrze i jeśli obrałem taką metodę to tak to ma wyglądać ? ...

Dobrze? Ledwo mizernie, bardzo daleko od dobrze...

W metodzie findbyid przypisujesz to id w ogole nie sprawdzajac czy sie powiodlo czy nie, czy znalazl czy nie. Na chwile obecna metoda ta jest bez sensu bo nic nie robi.
O przypisaniu name juz nie bede wspominal bo go w ogole nie ma. Skoro chcesz miec obiekt to niech on zawiera co ma zwierac a nie robisz pol srodki

W tej chwili masz tez dwie zmiennej, ktore wskazuja dokladnie na ten sam obiekt
$countryObject = $country
po co?