Question

我的项目中有以下方法：

public double CalculateDailyProjectMaxPumpSpm(DateTime date, string start = null, string end = null)
{
    Log("Calculating Daily Pump stroke Max...");

    var spm = new List<double>();

    if (start == null)
    {
        for (var i = 0; i < _pumpOneSpm.Count; i++)
        {
            if (_date[i].Equals(date))
            {
                spm.Add(_pumpOneSpm[i]);
                spm.Add(_pumpTwoSpm[i]);
            }
        }
    }
    else
    {
        for (var i = 0; i < _pumpOneSpm.Count; i++)
        {
            if (_date[i].Equals(date) && 
                DateTime.Compare(_time[i], DateTime.Parse(start)) > 0 && 
                DateTime.Compare(_time[i], DateTime.Parse(end)) < 0)
            {
                spm.Add(_pumpOneSpm[i]);
                spm.Add(_pumpTwoSpm[i]);
            }
        }
    }

    return _dailyProjectMaxSpm = Math.Round(spm.Max(), 2, MidpointRounding.AwayFromZero);
}

我正试图让这个方法看起来不那么笨拙。我试过了：

public double CalculateDailyProjectMaxPumpSpm(DateTime date, string start = null, string end = null)
{
    Log("Calculating Daily Pump stroke Max...");

    var spm = start == null ? _pumpOneSpm.Concat(_pumpTwoSpm).Where((t, i) => _date[i].Equals(date)).ToList()
                            : _pumpOneSpm.Concat(_pumpTwoSpm).Where((t, i) => _date[i].Equals(date) && 
                                                                              DateTime.Compare(_time[i], DateTime.Parse(start)) > 0 && 
                                                                              DateTime.Compare(_time[i], DateTime.Parse(end)) < 0).ToList();

    _dailyProjectMaxSpm = Math.Round(spm.Max(), 2, MidpointRounding.AwayFromZero);

    return _dailyProjectMaxSpm;
}

但是当我运行程序时出现Index out of range. Must be non-negative and less than the size of the collection. Parameter name: index错误。现在，我不在乎将元素添加到新列表的顺序，只要符合条件，就会添加它们。任何人都可以帮我解决这个错误吗？感谢。

更新

_date是从数据库中提取的日期列表，_time是从同一数据库中提取的时间戳列表。 _的所有变量都是从数据库中提取的列表。每个列表的Count将始终等于彼此列表的Count。

Answer 1

原始方法i的范围从0到_pumpOneSpm.Count，但现在从0到 _pumpOneSpm.Count + _pumpTwoSpm.Count

结果列在_pumpOneSpm.Count + _pumpTwoSpm.Count个项目列表中：

_pumpOneSpm.Concat(_pumpTwoSpm).Where((t, i) => _date[i]

我看不到LINQ模拟比使用for循环的第一个方法示例更清晰。

Answer 2

就像sil说的那样，你要连接两个列表，从而产生一个索引范围更大的列表。如何使用Enumerable.Range()生成索引的解决方案，然后使用两个谓词的组合版本进行过滤，最后使用SelectMany()展开列表：

public double CalculateDailyProjectMaxPumpSpm(DateTime date, string start = null, string end = null)
{
    Log("Calculating Daily Pump stroke Max...");

    var spm = Enumerable
         .Range(0, _pumpOneSpm.Count)
         .Where(x => _date[x].Equals(date) &&
                     (start == null ||
                      (DateTime.Compare(_time[x], DateTime.Parse(start)) > 0 && 
                       DateTime.Compare(_time[x], DateTime.Parse(end)) < 0)))
         .SelectMany(x => new [] { _pumpOneSpm[x], _pumpTwoSpm[x] });

    return _dailyProjectMaxSpm = Math.Round(spm.Max(), 2, MidpointRounding.AwayFromZero);
}

索引超出范围错误

2 个答案: