Laravel5-接口不可实例化

我知道这个问题被问了很多次，但是没有一个答案对我有帮助。

我在 Laravel 5得到了例外

BindingResolutionException in Container.php line 785:
Target [App\Contracts\CustomModelInterface] is not instantiable.

我所做的没有成功:

  • 在 app.php提供程序中注册 App\Providers\AppRepositoryProvider
  • php artisan clear-compiled
  • 如果我在 MyService 中替换存储库上的接口，一切都会正常工作，但我觉得这是错误的(应该由 IoC 容器来处理吗?).

结构:

app
- Contracts
- CustomModelInterface.php
- Models
- CustomModel.php
- Repositories
- CustomModelRepository.php
- Providers
- AppRepositoryProvider.php
- Services
- MyService.php

App Contracts CustomModelInterface.php

<?php namespace App\Contracts;


interface CustomModelInterface {
public function get();
}

应用程序库 CustomModelRepository.php

<?php namespace App\Repositories;


use App\Contracts\CustomModelInterface;
use App\Models\CustomModel;


class CustomModelRepository implements CustomModelInterface {


private $Model;


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


public function get() {
return 'result';
}
}

Php (在控制器和存储库之间保持业务逻辑/层)

<?php namespace App\Services;


use App\Contracts\CustomModelInterface;


class MyService {


private $Model;


public function __construct(CustomModelInterface $customModel) {
$this->Model= $customModel;
}


public function getAll() {
return $this->Model->get();
}
}

应用程序提供商 AppRepositoryProvider.php

<?php namespace App\Providers;


use Illuminate\Support\ServiceProvider;


class AppRepositoryProvider extends ServiceProvider {


public function boot() {}


public function register() {
$models = array(
'CustomModel'
);


foreach ($models as $idx => $model) {
$this->app->bind("App\Contracts\{$model}Interface", "App\Repositories\{$model}Repository");
}
}
}

我的控制器看起来像:

<?php namespace App\Http\Controllers;


use App\Services\MyService;


class SuperController extends Controller {


private $My;


public function __construct(MyService $myService) {
$this->My = $myService;
}


public function getDetails() {
return $this->My->getAll();
}
}

作曲家 Json

"autoload": {
"classmap": [
"database"
],
"psr-4": {
"App\\": "app/",
"App\\Models\\": "app/Models/",
"App\\Contracts\\": "app/Contracts/",
"App\\Repositories\\": "app/Repositories/"
}
},
120612 次浏览
小开

On App\Services\MyService.php you are passing that interface with dependency injection which tries to instantiate that - 

public function __construct(CustomModelInterface $customModel) {
$this->Model= $customModel;
}

which is wrong.

Try implement that in that class - class MyService implements CustomModelInterface { and use the function of that interface like - 

$this->get();

Or you are using it - class CustomModelRepository implements CustomModelInterface {

So if you do - 

public function __construct(CustomModelRepository $customModel) {
$this->Model= $customModel;
}

then also you can access the interface methods.

小开

I think the problem here is that you don't bind App\Contracts\CustomModelInterface to anything so Laravel tries to create instance of interface.

In App\Providers\AppRepositoryProvider.php you have only:

$models = array(
'Model'
);

but you should have in this array CustomModel also, so it should look like this:

$models = array(
'Model',
'CustomModel',
);
小开
最佳答案

Thank you everyone, but problem was in my AppRepositoryProvider. As it's binding exception, then obviously the problem was with binding :)

Correct file is:

<?php namespace App\Providers;


use Illuminate\Support\ServiceProvider;


class AppRepositoryProvider extends ServiceProvider {


public function boot() {}


public function register() {
$models = array(
'CustomModel',
'CustomModel2',
'CustomModel3'
);


foreach ($models as $model) {
$this->app->bind("App\Contracts\\{$model}Interface", "App\Repositories\\{$model}Repository");
}
}
}

Note, that I'm using "App\Contracts\\{$model}Interface" (not escaping "{" symbol) and it generate correct string App\Contracts\CustomModelInterface instead of App\Contracts\{$model}Interface (with unexpected escaping).

小开

Every time I create a new repository/contract pair I make sure I do the following:

  1. check the classes used in the service provider (copy/paste the namespaces)
  2. register a new binding in config/app.php
  3. php artisan optimize

Many hours of useless debugging led me to this short checklist.

小开

Note that this can also be caused by the _constructor on the class being declared private, or otherwise being blocked... If it cant call the constructor, the binding will fail

小开

The last thing you do is to use the interface you bound to the repository.

Set it up and try running your laravel app to make sure you get no errors.

In my case I had a mismatch between my repository and interface.

interface UserRepositoryInterface{
public function get($userId);
}


class UserRepository implements UserRepositoryInterface{
public function get(int $userId);
}

As you can see the interface get method does not include a type hint but the UserRepository class' get method has a type hint.

You won't get this error if you immediately start to use your Interface Binding.

小开

For me, I forgot to bind in app->providers->RepositoryServiceProvider the repository like this in the register method

public function register()
{
$this->app->bind(
\App\Play\Contracts\PatientRepository::class,
\App\Play\Modules\PatientModule::class
);
}

Make sure your RepositoryServiceProvider is registered in AppServiceProvider.

public function register()
{
$this->app->register(RepositoryServiceProvider::class);
}
小开

I got past this error running:

php artisan config:clear
php artisan clear-compiled
php artisan optimize
php artisan config:cache

Related to:

Target is not instantiable. Laravel 5 - App binding service provider

小开

I've just experienced an issue similar to this and the cause of my error was that I had set $defer to true in the service provider class but I had not implemented the required provides() method.

If you have deferred the creation of your class until it is need rather than it being loaded eagerly, then you need to also implement the provides method which should simply return an array of the classes that the provider provides. In the case of an interface, I believe it should be the name of the interface rather than the concrete class.

E.g.

public method provides(): array
{
return [
MyInterface::class,
];
}

Current documentation: https://laravel.com/docs/5.5/providers#deferred-providers

I hope this helps somebody else.

小开

Don't worry guys. I have a solution to your problem.

I have an example for you.

Step1: php artisan make:repository Repository/Post //By adding this command you can create a repository and eloquent files

Step2: After adding that file you have to add/use this repository in the controller in which you want to use.

for eg: use App\Repositories\Contracts\PostRepository;

Step3: After adding that repo in your controller if you will run the app you will get an error like " Interface is not instantiable". It comes because you have created a repo and used in a controller, but laravel don't know where this repository is register and bind with which eloquent. So that it throws an error.

Step4: To solve this error you have to bind your repo with your eloquent in AppServiceProvider. E.g:

AppServiceProvider.php file

<?php
namespace App\Providers;


// **Make sure that your repo file path and eloquent path must be correct.**


use App\Repositories\Contracts\PostRepository;         // **Use your repository here**


use App\Repositories\Eloquent\EloquentPostRepository;  **// Use your eloquent here**


use Illuminate\Support\ServiceProvider;


class AppServiceProvider extends ServiceProvider {
/**
* Register any application services.
*
* @return void
*/
public function register() {


**// And bind your repository and eloquent here. **


$this->app->bind(PostRepository::class, EloquentPostRepository::class);
}
}

Step5: After binding repo and eloquent you can use all method of repo in your controller. Enjoy.....

Please let me know if you have any query.

小开

The problem is solved by adding your repository in app/providers/AppServiceProvider like the example below.

public function register()
{
$this->app->singleton(UserRepository::class, EloquentUser::class);
}

Dont forget the name space

use Test\Repositories\EloquentUser;
use Test\Repositories\UserRepository;

It worked for me

小开

execute this command :

composer dump-autoload

this command will remap your laravel autoload classes together with all other vendor's i had same issue before and this did the trick you can use it together with "-o" param for optimization .

小开

register a new binding in config/app.php


提交答案