Допустимо ли использовать наследование в этой ситуации?

Question

[Anonymous85966]

Существует два класса для интернет магазина.Первый класс называется User.В нем содержатся переменные и методы для определения email, идентификатора пользователя и т.п. Второй класс называется basket.В нем содержатся методы для вывода товаров из корзины, добавления товаров в корзину и т.п. В классе basket функция для вывода товаров из бд принимает в аргументе идентификатор пользователя, который записывается в sql запрос.Идентификатор пользователя можно получить из метода в классе User.Поэтому, я реализовал наследование class Basket extends User.В нем создается конструктор, в который записываются переменные из класса User parent::__construct($email, $password).Допустимо ли использовать наследование в этой ситуации?

Comments

[Demian Smith]

За последние лет 5 не могу вспомнить случая когда наследование было бы оправданным (если не считать случаев уродского дизайна когда без наследования просто невозможно обойтись).

[FanatPHP]

Demian Smith, на самом деле можешь, если наследоваться от абстрактного класса - то как это и было задумано.
С другой стороны, с появлением интерфейсов надобность в абстрактных классах сильно упала.

Answer 1

[Stalker_RED]

То есть теперь у корзины будут методы определения email, идентификатора пользователя, проверки пароля, и всякого такого?

Корзина может иметь свой метод getUserId, а наследовать корзину от юзера как-то странновато.

Comments

[Anonymous85966]

Мне показалось, что создание нового метода getUserId будет дублированием кода, прописанным в классе User

[FanatPHP]

Anonymous85966, в каком смысле дублированием?

[Anonymous85966]

FanatPHP, метод getUserId прописан в классе User и я заново прописываю этот метод в классе Bakset,что является лишним дублированием кода

[FanatPHP]

А что делают эти два метода?

[Anonymous85966]

FanatPHP, возвращают id пользователя

[Anonymous85966]

FanatPHP, они делают одно и тоже

[FanatPHP]

Но что конкретно?

[Anonymous85966]

FanatPHP, возвращают id пользователя

[FanatPHP]

Anonymous85966, откуда-то они его берут?
Здась форум не по литературе. А по прграммированию
Никому не нужен твой пересказ кода своими словами.
если ты хочешь совет по коду, то показывай КОД. что ты там имеешь в виду своим "возвращают" никто не знает
и я, со своими 20 годами опыта, не могу себе представить, чтобы корзина и юзер имели на 100% идентичный код метода getUserId.

[Anonymous85966]

class User{
function __construct($email,$password){
$this->email=$email;
$this->password=$password;
}
function get_user_id($email){
$sql="SELECT FROM users WHERE email='$email' ";
//выполняю запрос и сохраняю результат в переменную $id
return $id;

}
}
class Basket{
function get_user_id($email){
$sql="SELECT FROM users WHERE email='$email' ";
//выполняю запрос и сохраняю результат в переменную $id
return $id;

}
}
Я уже понял, что это бессмысленный код.Решил проблему так:При авторизации сохраняю в сессию два экземпляр класса-User и Basket($_SESSION['user']=new User(),$_SESSION['basket']=new Basket()).Далее при необходимости через экземпляр класса User получаю id,email пользователя.Потом при использовании методов класса Basket просто передаю эти данные в аргумент функции и получаю нужный результат

[FanatPHP]

Anonymous85966, это всё никуда не годится
хранить экзампляры классов в сессии совсем плохая идея

В сессии должен храниться только айди юзера.
И вот изсессии и бери его сразу.
А вот если тебе понадобится емейл - то его можно получить из класса юзер по айди.

.

[Anonymous85966]

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

[FanatPHP]

Ну, для примитивных объектов, которые не содержат ссылки на другие объекты, наверное ничего страшного.

[Антон Р.]

Anonymous85966,

class User{

	public $id = null;
	private $email;
	private $password;

	function __construct($email, $password){
		$this->email=$email;
		$this->password=$password;
	}
	function get_user_id($email, $pdo){
		$sql="SELECT FROM users WHERE email='$email'";
		$result = $pdo->exec($sql);
		// Тут еще нужна проверка что запрос вернул что надо.
		return $this->id = $result;

	}
}

class Basket{

	private $user;
	private $items = [];
	private $summ = null;

	function __construct(User $user){
		$this->user=$user;
	}

	function get_user_id(){
		return $this->user->get_user_id();
	}
}

Как то так наверно.

Answer 2

[Антон Антон]

В данном случае правильно сделать не наследование, а композицию. Ну, или агрегацию.

Answer 3

[Tendor]

Думаю у юзера должен быть метод getBasket()

Comments

[Антон Р.]

Скорее сущность Shop которая имеет метод checkout который принимает два параметра User и Basket.