关于 C#“使用”的风格/可读性问题陈述

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。

我内心的整洁主义者想将其更改为：

{
    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();
    ...
}

出于某种原因，我觉得这样做有点肮脏。你觉得这样不好吗？如果您认为这不好，为什么？如果好的话，为什么？

编辑：请注意，这只是众多示例中的一个（有些人为的）。请不要关注这样一个事实：这恰好表明记录器类的接口考虑不周。这与我的问题无关，而且我也无权改进课程本身。

Answer 1

他们都很可怕。两者都不做。

您正在制作我所说的“高维护等级”在这里。高维护班有一份合同，上面写着“我要求你给我一堆资源，你需要知道我什么时候用完它们并适当地清理它们”。这个契约意味着类的用户必须知道类是如何实现的，从而违反了最初创建类的封装和抽象原则。

您可以通过评论来判断这一点：这是使用连接的地方，我知道不会再次使用该连接。你怎么知道的？您只知道这是否是该类的记录在案的合同。这对于强加给一个类的消费者来说并不是一个好的契约。

一些改进的方法：

1）使记录器成为一次性的。完成后让它清理连接。这样做的缺点是记录器保持连接的时间超过了必要的时间。

2）使InsertData将连接作为参数。调用者仍然可以负责清理连接，因为记录器不会保留它。

3）创建第三个类“Inserter”，它是一次性的，并在其构造函数中接受日志和连接。插入器在废弃连接时将其废弃；然后调用者负责处理插入器。

Answer 2

我同意理想情况下 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();
    ...
}

Answer 3

我会倾听你内心的整洁。我个人更喜欢他的方式。

Answer 4

两种使用using的方式是完全等价的。无论哪种方式，您最终都会得到一个仍在范围内但具有已释放连接的日志。更好的方法是使日志成为一次性日志，并让其 dispose 方法处理其连接，将日志放入 using 语句中，而不是连接本身。

Answer 5

如果 log 实现 IDisposable，则执行第二个选择，因为大括号是显式的。在某些情况下，您可以使用多个 using 语句：

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



}

您可以只使用一组大括号。这可以避免疯狂的缩进。