这段python代码是我想要的吗？

Question

我有这个代码

import random
b = 20
x = random.randrange(0,b)
y = random.randrange(0,b)
ab = 0
xc = 0

while ab != 10:
    if x != y:
        x = random.randrange(0,b)
        y = random.randrange(0,b)
        xc = xc + 1
    elif x == y:
        print ab
        print 'number of tries out of', b, ' : ', xc
        ab = ab + 1
        xc = 0
        y = 999999

它本质上是一个统计程序。 我想查看10次尝试中需要生成多少次随机数才能匹配。 到目前为止，我在运行该程序后获得的数字都得到了这些值，我运行了5次该程序，因此总共进行了50次尝试。

9
26
6
1
5
109
5
42
12
63

所有这些低数字使我想知道我的程序是否非常幸运，或者我的代码有什么问题。 谢谢！

注意：有没有一种方法可以让XC在每个while循环后以及while循环为true时加起来以显示总数？

Answer 1

我会说你有点倒霉。

平均为27.8。 但是，如果您选择两个介于0到20之间的数字，则期望它们匹配大约1/20的时间，因此，您希望等待大约20次才能获得匹配。

Answer 2

进行大量迭代检查：

#!/usr/bin/env python
import random

max_num = 20
x = random.randrange(0, max_num)
y = random.randrange(0, max_num)
tries = 0

i = 0
iterations = 1000
total_tries = 0
while i < iterations:
    if x != y:
        x = random.randrange(0, max_num)
        y = random.randrange(0, max_num)
        tries += 1
    else:
        print(i)
        print('number of tries out of %s : %s ' % (max_num, tries))
        total_tries += tries
        i += 1
        tries = 0
        y = -1

print("Average tries: %s" % (total_tries / iterations))

如果我运行这个程序，每次都会得到20左右。

注意：这可以简化为：

#!/usr/bin/env python
import random

max_num = 20
iterations = 1000
total_tries = 0

for i in range(iterations):
    print(i)

    tries = 0
    x = 0
    y = -1
    while x != y:
        x = random.randrange(0, max_num)
        y = random.randrange(0, max_num)
        tries += 1

    print('number of tries out of %s : %s ' % (max_num, tries))
    total_tries += tries
    i += 1
    tries = 0

print("Average tries: %s" % (total_tries / iterations))

Answer 3

你的逻辑很奇怪。

• 您不想使用人工前哨值来打破循环。 只是明确地打破它。

• 从根本上讲，您有两个循环：一个循环进行试验，另一个循环查看给定试验中需要进行多少次尝试。 不要隐藏那个结构。

• 不要用if / else if涵盖所有情况。 那就是else目的。

• 在循环中，请在测试之前（而不是之后）生成数字。 这样一来，您就可以清楚地知道自己在做什么，并且在循环之前不需要额外的生成步骤。 同样，在顶部而不是底部的循环中重新初始化值。

• 使用明确的变量名。 如果没有有意义的变量名，请避免创建变量。 您实际上不需要将两个变量设置为random.randrange(0, b)来比较结果。 OTOH，如果您想简化逻辑并避免对random.randrange(0, b)进行看似奇怪的比较，那么您可以注意（并且应该能够证明，如果您对此类型足够感兴趣的话）编写程序的东西），您可以任意选择目标值并获得相同的结果。 同样，使用变量来命名任意选择的数字常量。

• 您可以在Python中使用+=更新变量。

• 使用打印格式。

---

import random
range_size = 20
total = 0
iterations = 1000 # or however many

for trial in xrange(iterations):
    attempts = 0
    while random.randrange(0, range_size) != 0: attempts += 1
    print "Trial #{0}: Took {1} tries to match one of {2} numbers.".format(
        trial, attempts, range_size
    )
    total += attempts

print "Average trials: {0}".format(float(total) / iterations)

如果您不需要调试信息，我们可以使用内置函数为我们做求和和循环逻辑，从而使事情变得更加干净：

from random import randrange
from itertools import *
range_size = 20
total = 0
iterations = 1000 # or however many

print "Average trials: {0}".format(sum(
    sum(takewhile(lambda x: randrange(0, range_size) != 0, repeat(1)))
    # I tested that way, but this is probably more logical
    # even if it's more verbose:
    # len(list(takewhile(
    #     lambda x: x != 0,
    #     imap(randrange, repeat(0), repeat(range_size))
    # )))
    # 'lambda x: x != 0' can also be spelled 'bool' in this context,
    # but explicit is better than implicit...
    for i in xrange(iterations)
) / float(iterations))