Sto usando correttamente Unità di lavoro qui? (Entityi Framework 4 POCO)
-
25-09-2019 - |
Domanda
Ho trovato alcuni esempi di come creare unità di lavoro con EF4, non ho usato di / CIO e vorrei mantenere le cose semplici, e questo un esempio (il 90% di ispirazione), e penso che sia ok, ma da quando ho sto cercando in un modello da utilizzare da ora in poi vorrei chiedere un parere per l'ultima volta.
public interface IUnitOfWork
{
void Save();
}
public partial class TemplateEntities : ObjectContext, IUnitOfWork
{
....
public void Save()
{
SaveChanges();
}
}
public interface IUserRepository
{
User GetUser(string username);
string GetUserNameByEmail(string email);
void AddUser(User userToAdd);
void UpdateUser(User userToUpdate);
void DeleteUser(User userToDelete);
//some other
}
public class UserRepository : IUserRepository, IDisposable
{
public TemplateEntities ctx;
public UserRepository(IUnitOfWork unit)
{
ctx = unit as TemplateEntities;
}
public User GetUser(string username)
{
return (from u in ctx.Users
where u.UserName == username
select u).SingleOrDefault();
}
public string GetUserNameByEmail(string email)
{
return (from u in ctx.Users
where u.Email == email
select u.UserName).SingleOrDefault();
}
public void AddUser(User userToAdd)
{
ctx.Users.AddObject(userToAdd);
}
public void UpdateUser(User userToUpdate)
{
ctx.Users.Attach(userToUpdate);
ctx.ObjectStateManager.ChangeObjectState(userToUpdate, System.Data.EntityState.Modified);
}
public void DeleteUser(User userToDelete)
{
ctx.Users.Attach(userToDelete);
ctx.ObjectStateManager.ChangeObjectState(userToDelete, System.Data.EntityState.Deleted);
}
public void Dispose()
{
if (ctx != null)
ctx.Dispose();
}
}
E infine
public class BogusMembership : MembershipProvider
{
public MembershipCreateStatus CreateUser(string username, string password, string email, bool autoemail, string fullname)
{
IUnitOfWork ctx = new TemplateEntities();
using (UserRepository rep = new UserRepository(ctx))
{
using (TransactionScope tran = new TransactionScope())
{
if (rep.GetUser(username) != null)
return MembershipCreateStatus.DuplicateUserName;
if (requiresUniqueEmail && !String.IsNullOrEmpty(rep.GetUserNameByEmail(email)))
return MembershipCreateStatus.DuplicateEmail;
User userToCreate = new User
{
UserName = username,
PassWord = EncodePassword(password),
FullName = fullname,
Email = email,
AutoEmail = autoemail
};
try
{
rep.AddUser(userToCreate);
ctx.Save();
tran.Complete();
return MembershipCreateStatus.Success;
}
catch
{
return MembershipCreateStatus.UserRejected;
}
}
}
}
}
Dopo sbarazzarsi se l'IUnitOfWork e IDisposal gli sguardi CreateUser come questo:
public MembershipCreateStatus CreateUser(string username, string password, string email, bool autoemail, string fullname)
{
using (TransactionScope tran = new TransactionScope())
{
using (TemplateEntities ctx = new TemplateEntities())
{
UserRepository rep = new UserRepository(ctx);
//OtherRepository rep2 = new OtherRepository(ctx);
if (rep.GetUser(username) != null)
return MembershipCreateStatus.DuplicateUserName;
if (requiresUniqueEmail && !String.IsNullOrEmpty(rep.GetUserNameByEmail(email)))
return MembershipCreateStatus.DuplicateEmail;
User userToCreate = new User
{
UserName = username,
PassWord = EncodePassword(password),
FullName = fullname,
Email = email,
AutoEmail = autoemail
};
try
{
rep.AddUser(userToCreate);
ctx.SaveChanges();
tran.Complete();
return MembershipCreateStatus.Success;
}
catch
{
return MembershipCreateStatus.UserRejected;
}
}
}
}
Soluzione
Questo sembra fondamentalmente OK. Alcuni suggerimenti però:
- Non dovrebbe lasciare il repository smaltire il
TemplateEntities
. La ragione di questo è che quando hai bisogno di due repository all'interno di una transazione, hai un problema. Si dovrebbe spostare la responsabilità dello smaltimento delTemplateEntities
allo stesso livello dellaTransactionScope
; - Il
TransactionScope
dovrebbe essere spostato a un livello superiore. Preferibilmente, ilTemplateEntities
occorre istanziare entro unTransactionScope
; - Non è necessario creare l'involucro
Save
se non contiene la funzionalità. Se si specifica ilvoid SaveChanges()
sull'interfacciaIUnitOfWork
, questo prenderà laSaveChanges
delTemplateEntities
; - Personalmente non avrei
string GetUserNameByEmail(...)
ma piuttostoUser GetUserByEmail(...)
perché allora questo servirà anche il vostro scopo e si ha il vantaggio di non avere due metodi che di ricerca per indirizzo e-mail quando più tardi ha bisogno delUser GetUserByEmail(...)
; - Si consiglia di pensare di fare
ctx
privata, o almeno un setter privata comepublic TemplateEntities Ctx { get; private set; }
; - Si potrebbe creare un repository astratto con metodi come nell'esempio qui sotto. Questo vi farà risparmiare un sacco di battitura noiosa a lungo andare:
-
public interface IRepository<TEntity>
{
void Delete(TEntity entity);
/* ... */
}
public abstract class AbstractRepository<TEntity> : IRepository<TEntity>
{
public TemplateEntities ctx;
public AbstractRepository(IUnitOfWork unit)
{
ctx = unit as TemplateEntities;
}
protected abstract ObjectSet<TEntity> Entites { get; }
public virtual void Delete(TEntity entity)
{
Entities.Attach(entity);
ctx.ObjectStateManager.ChangeObjectState(entity, System.Data.EntityState.Deleted);
}
/* ... */
}
public interface IUserRepository : IRepository<User>
{
User GetUser(string username);
/* ... */
}
public class UserRepository : AbstractRepository<User>, IUserRepository
{
public UserRepository(IUnitOfWork unit)
: base(unit)
{
}
protected override ObjectSet<User> Entites
{
get { return ctx.Users; }
}
public User GetUser(string username)
{
return (from u in ctx.Users
where u.UserName == username
select u).SingleOrDefault();
}
/* ... */
}
Autorizzato sotto: CC-BY-SA insieme a attribuzione
Non affiliato a StackOverflow