class MyController
{
    [ThreadStatic] private DbInterface db;

    public void ImportAllData()
    {
        using (db = new DbInterface())
        {
            var records = PullData();
            PushData(records);
        }
    }

    private DbRecord[] PullData()
    {
        return db.GetFromTableA();
    }

    private void PushData(DbRecord[] records)
    {
        db.InsertIntoTableB(records);
    }
}

替代方案是更加麻烦维护.

class MyController
{
    public void ImportAllData()
    {
        using (var db = new DbInterface())
        {
            var records = PullData(db);
            PushData(records,db);
        }
    }

    private DbRecord[] PullData(DbInterface db)
    {
        return db.GetFromTableA();
    }

    private void PushData(DbRecord[] records,DbInterface db)
    {
        db.InsertIntoTableB(records);
    }
}

据我所见,我的第一个实现：

>是线程安全的(假设DbInterface是线程安全的),
>阻止任何其他进程触摸db变量,
>确保db将始终处理,即使在异常.

在具有类范围的变量上使用using语句是不好的做法吗？我错过了什么吗？