Witam

mam np taki oto kod:

[PHP] pobierz, plaintext 
<?php
public function getOnePerson($id) {
        
        try {
            
            $pdo = new PDO('mysql:host=localhost;dbname=contactmanager', 'root', 'root');
            $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $row = $pdo -> query("SELECT * FROM persons WHERE id = $id");
 
            return $row;
            
        } catch (PDOException $e) {
            echo 'Error:' . $e->getMessage();    
        }
        
        $row -> closeCursor();
        
    }
?>
[PHP] pobierz, plaintext 

i nasuwa się pytanie, co zrobić żeby tego tyle razy nie powtarzać ?
mam jakąś tam klase do zarządzania Osobami i praktycznie w każdej metodzie musze powtarzać ten sam fragment kodu...

jak wy to rozwiązujecie?
czy poprostu zrobić dodatkową metodę do której będę przekazywać coś takiego:

[PHP] pobierz, plaintext 
<?php
$row = $pdo -> query("SELECT * FROM persons WHERE id = $id");
?>
[PHP] pobierz, plaintext 

choć to pewnie nia zawsze się sprawdzi...

pisząc obiektowo raczej nie powinno się powtarzać kilka razy (DRY;))

jakieś rozwiązanie ?

z góry dzięki za pomoc

Czy ja dobrze widzę, że łączysz się z bazą za każdym razem, gdy chcesz coś z niej pobrać? Jeśli tak, to czym prędzej skasuj ten kod i przepisz go porządnie . Z bazą skrypt powinien się łączyć jedynie raz i tyle. Odpadnie Ci przy okazji problem "powtarzania", do którego mam taką uwagę, że jest pewna granica, do której DRY ma zastosowanie.

Czy tak będzie lepiej ? :

[PHP] pobierz, plaintext 
<?php
public $pdo;
    
    public function __construct() {
 
        $this->pdo = new PDO('mysql:host=localhost;dbname=.....', 'root', 'root');
        $this->pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        
    }
    
 
    public function getOnePerson($id) {
 
        try {
 
            $row = $this->pdo -> query("SELECT * FROM persons WHERE id = $id");
            return $row;
            
        } catch (PDOException $e) {
            echo 'Error:' . $e->getMessage();    
        }
        
    }
 
 
    public function __destruct() {
        
        $row -> closeCursor();
        
    }
?>
[PHP] pobierz, plaintext 

W __construct() też musisz mieć try ... catch() bo tak nakazuje manual No i po wywołaniu zapytania powinieneś robić $row -> closeCursor(); a nie dopiero w destruktorze (uruchamia on się dopiero wtedy, kiedy niszczony jest obiekt/kończy działanie aplikacja)

[PHP] pobierz, plaintext 
<?php
class PdoFactory {
    private static $instance = NULL;
    private function __construct() {}
    public static function get() {
        if (self::$instance === NULL) {
            $dsn = 'mysql:dbname=***;host=***;port=***;'; //
            $user = 'nobody';                              //zmien na swoje
            $password = 'nobody';                         //
            try {
                $pdo = new PDO($dsn, $user, $password);
                $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            } catch (PDOException $e) {
                throw new Exception ($e->getMessage());
            }
            self::$instance = $pdo;
        }
        return self::$instance;
    }
}
?>
[PHP] pobierz, plaintext 

Nazwa klasy wskazuje że jest to fabryka bo bardzo łatwo przerobić tą klasę tak żeby fabryką była (fabryka czyli klasa która serwuje różne obiekty w zależności od parametrów) i jeśli w projekcie korzystasz z więcej niż jednej bazy danych należałoby przekazać do metody PdoFactory::get() jeszcze jeden parametr dzięki któremu będziesz mógł uzyskać połączenie z określoną bazą danych

Zaprezentowany wyżej kod jest implementacją wzorca projektowego o nazwie Singleton i sprawdza się bardzo do kodu który ma być dostępny w dowolnym miejscu twojego skryptu. Jest to tak naprawdę opakowana w klasę zmienna globalna dlatego radze nie nadużywać tego wzorca.

Mając tak zdefiniowaną klasę obiekt klasy PDO pobierasz przez:

[PHP] pobierz, plaintext 
<?php
$pdo = PdoFactory::get();
?>
[PHP] pobierz, plaintext 

Metoda z twojego pierwszego postu może wyglądać teraz np tak:

[PHP] pobierz, plaintext 
<?php
public function getOnePerson($id) {
        
        $pdo = PdoFactory::get();
           //Jeżeli do zapytania przekazujesz jakikolwiek parametr radze zamiast z PDO::query() skorzystać z PDO::prepare()
            $row = $pdo -> query("SELECT * FROM persons WHERE id = $id"); 
 
            return $row;
        
        $row -> closeCursor(); //to jest bez sensu bo nigdy nie będzie wykonane przez to że umieściłeś to za "return"
        
    }
?>
[PHP] pobierz, plaintext 

albo obiekt klasy PDO pobrać już w samym konstruktorze i przypisać go do prywatnej składowej tej klasy.

Pozwoliłem dodać sobie dwa komentarze do kodu na jego temat.