Вопрос о стиле/удобочитаемости в отношении оператора C# «using»

Question

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

Я иду через решение, добавляя using заявления в довольно многих местах. Часто натыкаюсь что-то вроде этого:

{ 
    log = new log(); 
    log.SomeProperty = something; // several of these 
    log.Connection = new OracleConnection("..."); 
    log.InsertData(); // this is where log.Connection will be used 
    ... // do other stuff with log, but connection won't be used again 
} 

где log.Connection является OracleConnection, который реализует IDisposable.

neatnik во мне хочет изменить его на:

{ 
    using (OracleConnection connection = new OracleConnection("...")) 
    { 
     log = new log(); 
     log.SomeProperty = something; 
     log.Connection = conn; 
     log.InsertData(); 
     ... 
    } 
} 

Но любитель краткости и получить без отрыва от работы сделано, слегка быстрее хочет сделать:

{ 
    log = new log(); 
    log.SomeProperty = something; 
    using (log.Connection = new OracleConnection("...")) 
     log.InsertData(); 
    ... 
} 

Для по какой-то причине я чувствую себя немного грязно, делая это. Считаете ли вы это плохо или нет? Если вы считаете, что это плохо, почему? Если это хорошо, почему?

EDIT: Обратите внимание, что это всего лишь один (несколько надуманный) пример многих. Пожалуйста, не зацикливайтесь на том, что это происходит, чтобы указать класс журнала с плохо продуманным интерфейсом. Это не относится к моему вопросу, и я не вправе вообще улучшать классы.

Answer 1

Я согласен с тем, что в идеале log сам должен реализовать IDisposable, но предположим, что это невозможно, и обратимся к вопросу, который фактически задал ОП.

Второй способ лучше, просто потому, что это меньше кода, который выполняет одно и то же. Нет смысла вводить здесь дополнительную переменную connection.

Также обратите внимание, что вы можете выполнить другую инициализацию вне блока using. Здесь это не имеет значения, но может иметь значение, если вы «используете» какой-то действительно дорогой ресурс. То есть:

log = new log(); 
log.SomeProperty = something; // This can be outside the "using" 
using (OracleConnection connection = new OracleConnection("...")) 
{ 
    log.Connection = conn; 
    log.InsertData(); 
    ... 
} 

Answer 2

Я бы прослушал неатик в вас. Мне нравится его путь лучше лично.

Answer 3

Они оба ужасные. Ни одного из них.

Вы делаете то, что я называю «high maintenance class» здесь. В классе высокого обслуживания есть контракт, в котором говорится: «Я требую, чтобы вы дали мне кучу ресурсов, и вам нужно знать, когда я с ними поработаю, и тщательно их очистить». Этот контракт означает, что пользователь класса должен знать, как реализуется класс, тем самым нарушая принцип инкапсуляции и абстракции, который мотивировал создание класса в первую очередь.

Вы можете сообщить об этом на ваш комментарий: Здесь используется соединение, я знаю, что соединение не будет использоваться снова. Откуда ты это знаешь? Вы только знаете, что если это документально подтвержденный договор класса. Это не хороший контракт, чтобы навязать потребителю класс.

Некоторые способы сделать это лучше:

1) сделать регистратор для одноразового использования. Очистите соединение, когда это будет сделано. Нижняя сторона этого заключается в том, что регистратор удерживает соединение дольше, чем необходимо.

2) make InsertData принимает соединение в качестве параметра. Вызывающий абонент все еще может нести ответственность за очистку соединения, потому что регистратор не удерживает его.

3) сделать третий класс «вставкой», который является одноразовым и берет журнал и соединение в его конструкторе. Вставка устанавливает соединение, когда оно расположено; тогда вызывающий абонент несет ответственность за удаление вставки.

Answer 4

Если log реализует IDisposable, то сделайте второй выбор, поскольку фигурные скобки явны.В некоторых случаях вы можете использовать несколько using заявления:

using (Graphics g = ...) 
using (Pen p = new Pen ...) 
using (Font f = new Font ...) 
{ 



} 

, где вы можете уйти с использованием только 1 набор скобок. Это позволяет избежать сумасшедших отступов.

Answer 5

Два способа использования using полностью эквивалентны. В любом случае вы получаете журнал, который все еще находится в области видимости, но имеет удаленное соединение. Лучше сделать лог одноразового использования, и его метод удаления утилизирует его соединение, помещая журнал в оператор using, а не само соединение.