попытка поймать блок место

Question

У меня есть следующий сценарий, когда я называю метод в моем коде доступа к данным из моего бизнес-слоя:

//Call the method from BL 
SqlHandler sh = new SqlHandler(); 
var names = sh.GetNames(); 

Метод например в DAL:

public IEnumerable<string> GetNames() 
{ 
    string conString = GetOpenConnection(); 
    using (SqlConnection connection = new SqlConnection(conString)) 
    { 
     connection.Open(); 
     using(SqlCommand cmd = new SqlCommand("SELECT Name From Names")) 
     { 
      //... 
     } 
    } 
} 

Моя потребность - регистрировать любое исключение, которое происходит в моем DAL, и отображать сообщение пользователю с соответствующим сообщением. Насколько я могу видеть, у меня есть следующие варианты:

1) Окружите только вызов BL и журнал и вывести на экран сообщение оттуда:

try 
{ 
    SqlHandler sh = new SqlHandler(); 
    var names = sh.GetNames(); 
} 
catch (Exception ex) 
{ 
    ex.LogException(); 
    MessageBox.Show(ex.Message); 
} 

2) объемного звучания обоих вызовов и расколоть протоколирования часть и уведомляющее часть на 2 секции:

try 
{ 
    SqlHandler sh = new SqlHandler(); 
    var names = sh.GetNames(); 
} 
catch (Exception ex) 
{ 
    MessageBox.Show(ex.Message); 
} 

public IEnumerable<string> GetNames() 
{ 
    try 
    { 
     string conString = GetOpenConnection(); 
     using (SqlConnection connection = new SqlConnection(conString)) 
     { 
      connection.Open(); 
      using (SqlCommand cmd = new SqlCommand("SELECT Name From Names")) 
      { 
       //... 
      } 
     } 
    } 
    catch (Exception ex) 
    { 
     ex.LogException(); 
     //propagate the exception to the caller 
     throw; 
    } 
} 

3) я, конечно, можно было бы что-то пропустил, что я был бы рад узнать, о

что является предпочтительным APPRO в плане архитектуры приложения здесь? Есть ли разница между двумя верхними?

Answer 1

Я бы предложил третий вариант: Положите регистрацию ошибок и отображение ошибок в слое пользовательского интерфейса.

Зарегистрируйте central error handler в своем пользовательском интерфейсе. Там вы можете

• дисплей ошибка (либо простой MessageBox или фантазии окно с «более подробно» и «Отчет для разработчиков» кнопки) и

• регистрировать все,

без помех каждый BL-метод с уловками-catch-all try-catch.

Это имеет много преимуществ:

• Это будет сделать легче читать ваш код BL.
• Вы не можете забыть a BL метод.
• Легче сделать (централизованное) изменение кода обработки ошибок.
• Вы не связываете свою бизнес-логику с конкретным типом пользовательского интерфейса, который имеет множество преимуществ самостоятельно: например, вы можете повторно использовать его в веб-проекте позже, и вы можете проверить свою бизнес-логику с модульные испытания.

Answer 2

Прежде всего: Всегда поймать только исключения можно обрабатывать (See Eric Lippert Post)
Вы можете следовать этой стратегии:

• Если вы не можете оправиться от исключения сделать это в DAL слоя только и запишите его.
• try {} catch{} в слое BAL, если пользователь может его обработать.