必要に応じてオブジェクトを作成する正しい方法 - asp.net-mvc、dependency-injection、ninject

私は最初のDIコンテナーとしてNinjectを学んでいます。すべてのコンテナーが同様のアイデアを持っていると思うので、ここに私の質問があります。

関数を持つクラスUserRepositoryがあります：

//constructor
public UserRepository(Kernel kernel) { _kernel = kernel; }

public UserDataModel CreateNewUser(string title, string firstname,
string lastname, int age ...)
{
//I have the kernel registered everything properly
//Getting a new user object from the kernel
var user = _kernel.Get<UserDataModel>();

user.Title = title;
user.Firstname = firstname;
user.Lastname = lastname;
...
return user;
}

カーネルを使用する正しい方法ですか？他の投稿から、カーネルを公開して渡すことは悪い悪い考えだと読んでいます。もしそうなら、それを行う正しい方法は何ですか？

EDIT

tempUserオブジェクトの使用をデモするコードの一部

public int UpdateSaUser(Guid msuserid, UserProfile saprofile,
string accesstoken = "")
{
using (var db = new SaModelContainer())
{
var row1 = MakeNewRow(msuserid, saprofile, accesstoken);
var row2 = db.SaUser.SingleOrDefault(
r => r.MsUserId.Equals(msuserid));
if (row2 == null) { //add new
db.sauth_User.Add(row1);
} else { //modify
var entry = db.Entry(row2);
entry.OriginalValues.SetValues(row2);
entry.CurrentValues.SetValues(row1);
}
return db.SaveChanges();
}
}

private SaUser MakeNewRow(Guid msuserid, UserProfile saprofile,
string accesstoken = "")
{
var row = new SaUser {
MsUserId = msuserid,
Provider = (int)saprofile.Provider,
ProfileId = saprofile.ID,
Email = saprofile.Email,
AccessToken = accesstoken,
FirstName = saprofile.FirstName,
LastName = saprofile.LastName,
FullName = saprofile.FullName,
UserName = saprofile.Username,
DisplayName = saprofile.DisplayName,
Country = saprofile.Country,
DoB = StringParser.ParseDate(saprofile.DateOfBirth),
Gender = (int)saprofile.GenderType,
Language = saprofile.Language,
ProfileURL = saprofile.ProfileURL,
ProfilePictureURL = saprofile.ProfilePictureURL
};

return row;
}

回答：

依存性を注入することも、依存性注入として知られています。カーネルを注入する代わりに。

例（これはコンストラクターインジェクションとして知られています）：

private UserDataModel _userDataModel;

[Inject]
public UserRepository(UserDataModel userDataModel)
{
_userDataModel = userDataModel;
}

//Any other method can now use _userDataModel

あなたはおそらくあなたのインターフェースを作るべきです UserDataModel そのため、単体テストでモックまたはスタブできます。そして、ninjectを使用して、そのインターフェイスを正しい実装にバインドします。例を参照してください。その場合、属性は必要ありません。

Ninject MVCドキュメント

良いmvcの例

UserDataModelsのリストは依存関係ではありません。 I.E. 何らかの種類のデータベースまたはその他の依存関係（WCFサービス、Webサービス、Entity Framework、nHibernateなど）があり、そこからユーザーが依存しているのです。

public UserRepository(EntityFrameworkContext context)
{
//The entity framework context is the dependency that you don"t want to test.
_context = context;
}

public List GetAllUsers()
{
return _context.Users.ToList();
}