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

Question

[StasRomeo]

Ребята, почему не получается наследование? Учусь.
Вот мой простой класс для работы с книгами

$obj = new Book();

$post = array('name' => 'Книга1', 'author_id' => 10, 'price' => 1000);
$obj->create($post); //создаем книгу

$post = array('id' => 100, 'name' => 'Книга11', 'author_id' => 11, 'price' => 1111);
$obj->edit($post); //редактируем

$obj->delete($id); //удаляем
$obj->showAll(); //получаем весь массив книг
$obj->showOne(100); //получаем только одну книгу

И один приватный select(array $param) который взаимодействуя с showAll и showOne, меняет запрос к бд.
Со временем появляются новые задачи, функционал (например добавить обложку, добавить аннотацию и тд.)
Куда дописывать новый код в конкретном случае? Я могу добавить его в основной класс, или создать наследника. Только вот что тут и от чего наследовать я не въеду. Все методы кроме select-ов так то не связаны друг с другом. Каждый сделает свои проверки, запросы к бд и всё.
Если бы задача стояла вывести какой нибудь иной selectByAuthor(), тогда да, есть смысл наследовать тот же private select. Но тут совершенно нечего наследовать.

Comments

[Дмитрий Ларин]

Эм...наследуются классы, а не методы. Они идут в нагрузку от класса родителя к наследнику. Методы private не наследуются. Рекомендую почитать ООП PHP.

[Decadal]

приведите пожалуйста исходные коды классов.

[Герман Мартин]

$post = array('id' => 100, 'name' => 'Книга11', 'author_id' => 11, 'price' => 1111);
$obj->edit($post); //редактируем

Редактируете вы тут что? Сначала надо получить и менять ИД т.е. первичный ключ у записи нехорошо.

$obj->showAll(); //получаем весь массив книг
$obj->showOne(100); //получаем только одну книгу

следовало бы заменить на

$obj->getAll(); //получаем весь массив книг
$obj->getOne(100); //получаем только одну книгу

вывод чего либо, это отдельная история, т.к. там будет своя разметка и сопутствующее...

И один приватный select(array $param) который взаимодействуя с showAll и showOne, меняет запрос к бд.
Со временем появляются новые задачи, функционал (например добавить обложку, добавить аннотацию и тд.)

Создаете новый класс, например:

// BookI
class BookI extends Book{
   // описываете новые свойства
   // добавляете новые и/или 'перегружаете' старые методы
  
}

вот так это примерно выглядит, а так вам на офф.сайт php - изучать материалы, как советовал оратор выше.
И, напоследок, раз вы спрашиваете, приводите весь свой код, или ссылку на него.

Если бы задача стояла вывести какой нибудь иной selectByAuthor(), тогда да, есть смысл наследовать тот же private select.

Заодно посерьезнее следует изучить модификаторы доступа. Нельзя просто так взять и 'наследовать тот же private select'

[MIK Ek]

Герман Мартин, только не "перегружаете" а "переопределяете".
Это разные вещи и в PHP нет перегрузки.

[Герман Мартин]

MIKEk8, я специально занес это в кавычки имея ввиду переопределение

[StasRomeo]

Decadal,

class Book{

protected $db;

public function __construct($db){
  $this->db = $db;
}

private function error($message){
  throw new Exception($message);
}

public function create(array $post){
	
  $param = array('name', 'author_id', 'price');
  $add = array();
  foreach($param as $val){
    if(!empty($post[$val])){
      $add[':'.$val] = trim($post[$val]);
    }else{
      $this->error('Отсутствует значение: '.$val);
    }
  }

  $stmt = $this->db->prepare("INSERT INTO `book` (`date_add`, `name`, `author_id`, `price`) VALUE (NOW(), :name, :author_id, :price)");
  return $stmt->execute($add);
}

public function update(array $post){
  if(empty($post['id']) && !is_numeric($post['id'])){
    $this->error('empty ID');
  }
  $e= array();
  $param = array('id', 'name', 'author_id', 'price');
  foreach($param as $val){
    //...
  }
	
  $stmt = $this->db->prepare("UPDATE `book` SET ........ WHERE `id` = :id");
  return $stmt->execute($e);
}

public function delete($id){
  $stmt = $this->db->prepare("DELETE FROM `book` WHERE `id` = ?");
  return $stmt->execute(array($id));
}

private function select(array $param){
	
  $where = array();
  $z = array();
	
  if(!empty($param['id'])){
    $where[] = "`id` = :id";
    $z[':user'] = $param['author_id'];
  }
  
  if(!empty($param['author_id'])){
    $where[] = "`author_id` = :author_id";
    $z[':author_id'] = $param['author_id'];
  }
	
  $query = "SELECT * FROM `book`";
  if(!empty($where)){
    $query .= " WHERE ".implode(' AND ', $where);
  }
	
  $stmt = $this->db->prepare($query);
  $stmt->execute($z);
  return $stmt->fetchAll();
}

public function showAll(){
  return $this->select(array());
}

public function showOne($id){
  return $this->select(array('id' => $id));
}

}


class Book2 extends Book{

public function showOnlyAuthor($id){
  return $this->select(array('author_id' => $id));
}

}

Парни, пожалуйста, укажите грубые ошибки. Если ни есть :)
Кстати я расширил класс.

[StasRomeo]

Герман Мартин, Спасибо за такой расширеный ответ. Я пока учусь :)
На счет наследования приватного метода, согласен, забыл про это. Там ошибка в наследнике будет.

[StasRomeo]

Герман Мартин,

Редактируете вы тут что? Сначала надо получить и менять ИД т.е. первичный ключ у записи нехорошо.

Не очень понял, что тут имелось ввиду?

[Герман Мартин]

вот код:

$post = array('id' => 100, 'name' => 'Книга11', 'author_id' => 11, 'price' => 1111);
$obj->edit($post); //редактируем

увидев исходники, теперь понимаю, что код рабочий)

Имеет смысл немного улучшить код, а именно добавить обработчики к $stmt->execute($z); и подобным

//$stmt->execute($z);
try {
   $stmt->execute($z);
   $rs_stmt = $stmt->fetchAll(); // тут наверное стоит взять либо fetchAssoc либо  fetchNum. 
}catch(e){
   // обработка исключения... например
   $rs_stmt = null;
}
// банально запрос может не выполниться, ниже следующая строка не сработает
// return $stmt->fetchAll();
return $rs_stmt;

[StasRomeo]

Спасибо за советы!
Не так всё плохо, как я думал :)

Answer 1

[SwoDs]

Методы не наследуются, все методы родителя доступны классу который его наследует.

В вашем случае я бы просто расширил функционал класса Book, так как работа связана с ним.

Если у вас был бы класс "Печатная продукция", то большая часть методов была там, а классы Book, Newspaper и т.д.
наследовались от него. В этих класса была бы реализация только того что относится к ним, к примеру у книги есть автор, а у газеты нет... как то так