LINQify我的代码值得在闭包中访问foreach变量吗?
作者:互联网
令人想起过去的盗版现场滚石唱片的标题,Resharper比我以往任何时候都更加清晰.当我检查代码时,它告诉我有关闭包的信息:
1)“循环:
foreach (var item in PlatypiIds)
{
var query = db.Table<Locations>().Where(l => l.PlatypusId == item).
Where(l=> l.SentTimeUTC >= EarliestToShow).
Where(l=> l.SentTimeUTC <= LatestToShow).
OrderBy(l => l.SentTimeUTC);
if (query != null)
{
foreach (var q in query)
{
listLocs.Add(q);
}
}
}
…可以转换为LINQ表达式:
listLocs.AddRange(from item in PlatypiIds select db.Table<Locations>().Where(l => l.PlatypusId == item).Where(l => l.SentTimeUTC >= EarliestToShow).Where(l => l.SentTimeUTC <= LatestToShow).OrderBy(l => l.SentTimeUTC) into query
where query != null from q in query select q);"
…但是后来Resharper告诉我有关“新的和改进的”代码的信息:“访问闭包中的foreach变量.当使用不同版本的编译器进行编译时,可能会有不同的行为”
那么使用不同版本的编译器进行编译的可能性是什么?我的意思是,例如,从VS2012到VS2010,我不会往后退.
2)在这些行上:
if (db != null)
db.Insert(new PlatypiRequested()
…此代码:
using (var db = new SQLiteConnection(SQLitePath))
{
db.CreateTable<PlatypiRequested>();
db.RunInTransaction(() =>
{
if (db != null)
db.Insert(new PlatypiRequested()
{
PlatypusId = PlatypusId,
PlatypusName = PlatypusName,
InvitationSentLocal = invitationSentLocal
});
});
}
… Resharper通知我,“可以进入已处置的封闭装置”
这是什么意思,我该怎么办?
解决方法:
这里有两个不同的问题,一个是LINQ vs foreach,另一个是不同的情况.
关于ReSharper,当代码为LINQified时通知您“正在访问闭包中的foreach变量…”-我只是从不碰运气,而将其保留为foreach循环.在大多数情况下,它也更具可读性和可维护性,实际上,缩短代码并不是什么大问题.
关于第二种情况-您将需要丢失using语句,因为db对象将被过早处置.您应该在“ RunInTransaction” lambda表达式的结尾处以“老式风格”关闭并处理它.
标签:sqlite,closures,resharper,linq,c 来源: https://codeday.me/bug/20191031/1976637.html