如何检测double数组中的异常值？现有代码处理负数存在问题求优化

首先，你的代码处理负数场景失效的核心原因是不必要的绝对值转换——你把result和limits.Low都转成了绝对值，但计算出的upperThreshold和lowerThreshold是基于原始限值的（可能为负数），用绝对值后的结果和原始阈值比较，直接破坏了负数的数值大小关系，导致异常判断完全混乱。

举个直观的例子：假设limits.Low = -10，limits.High = 0，正常的低阈值应该是-9，高阈值是-1。如果有一个结果是-11（明显低于低阈值的异常值），你的代码会把它转成绝对值11，然后判断11 <= -9？这显然不成立，直接漏掉了这个异常。

另外，你的代码还没有处理limits.High < limits.Low的倒序区间情况，这也会导致阈值计算颠倒，判断逻辑出错。

修正后的代码（处理负数与倒序区间） #

下面是修复了逻辑问题的代码，同时保留了你的异常提示逻辑：

public bool DataHasOutliers(IEnumerable<double> results, Limits limits)
{
    double range = limits.High - limits.Low;
    double lowThreshold;
    double highThreshold;
    bool isRangeReversed = range < 0;

    // 根据区间正/倒序计算阈值
    if (isRangeReversed)
    {
        // 倒序区间：High < Low，10%阈值靠近High，90%阈值靠近Low
        lowThreshold = limits.Low + 0.9 * range;
        highThreshold = limits.Low + 0.1 * range;
    }
    else
    {
        // 正序区间：Low < High，10%阈值靠近Low，90%阈值靠近High
        lowThreshold = limits.Low + 0.1 * range;
        highThreshold = limits.Low + 0.9 * range;
    }

    foreach (double result in results)
    {
        bool isOutlier;
        if (isRangeReversed)
        {
            // 倒序区间：异常是大于90%阈值（靠近Low）或小于10%阈值（靠近High）
            isOutlier = result > lowThreshold || result < highThreshold;
        }
        else
        {
            // 正序区间：异常是小于10%阈值或大于90%阈值
            isOutlier = result < lowThreshold || result > highThreshold;
        }

        if (isOutlier)
        {
            var message = result < lowThreshold ? "Lower threshold violated" : "Upper threshold violated";
            message.Dump();
            return true;
        }
    }

    return false;
}

public class Limits
{
    public double High { get; set; }
    public double Low { get; set; }
    public string Error { get; set; }
}

更简洁的LINQ实现 #

如果你追求代码简洁性，可以用LINQ的Any方法直接判断是否存在异常值，逻辑完全一致：

public bool DataHasOutliers(IEnumerable<double> results, Limits limits)
{
    double range = limits.High - limits.Low;
    bool isReversed = range < 0;
    // 根据区间顺序计算两个阈值
    double threshold1 = limits.Low + (isReversed ? 0.9 : 0.1) * range;
    double threshold2 = limits.Low + (isReversed ? 0.1 : 0.9) * range;

    // 一行代码判断是否存在异常值
    return results.Any(result => 
        isReversed ? result > threshold1 || result < threshold2 
                   : result < threshold1 || result > threshold2);
}

边界情况说明 #

1. 当limits.Low == limits.High时：此时区间是一个点，两个阈值都会等于这个点，任何不等于该点的结果都会被判定为异常，符合业务逻辑。
2. 浮点数精度问题：如果需要严格的精度判断，可以用Math.Abs(result - threshold) < 1e-9这类方式代替直接比较，避免浮点数误差导致的误判。

内容的提问来源于stack exchange，提问作者Kelly Kleinknecht