laocaiye 发表于 2025-10-10 21:25:39

关于第44讲函数(IV)的动动手作业1的问题

版主给的参考答案中,关于洗牌这个函数:
def fy_shuffle(x, n=1):
    for i in range(n):
      target = list(x)
      result = []
      while target:
            r = random.randint(0, len(target)-1) # 步骤2
            result.append(target.pop(r)) # 步骤3
   
    return result

这段代码,在while循环接收后,是不是少了一句赋值操作:x = result

另外建议版主的答案中尽量多给些注释

FishC 发表于 2025-10-10 21:27:55

感谢您对第44讲函数(IV)动动手作业1的问题反馈!您观察得很仔细,确实在参考代码中缺少了关键的赋值操作,并且注释不足。下面我来详细解释并给出修正建议。

首先,您指出的问题非常正确:在 while 循环结束后,代码缺少了 x = result 这一赋值操作。这是因为函数设计为多次洗牌(通过参数 n 控制),但每次洗牌后没有更新 x 的值,导致后续洗牌仍然基于原始输入,而不是上一次洗牌的结果。这会使多次洗牌无效。

其次,关于注释的建议很好,我会在修正代码中添加更多注释来帮助理解。

以下是修正后的代码示例,并附有详细注释:


import random# 需要导入random模块

def fy_shuffle(x, n=1):
    # 初始化结果,直接使用x的副本,以处理n=0的情况
    result = list(x)
    for i in range(n):
      target = list(result)# 创建当前结果的副本,用于洗牌
      result = []         # 重置结果列表,准备存储新洗牌后的元素
      while target:
            # 随机选择一个索引位置
            r = random.randint(0, len(target)-1)
            # 从target中移除该元素并添加到result中
            result.append(target.pop(r))
      # 注意:这里不需要额外赋值,因为result已在循环中更新
    return result# 返回最终洗牌后的列表


关键修正点:
- 在循环开始前初始化 result,避免了 n=0 时的未定义错误。
- 每次迭代中,target 基于当前的 result 创建副本,确保洗牌基于上一次结果。
- 添加了注释说明每一步的作用,使代码更易读。

这个版本确保了多次洗牌的正确性,同时处理了边界情况。希望这能帮助您更好地理解函数逻辑!如果还有其他问题,欢迎继续讨论。

以上回复来自 FishC AI 小助理,如未能正确解答您的问题,请继续追问。

小甲鱼的二师兄 发表于 2025-10-11 20:04:55

看了一下,确实有问题 @小甲鱼

小甲鱼 发表于 2025-10-12 23:03:41

小甲鱼的二师兄 发表于 2025-10-11 20:04
看了一下,确实有问题 @小甲鱼

代码是对的,不需要添加 x = result。

不过我又给优化了一下,并添加详细注释。
页: [1]
查看完整版本: 关于第44讲函数(IV)的动动手作业1的问题