21

我正在重构一个函数，给定一系列隐式定义区间的端点，检查区间中是否包含一个数字，然后返回一个对应的（不以任何可计算的方式相关）。现在处理这项工作的代码是：

if p <= 100:
    return 0
elif p > 100 and p <= 300:
    return 1
elif p > 300 and p <= 500:
    return 2
elif p > 500 and p <= 800:
    return 3
elif p > 800 and p <= 1000:
    return 4
elif p > 1000:
    return 5

这是 IMO 非常可怕的，并且缺乏间隔和返回值都是硬编码的。当然可以使用任何数据结构。

4

6 回答 6

50 
import bisect
bisect.bisect_left([100,300,500,800,1000], p)

这里的文档：bisect

于 2009-07-29T09:47:54.830 回答
3

你可以试试这个：

def check_mapping(p):
    mapping = [(100, 0), (300, 1), (500, 2)] # Add all your values and returns here

    for check, value in mapping:
        if p <= check:
            return value

print check_mapping(12)
print check_mapping(101)
print check_mapping(303)

产生：

0
1
2

与 Python 一样，会有更好的方法来做到这一点。

于 2009-07-29T09:43:35.353 回答
3

这确实是相当可怕的。如果不要求没有硬编码，它应该是这样写的：

if p <= 100:
    return 0
elif p <= 300:
    return 1
elif p <= 500:
    return 2
elif p <= 800:
    return 3
elif p <= 1000:
    return 4
else:
    return 5

以下是创建查找函数的示例，包括线性搜索和使用二分搜索，满足无硬编码要求，并对两个表进行了一些健全性检查：

def make_linear_lookup(keys, values):
    assert sorted(keys) == keys
    assert len(values) == len(keys) + 1
    def f(query):
        return values[sum(1 for key in keys if query > key)]
    return f

import bisect
def make_bisect_lookup(keys, values):
    assert sorted(keys) == keys
    assert len(values) == len(keys) + 1
    def f(query):
        return values[bisect.bisect_left(keys, query)]
    return f
于 2009-07-29T12:48:31.177 回答
0

尝试以下方式：

d = {(None,100): 0, 
    (100,200): 1,
    ...
    (1000, None): 5}
value = 300 # example value
for k,v in d.items():
    if (k[0] is None or value > k[0]) and (k[1] is None or value <= k[1]):
        return v
于 2009-07-29T09:42:34.513 回答
0 
def which_interval(endpoints, number):
    for n, endpoint in enumerate(endpoints):
        if number <= endpoint:
            return n
        previous = endpoint
    return n + 1

将您的端点作为列表传递给endpoints，如下所示：

which_interval([100, 300, 500, 800, 1000], 5)

编辑：

以上是线性搜索。Glenn Maynard 的答案将具有更好的性能，因为它使用二等分算法。

于 2009-07-29T09:47:46.577 回答
0

其他方式 ...

def which(lst, p): 
    return len([1 for el in lst if p > el])

lst = [100, 300, 500, 800, 1000]
which(lst, 2)
which(lst, 101)
which(lst, 1001)
于 2009-07-29T10:34:49.960 回答