基本上我有一组对象,每个对象都实现了Type IValueCollection
的成员public interface IValueCollection : IEnumerable<decimal>
{
decimal this[int index] { get; set; }
}
MeasurementCollection.Values的类型为IValueCollection。
使用下面的逻辑,我想转动IValueCollection的集合并在下面编写扩展方法。
public static IEnumerable<IValueCollection> PivotValues(this MeasurementCollection items)
{
if(items.IsQuantized())
{
int s = (int)items.First().Template.Frequency; //
int c = items.Count;
for (int n = 0; n < s; n++)
{
IValueCollection v = new MeasurementValueCollection(c);
for (int m = 0; m < c; m++)
{
v[m] = items.ElementAt(m).Values[n];
}
yield return v;
}
}
}
应该这样做 {{1,2,3} {4,5,6} {7,8,9}}导致{{1,4,7},{2,5,8},{3,6,9}} 但是我觉得有一些更好,更苗条,更易读的表达方式 有人能指出我正确的方向吗?
修改 有关基础类的信息
interface IValueCollection : IEnumerable<decimal>
class MeasurementCollection : ICollection<IMeasurement>
interface IMeasurement
{
IMeasurementTemplate Template { get; }
......
}
interface IMeasurementTemplate
{
.....
MeasurementFrequency Frequency { get; }
}
答案 0 :(得分:2)
我个人会提前对您的收藏集进行评估,并将其用作数组。现在,每次拨打ElementAt时,您都会再次评估IEnumerable<T>
,导致大量重复搜索。
通过强制它提前评估数组,可以简化整个过程。类似的东西:
public static IEnumerable<IValueCollection> PivotValues(this MeasurementCollection items)
{
if(items.IsQuantized())
{
int elementLength = (int)items.First().Template.Frequency;
var itemArray = items.ToArray();
for (int n = 0; n < itemArray.Length; n++)
{
IValueCollection v = new MeasurementValueCollection(elementLength);
for (int m = 0; m < elementLength; m++)
{
v[m] = itemArray[m].Values[n];
}
yield return v;
}
}
else
yield break; // Handle the case where IsQuantized() returns false...
}
如果你控制MeasurementValueCollection
,我会添加一个构造函数,它也需要IEnumerable<decimal>
作为输入。然后你可以这样做:
public static IEnumerable<IValueCollection> PivotValues(this MeasurementCollection items)
{
if(items.IsQuantized())
{
var elements = Enumerable.Range(0, (int)items.First().Template.Frequency);
var itemArray = items.ToArray();
foreach(var element in elements)
yield return new MeasurementValueCollection(
itemArray.Select(
(item,index) => itemArray[index].Value[element]
)
);
}
else
yield break; // Handle the case where IsQuantized() returns false...
}