repozytorium XML; do Save() lub nie do Save()

Question

Co byłoby lepsze podejście do repozytorium oparte na XML:

1) Zapisz zmiany bazowego dokumentu XML na każdym wywołaniu repozytorium ...

public class XmlRepository1 
{ 
    private XDocument xDocument; 

    public void CrudOp() 
    { 
     // Perform CRUD operation... 

     // Call Save() 
     xDocument.Save(path); 
    } 
} 

lub

2) podać użytkownikowi końcowemu metodą SaveChanges() ...

public class XmlRepository2 
{ 
    private XDocument xDocument; 

    public void CrudOp() 
    { 
     // Perform CRUD operation... 

     // DON'T call save 
    } 

    // Provide a SaveChanges() method to the end-user... 
    public void SaveChanges() 
    { 
     xDocument.Save(path); 
    } 
} 

Moja skłonność opiera się na opcji 1, ponieważ dostarczenie metody SaveChanges() nie wydaje się odpowiedzialnością repozytoriów. Jednakże, po drugie, zgaduję tę decyzję z kilku powodów:

a) W środowisku wielowątkowym daje to użytkownikowi końcowemu łatwy sposób na wycofanie zmian w przypadku niepowodzenia połączenia z repozytorium , pozostawiając obiekty w stanie częściowo zmutowanym.

b) Opcja 2 zapewnia paradygmat typu "batch-like", który z różnych powodów uważam za bardziej elastyczny.

Answer 1

Zastanówcie się nad dodaniem obsługi transakcji (w pobliżu drugiej aplikacji).

public class XmlRepository2 
    { 
    public void CrudOp() 
    { 
     // DON'T call save 
    } 

    public MakeTransacedChanges(Action<XmlRepository2> makeChanges) 
    { 
     try{ 
      makeChanges(this); 
      saveChanges(); 
     } 
     catch (RepositoryException e) 
     { 
      //revert changes 
     } 
    } 

    private void saveChanges() 
    { 
     xDocument.Save(path); 
    } 
    } 

Answer 2

Wolę mieć oddzielną metodę zapisu w repozytorium, aby mieć szansę na cofnięcie zmian, jeśli coś pójdzie nie tak.

Znalazłem ten artykuł Repositories and the Save Method. Mam nadzieję, że to pomoże.