Design and Architecture

Kevin Marois wrote:

I've never worked with concurrency in SQL before.

Member 13897152 wrote:

What if the state includes resources that have to be explicitly released (like file handles, OpenGL textures etc)?

Member 13897152 wrote:

How would you implement Memento when state has resources to be released?

Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.

if XMLTV = 'something':
    import xmltva
    # remaining code here
else:
    import xmltvb
    # remaining code here

if os.environ['XMLTV'] == 'bbc':
   import xmltv.bbc

Kevin Marois wrote:

var test = projectRepo.FindProjectsByName(entity.ProjectName);

if (test != null)
{
    projectRepo.Add(entity);

    foreach (var task in entity.Tasks)
    {
        projectTaskRepo.Add(task);
    }

    dc.SaveChanges();
}
else
{
    throw new DuplicateProjectException(entity.ProjectName);
}

var test = projectRepo.FindProjectsByName(entity.ProjectName);
if (test.Any()) throw new DuplicateProjectException(entity.ProjectName);

projectRepo.Add(entity);

foreach (var task in entity.Tasks)
{
    projectTaskRepo.Add(task);
}

dc.SaveChanges();

Kevin Marois wrote:

IEnumerable<TEntity> GetAll();
IEnumerable<TEntity> Find(Expression<Func<TEntity, bool>> predicate);

Richard Deeming wrote:

Purists frown on it,

interface IUnitOfWork : IDisposable
{
    void Save();
    Task SaveAsync();
}

// NB: Could implement the IUnitOfWork interface directly on the FalconContext class.
class FalconContextUnitOfWork : IUnitOfWork
{
    private readonly FalconContext _context;
    
    public FalconContextUnitOfWork(FalconContext context)
    {
        _context = context;
    }
    
    public void Save()
    {
        _context.SaveChanges();
    }
    
    public Task SaveAsync()
    {
        return _context.SaveChangesAsync();
    }
    
    public void Dispose()
    {
        _context.Dispose();
    }
}

interface IBizObject : IDisposable
{
    void AddProject(Project entity);
}

class BizObject : IBizObject
{
    private readonly IProjectsRepository _projectsRepository;
    private readonly IProjectsTaskRepository _projectsTaskRepository;
    private readonly IUnitOfWork _unitOfWork;
    
    public BizObject(IProjectsRepository projectsRepository, IProjectsTaskRepository projectsTaskRepository, IUnitOfWork unitOfWork)
    {
        _projectsRepository = projectsRepository;
        _projectsTaskRepository = projectsTaskRepository;
        _unitOfWork = _unitOfWork;
    }
    
    public void AddProject(Project entity)
    {
        if (_projectsRepository.FindProjectsByName(entity.ProjectName).Any())
        {
            throw new DuplicateProjectException(entity.ProjectName);
        }
        
        _projectsRepository.Add(entity);

        foreach (var task in entity.Tasks)
        {
            _projectsTaskRepository.Add(task);
        }
        
        _unitOfWork.Save();
    }
    
    public void Dispose()
    {
        _unitOfWork.Dispose();
    }
}

public class ProjectsController : ApiController
{
    private IBizObject _bo;
    
    public ProjectsController(IBizObject bo)
    {
        _bo = bo;
    }

    public IHttpActionResult AddProject(Project entity)
    {
        _bo.AddProject(entity);
        return Ok();
    }
    
    protected override void Dispose(bool disposing)
    {
        base.Dispose(disposing);
        if (disposing) _bo.Dispose();
    }
}

Richard Deeming wrote:

Register all of the concrete implementations as transient services in your DI container