desde hace poco que estoy desarrollando con netcore y aún hay temas que no entiendo del todo. En el siguiente código tengo un controlador al que por Inyección de Dependencias le paso un repositorio.
En el repositorio hago una serie de acciones (entre ellas llamar a otro repositorio).
public class AppController : ControllerBase
{
public virtual async Task<IActionResult> Put(int id, TEntity data)
{
//AppRepo
await repository.Update(data);
return NoContent();
}
}
public class AppRepo<App, DataContext> {
private DataContex _ctx ;
public async Task<App> Add(App entity)
{
await _ctx.Apps.AddAsync(entity);
await _ctx.SaveChangesAsync()
return entity;
}
public async Task<App> Update(App data){
_ctx.Entry(data).State = EntityState.Modified;
App newData = await Add(data);
var dependiente = new AppDependenciasRepo(_ctx);
await dependiente.updateChangesApp(data.Id, data.Dependencias);
await _ctx.SaveChangesAsync()
}
}
public class AppDependenciasRepo<Dependencia, DataContext>
{
public async Task updateChangesApp(int ApplicationId, List<Dependencia> nuevos){
Dependencias old = new Dependencias () { ApplicationId = ApplicationId };
_ctx.Dependencias.Attach(old);
_ctx.Dependencias.Remove(old);
await _ctx.Dependencias.AddRangeAsync(nuevos);
await _ctx.SaveChangesAsync()
}
}
He puesto en cada método de los repositoris _ctx.SaveChangesAsync(), para cuando se hagan llamado independiente se escriban los datos en la db.
Pero cuando hago operaciones mas complejas comienzan las dudas:
_ctx.Entry(data).State = EntityState.Modified;
App newData = await Add(data);
En las sentencias anteriores quiero modificará el estado y luego agrego un nuevo dato, entiendo que cuando termine Add(data), también se escribirá el nuevo estado de la entidad; por lo que el contexto _ctx, ya no tendrá cambios pendientes.
En el método Update de AppRepo, llamo a updateChangesApp, este primero borra datos para luego ingresar datos substitutos. Como estoy utilizando Async tengo el temor de que primero agregue los datos y luego los borre todos.
_ctx.Dependencias.Remove(old);
await _ctx.Dependencias.AddRangeAsync(nuevos);
await _ctx.SaveChangesAsync()
¿Son correctos mis suposiciones, es así como se debe trabajar o tal vez debería crear una Unidad de Trabajo para encapsular el llamado a SaveChangesAsync?.
SaveChanges
al repository porque no es su trabajo. Y tampoco lo es del controlador, que no debería decidir cuándo guardar datos. El controlador sólo debería mantener la comunicación con terceros (control de errores, mapping de datos, ...).AddRange
). Si en algún punto del loop quiero parar, puedo, y cancelo toda la transacción. Si para cadaAdd
se llamada a unSave
, no podría.Add
implicaría un viaje a la base de datos. Si agrupas varias unit of work te permites el lujo de volcar varios cambios en el contexto virtual de EF de una sola llamada a la base de datos. Las últimas líneas de código son claramente un ejemplo de cómo se vería un método de un servicio, no de un repository. Eso es lógica de negocio, no lógica de la capa de acceso a datos.