Какой код написан лучше?

Question

Я побежал инструмент статического анализа кода на нашем инструменте и, глядя на его результаты ниже код был один из вещей, он говорил о:

SpreadSnapshot oSnap = new SpreadSnapshot(); 
    using (oSnap.SetRowCol(fpSpread, row, col)) 
    { 
     SpreadSetComboBox(fpSpread, list, displayProperty); 
    } 

Так что я изменил его на код ниже и исправлена ​​ошибка что инструмент говорил:

using (SpreadSnapshot oSnap = new SpreadSnapshot()) 
    { 
     oSnap.SetRowCol(fpSpread, row, col); 
     SpreadSetComboBox(fpSpread, list, displayProperty); 
    } 

Так, по вашему мнению Какого стиля кодирования вы считаете более подходящие и менее подвержены ошибкам?

Благодаря

Answer 1

Эти два значения означают совершенно разные вещи, если SetRowCol не возвращает this в конце. Во-первых, вы удаляете результаты SetRowCol. Во втором вы удаляете SpreadSnapshot.

Если оба являются одноразовыми, вы должны сделать, используя для обоих:

using (SpreadSnapshot oSnap = new SpreadSnapshot()) 
using (oSnap.SetRowCol(fpSpread, row, col)) 
{ 
    SpreadSetComboBox(fpSpread, list, displayProperty); 
} 

Answer 2

Последнее - это гарантирует, что вы не в конечном итоге с помощью oSnapпослеusing заявление.

Помимо всего прочего, было бы довольно странно для SetRowCol вернуть что-то одноразовое ... что бы это означало?