WebAPI 2.1のNinjectエラー-コントローラーにパラメーターなしのパブリックコンストラクターがあることを確認してください
WebAPIプロジェクトに次のパッケージとその依存関係がインストールされています。
Ninject.Web.WebApiNinject.Web.WebApi.OwinHost
これは純粋にweb-apiプロジェクトとして実行しています。 MVCはありません。
アプリケーションを実行し、POSTをAccountControllerのRegisterアクションに送信すると、次のエラーが返されます。
{
"message":"An error has occurred.",
"exceptionMessage":"An error occurred when trying to create a controller of type 'AccountController'. Make sure that the controller has a parameterless public constructor.",
"exceptionType":"System.InvalidOperationException",
"stackTrace":" at System.Web.Http.Dispatcher.DefaultHttpControllerActivator.Create(HttpRequestMessage request, HttpControllerDescriptor controllerDescriptor, Type controllerType)\r\n at System.Web.Http.Controllers.HttpControllerDescriptor.CreateController(HttpRequestMessage request)\r\n at System.Web.Http.Dispatcher.HttpControllerDispatcher.SendAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)\r\n at System.Web.Http.Dispatcher.HttpControllerDispatcher.<SendAsync>d__0.MoveNext()",
"innerException":{
"message":"An error has occurred.",
"exceptionMessage":"Type 'RPT.Api.Controllers.AccountController' does not have a default constructor",
"exceptionType":"System.ArgumentException",
"stackTrace":" at System.Linq.Expressions.Expression.New(Type type)\r\n at System.Web.Http.Dispatcher.DefaultHttpControllerActivator.GetInstanceOrActivator(HttpRequestMessage request, Type controllerType, Func`1& activator)\r\n at System.Web.Http.Dispatcher.DefaultHttpControllerActivator.Create(HttpRequestMessage request, HttpControllerDescriptor controllerDescriptor, Type controllerType)"
}
}
Googleで見つけることができる唯一の詳細は2012年のように思えるので、誰でも私を助けることができます。
注：Ninjectの代わりにAutoFacも試してみましたが、同じエラーが発生します。最もイライラします。
NinjectWebCommon.csは次のとおりです。
using Microsoft.AspNet.Identity;
using Microsoft.AspNet.Identity.EntityFramework;
using RPT.Data;
using RPT.Services;
using RPT.Services.Interfaces;
[Assembly: WebActivatorEx.PreApplicationStartMethod(typeof(RPT.Api.NinjectWebCommon), "Start")]
[Assembly: WebActivatorEx.ApplicationShutdownMethodAttribute(typeof(RPT.Api.NinjectWebCommon), "Stop")]
namespace RPT.Api
{
using System;
using System.Web;
using Microsoft.Web.Infrastructure.DynamicModuleHelper;
using Ninject;
using Ninject.Web.Common;
public static class NinjectWebCommon
{
private static readonly Bootstrapper bootstrapper = new Bootstrapper();
/// <summary>
/// Starts the application
/// </summary>
public static void Start()
{
DynamicModuleUtility.RegisterModule(typeof(OnePerRequestHttpModule));
DynamicModuleUtility.RegisterModule(typeof(NinjectHttpModule));
bootstrapper.Initialize(CreateKernel);
}
/// <summary>
/// Stops the application.
/// </summary>
public static void Stop()
{
bootstrapper.ShutDown();
}
/// <summary>
/// Creates the kernel that will manage your application.
/// </summary>
/// <returns>The created kernel.</returns>
private static IKernel CreateKernel()
{
var kernel = new StandardKernel();
try
{
kernel.Bind<Func<IKernel>>().ToMethod(ctx => () => new Bootstrapper().Kernel);
kernel.Bind<IHttpModule>().To<HttpApplicationInitializationHttpModule>();
RegisterServices(kernel);
return kernel;
}
catch
{
kernel.Dispose();
throw;
}
}
/// <summary>
/// Load your modules or register your services here!
/// </summary>
/// <param name="kernel">The kernel.</param>
private static void RegisterServices(IKernel kernel)
{
kernel.Bind<RptContext>().ToSelf();
kernel.Bind<IUserStore<IdentityUser>>().To<UserStore<IdentityUser>>();
kernel.Bind<UserManager<IdentityUser>>().ToSelf();
kernel.Bind<IAccountService>().To<AccountService>();
}
}
}
これが私のAccountControllerです。
using System.Threading.Tasks;
using System.Web.Http;
using System.Web.ModelBinding;
using Microsoft.AspNet.Identity;
using RPT.Api.Models;
using RPT.Services.Interfaces;
namespace RPT.Api.Controllers
{
[RoutePrefix("api/account")]
public class AccountController : ApiController
{
#region Initialisation
private readonly IAccountService _accountService;
public AccountController(IAccountService accountService) : base()
{
_accountService = accountService;
}
#endregion
#region Actions
[AllowAnonymous]
[Route("register")]
public async Task<IHttpActionResult> Register(UserRegistrationViewModel model)
{
if (!ModelState.IsValid)
{
return BadRequest(ModelState);
}
var result = await _accountService.RegisterUser(model.UserName, model.Password);
var errorResult = GetErrorResult(result);
return errorResult ?? Ok();
}
#endregion
#region Internal
protected override void Dispose(bool disposing)
{
if (disposing)
{
_accountService.Dispose();
}
base.Dispose(disposing);
}
private IHttpActionResult GetErrorResult(IdentityResult result)
{
if (result == null)
{
return InternalServerError();
}
if (result.Succeeded) return null;
if (result.Errors != null)
{
foreach (var error in result.Errors)
{
ModelState.AddModelError("", error);
}
}
if (ModelState.IsValid)
{
// No ModelState errors are available to send, so just return an empty BadRequest.
return BadRequest();
}
return BadRequest(ModelState);
}
#endregion
}
}
OWIN Startupクラスを変更して、app.UseNinjectWebApiを呼び出すのではなく、app.UseNinjectMiddlewareとapp.UseWebApiを呼び出すようにしましたか？
Startup.cs Ninject Web APIサンプルではこれを行います...
私の場合、理由はリゾルバーがマッピングを見つけられなかったためです。 HomeControllerがIDumbに依存していると仮定すると、リゾルバーはIDumbを実装したDumbの具体的な実装を見つけることができませんでした。言い換えれば、エラーメッセージ
**No parameterless constructor defined for this object
An error occurred when trying to create a controller of type 'ToDoListT1.WebApp.Controllers.HomeController'. Make sure that the controller has a parameterless public constructor**
完全に誤解を招く。私の場合、クラスDumbのプロジェクトへの参照を追加して解決しました。 「IDumbのマッピングが見つかりませんでした。」のようなものであるべきでした。問題がNInjectまたはMSにあるかどうかはわかりません。これを見つけるのに何時間もかかった。
私の解決策は、コンストラクタに「public」キーワードを追加することです。
本当に基本的な実装である依存関係リゾルバーがありません：
public class NinjectHttpDependencyResolver : IDependencyResolver, IDependencyScope
{
private readonly IKernel _kernel;
public NinjectHttpDependencyResolver(IKernel kernel)
{
_kernel = kernel;
}
public IDependencyScope BeginScope()
{
return this;
}
public void Dispose()
{
//Do nothing
}
public object GetService(Type serviceType)
{
return _kernel.TryGet(serviceType);
}
public IEnumerable<object> GetServices(Type serviceType)
{
return _kernel.GetAll(serviceType);
}
}
次に、カーネルを作成するときに登録します。
var httpResolver = new NinjectHttpDependencyResolver(kernel);
GlobalConfiguration.Configuration.DependencyResolver = httpResolver;
私にとって、このエラーの原因は、コンストラクターに渡すインターフェイスに対して、名前付きバインディングがあり、間違った名前で呼び出したことです。以下を見るとわかるように、バインディングには「Warehouse」という名前が付けられていますが、私は物事を混乱させ、コンストラクターに「DataWarehouse」を入れました。これを修正すると、パラメーターのないコンストラクターに関するエラーがなくなります。
private static void RegisterServices(IKernel kernel)
{
kernel.Bind<IVisitProcessor>().To<VisitProcessor>().InThreadScope();
kernel.Bind<IPatientProcessor>().To<PatientProcessor>().InThreadScope();
kernel.Bind<IDbConnectionFactory>().To<SqlConnectionFactory>().Named("Warehouse").WithConstructorArgument("connectionString", ConfigurationManager.ConnectionStrings["DataWarehouse"].ConnectionString);
}
コンストラクタ：
//WRONG NAME
public VisitProcessor([Named("DataWarehouse")] IDbConnectionFactory connection)
{
_database = connection;
}
コントローラーが使用するすべてのタイプをデータベースに至るまで登録していることを確認してください。
私の場合、コントローラー自体が使用したインターフェースのみを追加しましたが、実際にデータベースを照会するために使用したインターフェースは追加しませんでした。
以下のコードで、AddUserMaintenanceProcessorクラスには、コントローラーが認識していない依存関係があることに注意してください。これらの依存関係のUnity（または使用するIoCツール）タイプマッピングを省略すると、コントローラーの構築は失敗します。
私のソリューションはUnityを使用していますが、私がしようとしている点は、すべての依存関係の型マッピングを作成する必要があるということです。
Startup.cs
public void Configuration(IAppBuilder app)
{
var config = new HttpConfiguration();
// Configure Unity
var resolver = new UnityDependencyResolver(UnityConfig.GetConfiguredContainer());
GlobalConfiguration.Configuration.DependencyResolver = resolver;
config.DependencyResolver = resolver;
// Do Web API configuration
WebApiConfig.Register(config);
app.UseWebApi(config);
}
nityConfig.cs
public class UnityConfig
{
private static readonly Lazy<IUnityContainer> Container = new Lazy<IUnityContainer>(() =>
{
var container = new UnityContainer();
RegisterTypes(container);
return container;
});
// Gets the configured Unity container
public static IUnityContainer GetConfiguredContainer()
{
return Container.Value;
}
// Register type mappings
public static void RegisterTypes(IUnityContainer container)
{
// LogManagerAdapter wrapping e.g. log4net
container.RegisterType<ILogManager, LogManagerAdapter>();
// AutoMapperAdapter wrapping e.g. AutoMapper (configuration omitted)
container.RegisterType<IAutoMapper, AutoMapperAdapter>();
// Interface for persisting the user
container.RegisterType<IAddUserQueryProcessor, AddUserQueryProcessor>();
// Interface for doing application logic in regards to adding a user
container.RegisterType<IAddUserMaintenanceProcessor, AddUserMaintenanceProcessor>();
}
}
sersController.cs
public class UsersController : ApiController
{
private readonly IAddUserMaintenanceProcessor _addUserProcessor;
public UsersV1Controller(IAddUserMaintenanceProcessor addUserProcessor)
{
_addUserProcessor = addUserProcessor;
}
public async Task<UserModel> Post(NewUser user)
{
return await _addUserProcessor.AddUserAsync(user);
}
// ...
}
AddUserMaintenanceProcessor.cs
public class AddUserMaintenanceProcessor : IAddUserMaintenanceProcessor
{
private readonly IAddUserQueryProcessor _queryProcessor;
private readonly ILog _logger;
private readonly IAutoMapper _mapper;
public AddUserMaintenanceProcessor(
IAddUserQueryProcessor queryProcessor,
ILogManager logManager,
IAutoMapper mapper)
{
_queryProcessor = queryProcessor;
_logger = logManager.GetLog(typeof(AddUserMaintenanceProcessor));
_mapper = mapper;
}
public async Task<UserModel> AddUserAsync(NewUser newUser)
{
_logger.Info($"Adding new user {newUser.UserName}");
// Map the NewUser object to a User object
var user = _mapper.Map<User>(newUser);
// Persist the user to a medium unknown to this class, using the query processor,
// which in turn returns a User object
var addedUser = await _queryProcessor.AddUserAsync(user);
// Map the User object back to UserModel to return to requester
var userModel = _mapper.Map<UserModel>(addedUser);
_logger.Info($"User {userModel.UserName} added successfully");
return userModel;
}
}
プロセッサには1つのメソッド（戦略パターン）しか含まれていないため、プロセッサのインターフェイスは省略しました。ロギングと自動マッピングのインターフェースは、この質問とは無関係です。
AddUserQueryProcessorクラスは、単にユーザーをデータベースに永続化します。この質問とは無関係です。