9

我有一个实体集合Readings。每个Reading都链接到一个名为 的实体Meter。(并且每个Meter都有多个读数)。每个都Reading包含一个仪表 ID (int) 字段和一个时间字段。

这是一些简化的代码来演示它:

public class Reading
{
    int Id;
    int meterId;
    DateTime time;
}

public class Meter
{
    int id;
    ICollection<Readings> readings;    
}

给定一个特定的时间段和meterids 列表,为每个仪表获取该时间段内的第一个和最后一个读数的最有效方法是什么?

我能够遍历所有仪表和每个仪表以获取该期间的第一个和最后一个读数,但如果有更有效的方法来实现这一点,我正在徘徊。

还有一个额外的问题:同一个问题,但需要多个时间段来获取数据,而不仅仅是一个时间段。

4

7 回答 7

3

我不完全确定您想要这些数据,但您可以将其投影为匿名类型:

var metersFirstAndLastReading = meters.Select(m => new 
    {
        Meter = m,
        FirstReading = m.readings.OrderBy(r => r.time).First(),
        LastReading = m.readings.OrderBy(r => r.time).Last()
    });

然后你可以像这样阅读你的结果列表(这个例子只是为了说明):

foreach(var currentReading in metersFirstAndLastReading)
{
    string printReadings = String.Format("Meter id {0}, First = {1}, Last = {2}", 
                               currentReading.Meter.id.ToString(),
                               currentReading.FirstReading.time.ToString(),
                               currentReading.LastReading.time.ToString());

    // Do something...
}

另一种选择是在 Meter 中创建动态返回第一个和最后一个读数的属性:

public class Meter
{
    public int id;
    public List<Reading> readings;

    public Reading FirstReading 
    {
        get
        {
            return readings.OrderBy(r => r.time).First();
        }
    }

    public Reading LastReading
    {
        get
        {
            return readings.OrderBy(r => r.time).Last();
        }
    }
}

编辑:我有点误解了这个问题。

这是确定包含日期范围的仪表的第一个和最后一个读数的实现(假设meterIdList是一个ICollection<int>ID,begin并且end是指定的日期范围)

var metersFirstAndLastReading = meters
    .Where(m => meterIdList.Contains(m.id))
    .Select(m => new 
    {
        Meter = m,
        FirstReading = m.readings
                        .Where(r => r.time >= begin && r.time <= end)
                        .OrderBy(r => r.time)
                        .FirstOrDefault(),
        LastReading = m.readings
                        .Where(r => r.time >= begin && r.time <= end)
                        .OrderByDescending(r => r.time)
                        .FirstOrDefault()
    });

您现在将无法使用属性(因为您需要提供参数),因此方法可以作为替代方法正常工作:

public class Meter
{
    public int id;
    public List<Reading> readings;

    public Reading GetFirstReading(DateTime begin, DateTime end)
    {
        var filteredReadings = readings.Where(r => r.time >= begin && r.time <= end);

        if(!HasReadings(begin, end))
        {
            throw new ArgumentOutOfRangeException("No readings available during this period");
        }

        return filteredReadings.OrderBy(r => r.time).First();
    }

    public Reading GetLastReading(DateTime begin, DateTime end)
    {
        var filteredReadings = readings.Where(r => r.time >= begin && r.time <= end);

        if(!HasReadings(begin, end))
        {
            throw new ArgumentOutOfRangeException("No readings available during this period");
        }

        return filteredReadings.OrderBy(r => r.time).Last();
    }

    public bool HasReadings(DateTime begin, DateTime end)
    {
        return readings.Any(r => r.time >= begin && r.time <= end);
    }
}
于 2013-05-27T07:32:58.637 回答
1

我有一个非常相似的数据模型,该代码用于获取最旧的读数,我只是将其更改为也包括最新的。

我使用查询语法来做这样的事情:

var query = from reading in db.Readings
            group reading by reading.meterId
            into readingsPerMeter
            let oldestReadingPerMeter = readingsPerMeter.Min(g => g.time)
            let newestReadingPerMeter = readingsPerMeter.Max(g => g.time)
            from reading in readingsPerMeter
            where reading.time == oldestReadingPerMeter || reading.time == newestReadingPerMeter 
            select reading; //returns IQueryable<Reading> 

这将导致每个仪表只有最新和最旧的读数。

我认为这是有效的原因是因为它对数据库的一次查找以获取每个仪表的所有读数,而不是对每个仪表进行多次查找。我们有约 40000 米,读数约 3000 万。我刚刚测试了对我们数据的查找大约需要 10 秒

执行的 sql 是针对每个最小和最大日期的两个子选择之间的交叉连接。

更新:

由于这是可查询的,您应该能够在之后提供一段时间,如下所示:

query.Where(r=>r.time > someTime1 && r.time < someTime2)

或者把它放到原来的查询中,我就是喜欢这样分开。查询尚未执行,因为我们尚未执行获取数据的操作。

于 2013-05-27T09:25:55.607 回答
0

创建一个名为的新类作为返回类型Result,如下所示

public class Result
{
    public int MeterId;
    public Readings Start;
    public Readings Last;
}

我通过列出仪表并填充一些数据来模拟您的情况,但您的查询应该几乎相同

var reads = Meters.Where(x => x.readings != null)
                  .Select(x => new Result
                          {
                              MeterId = x.id,
                              Start = x.readings.Select(readings => readings).OrderBy(readings=>readings.time).FirstOrDefault(),
                              Last = x.readings.Select(readings=>readings).OrderByDescending(readings=>readings.time).FirstOrDefault()
                          });
于 2013-05-27T07:33:26.827 回答
0
public IEnumerable<Reading> GetFirstAndLastInPeriod
    (IEnumerable<Reading> readings, DateTime begin, DateTime end)
{
    return
        from reading in readings
        let span = readings.Where(item => item.time >= begin && item.time <= end)
        where reading.time == span.Max(item => item.time) 
            || reading.time == span.Min(item => item.time)
        select reading;            
}
于 2013-05-27T07:34:14.800 回答
0
meters.Where(mt=>desiredMeters.Contains(mt)).Select(mt=>
   new{
     mt.Id,
     First = mt.Readings.Where(<is in period>).OrderBy(rd=>rd.Time).FirstOrDefault(),
     Last = mt.Readings.Where(<is in period>).OrderBy(rd=>rd.Time).LastOrDefault()
   });

如果每米有很多读数,这将不会很好,你应该考虑读数是 SortedList 类。

于 2013-05-27T07:45:57.767 回答
0

我的解决方案将准确返回您想要的(包含给定时间段内读数的所有仪表的列表)

public IList<Reading[]> GetFirstAndLastReadings(List<Meter> meterList, DateTime start, DateTime end)
     {       
        IList<Reading[]> fAndlReadingsList = new List<Reading[]>();

            meterList.ForEach(x => x.readings.ForEach(y =>
            {
                var readingList = new List<Reading>();
                if (y.time >= startTime && y.time <= endTime)
                {
                      readingList.Add(y);
                      fAndlReadingsList.Add(new Reading[] { readingList.OrderBy(reading => reading.time).First(), readingList.OrderBy(reading => reading.time).Last() });
                }
            }));

       return fAndlReadingsList;
    }
于 2013-05-27T08:28:53.237 回答
0

感谢所有响应者,我得到了一些非常好的线索。这是对我有用的解决方案:

        /// <summary>
        /// Fills the result data with meter readings matching the filters.
        /// only take first and last reading for each meter in period.
        /// </summary>
        /// <param name="intervals">time intervals</param>
        /// <param name="meterIds">list of meter ids.</param>
        /// <param name="result">foreach meter id , a list of relevant meter readings</param>
        private void AddFirstLastReadings(List<RangeFilter<DateTime>> intervals, List<int> meterIds, Dictionary<int, List<MeterReading>> result)
        {
            foreach (RangeFilter<DateTime> interval in intervals)
            {
                var metersFirstAndLastReading = m_context.Meter.Where(m => meterIds.Contains(m.Id)).Select(m => new
                {
                    MeterId = m.Id,
                    FirstReading = m.MeterReading
                                    .Where(r => r.TimeStampLocal >= interval.FromVal && r.TimeStampLocal < interval.ToVal)
                                    .OrderBy(r => r.TimeStampLocal)
                                    .FirstOrDefault(),
                    LastReading = m.MeterReading
                                    .Where(r => r.TimeStampLocal >= interval.FromVal && r.TimeStampLocal < interval.ToVal)
                                    .OrderByDescending(r => r.TimeStampLocal)
                                    .FirstOrDefault()
                });

                foreach (var firstLast in metersFirstAndLastReading)
                {
                    MeterReading firstReading = firstLast.FirstReading;
                    MeterReading lastReading = firstLast.LastReading;

                    if (firstReading != null)
                    {
                        result[firstLast.MeterId].Add(firstReading);
                    }

                    if (lastReading != null && lastReading != firstReading)
                    {
                        result[firstLast.MeterId].Add(lastReading);
                    }

                }

            }
        }


    }
于 2013-05-28T08:37:33.550 回答