Czy identyfikator obiektu biznesowego powinien być tylko do odczytu, czy nie?

Question

Moim zdaniem pole ID obiektu biznesowego powinno być tylko do odczytu (publiczne pobranie i prywatny zestaw), ponieważ z definicji identyfikator nigdy się nie zmieni (ponieważ jednoznacznie identyfikuje rekord w bazie danych).

Powoduje to problem podczas tworzenia nowego obiektu (identyfikator jeszcze nie ustawiony), zapisz go w bazie danych za pomocą procedury przechowywanej na przykład, która zwraca nowo utworzony identyfikator, a następnie w jaki sposób zapisać go z powrotem w obiekcie, jeśli Właściwość ID jest tylko do odczytu?

Przykład:

Employee employee = new Employee(); 
employee.FirstName="John"; 
employee.LastName="Smith"; 

EmployeeDAL.Save(employee); 

Jak działa metoda Save (które w rzeczywistości łączy się z bazą danych, aby zapisać nowego pracownika) zaktualizować właściwość IDpracownika w obiekcie pracownika, jeśli ta właściwość jest tylko do odczytu (który powinien być ponieważ EmployeeId nigdy się nie zmieni, gdy zostanie utworzony).

Wygląda na to, że Id powinien być zapisywalny przez DAL i tylko do odczytu dla reszty świata. Jak to zaimplementować, szczególnie jeśli klasy DAL i obiekty biznesowe są w różnych złożeniach?

Nie chcę tworzyć metody Save w klasie Employee, ponieważ ta klasa nie powinna mieć nic wspólnego z bazą danych.

Answer 1

Innym możliwym rozwiązaniem jest zadeklarować jako pracownika: -

public class Employee 
{ 
    public int Id { get; internal set; } 
} 

... pod warunkiem, że zajęcia pracownicze i DAL są w tym samym zespole

Nie twierdzę się podoba, ale mam Użyj tego.

Answer 2

Jak o:

Employee employee = new Employee(EmployeeDAL.GetNextID()); 

ten powinien również dokonać zapisać kod prostsze.

Answer 3

można dokonać tylko metoda DAL powrócić zaktualizowaną obiektu:

public class EmployeeDAL 
{ 
    Employee EmployeeDAL.Save (Employee employee) 
    { 
     // Save employee 
     // Get the newly generated ID 
     // Recreate the object with the new ID and return it 
    } 
} 

Alternatywnie, można wygenerować nowy identyfikator w kodzie, instancję obiektu z tego identyfikatora następnie zapytać DAL aby go zapisać.

Jeśli chcesz, aby Twój obiekt był aktualizowany podczas operacji zapisu, musisz ustawić tę właściwość jako publiczną.

Ja osobiście lubię tworzyć niezmienne obiekty, które można ustawić tylko raz, przekazując wszystkie wartości do konstruktora. Przy takim podejściu wystarczy utworzyć obiekt do zapisania, a następnie pobrać go z powrotem wraz z przypisanym identyfikatorem z bazy danych i zwrócić go do osoby dzwoniącej.

Answer 4

Można dokonać setter dla identyfikatora pozwalają ustawić tylko wtedy, gdy nie jest już ustawiony wcześniej:

public class Employee 
{ 
    private int? m_ID; 

    public int? ID 
    { 
     get { return m_ID; } 
     set 
     { 
      if (m_ID.HasValue()) 
       throw ... 
      m_ID = value; 
     } 
    } 
} 

Alternatywnie, myślę, że pewne ramy wsparcia tego typu funkcjonalności (na przykład, myślę, NHibernate będzie pozwala na ustawienie prywatnego settera w polu ID).

Answer 5

Jak o tylko pozwalając kodu poza DAL odnieść się do obiektu poprzez interfejs, który nie dostarcza setter dla pola ID (i wszelkie inne pola niezmienne):

public interface IEmployee 
{ 
    Int32 Id {get;} 
    String Name {get;set;} 
    // ... and so on ... 
} 

public class Employee: IEmployee 
{ 
    Int32 Id {get;set;} 
    String Name {get;set;} 
} 

DAL może ustawić w razie potrzeby, ale kod zużywający się nie może.