Как избежать частной собственности null проверить сделать ленивую загрузку?

Question

У меня есть класс с 20 полями, которые заполняются из базы данных SQL при загрузке. В настоящее время я вызываю метод данных загрузки сразу после конструктора, который вызывает SQL proc и заполняет все необходимые поля. Время от времени я вообще не могу получить доступ к этим 20 полям, добавляя дополнительную стоимость вызова SQL, даже если это не требуется. Поэтому я изменил все свойства, чтобы иметь связанное личное свойство, и когда программа вызывает общедоступное свойство, сначала проверяю частное свойство, и если оно равно null, это означает, что нам нужно загрузить данные из sql, поэтому я вызываю метод загрузки. Он отлично работает, но когда я вижу код, повторяется шаблон нулевой проверки и загрузка SQL-запроса. Есть ли лучший способ сделать это?

private string _name; 
public string Name 
{ 
    get { 
     if (_name == null) 
     LoadData(); //this popultes not just but all the properties 
     return _name; 
    } 
} 

Answer 1

Нет, это право. Here is the wikipedia article. Накладные расходы нулевой проверки будут очень минимальными по сравнению с ненужными вызовами базы данных. Теперь, если пользователи программы фактически используют значения в 99% случаев, я бы сказал, что этот шаблон не нужен.

Только одно примечание: если какое-либо из ваших значений может быть пустым, тогда вы будете делать ненужные вызовы базы данных. Может быть, лучше сделать что-то вроде этого (который будет еще быстрее проверить, так как это просто немного проверка):

//Constructor default to not loaded 
bool isLoaded = false; 

private string _name; 
public string Name 
{ 
    get { 
     if (!isLoaded) 
     LoadData(); //this popultes not just but all the properties 
     return _name; 
    } 
} 

private LoadData() 
{ 
    //Load Data 
    isLoaded = true; 
} 

Answer 2

Ну вы могли бы изменить его на:

if (!initialized) 
    LoadData(); 

И в вашем LoadData установлен в значение true, но это действительно не меняет семантики.

Answer 3

Одна вещь, которую вы можете сделать, это извлечь, если в отдельный метод, так каждое свойство содержит только один дополнительный вызов:

void EnsureData() 
{ 
     if (!dataLoaded) 
     LoadData(); //this populates all the properties 
} 

public string Name { 
    get { 
     EnsureData(); 
     return _name; 
    } 
} 

Answer 4

Я думаю, вы должны рассмотреть структуру приложения. Почему бы вам даже создать экземпляр класса, если вы не собираетесь использовать свойства? Я считаю, что на самом деле чище вызывать SQL после кода конструктора, но только создавайте объекты своего класса, если собираетесь его использовать. Другое более гибкое решение делает публикацию LoadData общедоступной и вызывает ее по мере необходимости из экземпляра объекта по мере необходимости.

Answer 5

Я участвую в процессе обучения шаблонов проектирования. У меня есть одно предложение, если вы загружаете данные только после того, как сможете попробовать с шаблоном проектирования singleton.

public class Singleton123 
{ 
    private static readonly string _property1 = ClassLoadData.LoadData(); 

    public static string MyProperty1 
    { 
     get 
     { 
      return _property1; 
     } 
    } 

} 

public class ClassLoadData 
{ 
    public static string LoadData() 
    { 
     // any logic to load data 
     return "test"; 
    } 
} 

вызовов свойство, как показано ниже

Singleton123 obj = new Singleton123(); 
     string stra = Singleton123.MyProperty1; 
     string strb = Singleton123.MyProperty1; 

это свойство будет загружен только один раз.

Answer 6

Btw C# имеет стандартную реализацию ленивых погрузчиков. Почему бы не использовать его, вместо того, чтобы предоставлять флаги isSomethingLoaded? :)

public class Bar 
{ 
    private Lazy<string> _name = new Lazy<string>(() => LoadString()); 

    public string Name 
    { 
     get { return _name.Value; } 
    } 
} 

В случае не статический метод LoadString, ленивый-загрузчик должен быть инициализирован конструктором;