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

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

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

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

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

Они оба ужасны. Не делайте ни того, ни другого.

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

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

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

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

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

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

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

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

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

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



}

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