是否有更优雅的方式一次实现5个项目而不是像这样的for循环?
var q = Campaign_stats.OrderByDescending(c=>c.Leads).Select(c=>c.PID).Take(23);
var count = q.Count();
for (int i = 0; i < (count/5)+1; i++)
{
q.Skip(i*5).Take(5).Dump();
}
答案 0 :(得分:12)
for(int i = 0; i <= count; i+=5)
{
}
答案 1 :(得分:5)
因此,您希望在Dump()
中的每5个项目上有效地致电q
。
您现在拥有的解决方案每次都会通过IEnumerable<T>
循环重新迭代for
。做这样的事情可能更有效:(我不知道你的类型是什么,所以我正在使用T
)
const int N = 5;
T[] ar = new T[N]; // Temporary array of N items.
int i=0;
foreach(var item in q) { // Just one iterator.
ar[i++] = item; // Store a reference to this item.
if (i == N) { // When we have N items,
ar.Dump(); // dump them,
i = 0; // and reset the array index.
}
}
// Dump the remaining items
if (i > 0) {
ar.Take(i).Dump();
}
这只使用一个迭代器。考虑到你的变量名为q
,我假设它是“查询”的缩写,这暗示这是针对数据库的。所以只使用一个迭代器可能非常有用。
我可以保留此代码,并将其包装在扩展方法中。 “丛”怎么样?
public static IEnumerable<IEnumerable<T>> Clump<T>(this IEnumerable<T> items, int clumpSize) {
T[] ar = new T[clumpSize];
int i=0;
foreach(var item in items) {
ar[i++] = item;
if (i == clumpSize) {
yield return ar;
i = 0;
}
}
if (i > 0)
yield return ar.Take(i);
}
在代码的上下文中调用它:
foreach (var clump in q.Clump(5)) {
clump.Dump();
}
答案 2 :(得分:1)
尝试迭代5来代替!
for(int i = 0; i < count; i += 5)
{
//etc
}
答案 3 :(得分:1)
使用GroupBy和Zip添加更多LINQ:
q
// add indexes
.Zip(Enumerable.Range(0, Int32.MaxValue),(a,index)=> new {Index=index, Value=a})
.GroupBy(m=>m.Index /5) // divide in groups by 5 items each
.Select(k => {
k.Select(v => v.Value).Dump(); // Perform operation on 5 elements
return k.Key; // return something to satisfy Select.
});