有时,我的用户会遇到这样的问题,我可以在日志文件中看到抛出此异常(序列不包含任何元素)
我四处搜索,发现当您尝试在空列表上访问或使用聚合时会发生此异常。
我搜索了围绕此异常的代码(太糟糕了,没有记录堆栈跟踪),唯一的“潜在”罪魁祸首是以下几行(使用 Fist()、Last()、Single() 或任何聚合) .但是我不明白为什么也不能在我的本地重现。请帮忙指教。
if (data.Any())
return data.OrderByDescending(d => d.UpdatedTime).First().UpdatedTime;
在这里,data
是 List<MyObject>
和 MyObject
有DateTime
属性名为 UpdatedTime
=====更多周边代码=====
这是我在日志中得到未处理异常的地方。 GetRecentUpdates 方法有自己的 try catch block ,因此排除。
public ActionResult GetUpdatedTime(long lastUpdated) {
var data = dataAccess.GetRecentUpdates(lastUpdated);
var html = htmlBuilder.Build(data);
return Content(html);
}
public List<MyObject> GetRecentUpdates(long lastUpdatedInTicks) {
var list = _cache.GetRecentRequests(_userCache.UserId);
if (list != null) {
var lastUpdated = new DateTime(lastUpdatedInTicks);
list = list.Where(l => l!=null && l.UpdatedTime > lastUpdated).ToList();
}
return list ?? new List<MyObject>();
}
public List<MyObject> GetRecentRequests(string userId) {
List<MyObject> requests = null;
try {
// simplied but the idea stays
requests = dictionary.Get(userId);
commonRequests = dictionary.Get("common");
if (requests != null) {
if (commonRequests != null)
requests = requests.Union(commonRequests).ToList();
} else {
request = commonRequests;
}
if (requests != null) {
requests = requests.OrderByDescending(r => r.CreatedDateTime).ToList();
}
catch (Exception ex) {
// log the exception (handled)
}
return requests;
}
public string Build(List<MyObject> data) {
var lastUpdated = DateTime.MinValue;
if (data.Any())
lastUpdated = data.OrderByDescending(d => d.UpdatedTime).First().UpdatedTime;
return String.Format("<tr style=\"display:none\"><td><div Id='MetaInfo' data-lastUpdated='{0}' /></td></tr>", lastUpdated.Ticks);
}
javascript 调用 GetUpdatedTime
每 10 秒。通常一切顺利,但偶尔会抛出此异常。一旦抛出,它就会每 10 秒持续抛出一次,直到用户刷新页面。
最佳答案
更新:
经过一些调查后的另一个版本:正如您所说,您的代码在多线程环境中运行,并且 data
对象可以被两个或多个线程访问。因为它是一个reference
类型的变量,它的引用是可以修改的。所以,考虑这样的情况:
第一个线程进入Build
方法并检查条件:
if (data.Any())
此时data
不为空,所以进入true
block 。 恰好在这个时候另一个线程进入了Build
方法,但是此时data
变量是空的,它的所有引用都指向清空 List
。但是第一个线程已经进入 true
block :
lastUpdated = data.OrderByDescending(d => d.UpdatedTime).First().UpdatedTime;
它失败了,你除外。现在有个好消息:您可以通过多种方式修复它:
- 首先,检查创建
data
的逻辑。可能是,它是一个静态或共享变量,或者它被填充的对象是一个静态或共享变量,并且您有此资源的竞争条件。您可以更改它的创建逻辑或将其包装到某种同步原语中,这样只有一个线程可以同时Build
(但这会影响程序的性能). - 更改
GetRecentRequests
的逻辑 - 不能肯定地说,但我认为情况是这样的:commonRequests
一直是空的,并且对于第一个线程dictionary
获得了一些数据,但第二个线程没有数据,并且data
对象被覆盖并且为空。 调试方法:在测试运行时向您的程序添加Barrier
原语,并等待10-15个线程等待屏障。之后,他们将同时开始构建您的数据,并且很可能会发生错误(不要插入断点 - 他们会同步您的线程)。 制作
data
对象的本地副本,如下所示:var localData = data.Select(d => d).ToList();
希望这对您有所帮助。
您的代码正在检查某些数据是否可用,然后按日期过滤数据。当您使用 LINQ 扩展方法时,我认为 data
是一个 IEnumerable
对象,而不是 List
,因此,当您调用 Any()
方法,正在枚举,然后调用 First()
方法,也在枚举它.
因此,如果您的数据是某些yield return
方法的结果,它会被枚举一次,而第二次那里没有数据,序列为空。
考虑更改您的代码以将数据作为 List
或 Array
使用,或者使用 FirstOrDefault
方法让 null
对象,如果没有数据,像这样:
//var dataList = data.OrderByDescending(d => d.UpdatedTime).ToList();
if (data.Count > 0)
return dataList[0].UpdatedTime;
或
var firstElement = data.OrderByDescending(d => d.UpdatedTime).FirstOrDefault();
return firstElement != null ? firstElement.UpdatedTime : DateTime.MinValue;
关于c# - LINQ : Sequence contains no elements 错误,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/29004258/