Error __construct() must be an instance of Interface, none given

Question

I'm developing a package for Laravel and I'm getting an error which I can't figure out how to fix:

Argument 1 passed to Cartalini\Drayman\Drayman::__construct() must be an instance of Cartalini\Drayman\Repositories\UserRepositoryInterface, none given, called in /Applications/MAMP/htdocs/l4/app/controllers/HomeController.php on line 10 and defined

Here's my code...

namespace Cartalini\Drayman;

use Cartalini\Drayman\Repositories\UserRepositoryInterface;

class Drayman
{
    protected $user;

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

    public function deliverBeer()
    {
        return $this->user->all();
    }
}

UserRepository...

namespace Cartalini\Drayman\Repositories;

class UserRepository implements UserRepositoryInterface
{
    public function all()
    {
        return User::all();
    }
}

UserRepositoryInterface...

namespace Cartalini\Drayman\Repositories;

interface UserRepositoryInterface
{
    public function all();
}

Service provider...

public function register()
{
    $this->app->bind('Cartalini\Drayman\Repositories\UserRepositoryInterface', 'Cartalini\Drayman\Repositories\UserRepository');
}

And finally my controller...

use Cartalini\Drayman\Drayman as Drayman;

class HomeController extends BaseController 
{
    public function showWelcome()
    {
        $drayman = new Drayman;
        return $drayman->deliverBeer();
    }
}

Can anyone help me to debug this please?

The message says it all. Daryamn requires a parameter of the type UserRepositoryInterface but you don't provide one. — John Conde
– John Conde, Commented Nov 26, 2013 at 21:02
Follow this and this article for package development in Laravel, I think you missed couple of things. — The Alpha
– The Alpha, Commented Nov 26, 2013 at 21:34

qwertynl · Accepted Answer · 2013-11-26 21:02:56Z

1

In your showWelcome function:

public function showWelcome()
{
    // need to pass a UserRepositoryInterface object here:
    $drayman = new Drayman;
    return $drayman->deliverBeer();
}

Since you did not pass a UserRepositoryInterface object that your code requires you get that error.

answered Nov 26, 2013 at 21:02

qwertynl

3,9331 gold badge24 silver badges43 bronze badges

Sign up to request clarification or add additional context in comments.

8 Comments

The Alpha Over a year ago

This is not the way Laravel works, it's about Laravel package and this is theoretically right but not right answer in this context.

The Alpha Over a year ago

Do you know how Laravel works ? Check the link I gave in my previous comment.

The Alpha Over a year ago

It doesn't matter what I know but it's a fact that you are not aware of Laravel and what do you answered, how to pass the dependency here, you even didn't give any example, can you make it right ?

The Alpha Over a year ago

What is your advise here to fix the problem, any code example ? You even didn't give the right code according to plain PHP, where did you pass the dependency in your answer ?

reikyoushin Over a year ago

^ the advice is on the answer // need to pass a UserRepositoryInterface object here:

|

Faramarz Qoshchi · Accepted Answer · 2021-10-22 14:10:22Z

1

it might be a to late, but i had the same problem and the reason was my concrete class didn't implement it's corresponding interface class. after implementing it, all went well.
although you are correctly implementing it, so this error might have a few reason, which one of them is what i described.

edited Oct 22, 2021 at 14:10

answered Aug 11, 2021 at 4:51

Faramarz Qoshchi

1,7082 gold badges17 silver badges30 bronze badges

Collectives™ on Stack Overflow

Error __construct() must be an instance of Interface, none given

2 Answers 2

8 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related