为什么下面的代码返回一个空列表？

Question

def outlier(*args):
    outlist=[]
    def median(args1):
        if(len(args1)%2==1):
            return list(sorted(args1))[int((len(args1)/2)-0.5)]
        else:
            return (list(sorted(args1))[int(len(args1)/2)]+list(sorted(args1))[int(len(args1)/2)-1])/2
    def fmax(args2):
        sortargs=sorted(args2)
        return sortargs[-1]
    def fmin(args3):
        sortargs=sorted(args3)
        return sortargs[0]
    q1=median(list(range(fmin(args),floor(median(args))+1)))
    q3=median(list(range(floor(median(args)),fmax(args)+1)))
    for i in args:
        if(i<(q1-1.5*(q3-q1)) or i>(q3+1.5*(q3-q1)*(q3-q1))):
            outlist.append(i)
    return outlist

print(outlier(1,2,3,4,5,6,7,8,9,10,100000000))

我曾尝试在 Python 中获取列表的异常值，但每次尝试时，它都会返回一个空列表或引发错误。

Answer 1

如果列表返回空，原因是您的if 条件的任何部分都不满足，因此列表中没有附加任何内容：

if(i<(q1-1.5*(q3-q1)) or i>(q3+1.5*(q3-q1)*(q3-q1))):   # never met

如果您不反对使用更大的模块，您可以使用 numpy 计算四分位数，请参阅

• How to calculate 1st and 3rd quartiles?

或使用this answer that gives you a function for manual计算：

---

顺便说一句：

• sorted() 返回一个列表，所以 list(sorted(..)) 是多余的
• 虽然较小的函数很好，但多次排序相同的数据效率不高 - 排序一次 - 获取列表使用的最小值/最大值：
• 您的所有计算都需要排序输入 - 您也可以将其排序一次，然后将排序后的列表用作更下方的输入

您还可以在一次排序中获得最小值和最大值：

def minmax(data):
    if len(data) < 2: 
        raise ValueError("Must be iterable of len 2 or more")
    srt = sorted(data)

    return srt[0], srt[-1]

而不是

def fmax(args2):
    sortargs=sorted(args2)
    return sortargs[-1]

def fmin(args3):
    sortargs=sorted(args3)
    return sortargs[0]