Skip to content

Conversation

@Malcom1986
Copy link

No description provided.

@Malcom1986 Malcom1986 requested a review from fey August 6, 2024 11:35
@Malcom1986
Copy link
Author

Malcom1986 commented Aug 6, 2024

@fey Коль, посмотри плиз, норм ли такой подход. Будем ли класс для сущности создавать, или ассоциативный массив пусть остается? Если норм, уже урок на основе этого напишу

public/index.php Outdated
$filePath = implode('/', [dirname(__DIR__), 'init.sql']);
$initSql = file_get_contents($filePath);
$conn->exec($initSql);
$carRepository = new CarRepository($conn);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Malcom1986 репозиторий лучше положить в контейнер, как ниже.

$container->set(CarRepository::Class, function ()

Ключем будет имя класса, а отдаваться будет обьект уже сконфигурированный. По идее аналогично должен передаваться Репозиторию коннекшон. Тк у нас есть конейнер зависимостей. мы можем его использовать.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну или в качестве ключа можно просто строку использовать, но тогда это будет строка, а не константа класса)

@fey
Copy link
Contributor

fey commented Aug 6, 2024

@fey Коль, посмотри плиз, норм ли такой подход. Будем ли класс для сущности создавать, или ассоциативный массив пусть остается? Если норм, уже урок на основе этого напишу

Я думаю можно ассоц массив использовать

@Malcom1986
Copy link
Author

Сделал инжекцию репозитория через контейнер. Заодно сущность на класс переписал

@fey
Copy link
Contributor

fey commented Aug 7, 2024

@fey fey merged commit 3866286 into main Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants