Привязать модель к репозиторию в Laravel?

Question

[NDll]

в сервис провайдере связал интерфейс с репозиторием и моделью

public function register()
    {
        $this->app->singleton(UserRepositoryContract::class, function () {
            $repository = new UserReporitory(
                new User()
            );
            return new $repository;
        });
    }

как теперь в репозитории использовать эту модель?

class UserRepository implements UserRepositoryContract
{
    private $model;

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

Comments

[JhaoDa]

В ларавел нет никаких репозиториев, поэтому откуда нам знать, что ты там поставил/написал и как это юзать?

Answer 1

[dmitriy]

все правильно берешь и юзаешь получение данных через $this->model
реализуя контракт UserRepositoryContract

контракт было бы неплохо увидеть

Comments

[NDll]

вот контракт

interface UserRepositoryContract
{
    public function getAllUsersForAdmin($perPage);
    public function getNewUsersForThePeriod($date);
    public function getActiveUsersForThePeriod($date);
    public function getUsersOutOfTown($city_id);
}

[NDll]

ругается на $user в репо

[dmitriy]

в тконструкторе распечатайте $this->model
туда прилетает что то?

[NDll]

dmitriy, выдает вот это

Unresolvable dependency resolving [Parameter #0 [ <required> $user ]] in class App\Repositories\UserRepository

[dmitriy]

что то видимо с DI

[dmitriy]

так у вас ошибка

$repository = new UserReporitory(
                new User()
            );
            return new $repository;

[NDll]

dmitriy, исправил,

$this->app->singleton(UserRepositoryContract::class, function () {
            $repository = new UserRepository(
                new User()
            );
            return new $repository;
        });

но все ровно та же ошибка

[dmitriy]

как получилось?

[NDll]

dmitriy, нет, выдает

Unresolvable dependency resolving [Parameter #0 [ <required> $model ]] in class App\Repositories\UserRepository

[dmitriy]

код DI покажите?

[NDll]

заработало
вот сервис провайдер

$this->app->singleton(UserRepositoryContract::class, function () {
            return new UserRepository(
                new User()
            );
        });

контроллер

class UserController extends Controller
{
    private $userService;

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

    public function index(){
        $users = $this->userService->getAllUsersForAdmin(12);
        return view('admin.users.index', compact('users'));
    }
}

сервис

class UserService
{
    private $userRepository;

    public function __construct()
    {
        $this->userRepository = app(UserRepositoryContract::class);
    }

    public function getAllUsersForAdmin($perPage){
        return $this->userRepository->getAllUsersForAdmin($perPage);
    }
}

репозиторий

class UserRepository implements UserRepositoryContract
{
    private $user;
    public function __construct($user)
    {
        $this->user = $user;
    }
    public function getAllUsersForAdmin($perPage)
    {
        return $this->user->paginate($perPage);
    }
}

в UserService у меня было так

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

заменил репозиторий на контракт и заработало

public function __construct()
    {
        $this->userRepository = app(UserRepositoryContract::class);
    }

Правильно так, или все так где-то ошибка и не нужно app(UserRepositoryContract::class) а app(UserRepository::class);

[dmitriy]

корректировки рекомендую:

public function __construct(User $user)
.....
public function __construct(UserRepositoryContract $userRepository)

.....
Repository
public function findAllUsersForAdmin(int $perPage)


и что на выходе можно тоже указать

[NDll]

dmitriy, а для чего так?

public function __construct(User $user)

он разве не берет из того, что мы привязали через singlrton?

[dmitriy]

берет , но конкретизация принимаемого объекта улучшит читаемость кода

[dmitriy]

+ IDE будет знать что это объект

[NDll]

dmitriy, спасибо