Question

我想从列表的随机元素中列出固定数量的值。我得到了一个错误：

import random                                                               

ALL_COLORS = ("red", "green", "blue", "yellow", "orange", "white")          


def create_combination(nb_elements):                                        
    i = 0                                                                   
    combination = ""                                                        
    while i < nb_elements:                                                  
        combination += random.choice(ALL_COLORS)                            
        i += 1                                                              
    print(combination, sep=", ")                                            
    return (combination)                                                    


for i in range(10):                                                         
    combination = create_combination                                        
    assert len(combination) == i    

for color in combination:                                               
    assert color in ALL_COLORS

Answer 1

您的代码过于复杂。

以下是一种可以实现您可能需要的方法：

import random

colours = ("red", "green", "blue", "yellow", "orange", "white")

result = [random.choice(colours) for _ in range(10)]

# Example output
# ['orange', 'red', 'blue', 'yellow', 'green', 'red', 'yellow', 'white', 'orange', 'red']

现有代码存在许多问题，使其无法使用：

没有缩进。
连接字符串而不是添加到列表。如果需要连接字符串，可以轻松地将其作为字符串列表的最后一步执行。
然后检查结果的长度，如果有的话，将是一个字符串，例如“orangeredblueyellow ...”。
这样的简单循环可以用列表推导替换。

如何使用python

1 个答案: