Rob*_*Rob 2 c# t-sql performance transactions linq-to-sql
这是我在ASP.NET应用程序生命周期中定期运行的方法的第一次破解,以清理存储在我的数据库中的过期会话.它似乎工作得很好,但我的软件工程师对这段代码感觉不对.我已经使用LINQ to SQL几个月了,但我对以下代码不太自信.我担心一些事情:
以下代码是否可以在我的应用程序中的不同线程访问数据库的情况下运行?我对交易的想法有一个很好的理解,但我想确保我正确使用它们.
我的查询是否会导致性能问题?或者在这种情况下选择此特定表中的所有记录是否合适?此方法仅每15分钟运行一次,因此不会在短时间内反复进行查询.
有没有更好的方法可以做到这一点?我有一种唠叨的感觉.
码:
/// <summary>
/// Method, run periodically, to remove all sign in records that correspond to expired sessions.
/// </summary>
/// <param name="connectionString">Database connection string</param>
/// <returns>Number of expired sign in records removed</returns>
public static int Clean(String connectionString)
{
MyDatabaseDataContext db = new MyDatabaseDataContext(connectionString);
var signIns = db.SignIns.Select(x => x);
int removeCount = 0;
using (TransactionScope scope = new TransactionScope())
{
foreach (SignIn signIn in signIns)
{
DateTime currentTime = DateTime.Now;
TimeSpan span = currentTime.Subtract(signIn.LastActivityTime);
if (span.Minutes > 10)
{
db.SignIns.DeleteOnSubmit(signIn);
++removeCount;
}
}
db.SubmitChanges();
scope.Complete();
}
return removeCount;
}
Run Code Online (Sandbox Code Playgroud)
这听起来像你可以在sproc中轻松做的事情.SQLServer为您提供了一个返回当前时间的GETDATE()方法......我不明白为什么您不能这样做
DELETE * FROM tblSignIns
WHERE LastActivityTime < DATEADD("minute", -10, GETDATE());
Run Code Online (Sandbox Code Playgroud)
那会不会做同样的事情?