¿Es este diseño/implementación de mala o incorrecta de la unidad de trabajo y el patrón de repositorio (con transacción) utilizando petapoco?

Question

Soy nuevo en esta gran herramienta Micro-anm (Petapoco) y me pregunto cómo implementar el patrón de UOW y repositorio utilizando Petapoco en el proyecto web. Me han leído algunos artículos pero no tengo buenas ideas sobre cómo diseñar/implementar. ¿Podría alguien proporcionar algún ejemplo de producción o dirigirme a lograr esto?

Aquí está mi pensamiento y código de implementación de patateo, por favor conseguí o comente si me equivoco.

public interface IUnitOfWork
{
    void StartNew();
    void Commit();
    void Rollback();
}

public class PetaPocoUnitOfWork : IUnitOfWork
{
    private PetaPoco.Database _db = null;

    public PetaPocoUnitOfWork(PetaPoco.Database db)
    {
        _db = db;
    }
    public void StartNew()
    {
        _db.BeginTransaction();
    }

    public void Commit()
    {
        _db.CompleteTransaction();
    }

    public void Rollback()
    {
        _db.AbortTransaction();
    }
}

public class UnitOfWorkFactory
{
    public static IUnitOfWork GetInstance()
    {
        return new PetaPocoUnitOfWork(Project.Core.Domain.ProjectDb.GetInstance());
    }
}

interface IRepository<T>
{
    void Insert(T entity);
    void Update(T entity);
    void Delete(T entity);
    List<T> FetchAll();
    List<T> FetchAll(int startIndex, int endIndex, int count);
    T Fetch(int uid);
    void SaveChanges();
}

public class TireRepository : IRepository<Tire>
{
    private IUnitOfWork _uow = null;
    public TireRepository(IUnitOfWork uow)
    {
        _uow = uow;
    }
    public void Insert(Tire entity)
    {
        var db = ProjectDb.GetInstance();
        db.Save(entity);
    }

    public void Update(Tire entity)
    {
        throw new NotImplementedException();
    }

    public void Delete(Tire entity)
    {
        var db = ProjectDb.GetInstance();
    }

    public List<Tire> FetchAll()
    {
        throw new NotImplementedException();
    }

    public List<Tire> FetchAll(int startIndex, int endIndex, int count)
    {
        throw new NotImplementedException();
    }

    public Tire Fetch(int id)
    {
        throw new NotImplementedException();
    }

    public void SaveChanges()
    {
        _uow.Commit();
    }
}

Aquí está el caso de prueba simple, y probablemente sea el uso de la llamada de capa de servicio.

    [TestMethod()]
    public void InsertTest()
    {
        IUnitOfWork uow = Xyz.Core.UnitOfWorkFactory.GetInstance();
        TireRepository target = new TireRepository(uow);
        Tire entity = new Tire();
        entity.Description = "ABCD";
        entity.Manufacturer = 1;
        entity.Spec = "18R/V";
        try
        {
            target.Insert(entity);
            target.SaveChanges();
        }
        catch
        {
            uow.Rollback();
        }
    }

Planeo usar AutoFAC para ser mi solución IOC y inyectaré una instancia de UOW según la solicitud HTTP al objeto Repositorio.

Por favor, déme algún comentario o consejo si este código es incorrecto o malo. Muchas gracias.

Answer 1

El código es correcto. Solo un poco excesivo en mi humilde opinión. Una cosa: cambiaría el nombre de SaveChanges a CommitChanges Para una mejor claridad.