Как правильно расширить класс?

Question

[Max Ba]

Есть объект выводящий список всех машин. Как мне (в теории) расширить класс таким образом, чтобы я мог модифицировать запрос, чтобы вывести все машины, например начинающиеся на букву А. Это речь про наследование вообще?

class Car
{
  public $db;
	
  public function getAll(){
    $result = $this->db->query("SELECT * FROM `car`");
    $data = array();
    foreach($result->fetchAll() as $row){
      $data[$row['id']] = $row['model'];
    }
    return $data;
  }
}

$car = new Car();
$car->db = DB::instance();
var_dump($car->getAll());

Answer 1

[Lander]

Давайте представим что у вас помимо машин появились ещё самолёты. Тогда у вас появится второй класс:

class Aircraft
{
  public $db;
  
  public function getAll(){
    $result = $this->db->query("SELECT * FROM `aircraft`");
    $data = array();
    foreach($result->fetchAll() as $row){
      $data[$row['id']] = $row['model'];
    }
    return $data;
  }
}

То есть отличий от Car - только название таблицы. Соответственно, мы должны выделить весь код в отдельный суперкласс

class Transport
{
  public $db;
  protected $table = null;  

  public function getAll(){
    $result = $this->db->query("SELECT * FROM `{$this->table}`");
    $data = array();
    foreach($result->fetchAll() as $row){
      $data[$row['id']] = $row['model'];
    }
    return $data;
  }
}

class Car extends Transport
{
  protected $table = 'car';
}

class Car extends Transport
{
  protected $table = 'aircraft';
}

К чему это я? Ах да!
Вам нужно создать класс для доступа к базе данных отдельно. Без привязки к какой либо таблице и в нём добавить методы для выборки по конкретным условиям. И уже от него наследовать все классы, которые что то читают из таблицы.

Comments

[Max Ba]

Кстати, а разве можно и нужно так делать?
$query = "SELECT * FROM `{$this->table}`";

Я всегда был уверен, что лучше писать так
"SELECT * FROM `".$this->table."`"

[Lander]

Макс Васильев, Если до конца быть честным, то между приведенными вариантами нет разницы. А лучше всего делать так:
'SELECT * FROM `'.$this->table.'`'
Но так менее читаемо. Тут же главное задача до вас решение донести, а не оптимизацией заниматься. :) Ну или плейсхолдерами. Ну это потом. :)

Answer 2

[Andreo]

Нет. Это никакое не наследование. Делайте новый метод или модифицируйте имеющийся так, чтоб можно было манипулировать аргументом с опциями, собирая таким образом запрос в БД.
Что-то типа например:

...
public function get(Array $options = [])
{
    if (isset($options['by_name']) {
        $query = "WHERE `name` LIKE %". $options['by_name']  ."% ";
    }
...
}