Come si progetta un Data Access Layer in modo appropriato?

Question

Ho il seguente livello di accesso ai dati (DAL). Mi stavo chiedendo se è impostato correttamente, o se ho bisogno di migliorarlo?

public class User 
{ 

} 

//Persistence methods 
static class UserDataAccess 
{ 
    UsersDAL udal = // Choose SQL or FileSystem DAL impl. 


    InsertUser(User u) 
    { 
     // Custom logic , is 'u' valid etc. 

     udal.Insert(u); 
    } 
} 

abstract class UsersDAL 
{  
    GetUserByID(); 
    InsertUser(u); 
    ... 
} 

// implementaitons of DAL 

static class UsersSQLStore : UsersDAL 
{ 

} 

static class UsersFileSystemStore : UsersDAL 
{ 

} 

ho separato lo strato di stoccaggio dalla classe utente per accedere raccolta metodi che chiamano ulteriormente qualsiasi DAL personalizzato.

L'utilizzo di static nell'implementazione DAL è corretto?

Si prega di suggerire correzioni o modi per migliorarlo. Non ho molta esperienza con la scrittura di codice a strati.

Answer 1

Nessuna di queste classi dovrebbe essere static. Non penso che dovresti nominare le tue classi DAL, perché è un abbreviazione di Data Access Layer, e una classe in sé non è un livello (almeno nella mia mente). È possibile utilizzare invece il termine ampiamente adottato repository. Vi suggerisco di fare qualcosa di simile a quanto segue:

public class User{ 

} 

public abstract class UserRepository{ 
    public abstract void InsertUser(User user); 
} 

public class SqlUserRepository : UserRepository{ 
    public override void InsertUser(User user) 
    { 
     //Do it 
    } 
} 

public class FileSystemUserRepository : UserRepository{ 
    public override void InsertUser(User user) 
    { 
     //Do it 
    } 
} 

public class UserService{ 
    private readonly UserRepository userRepository; 

    public UserService(UserRepository userRepository){ 
     this.userRepository = userRepository; 
    } 

    public void InsertUser(User user){ 
     if(user == null) throw new ArgumentNullException("user"); 
     //other checks 
     this.userRepository.InsertUser(user); 
    } 
} 

Si noti che il UserService viene iniettato con un'istanza della classe astratta UserRepository nel suo costruttore. Puoi utilizzare un framework Dependency Injection (DI) per farlo automaticamente, come ad esempio il Castello di Windsor dal Castle Project. Permetterà di specificare una mappatura dall'astrazione (UserRepository) all'implementazione concreta (ad esempio SqlUserRepository) in un file di configurazione o nel codice.

Spero che questo ti guidi nella giusta direzione e ti chiedo se hai bisogno di ulteriori informazioni.

Answer 2

mio modesto Economia

1. utilizzare le interfacce al posto di classe astratta se utente non ha alcuna gerarchia.
2. Scrivere un DAL generico in modo da poterlo riutilizzare come facciata per il livello DAL.
3. risolvere il dals di cemento da un quadro DI

Answer 3

Davy Brion ha un'eccellente serie di post di blog su questo argomento: lo build your own data access layer series.