我有类似于以下的代码。
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
在异常期间,也将始终释放 ensure。
using
在具有类范围的变量上使用语句是不好的做法吗?我错过了什么吗?